Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Only warn list of tables which not covered by YANG #3589

Closed
wants to merge 1 commit into from

Conversation

wen587
Copy link
Contributor

@wen587 wen587 commented Oct 23, 2024

ADO: 29917681

What I did

Sent a warn syslog if there are tables not covered by yang.

How I did it

Sent a warn syslog if there are tables not covered by yang.

How to verify it

Manual test

Improve output from

2024 Oct 23 07:43:20.293672 bjw-can-7260-12 WARNING config: YANG failed to cover tables: {'TABLE_NOT_COVRED_BY_YANG': {'mux_tunnel_egress_acl': {'status': 'disabled'}, 'tunnel_qos_remap': {'status': 'enabled'}}, 'bgpraw': 'aa', 'bgpraw1': 'aa', 'bgpraw2': 'aa'}.

to

2024 Oct 23 07:45:21.755952 bjw-can-7260-12 WARNING config: YANG failed to cover tables: ['TABLE_NOT_COVRED_BY_YANG', 'bgpraw', 'bgpraw1', 'bgpraw2']

Previous command output (if the output of a command-line utility has changed)

New command output (if the output of a command-line utility has changed)

@wen587
Copy link
Contributor Author

wen587 commented Oct 23, 2024

Create seperate PR for 202311: #3591
and for master and 202405: #3590

@wen587 wen587 closed this Oct 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant