Skip to content

test: enforce strict replay sandboxing in e2e fixtures#141

Merged
sohankshirsagar merged 2 commits intomainfrom
sohan/require-strict-sandboxing
Mar 17, 2026
Merged

test: enforce strict replay sandboxing in e2e fixtures#141
sohankshirsagar merged 2 commits intomainfrom
sohan/require-strict-sandboxing

Conversation

@sohankshirsagar
Copy link
Contributor

Summary

Require strict replay sandboxing across the Docker-based e2e fixtures and update the test containers so the sandbox can actually start during replay. This also fixes a few compose files where the initial bulk edit misplaced build.args.

Changes

  • Set replay.sandbox.mode: strict in all e2e .tusk/config.yaml files so replay fails instead of silently falling back to unsandboxed execution.
  • Install socat and bubblewrap in all Docker-based e2e Dockerfiles, since those binaries are required for Linux replay sandboxing.
  • Add cap_add: [SYS_ADMIN] plus security_opt overrides (seccomp=unconfined, apparmor=unconfined) to each e2e docker-compose.yml app service.
    • Docker’s default seccomp/AppArmor restrictions were blocking bubblewrap from creating the namespaces it needs, which caused strict sandbox startup to fail with Operation not permitted.

@tusk-dev
Copy link

tusk-dev bot commented Mar 16, 2026

PR identified as a refactor

View output

Tip

New to Tusk Unit Tests? Learn more here.

View check history

Commit Status Output Created (UTC)
ec86df6 PR identified as a refactor Output Mar 16, 2026 11:18PM

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 90 files

Note: This PR contains a large number of files. cubic only reviews up to 75 files per PR, so some files may not have been reviewed.

Copy link
Contributor

@sohil-kshirsagar sohil-kshirsagar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice! worth updating the e2e testing docs explaining this?

@sohankshirsagar sohankshirsagar merged commit e86fae0 into main Mar 17, 2026
19 checks passed
@sohankshirsagar sohankshirsagar deleted the sohan/require-strict-sandboxing branch March 17, 2026 00:23
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