Skip to content

ci: fix bake build#4555

Merged
milosgajdos merged 1 commit into
distribution:mainfrom
crazy-max:ci-fix-bake
Jan 16, 2025
Merged

ci: fix bake build#4555
milosgajdos merged 1 commit into
distribution:mainfrom
crazy-max:ci-fix-bake

Conversation

@crazy-max

Copy link
Copy Markdown
Contributor

similar to docker/compose#12470

When building using git context, it does not fetch tags and therefore version would be broken in:

VERSION=$(git describe --match 'v[0-9]*' --dirty='.m' --always --tags) REVISION=$(git rev-parse HEAD)$(if ! git diff --no-ext-diff --quiet --exit-code; then echo .m; fi); \

Let's keep local context as before until we investigate this issue upstream.

Signed-off-by: CrazyMax <1951866+crazy-max@users.noreply.github.com>

@milosgajdos milosgajdos left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

PTAL @thaJeztah

@thaJeztah thaJeztah left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@milosgajdos milosgajdos merged commit 7271d88 into distribution:main Jan 16, 2025
@crazy-max crazy-max deleted the ci-fix-bake branch January 16, 2025 21:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants