Skip to content

feat(forwarding): add BridgeLearnAction builder, exact RemoveKey, and… - #757

Merged
hardikharti merged 3 commits into
openconfig:mainfrom
thineshvs-google:l2-chunk1-fwdtable-primitives
Oct 19, 2026
Merged

hardikharti merged 3 commits into
openconfig:mainfrom
thineshvs-google:l2-chunk1-fwdtable-primitives

Conversation

@thineshvs-google

Copy link
Copy Markdown
Contributor

… bridge learn callback

  • dataplane/forwarding/fwdconfig/action.go:
    • Add BridgeLearnActionBuilder and BridgeLearnAction helper for ACTION_TYPE_BRIDGE_LEARN.
  • dataplane/forwarding/fwdtable/exact/exact.go:
    • Reset stale list pointers on Table.Clear() to prevent dangling transient entry references.
    • Add Table.RemoveKey(key) to remove an exact-match entry by key and unlink it from the stale list.
  • dataplane/forwarding/fwdtable/bridge/bridge.go:
    • Add SetLearnCallback and invoke LearnCallback when a dynamic MAC entry is learned.
    • Add Table.Remove(mac) wrapper delegating to exact.Table.RemoveKey.

… bridge learn callback

- dataplane/forwarding/fwdconfig/action.go:
  - Add BridgeLearnActionBuilder and BridgeLearnAction helper for ACTION_TYPE_BRIDGE_LEARN.
- dataplane/forwarding/fwdtable/exact/exact.go:
  - Reset stale list pointers on Table.Clear() to prevent dangling transient entry references.
  - Add Table.RemoveKey(key) to remove an exact-match entry by key and unlink it from the stale list.
- dataplane/forwarding/fwdtable/bridge/bridge.go:
  - Add SetLearnCallback and invoke LearnCallback when a dynamic MAC entry is learned.
  - Add Table.Remove(mac) wrapper delegating to exact.Table.RemoveKey.
@thineshvs-google
thineshvs-google requested a review from a team as a code owner October 15, 2026 16:40
@hardikharti

Copy link
Copy Markdown
Contributor

/gcbrun

@hardikharti
hardikharti merged commit ff88698 into openconfig:main Oct 19, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants