test(e2e): remove schema parity campaign - #3864
Conversation
Signed-off-by: Evan Lezar <elezar@nvidia.com>
|
🌿 Preview your docs: https://nvidia-preview-pr-3864.docs.buildwithfern.com/openshell |
591e1af to
3d1fd5b
Compare
|
Analysis by agent found one cleanup item I think is worth addressing at some point: podman_config in crates/openshell-gateway/src/lib.rs reads OPENSHELL_PODMAN_HOST_GATEWAY_IP and OPENSHELL_PODMAN_USERNS from the environment. After this PR, nothing in the repo sets them and they are undocumented. The last mention was e2e/parity/test.sh, which set them only to verify they were ignored. Options: document them alongside OPENSHELL_PODMAN_SOCKET in container-deployment.mdx, or remove them in favour of TOML config. |
3d1fd5b to
576812b
Compare
|
Addressed in 7a80252. The in-tree gateway no longer reads |
576812b to
7a80252
Compare
Summary
Remove the completed schema-v1/schema-v2 parity campaign while retaining unique current-product regression coverage in the regular gateway configuration tests.
Related Issue
Follow-up cleanup for #2792.
Stack
Changes
OPENSHELL_PODMAN_HOST_GATEWAY_IPandOPENSHELL_PODMAN_USERNSoverrides in favor of the documented Podman TOML fields.test:gateway-config.test:e2e-parityfrom the aggregate test task and update testing, architecture, Windows, and agent documentation.Testing
mise run --jobs 1 pre-commitpasses after rebasing onto currentmainmise run test:gateway-configcargo test --locked --package openshell-gatewayreaches linking but cannot complete locally because the systemz3library is unavailableChecklist