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
The test_port_settings_basic_crud test is currently disabled because it requires that nexus talk to MGS to figure out which switches it needs to talk to land port settings on a particular switch.
When this test was originally written, I believe we only had support for one switch and MGS was not involved in helping nexus make decisions. But now that we have multi-switch support, things are more complicated. This may ultimately not be necessary depending on how the following issues pan out e.g., if nexus determines what switches to talk to in ways that do not involve MGS.
I think for this and other related issues, we should effectively pin the version of mgd etc. We could either build it within omicron, or have a nextest setup script that can ensure that the right version of mgd (etc) is present on disk (and if not then download it). We should also consider enforcing consistency for non-test environments as well -- again, using either approach.
The
test_port_settings_basic_crud
test is currently disabled because it requires that nexus talk to MGS to figure out which switches it needs to talk to land port settings on a particular switch.omicron/nexus/tests/integration_tests/switch_port.rs
Lines 28 to 33 in e0a1184
When this test was originally written, I believe we only had support for one switch and MGS was not involved in helping nexus make decisions. But now that we have multi-switch support, things are more complicated. This may ultimately not be necessary depending on how the following issues pan out e.g., if nexus determines what switches to talk to in ways that do not involve MGS.
HashMap<SwitchLocation, Client>
with on-demand lookups, and eventually DNS #5092The text was updated successfully, but these errors were encountered: