Skip to content

up: do not warn on successful optional dependency complete#10870

Merged
milas merged 1 commit into
docker:v2from
milas:required-cond-exit-warning
Aug 3, 2023
Merged

up: do not warn on successful optional dependency complete#10870
milas merged 1 commit into
docker:v2from
milas:required-cond-exit-warning

Conversation

@milas

@milas milas commented Aug 3, 2023

Copy link
Copy Markdown
Contributor

What I did
If an optional dependency exits successfully (exit code of 0), with a service condition of service_completed_successfully, don't log a warning.

Related issue

(not mandatory) A picture of a cute animal, if possible in relation to what you did
frog floating in water

@milas milas added the kind/bug label Aug 3, 2023
@milas
milas requested a review from a team August 3, 2023 12:49
@milas milas self-assigned this Aug 3, 2023
@milas
milas requested review from StefanScherer, glours, laurazard, ndeloof, nicksieger and ulyssessouza and removed request for a team August 3, 2023 12:49

@glours glours left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice 👍

If an optional dependency exits successfully (exit code of 0),
with a service condition of `service_completed_successfully`,
don't log a warning.

Signed-off-by: Milas Bowman <[email protected]>
@milas
milas force-pushed the required-cond-exit-warning branch from aa87f3f to 4e69e87 Compare August 3, 2023 18:50
@milas
milas enabled auto-merge (squash) August 3, 2023 20:48
@milas
milas merged commit c350f80 into docker:v2 Aug 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants