gNMI-1.3: update BGP import/export policy, MED, and IS-IS authentication configuration - #6048
LiamWalsh98 wants to merge 8 commits into
Conversation
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request updates the gNMI-1.3 implementation to align with revised configuration paths for BGP policies, MED actions, and IS-IS authentication. The changes simplify policy handling by introducing helper functions for peer-group modifications and ensure compatibility with current OpenConfig models by moving authentication settings to the interface level. Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize the Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counterproductive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request refactors BGP policy configuration and cleanup in the drained configuration convergence time test by introducing helper functions replacePeerGroupExportPolicy and deletePeerGroupApplyPolicy to handle platform deviations. It also removes a Cisco platform exception from metadata and updates IS-IS authentication in a shared setup helper. Review feedback advises against changing the default IS-IS authentication globally to prevent regressions, suggests using gnmi.Lookup instead of gnmi.Get to safely retrieve unconfigured policy paths, and warns against deleting unsupported AFI/SAFI policy paths to avoid gNMI errors on strict implementations.
|
/fptest virtual |
Summary
Updates gNMI-1.3 to replace complete BGP import/export policy roots, send
set-med-actionwithset-med, and configure IS-IS authentication at the interface level.Path changes
Previous BGP export-policy leaf-list update:
.../apply-policy/config/export-policyCurrent BGP import/export policy root replacement:
.../apply-policy/configExisting MED value:
/routing-policy/policy-definitions/policy-definition/statements/statement/actions/bgp-actions/config/set-medRequired companion leaf:
/routing-policy/policy-definitions/policy-definition/statements/statement/actions/bgp-actions/config/set-med-actionPrevious IS-IS authentication path:
/network-instances/network-instance/protocols/protocol/isis/interfaces/interface/levels/level/config/hello-authentication-typeCurrent IS-IS authentication path:
/network-instances/network-instance/protocols/protocol/isis/interfaces/interface/config/authentication-typeRemoved deviation assignment
bgp_set_med_action_unsupportedgNMI-1.3
Replaces and deletes the complete peer-group
apply-policyroot, enables the paired MED action, and uses interface-level IS-IS authentication as the default behavior.