Skip to content

Release: provider suite gates deploys; PR gate checks dev's head - #198

Merged
saqibmanan merged 4 commits into
mainfrom
dev
Sep 17, 2026
Merged

saqibmanan merged 4 commits into
mainfrom
dev

Conversation

@saqibmanan

Copy link
Copy Markdown
Contributor

Workflow-only release (2 files vs main): deploy-backend.yml, pr-gate.yml. From #197.

  • Provider suite is a gate. A provider failure rolls back a dev deploy, and Full Suite in this gate now includes provider-smoke.
  • Deployed-SHA check uses dev's head (PR gate fails every release PR not opened from dev #195), so a release built as a merge commit no longer fails it by construction.

This PR is the first run of the new gate. Branch protection on main requires Full Suite (dev) / api-smoke, consumer-smoke, provider-smoke and Django Tests / django-tests.

Verified: dev deploy 35201582293 green with provider gating (Functional 3/3, provider 11 passed, no retries). Merge into main is clean and keeps main's CodeQL permissions blocks.

Merge with Create a merge commit.

Provider functional ran report-only while it was flaky against dev. After the
sidebar, autosave and upload fixes in CivicDataSpace-test, three full provider
runs passed with no retries, so a provider failure now rolls back the deploy
like any other functional failure.
It ran as a separate report-only job that branch protection ignored. Folding it
into Full Suite makes a provider failure block the release PR.
The deployed-SHA check compared dev's live git_sha against the PR head, so any
release PR not opened straight from dev failed it by construction (#191). Resolve
dev's head in a small job and check against that instead. Fixes #195.
ci: provider suite becomes a gate; PR gate checks dev's head
@saqibmanan
saqibmanan merged commit abce96f into main Sep 17, 2026
30 of 31 checks passed

This branch was successfully deployed

1 active deployment
development — a5d6d679 Deployed Sep 17, 2026 by saqibmanan via Finalize Deploy #52
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