Skip to content

test: repair VPS release QA after safe runner rsync override - #27

Merged
appdevelopsk merged 4 commits into
mainfrom
fix/site-ci-rsync-guard-20261030
Oct 30, 2026
Merged

appdevelopsk merged 4 commits into
mainfrom
fix/site-ci-rsync-guard-20261030

Conversation

@appdevelopsk

Copy link
Copy Markdown
Owner

Reason

Public App Discovery and SEO QA failed after the deploy scripts switched to configurable APP_SITE_RSYNC_FLAGS for a non-root self-hosted runner. Two tests still asserted the old literal rsync -avR --itemize-changes command.

Change

Update both tests to validate the actual default safe flags, explicit --dry-run/--apply behavior, and invocation of rsync $rsync_flags without destructive --delete. No deployment script or production content is changed.

Evidence/gates

Previous failure: https://github.com/appdevelopsk/appdevelopsk.github.io/actions/runs/40718637111
Production release & live page verification occurred separately on 2026-10-29: https://github.com/appdevelopsk/appdevelopsk.github.io/actions/runs/40828987006 and https://github.com/appdevelopsk/appdevelopsk.github.io/actions/runs/40855155258 (0 stale / 0 unreachable).

Merge only after app-discovery-qa.yml succeeds.

@appdevelopsk
appdevelopsk merged commit 3fe47f7 into main Oct 30, 2026
1 check 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.

1 participant