Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Don't report image is pulled unless it's actually happening #10102

Closed
gunnarmorling opened this issue Jun 18, 2020 · 0 comments · Fixed by #10142
Closed

Don't report image is pulled unless it's actually happening #10102

gunnarmorling opened this issue Jun 18, 2020 · 0 comments · Fixed by #10142
Labels
good first issue Good for newcomers kind/enhancement New feature or request
Milestone
1.6.0.CR1

Comments

@gunnarmorling
Copy link
Contributor

Seeing this confusing message in the log:

[INFO] [io.quarkus.deployment.pkg.steps.NativeImageBuildStep] Pulling image quay.io/quarkus/ubi-quarkus-native-image:19.3.1-java11
19.3.1-java11: Pulling from quarkus/ubi-quarkus-native-image
Digest: sha256:9e2dd59c82db091bb90fd0ee60530291a5ecc201ebc718ffcc59ab0f406e47ea
Status: Image is up to date for quay.io/quarkus/ubi-quarkus-native-image:19.3.1-java11

It only should say "Pulling image..." if that's actually happening. Perhaps "Checking image status... or something initially"?
@gunnarmorling gunnarmorling added the kind/enhancement New feature or request label Jun 18, 2020
@geoand geoand added the good first issue Good for newcomers label Jun 19, 2020
@ejba ejba mentioned this issue Jun 21, 2020
Merged
@gsmet gsmet added this to the 1.6.0 - master milestone Jun 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Good for newcomers kind/enhancement New feature or request
Projects
None yet
Milestone
1.6.0.CR1
Development

Successfully merging a pull request may close this issue.

Enhance build image log
3 participants
@gunnarmorling @gsmet @geoand