You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
While investigating #5665, I ran into a few places where we're using Ipv6Network for types that we know should have a specific prefix size. Two examples are
Both of these should always be /56 networks, but the types don't enforce that, which allowed #5665 to sneak in in the first place. I think these should be Ipv6Subnet<RACK_PREFIX> (and the same for other places where we know we should have a specific prefix size). I don't think this is a trivial fix because at least the first of those examples above is serialized in the bootstore, so any change would either need to be backwards compatible or deal with migrating the format.
The text was updated successfully, but these errors were encountered:
While investigating #5665, I ran into a few places where we're using
Ipv6Network
for types that we know should have a specific prefix size. Two examples areomicron/common/src/api/internal/shared.rs
Line 162 in 9c90e4b
omicron/nexus/db-model/src/rack.rs
Line 19 in 9c90e4b
Both of these should always be
/56
networks, but the types don't enforce that, which allowed #5665 to sneak in in the first place. I think these should beIpv6Subnet<RACK_PREFIX>
(and the same for other places where we know we should have a specific prefix size). I don't think this is a trivial fix because at least the first of those examples above is serialized in the bootstore, so any change would either need to be backwards compatible or deal with migrating the format.The text was updated successfully, but these errors were encountered: