Commit 0f80a2f
committed
[multicast] Narrow underlay scope to ff04::/64 and add ASM source filtering
Re-opening this from #162, as that PR was getting pretty gnarly with a saturation of merge commits. This one deserves a re-review after some added changes to match the Omicron mcast-lifecycle work in oxidecomputer/omicron#9450.
Underlay changes:
- Restrict internal multicast from admin/site/org scoped (ff04, ff05, ff08) to just Omicron's reserved subnet (ff04::/64). These underlay addresses are made unique in Omicron.
- Rename `AdminScopedIpv6` to `UnderlayMulticastIpv6`.
- Simplify P4 to only match ff04::/64.
- Use omicron-common multicast constants for validation.
Source filtering changes:
- Replace IpSrc::Subnet with IpSrc::Any for any-source matching.
- Change IPv6 source filter from exact to LPM match.
- Allow source filters on ASM groups (previously SSM-only).
API changes:
- API v5 adds IpSrc::Any for ASM source filtering.
- API v6 enforces strict underlay subnet validation.
Reviewers: FelixMcFelix, Nieuwejaar, rcgoodfellow
Pull Request: #1891 parent 846348b commit 0f80a2f
File tree
33 files changed
+24844
-1342
lines changed- .cargo
- dpd-api/src
- dpd-client
- tests/integration_tests
- dpd-types/src
- dpd
- p4
- src
- mcast
- table/mcast
- openapi/dpd
- swadm/src
- xtask/src
33 files changed
+24844
-1342
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
0 commit comments