Skip to content

Commit fa210df

Browse files
[cleanup] Remove unnecessary mcast tests (#92)
The exclusion/prune and no_admin_ula mcast integration tests were unnecessary and part of previously configurable code that was made obsolete during the last review cycle of the large mcast PR. The exclusion/prune test was being flaky on CI, but didn't set up proper MAC linking anyway. It's not useful because we force 0 for the exclusion flag when we configure replication on the underlay groups. Includes: * use oxnet's derive mac facility for tests, updating the dep * mcast integration test comment cleanup * move P4 # comment to //
1 parent 6e66652 commit fa210df

File tree

4 files changed

+63
-255
lines changed

4 files changed

+63
-255
lines changed

Cargo.lock

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ omicron-common = { git = "https://github.com/oxidecomputer/omicron", branch= "ma
4444
oximeter = { git = "https://github.com/oxidecomputer/omicron", branch = "main" }
4545
oximeter-producer = { git = "https://github.com/oxidecomputer/omicron", branch = "main" }
4646
oximeter-instruments = { git = "https://github.com/oxidecomputer/omicron", branch = "main", default-features = false, features = ["kstat"] }
47-
oxnet = { version = "0.1.2", default-features = false, features = ["schemars", "serde"] }
47+
oxnet = { version = "0.1.3", default-features = false, features = ["schemars", "serde"] }
4848
propolis = { git = "https://github.com/oxidecomputer/propolis" }
4949
sled-agent-client = { git = "https://github.com/oxidecomputer/omicron", branch = "main" }
5050
smf = { git = "https://github.com/illumos/smf-rs" }

0 commit comments

Comments
 (0)