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

Persist images for all targets #1097

Merged
merged 1 commit into from
Mar 30, 2020

Conversation

zubron
Copy link
Contributor

@zubron zubron commented Mar 30, 2020

What this PR does / why we need it:
[When building the Sonobuoy images, we build images for multiple targets.
We push them in a separate step from where they were built so they need
to be persisted to the workspace. We were previously only persisting the
sonobuoy/sonobuoy tagged images, not the sonobuoy/sonobuoy-ARCH images.
This was causing CI failures as the image pushing step was trying to
push images for different architectures that did not exist locally.

Signed-off-by: Bridget McErlean bmcerlean@vmware.com

When building the Sonobuoy images, we build images for multiple targets.
We push them in a separate step from where they were built so they need
to be persisted to the workspace. We were previously only persisting the
sonobuoy/sonobuoy tagged images, not the sonobuoy/sonobuoy-ARCH images.
This was causing CI failures as the image pushing step was trying to
push images for different architectures that did not exist locally.

Signed-off-by: Bridget McErlean <bmcerlean@vmware.com>
@zubron zubron merged commit e915293 into vmware-tanzu:master Mar 30, 2020
@zubron zubron deleted the persist-all-image-targets branch March 30, 2020 20:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant