Skip to content

[codex] Restore container e2e workspace package#4459

Merged
proggeramlug merged 3 commits into
PerryTS:mainfrom
andrewtdiz:codex/container-tests-package-20260604
Jun 4, 2026
Merged

[codex] Restore container e2e workspace package#4459
proggeramlug merged 3 commits into
PerryTS:mainfrom
andrewtdiz:codex/container-tests-package-20260604

Conversation

@andrewtdiz

Copy link
Copy Markdown
Contributor

Summary

  • restore crates/perry-container-e2e as an explicit workspace member
  • record the newly visible package in Cargo.lock
  • leave default-members unchanged so default workspace builds do not pick up env-gated E2E tests

Root Cause

The Layer D container workflow still invokes cargo test -p perry-container-e2e --test e2e_container, and the crates/perry-container-e2e crate/test target still exists. The crate had fallen out of root workspace membership, so Cargo failed package resolution before the E2E skip gates or tests could run.

Validation

  • cargo test -p perry-container-e2e --test e2e_container -- e2e_redis_smoke --test-threads=1
  • cargo test -p perry-container-e2e --test e2e_container -- e2e_forgejo_stack --test-threads=1
  • cargo metadata --no-deps --format-version 1 | jq -r .packages[] | select(.name == "perry-container-e2e") | .manifest_path
  • git diff --check
  • ./scripts/check_file_size.sh

@proggeramlug proggeramlug marked this pull request as ready for review June 4, 2026 23:33
@proggeramlug proggeramlug merged commit 17fd9a6 into PerryTS:main Jun 4, 2026
20 checks passed
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.

2 participants