Skip to content

fix(ci): remove non-existent web image from rollback matrix#556

Open
einarwar wants to merge 1 commit into
mainfrom
fix/rollback-remove-stale-web-image
Open

fix(ci): remove non-existent web image from rollback matrix#556
einarwar wants to merge 1 commit into
mainfrom
fix/rollback-remove-stale-web-image

Conversation

@einarwar
Copy link
Copy Markdown
Contributor

The publish-image workflow no longer builds a standalone web image — the frontend is bundled into the nginx image. Attempting a rollback would therefore fail with a manifest unknown error on the web matrix entry.

This PR removes ghcr.io/equinor/template-fastapi-react/web from the rollback matrix so only the images that are actually published (api, nginx) are tagged.

@einarwar einarwar requested a review from a team as a code owner May 19, 2026 13:08
@einarwar einarwar requested a review from Copilot May 19, 2026 13:16
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Removes a stale web image entry from the rollback workflow matrix. Since publish-image.yaml only publishes api and nginx images (the frontend is bundled into the nginx image), attempting a rollback for web would fail with manifest unknown.

Changes:

  • Drop ghcr.io/equinor/template-fastapi-react/web from the rollback matrix in .github/workflows/rollback.yaml.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

The publish-image workflow no longer builds a 'web' image (the frontend
is bundled into the 'nginx' image), so attempting to rollback the
'web' image would fail with a manifest-not-found error.
@einarwar einarwar force-pushed the fix/rollback-remove-stale-web-image branch from 3040d07 to 750d11c Compare May 21, 2026 11:43
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