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

azure-podvm-image-build: Relogin before cleanup #1772

Merged

Conversation

surajssd
Copy link
Member

@surajssd surajssd commented Apr 6, 2024

The packer based build takes more than 10 minutes to cleanup. And after the build if we use az CLI to clean up the intermediate image, it fails because the assertion is valid for 5 minutes only.

So relogin before we move on to cleanup the intermediate image.

Fixes: #1771

The packer based build takes more than 10 minutes
to cleanup. And after the build if we use az CLI
to clean up the intermediate image, it fails
because the assertion is valid for 5 minutes only.

So relogin before we move on to cleanup the
intermediate image.

Fixes: confidential-containers#1771

Signed-off-by: Suraj Deshmukh <suraj.deshmukh@microsoft.com>
@kartikjoshi21
Copy link
Contributor

LGTM, can you please check lint failure ? Thanks

Copy link
Contributor

@mkulke mkulke left a comment

Choose a reason for hiding this comment

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

LGTM, the vuln lint seems unrelated

@mkulke mkulke merged commit a510064 into confidential-containers:main Apr 8, 2024
17 of 18 checks passed
@surajssd surajssd deleted the relogin-before-cleanup branch April 8, 2024 14:24
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.

CAA podvm build's cleanup step fails with authn errors
3 participants