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
Add survey for MPLS usage #16403
Add survey for MPLS usage #16403
Conversation
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.
Should remove updateMemento()
in favor of using built in diagnostic command API, LGTM otherwise.
Working on refactoring to use the commands and ditch the memento, testing is just not fun. |
Refactored. This relies on the assumption that |
@karrtikr Can you take a look to see if the current state is reasonable? I am not super familiar with this new diagnostic infrastructure. |
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.
LGTM
This is modeled on #16069 and the main survey banner.