updated readme for clarity regarding alpine images #971
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.
What I did
Update the readme to reflect the fact that the default documented docker run command will not work on an alpine based image of hadolint, as the default path to the config is not working here.
How I did it
Simple update of the readme. I though about fixing this issue in the container image build of hadolint, but that still leaves older versions with the same issue. So you're now aware, the users have a correct readme. Maybe a simple symlink will be enough for future builds.
How to verify it
Run the following commands and you'll see the first one will not have the configuration file loaded as hadolint is looking for environment variables which are not present in the alpine version:
docker run --rm -i -v /path/to/hadolint.yaml:/.config/hadolint.yaml ghcr.io/hadolint/hadolint:2.12.0-alpine hadolint -V
docker run --rm -i -v /path/to/hadolint.yaml:/root/.config/hadolint.yaml ghcr.io/hadolint/hadolint:2.12.0-alpine hadolint -V