Skip to content

ci: disable nightly build-images#456

Merged
piersy merged 1 commit into
celo-rebase-18from
palango/disable-nightly-build-images
Jun 17, 2026
Merged

ci: disable nightly build-images#456
piersy merged 1 commit into
celo-rebase-18from
palango/disable-nightly-build-images

Conversation

@palango

@palango palango commented Jun 17, 2026

Copy link
Copy Markdown

What

Removes the nightly schedule cron from the upstream build images workflow on the Celo fork. Kept workflow_dispatch for manual/upstream parity.

Why

The nightly run always fails:

  • On a schedule trigger the checkout forces ref: develop (build-images.yaml:52/:95), but on the fork develop is a stale upstream branch that predates the factory image pipeline and has no .github/images.json → the plan step fails with Config file not found: .github/images.json.
  • Even past that, the build job publishes to Optimism Labs' registry (oplabs-tools-artifacts, via vars.GCP_PROJECT_ID_OPLABS_TOOLS_ARTIFACTS) — infra the fork has no access to — so it can never succeed here.

Image builds for the fork are handled by docker-build-scan.yaml, which publishes to Celo's own registry. push/pull_request only target develop (never pushed to on the fork), so removing schedule stops the only trigger that actually fires.

🤖 Generated with Claude Code

The scheduled run checks out stale upstream 'develop' (no .github/images.json)
and targets Optimism Labs' registry, which the fork can't access. Image builds
for the fork are handled by docker-build-scan.yaml.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@palango palango changed the title ci: disable nightly build-images on the Celo fork ci: disable nightly build-images Jun 17, 2026
@piersy piersy merged commit 9b2017e into celo-rebase-18 Jun 17, 2026
41 of 45 checks passed
@piersy piersy deleted the palango/disable-nightly-build-images branch June 17, 2026 10:59
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