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

caa: Add option to skip Peer pod VM delete #2227

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

stevenhorsman
Copy link
Member

During development, several of us have hit times where being able to access a failing peer pod VM has been crucial to debugging issues. The general approach to this is to add a sleep before, or comment out DeleteInstance, but this involved rebuilding the CAA image, so instead we can provide an option to set PEERPODS_DEVELOPER_MODE=true in peer-pods-cm which will skip this peer pod vm delete.

During development, several of us have hit times where being able to
access a failing peer pod VM has been crucial to debugging issues.
The general approach to this is to add a sleep before, or comment out
`DeleteInstance`, but this involved rebuilding the CAA image, so instead
we can provide an option to set `PEERPODS_DEVELOPER_MODE=true`
in peer-pods-cm which will skip this peer pod vm delete.

Signed-off-by: stevenhorsman <[email protected]>
@stevenhorsman stevenhorsman added the test_e2e_libvirt Run Libvirt e2e tests label Jan 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test_e2e_libvirt Run Libvirt e2e tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant