Skip to content

Commit

Permalink
Reconfigurator: Teach planner about SNAT IPs
Browse files Browse the repository at this point in the history
Prior to this change, the planner expected all blueprints to have
fully-exclusive external IP addresses. This isn't compatible with #6037,
where RSS now hands out SNAT IPs with distinct port ranges but the same
IP address.

A big chunk of this work is necessary to support boundary NTP zone
planning, but that isn't included in this PR, so those bits are marked
with `#[cfg(test)]`.

Fixes #6194.
  • Loading branch information
jgallagher committed Jul 31, 2024
1 parent 35a5041 commit 85c8ba8
Show file tree
Hide file tree
Showing 3 changed files with 388 additions and 17 deletions.
2 changes: 2 additions & 0 deletions Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 2 additions & 0 deletions nexus/reconfigurator/planning/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,8 @@ oxnet.workspace = true
rand.workspace = true
sled-agent-client.workspace = true
slog.workspace = true
static_assertions.workspace = true
strum.workspace = true
thiserror.workspace = true
typed-rng.workspace = true
uuid.workspace = true
Expand Down
Loading

0 comments on commit 85c8ba8

Please sign in to comment.