Skip to content

Delete instances - #936

Open
Vova956 wants to merge 17 commits into
canonical:mainfrom
Vova956:delete-instances
Open

Delete instances#936
Vova956 wants to merge 17 commits into
canonical:mainfrom
Vova956:delete-instances

Conversation

@Vova956

@Vova956 Vova956 commented Mar 24, 2026

Copy link
Copy Markdown

Changes :

  • add Provider.delete_instances(...)
  • implement deletion for LXDProvider
  • implement deletion for MultipassProvider
  • add unit tests for LXD and Multipass providers
  • ruff check ...
  • unit tests for:
    • tests/unit/lxd/test_lxd_provider.py
    • tests/unit/multipass/test_multipass_provider.py

let me know if I did something wrong. Thanks!

@Vova956
Vova956 requested a review from a team as a code owner March 24, 2026 20:20
@bepri

bepri commented Mar 24, 2026

Copy link
Copy Markdown
Member

Hi @Vova956 - I believe this is duplicating the work being done by @Aeonoi on #925, and does not follow the specifications of the original issue (#893).

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.

2 participants