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

ci: reuse az nightly image in az e2e test #1715

Conversation

mkulke
Copy link
Collaborator

@mkulke mkulke commented Feb 26, 2024

draft until #1713 is merged

For nightly e2e-tests we can re-use the nightly-built podvm image
without rebuilding it from scratch.

This change will convert the e2e test into a reusable workflow and
invoke it nightly from a meta workflow "azure nightly build". this
workflow will call both the podvm-image-build and the e2e test
subsequently.

The caching keys will not cover any modifications that we do to the
build process, so we need to to rebuild the binaries on podvm builds.

Signed-off-by: Magnus Kulke <magnuskulke@microsoft.com>
For nightly e2e-tests we can re-use the nightly-built podvm image
without rebuilding it from scratch.

This change will convert the e2e test into a reusable workflow and
invoke it nightly from a meta workflow "azure nightly build". this
workflow will call both the podvm-image-build and the e2e test
subsequently.

Signed-off-by: Magnus Kulke <magnuskulke@microsoft.com>
@mkulke mkulke changed the title Mkulke/reuse az nightly image in e2e ci: reuse az nightly image in az e2e test Feb 26, 2024
@mkulke
Copy link
Collaborator Author

mkulke commented Feb 26, 2024

good idea, doesn't work. the output from one reusable wf doesn't seem to be usable as input for another, it will be in call cases. I don't think it's worth debugging.

@mkulke mkulke closed this Feb 26, 2024
@mkulke mkulke deleted the mkulke/reuse-az-nightly-image-in-e2e branch September 17, 2024 06:54
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