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
Include bootstrap.sh in main image #885
Conversation
I attempted to build the image locally as a sanity check for this trivial change, but it failed to build. I have no idea about the rust ecosystem 🙃
|
Oh, you're updating the Dockerfile in the root! Yeah, we don't regularly use this one, it's more for development. But it's nice to add it there as well, I guess? The image is a bit old. You can upgrade the base tag to alpine3.19 and it should fix the errors. The Dockerfile you want to edit is in .github/workflows |
Forgive me, I don't have much experience with multi stage builds, but do I need to add the |
You need to add it to the final image (the one with the comment "Final image" :) ). That's the one that contains the final binaries. The other image is just for compiling the binaries, so it contains the rust install, for instance. You don't need the bootstrap script there. |
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.
Let's move the bootstrap script to the scripts
folder at the root, instead of in example_configs
: now it's becoming official.
Done in f7689bb |
I think this is ready for merge now. And please could you push an updated image, even if it's not a fully-fledged stable tag? Thanks |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #885 +/- ##
==========================================
- Coverage 86.28% 86.13% -0.16%
==========================================
Files 50 50
Lines 10925 10791 -134
==========================================
- Hits 9427 9295 -132
+ Misses 1498 1496 -2 |
There you go! It'll be part of the daily image by tomorrow. |
Include
bootstrap.sh
in main image to allow Kubernetes to run a bootstrap job from the same imageFixes #884