Skip to content

test(e2e): remove schema parity campaign - #3864

Merged
elezar merged 1 commit into
mainfrom
codex/remove-parity
Oct 1, 2026
Merged

elezar merged 1 commit into
mainfrom
codex/remove-parity

Conversation

@elezar

@elezar elezar commented Sep 29, 2026 •

Copy link
Copy Markdown
Member

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

  • Remove the parity harness, task definitions, historical campaign manifests and results, and verifier-specific Python tests.
  • Simplify the Podman E2E configuration generator and remove parity-only environment, digest, provenance, and capture plumbing.
  • Remove the unused in-tree OPENSHELL_PODMAN_HOST_GATEWAY_IP and OPENSHELL_PODMAN_USERNS overrides in favor of the documented Podman TOML fields.
  • Move durable current-schema assertions for in-tree and external Podman configuration into test:gateway-config.
  • Remove test:e2e-parity from the aggregate test task and update testing, architecture, Windows, and agent documentation.

Testing

  • mise run --jobs 1 pre-commit passes after rebasing onto current main
  • mise run test:gateway-config
  • Focused Python configuration and release-formula tests (12 passed)
  • Focused Podman driver configuration tests (73 passed)
  • cargo test --locked --package openshell-gateway reaches linking but cannot complete locally because the system z3 library is unavailable
  • Live E2E tests were not run; lifecycle coverage remains in the regular driver E2E suites

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)

Signed-off-by: Evan Lezar <elezar@nvidia.com>
@github-actions

Copy link
Copy Markdown

@elezar
elezar force-pushed the codex/remove-parity branch from 591e1af to 3d1fd5b Compare September 30, 2026 06:32
@elezar
elezar added this pull request to stack #3939 September 30, 2026 06:32
krishicks
krishicks previously approved these changes Sep 30, 2026
@krishicks

krishicks commented Sep 30, 2026 •

Copy link
Copy Markdown
Collaborator

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.

@elezar

elezar commented Oct 1, 2026 •

Copy link
Copy Markdown
Member Author

Addressed in 7a80252. The in-tree gateway no longer reads OPENSHELL_PODMAN_HOST_GATEWAY_IP or OPENSHELL_PODMAN_USERNS; the documented Podman TOML fields remain the canonical configuration. OPENSHELL_PODMAN_SOCKET is retained because the E2E wrapper still uses that override.

@elezar
elezar force-pushed the codex/remove-parity branch from 576812b to 7a80252 Compare October 1, 2026 12:31
@elezar
elezar requested a review from krishicks October 1, 2026 12:38
@elezar
elezar added this pull request to the merge queue Oct 1, 2026
Merged via the queue into main with commit 0e8d9e5 Oct 1, 2026
73 checks passed
@elezar
elezar deleted the codex/remove-parity branch October 1, 2026 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants