Resolve "Add error handling in case the paths for gNMI Subscription for an MNE is empty"
Merged
requested to merge 333-add-error-handling-in-case-the-paths-for-gnmi-subscription-for-an-mne-is-empty into master
Description
Related Issue
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
-
Bug fix (non-breaking change which fixes an issue) -
New feature (non-breaking change which adds functionality) -
Breaking change (fix or feature that would cause existing functionality to change)
Checklist:
-
My code follows the code style of this project. -
My change requires a change to the documentation. -
I have updated the documentation accordingly. -
I have read the CONTRIBUTING document. -
I have added tests to cover my changes. -
All new and existing tests passed.
Related to #333
Merge request reports
Activity
added componentevent-system goSDN prioritylow statusavailable typeimprovement labels
added 1 commit
- 6591cc62 - added skip for mne monitoring in the case of no provided paths
After a short talk with @hda11597 we decided, the error handling as described in #333 is not desired. We thought that users don't need to be informed about subscriptions not being established, if there are no paths provided during the creation of an MNE and if there are no paths in the subscription txt. Instead, in the case that both are empty, we simply skip the monitoring stuff to prevent the use of unnecessary computation cycles.
added statusreview label and removed statusavailable label
assigned to @istmabauc and @hda11597
mentioned in commit f32a160c
mentioned in issue #333
Please register or sign in to reply