-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
don't apply default pull policy from command line if one is define in service configuration #9720
Conversation
fa743a3
to
6401f0d
Compare
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.
Don't think this is the right fix (see my comment)
cd63858
to
4bd6c61
Compare
478b6ce
to
7ac17c3
Compare
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 (assuming green)
We should probably have a test for this (as a follow-up?)
7ac17c3
to
2fe6e4b
Compare
Odd failures in CI during linting; I see we're not pinning golang-ci-lint to a version, which may mean it's not compatible with the go version we're using 🤔 |
unrelated; looks like CI is using a pretty old version of docker 20.10; https://github.com/docker/compose/blob/v2/.github/workflows/ci.yml#L104 |
e0929e5
to
b79b293
Compare
… service configuration Signed-off-by: Guillaume Lours <[email protected]>
b79b293
to
c47079e
Compare
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!
What I did
Don't override service pull policy if the value from the command line is the default one
Related issue
fix #9717
(not mandatory) A picture of a cute animal, if possible in relation with what you did
