Skip to content

test(quickstart): fail when the project cannot be removed without root #289

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Bccorb merged 1 commit into from
Oct 20, 2026
Merged

test(quickstart): fail when the project cannot be removed without root #289

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 12 additions & 2 deletions scripts/quickstart-clean-run.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,8 +39,8 @@ cleanup() {
fi
(cd "$APP_DIR" && docker compose down -v --rmi local --remove-orphans >/dev/null 2>&1) || true
fi
# On Linux the api container runs as root over the bind-mounted api/, so files it
# wrote (api/logs) are root's. Removing them from a container needs no sudo.
# Only for a run that failed before the removal check below. Files a container
# wrote as root over a bind mount can still be removed from a container.
if ! rm -rf "$WORKDIR" 2>/dev/null; then
docker run --rm -v "$WORKDIR:/w" alpine:3 rm -rf /w/my-app >/dev/null 2>&1 || true
rm -rf "$WORKDIR" || true
Expand Down Expand Up @@ -124,6 +124,16 @@ end_phase "Owner signed in through /auth"

total=$(($(date +%s) - run_start))

# On a Linux host the dev containers run as root over bind mounts, so anything they
# write into the project is root's, and a user cannot delete the project without sudo.
echo "==> Removing the project"
docker compose down -v --rmi local --remove-orphans >/dev/null 2>&1
cd "$WORKDIR"
if ! rm -rf "$APP_DIR"; then
find "$APP_DIR" -user 0 >&2 || true
fail "the project could not be removed without root. The files above were written by a container."
fi

{
echo "### Quickstart clean run: $CLI ($cli_version)"
echo
Expand Down
Loading