Skip to content

CI: sanitize branch ref before using it as a Docker image tag#333

Merged
boddumanohar merged 2 commits into
mainfrom
ci/sanitize-operator-docker-tag
Jul 15, 2026
Merged

CI: sanitize branch ref before using it as a Docker image tag#333
boddumanohar merged 2 commits into
mainfrom
ci/sanitize-operator-docker-tag

Conversation

@boddumanohar

@boddumanohar boddumanohar commented Jul 15, 2026 •
edited
Loading

Copy link
Copy Markdown
Member
  • Sanitize Docker image tag name
  • Parallelize the build and push for operator and rebalancer images

@boddumanohar @claude
Sanitize branch ref before using it as a Docker image tag
3064639 
operator_build.yaml tags images with the raw github.ref_name, but Docker
tags can't contain "/". Any branch with a slash (e.g. feature/foo) fails
build-and-push with "invalid reference format". Compute a sanitized tag
(replace "/" with "-") once and reuse it for both image builds.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
boddumanohar added a commit that referenced this pull request Jul 15, 2026
@boddumanohar @claude
Revert operator_build.yaml Docker tag fix, split into #333
ed3d533 
Splitting the CI workflow fix out into its own PR
(#333) since it's independent of this
feature. Note: build-and-push will fail again on this branch until #333
merges and this branch is rebased on top of it, or #333 merges before
this PR does.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@boddumanohar @claude
Parallelize operator and rebalancer image builds via a matrix
7b5a064 
The two docker/build-push-action steps ran sequentially in one job.
Switch to a matrix strategy (fail-fast: false) so the operator and
simplyblock-rebalancer multi-arch images build and push as independent,
concurrent jobs instead. Each matrix entry gets its own GHA cache scope
to avoid cross-image cache key collisions.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@boddumanohar
boddumanohar merged commit 93f67fc into main Jul 15, 2026
3 checks passed
@boddumanohar
boddumanohar deleted the ci/sanitize-operator-docker-tag branch July 15, 2026 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@noctarius noctarius noctarius approved these changes
@geoffrey1330 geoffrey1330 geoffrey1330 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@boddumanohar @noctarius @geoffrey1330