-
Notifications
You must be signed in to change notification settings - Fork 3.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Build loki-build-image for multiple architectures. #11130
Merged
Merged
Changes from all commits
Commits
Show all changes
53 commits
Select commit
Hold shift + click to select a range
8afc9f3
Build loki-build-image for multiple architectures.
jeschkies ac6632f
Merge remote-tracking branch 'grafana/main' into karsten/multi-arch-b…
jeschkies 435b489
Mount Docker socket.
jeschkies b0507b7
Provide docker sock volume.
jeschkies 81b4c41
Use buildx
jeschkies ffd82a2
Ensure buildx is setup.
jeschkies 9f8340c
Checking buildx builder script.
jeschkies 381a128
Only build image when there are changes.
jeschkies 963089b
Install arm packages where required.
jeschkies 07decbf
Install the correct Helm and Helm Docs arch.
jeschkies ab5ccf3
Use make target to push build image.
jeschkies 6923a52
Merge remote-tracking branch 'grafana/main' into karsten/multi-arch-b…
jeschkies c0dd62a
Login Docker CLI.
jeschkies a011576
update drone.yml
jeschkies a881707
Merge remote-tracking branch 'grafana/main' into karsten/multi-arch-b…
jeschkies ab49503
Use new build image.
jeschkies 7669335
Merge remote-tracking branch 'grafana/main' into karsten/multi-arch-b…
jeschkies a456e7a
Test image push.
jeschkies 006e831
Migrate to two step.
jeschkies 1f3843c
Correct Docker builds.
jeschkies 1746ed0
Pass platform parameter
jeschkies 2492df8
Use buildkit to get target architecture.
jeschkies 3dbf3d2
Format jsonnet
jeschkies 50200ae
Add todo
jeschkies 31e4129
Add manifest for build image.
jeschkies 2ec99de
Merge remote-tracking branch 'grafana/main' into karsten/multi-arch-b…
jeschkies 1de5317
Try with env var.
jeschkies c9fff01
Merge remote-tracking branch 'grafana/main' into karsten/multi-arch-b…
jeschkies ae9d5c0
Only run on amd64
jeschkies 1923d10
Reduce further.
jeschkies 879159f
Use Docker config instead of user and password.
jeschkies 6dc1dea
Remove buildkit
jeschkies 509f2cd
Use 'args'
jeschkies cafdd65
try other format
jeschkies 82cbe25
Use secrets
jeschkies fb81a16
Back to square one
jeschkies 3e5b926
Use buildkit
jeschkies 546e099
Build for arm as well
jeschkies e9a31cb
publish manifest
jeschkies 58e2711
set tag
jeschkies 37e8053
Format jsonnet
jeschkies b0324b6
Try different tagging
jeschkies d52b186
Set version manually for now
jeschkies 6d66011
make manifest version dynamic
jeschkies b2b1e33
Remove unused script
jeschkies a0e6a92
add test stage
jeschkies b5796d5
Merge remote-tracking branch 'grafana/main' into karsten/multi-arch-b…
jeschkies ee26621
fix jsonnet format
jeschkies e6c4ee4
Make complete pipeline conditional
jeschkies 6196073
Revert previous change
jeschkies b552ad1
Generate drone yaml
jeschkies 4ff6892
Fun back and forth with condition
jeschkies 4a96bf4
Update patch Go version documentation.
jeschkies File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,17 @@ | ||
image: grafana/{{config.target}} | ||
tags: | ||
{{#if build.tags}} | ||
{{#each build.tags}} | ||
- {{this}} | ||
{{/each}} | ||
{{/if}} | ||
manifests: | ||
- image: grafana/{{config.target}}-amd64 | ||
platform: | ||
architecture: amd64 | ||
os: linux | ||
- image: grafana/{{config.target}}-arm64 | ||
platform: | ||
architecture: arm64 | ||
os: linux | ||
variant: v8 |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like we are just building and not pushing now?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Look at line 106.
PUSH_OCI
is still defined. This is just for defining variables. There's no make target here.