Skip to content

ci: fix CircleCI apt + Slack notification hard-fail - #507

Merged
ch4r10t33r merged 1 commit into
stagingfrom
fix/ci-apt-and-slack-checks
Aug 4, 2026
Merged

ci: fix CircleCI apt + Slack notification hard-fail#507
ch4r10t33r merged 1 commit into
stagingfrom
fix/ci-apt-and-slack-checks

Conversation

@ch4r10t33r

@ch4r10t33r ch4r10t33r commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • CircleCI e2e was dying at Install jq because apt-get update fails on the Cypress image's Chrome apt source (NO_PUBKEY FD533C07C264648F). Remove those sources before updating.
  • Slack/Cloudflare Pages notification job was failing the build check with Cloudflare API auth error 10000. Mark that step continue-on-error: true so a bad/expired CF_API_TOKEN does not block releases (token still needs rotation in repo secrets).

Test plan

  • CircleCI e2e on staging gets past Install jq
  • Notifications workflow no longer fails the commit status when CF auth errors

Summary by CodeRabbit

  • Chores
    • Improved CI/CD pipeline stability and workflow reliability to enhance deployment processes.

Remove the Cypress image Chrome apt source before apt-get update so
jq install is not blocked by an expired Google GPG key. Allow the
Cloudflare Pages Slack notification step to fail without failing the
workflow when CF_API_TOKEN auth is invalid.
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying x with  Cloudflare Pages  Cloudflare Pages

Latest commit: 2370996
Status:⚡️  Build in progress...

View logs

@ch4r10t33r
ch4r10t33r merged commit 2cf0ae2 into staging Aug 4, 2026
4 of 6 checks passed
@coderabbitai

coderabbitai Bot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 787cfd5a-c6f6-4fe9-9347-6d66660b7a33

📥 Commits

Reviewing files that changed from the base of the PR and between e52be72 and 2370996.

📒 Files selected for processing (2)
  • .circleci/config.yml
  • .github/workflows/notifications.yml

📝 Walkthrough

Walkthrough

The CircleCI end-to-end test job cleans Chrome APT source files before package installation. The deployment notification workflow continues when the Slack action fails.

Changes

CircleCI package setup

Layer / File(s) Summary
Chrome APT source cleanup
.circleci/config.yml
The end-to-end test job removes Google Chrome APT source list files before updating package metadata and installing curl and jq. The cleanup tolerates missing files.

Deployment notifications

Layer / File(s) Summary
Slack failure handling
.github/workflows/notifications.yml
The Slack notification step ignores action failures and allows subsequent workflow execution to continue.

Estimated code review effort: 2 (Simple) | ~5 minutes

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/ci-apt-and-slack-checks

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@ch4r10t33r ch4r10t33r mentioned this pull request Aug 4, 2026
5 tasks
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