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.

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

Add docker build output to task result #805

Merged
merged 1 commit into from
Nov 15, 2020

Conversation

raags
Copy link
Contributor

@raags raags commented Aug 20, 2020

SUMMARY

The build output is only added during failures, but its useful to have this available during normal task execution as well.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

docker_image

ADDITIONAL INFORMATION

This change adds a new key build_output to the task result which includes all the docker build steps. This is useful to understand for e.g. what steps were run for the build, as well as steps that used / not used the image cache.

@ansibullbot ansibullbot added affects_2.10 cloud community_review docker feature This issue/PR relates to a feature request module module needs_triage new_contributor Help guide this first time contributor plugins plugin (any type) labels Aug 20, 2020
@felixfontein
Copy link
Collaborator

You need a changelog fragment. Will look at the change itself later...

Copy link
Contributor

@martinm82 martinm82 left a comment

Choose a reason for hiding this comment

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

What about adding some unit/integration tests (eg. here tests/integration/targets/docker_image)?

@ansibullbot ansibullbot added the stale_ci CI is older than 7 days, rerun before merging label Sep 29, 2020
@felixfontein
Copy link
Collaborator

@raags could you please update your PR?

needs_info

@Andersson007 Andersson007 added the needs_info This issue requires further information. Please answer any outstanding questions label Oct 21, 2020
@ansibullbot ansibullbot added integration tests/integration tests tests and removed stale_ci CI is older than 7 days, rerun before merging labels Nov 15, 2020
The build output is only added during failures, but its useful to
have this available during normal task execution as well.
Copy link
Collaborator

@felixfontein felixfontein left a comment

Choose a reason for hiding this comment

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

LGTM

@felixfontein felixfontein removed the needs_info This issue requires further information. Please answer any outstanding questions label Nov 15, 2020
@felixfontein felixfontein merged commit 33b8d1c into ansible-collections:main Nov 15, 2020
patchback bot pushed a commit that referenced this pull request Nov 15, 2020
The build output is only added during failures, but its useful to
have this available during normal task execution as well.

(cherry picked from commit 33b8d1c)
@felixfontein
Copy link
Collaborator

@raags thanks a lot for contributing this!
@martinm82 thanks for reviewing! Proper tests will be added in community.docker.

felixfontein pushed a commit that referenced this pull request Nov 15, 2020
The build output is only added during failures, but its useful to
have this available during normal task execution as well.

(cherry picked from commit 33b8d1c)

Co-authored-by: Raghu Siddarth Udiyar <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cloud docker feature This issue/PR relates to a feature request integration tests/integration module module new_contributor Help guide this first time contributor plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants