-
Notifications
You must be signed in to change notification settings - Fork 607
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
Improve Dockerfile #1071
Improve Dockerfile #1071
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1071 +/- ##
=======================================
Coverage 20.64% 20.64%
=======================================
Files 193 193
Lines 25270 25270
=======================================
Hits 5218 5218
Misses 19096 19096
Partials 956 956 Continue to review full report at Codecov.
|
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.
Oh awesome! Should we add the docker alias somewhere to the docs /README as well, to make people more aware of it? cc @czarcas7ic
Will sync with Nicco on this to make it happen, thanks |
* Use nonroot docker image * Update CHANGELOG.md (cherry picked from commit 8bbfd24)
* Use nonroot docker image * Update CHANGELOG.md (cherry picked from commit 8bbfd24) Co-authored-by: Niccolo Raspa <[email protected]>
Description
This repository improves our Dockerfile by:
base-debian11:nonroot
overbase-debian11
to avoid running container as root/osmosis
folder as$HOME
directory for the containerlibwasmvm_muslc.a
as in https://github.com/CosmWasm/wasmd/blob/master/DockerfileUsage:
Example:
docker run --rm -it -v $HOME/.osmosisd/:/osmosis/ osmolabs/osmosis:v7.0.4 init my-moniker
You can create an alias to simplify the command:
and then simply:
For contributor use:
docs/
) or specification (x/<module>/spec/
)Unreleased
section inCHANGELOG.md
Files changed
in the Github PR explorer