Skip to content

v11 - feat!: add ability to retain snapshot after cleanup #480

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

Open
wants to merge 1 commit into
base: v11
Choose a base branch
from

Conversation

a-dubs
Copy link
Contributor

@a-dubs a-dubs commented Mar 12, 2025

Description

feat!: add ability to retain snapshot after cleanup

This allows for marking snapshots to be kept at time of creation
instead of being cleaned up automatically. This allows for still using
context managers for easy cleanup while retaining snapshots where
desired. This functionality can also be used when not using context
managers and instead the cloud.clean() method is manually called,
like in cloud-init's integration tests.

BREAKING CHANGE: signature of cloud.snapshot() public method has
been changed and now all args except for the first (instance)
must be passed as named args.

Additional Context and Relevant Issues

Test Steps

@a-dubs a-dubs changed the title feat!: add ability to retain snapshot after cleanup v11 - feat!: add ability to retain snapshot after cleanup Mar 12, 2025
This allows for marking snapshots to be kept at time of creation
instead of being cleaned up automatically. This allows for still using
context managers for easy cleanup while retaining snapshots where
desired. This functionality can also be used when not using context
managers and instead the cloud.clean() method is manually called,
like in cloud-init's integration tests.

BREAKING CHANGE: signature of cloud.snapshot() public method has
been changed and now all args except for the first (instance)
must be passed as named args.
@a-dubs a-dubs force-pushed the v11-overhaul-snapshots branch from 12bb625 to 4b62ded Compare April 10, 2025 14:04
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