Skip to content

Fix image tag name for devel#16

Open
cigamit wants to merge 1 commit into
develfrom
devel_image2
Open

Fix image tag name for devel#16
cigamit wants to merge 1 commit into
develfrom
devel_image2

Conversation

@cigamit
Copy link
Copy Markdown
Collaborator

@cigamit cigamit commented Jun 2, 2026

github doesn't like + in the tag name, even though it does over on Ascender.

@cigamit cigamit requested a review from TheWitness June 2, 2026 20:17
@cigamit cigamit self-assigned this Jun 2, 2026
Copilot AI review requested due to automatic review settings June 2, 2026 20:17
@cigamit cigamit added the bug Something isn't working label Jun 2, 2026
Copy link
Copy Markdown
Contributor

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

Adjusts the development image publishing workflow to use a GHCR-compatible Docker tag format (avoiding + in the tag), aligning the generated image reference with registries that enforce Docker tag character constraints.

Changes:

  • Replace +git<sha> with _git<sha> in the generated version/tag string used for devel image builds.
  • Update the computed IMG reference used by make docker-buildx when pushing devel images.

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

--build-arg OPERATOR_VERSION=${{ steps.date.outputs.year }}.${{ steps.date.outputs.month }}.${{ steps.date.outputs.day }}+git${{ steps.short-sha.outputs.sha }}" \
IMG=ghcr.io/${{ github.repository }}_devel:${{ steps.date.outputs.year }}.${{ steps.date.outputs.month }}.${{ steps.date.outputs.day }}+git${{ steps.short-sha.outputs.sha }} \
--build-arg OPERATOR_VERSION=${{ steps.date.outputs.year }}.${{ steps.date.outputs.month }}.${{ steps.date.outputs.day }}_git${{ steps.short-sha.outputs.sha }}" \
IMG=ghcr.io/${{ github.repository }}-devel:${{ steps.date.outputs.year }}.${{ steps.date.outputs.month }}.${{ steps.date.outputs.day }}_git${{ steps.short-sha.outputs.sha }} \
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Development

Successfully merging this pull request may close these issues.

2 participants