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

[202311] Set warn msg for list of tables which not covered by YANG #3591

Merged
merged 1 commit into from
Oct 24, 2024

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. Only show the list of keys of table name

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 wen587 requested a review from yxieca October 23, 2024 08:38
@yxieca yxieca merged commit 4907fba into sonic-net:202311 Oct 24, 2024
5 checks passed
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.

2 participants