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: Unify custom dockerfiles, fix missing semicolon #59615
Conversation
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.
A suggestion and a question. Thank you for the contribution!
Co-authored-by: Christopher Moyer <35463610+chri2547@users.noreply.github.com>
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.
lgtm! Thanks!
@DanCech if I run this with
I should expect this to build the docker image and skip all of the yarn install/yarn build/go gen/go build stuff and just use the assets from the tar file I provide, right? that doesn't seem to be happening 🤔 |
@kminehart this is in packaging/docker/custom/ which doesn't use GRAFANA_TGZ, it builds on top of an official image. To use the standard Dockerfile to build from TGZ you have to use buildkit and set JS_SRC and GO_SRC build args like the build scripts do |
doh. for some reason I thought we also were able to get rid of the ones in |
The backport to
To backport manually, run these commands in your terminal: # Fetch latest updates from GitHub
git fetch
# Create a new branch
git switch --create backport-59615-to-v9.3.x origin/v9.3.x
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x c4bb9d637ea2a4cd16d90b64cf6cea88539edd6b
# Push it to GitHub
git push --set-upstream origin backport-59615-to-v9.3.x
git switch main
# Remove the local backport branch
git branch -D backport-59615-to-v9.3.x Then, create a pull request where the |
* unify custom dockerfiles, fix missing semicolon * tweak verbiage Co-authored-by: Christopher Moyer <35463610+chri2547@users.noreply.github.com> * update example description Co-authored-by: Christopher Moyer <35463610+chri2547@users.noreply.github.com> (cherry picked from commit c4bb9d6)
* unify custom dockerfiles, fix missing semicolon * tweak verbiage Co-authored-by: Christopher Moyer <35463610+chri2547@users.noreply.github.com> * update example description Co-authored-by: Christopher Moyer <35463610+chri2547@users.noreply.github.com>
* unify custom dockerfiles, fix missing semicolon * tweak verbiage Co-authored-by: Christopher Moyer <35463610+chri2547@users.noreply.github.com> * update example description Co-authored-by: Christopher Moyer <35463610+chri2547@users.noreply.github.com> (cherry picked from commit c4bb9d6)
While fixing the typo @grongor pointed out in #58101 I realized that we don't really need 2 custom dockerfiles either, this PR unifies them into a single file that can be used to build alpine or ubuntu-based custom images.