oxidecomputer / omicron

Omicron: Oxide control plane
Mozilla Public License 2.0
252 stars 40 forks source link

[reconfigurator] BlueprintBuilder cleanup 5/5 - Convert discretionary ensure_zone_multiple_* to add_zone_* #7107

Closed jgallagher closed 9 hours ago

jgallagher commented 6 days ago

All of the callers of the ensure_zone_multiple_* methods to add discretionary zones were, in practice, wanting to add a specific number of zones. This rewords all of those methods to be imperative, allowing us to remove a pretty significant chunk of now-unnecessary checks and logic.

Builds on and is staged on top of #7106.