-
Notifications
You must be signed in to change notification settings - Fork 46
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
added a new chart static-site that build and host static sites. #64
Conversation
laszlocph
commented
Mar 27, 2023
•
edited
Loading
edited
charts/static-site/values.yaml
Outdated
gitCloneUrl: https://github.com/YoucefGuichi/hugo-site.git | ||
buildImage: node | ||
gitCloneUrl: https://github.com/gimlet-io/hugo-site.git | ||
buildImage: node # the image must have git installed |
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.
👏
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.
Also, can you find something smaller with git. This image is 999MB 😨
docker images | grep node
node latest 0e0ab07dbedd 4 days ago 999MB
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.
using alpine/git
now,
➜ ~ git:(main) ✗ docker images | grep alpine
alpine/git latest 9793ee61fc75 4 months ago 43.4MB
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.
Not bad. I actually fear that alpine is harming usability. Noone knows how to use alpine images.. at least even after 7 years of using them occasionally I still have to look up stuff and curse everytime. Can you find an alternative.. some debian or ubuntu. 200-300MB is a good upper limit.
…et-io#64) Co-authored-by: Youcef Guichi <[email protected]>