Skip to content

Pass the canary's paid secrets only on a weekly run; the daily run stays free - #13

Merged
jehrr merged 1 commit into
mainfrom
ci/weekly-paid-canary-2026-10-16
Oct 16, 2026
Merged

jehrr merged 1 commit into
mainfrom
ci/weekly-paid-canary-2026-10-16

Conversation

@jehrr

@jehrr jehrr commented Oct 16, 2026

Copy link
Copy Markdown
Collaborator

Problem: the canary runs daily and its job(s) (live-run) work from a bare runner with no secret, but each passes the optional paid secret(s) SPINNY_CDP_ENDPOINT, TWOCAPTCHA_KEY into the run when set. With a secret configured, every daily run would bill proxy traffic or a Scraping Browser session.

Change: a second schedule, 23 6 (Mondays, at the daily run's hour and minute), is added, and each of those secrets is now mapped as ${{ (github.event_name != 'schedule' || github.event.schedule == '23 6') && secrets.X || '' }} (and the presence flags likewise). On the daily run the secrets read as empty, so the job takes its free no-secret route exactly as it does today without them; on the Monday run and on any manual dispatch they reach the run as before. No job is skipped and no job is made weekly, because the jobs themselves are free paths. No secret named here is currently set in this repository, so today's daily runs are unchanged. CONTRIBUTING now says the secret is used on the Monday run only. smoke_test.py pinned the old mapping string SPINNY_CDP_ENDPOINT: ${{ secrets.SPINNY_CDP_ENDPOINT }} ("the secret still reaches the run"); it now pins the gated mapping. Control: restoring main's canary.yml turns that check red; with this change the suite is green in a core-only venv.

Not changed: what each canary job runs and asserts, its thresholds, and its behaviour when a secret is missing (a paid job without its secret still fails with an ::error:: naming it, now on its weekly run). workflow_dispatch still runs every job.

Evidence: the edited workflow parses as YAML and its on.schedule / job if: values were printed from the parsed file (listed above). The offline suite runs in this PR's tests workflow on the head commit. No paid call was made to check this change.

🤖 Generated with Claude Code

https://claude.ai/code/session_01CKsd7q25N19WzUZjrAdzx8

…ays free

Paid canary paths (proxy, Scraping Browser, captcha) run weekly on Mondays;
free paths keep their daily schedule. A manual dispatch runs everything.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01CKsd7q25N19WzUZjrAdzx8
@jehrr
jehrr merged commit 2c1e52d into main Oct 16, 2026
7 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.

1 participant