Skip to content

Pass image tag as build argument#2445

Merged
nluaces merged 2 commits into
skupperproject:mainfrom
nluaces:fix-version-generation-docker-images
May 19, 2026
Merged

Pass image tag as build argument#2445
nluaces merged 2 commits into
skupperproject:mainfrom
nluaces:fix-version-generation-docker-images

Conversation

@nluaces
Copy link
Copy Markdown
Member

@nluaces nluaces commented May 14, 2026

Fixes #2442

@nluaces nluaces self-assigned this May 14, 2026
@nluaces nluaces requested a review from pwright May 14, 2026 18:44
@pwright
Copy link
Copy Markdown
Member

pwright commented May 15, 2026

@nluaces working from your branch:

$ make build-cli
GOOS=linux GOARCH=amd64 go build -ldflags="-s -w  -X github.com/skupperproject/skupper/internal/version.Version="  -o skupper ./cmd/skupper
$ ./skupper version

ie no version

Is the intention that devs must do something like:

export IMAGE_TAG=fix-big-bug  
export REGISTRY=quay.io/yourname   # if pushing images  
  
make build-cli  
make docker-build  
make podman-push

Maybe this shows we need a CONTRIBUTING.md

@nluaces
Copy link
Copy Markdown
Member Author

nluaces commented May 15, 2026

ie no version

@pwright that was wrong, thanks for noticing.
I was checking how the version was set while generating the docker image, but I've missed this.

Copy link
Copy Markdown
Member

@fgiorgetti fgiorgetti left a comment

Choose a reason for hiding this comment

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

LGTM

@nluaces nluaces merged commit 7d9d742 into skupperproject:main May 19, 2026
2 checks passed
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.

Wrong version is shown in controllers

4 participants