-
Notifications
You must be signed in to change notification settings - Fork 82
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
Interface and Ethernet Counter support #139
Interface and Ethernet Counter support #139
Conversation
Co-authored-by: Satoru-Shinohara <satoru.shinohara@dell.com> Co-authored-by: Nikita Agarwal <nikita_agarwal1@dell.com>
Co-authored-by: Satoru-Shinohara <satoru.shinohara@dell.com> Co-authored-by: Nikita Agarwal <nikita_agarwal1@dell.com>
* add support for counters - top level and ethernet * Code Clean-up * Add subscription support for interface and ethernet counters * Update test-cases to support interface and ethernet state counters
* add support for counters - top level and ethernet * Code Clean-up * Add subscription support for interface and ethernet counters * Update test-cases to support interface and ethernet state counters * address azure-pipelines error
0b5b2c4
to
fa5021c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The clear counter seems to be pending. I presume that would be coming in as a separate pull request.
Yes, clear counters will be supported in a later PR. Thank you for the review! |
Support for Interface and Ethernet counters.
This should address the pipeline test error that is caused by missing path.
Fail on UT-log are for tests that leave out ipv4-state as pipelines do not have APPL_DB while local switches do.
UT_run.log
translib_run.log
manual_GET_interface.log
manual_GET_interface_ethernet.log
Ethernet257 has a switch connected to it, and will have non-0 values.
Fail on on-change is because these attributes has on-change disabled in deviation.
on_change_subscription.log
target_defined_subscription.log
sample_subscription.log