Add policy forwarding for next hop match and GUE encapsulation action #1203
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add support for policy forwarding match on next hop and GUE encapsulation action. * Update container name to be address specific
* Import oc-types module before using it.
|
/gcbrun |
|
Major YANG version changes in commit 8702a2d: |
earies
reviewed
Oct 21, 2024
| @@ -303,7 +303,7 @@ submodule openconfig-aft-common { | |||
| } | |||
|
|
|||
| container gre { | |||
| when "../state/type = 'oc-aftt:GRE'"; | |||
| when "../state/type = 'oc-types:GRE'"; | |||
There was a problem hiding this comment.
Since there is a relocation of types to a separate model - the openconfig-version and revision must be updated to reflect this backwards incompatible change
There was a problem hiding this comment.
+1, but I don't see why we need to make this change. Why not leave these in AFT types?
There was a problem hiding this comment.
The next PR from @danameme will use them in policy-forwarding
There was a problem hiding this comment.
It's used in policy-forwarding in this same PR.
…ute target (openconfig#1109) * Add a new leaf "es-import-route-target" to the Ethernet Segment config model to allow the explicit configuration of an ES-import route target as per RFC7432 Section 7.6
…empt` annotation to AFT counters containers (openconfig#1186) * Add atomic-exempt and use it in counters AFT
… (updated versions also)
Resolved version conflict in openconfig-aft-common.yang
|
/gcbrun |
Signed-off-by: Emmanuel Ferdman <[email protected]>
… (with version updates)
|
/gcbrun |
… (use correct versions)
|
/gcbrun |
…onfig#1165) * Refactor non-compliant/overlapping YANG definition for BGP MED * (M) release/models/bgp/openconfig-bgp-policy.yang - Remove string type from bgp-set-med-type union - Add new bgp-set-med-action typedef to separate out action on MED values - Add new set-med-action leaf to be optionally utilized with set-med * Add constraints between med/action + add an explicit SET action --------- Co-authored-by: Darren Loher <[email protected]>
… (resolve file conflicts)
|
/gcbrun |
* Delete idle-timeouts and counters-last-cleared leaves * Split source-counters and client counters to be under separate augments Co-authored-by: Darren Loher <[email protected]>
… (resolve conflicts)
|
/gcbrun |
|
Create a new #1208 due to merge conflicts from committed changes. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Change Scope