lxc: remove TODO for copying snapshots

Metadata

Project: craft-providers
Number: #181
Type: pull request
State: merged
Author: mr-cal
Labels: —
Created: 2022-12-01 09:46 PM UTC
Updated: 2022-12-02 04:40 PM UTC
Closed: 2022-12-02 04:40 PM UTC

Current evaluation

Merged #181 via commit 8a8e93d, removing the lxc.copy() snapshot TODO because the LXC launch path no longer uses snapshots (CRAFT-1340).

Resolution: closed resolved

Reason: PR #181 was approved and merged. Commit 8a8e93d removed the obsolete three-line TODO after the launch mechanism stopped using snapshots.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [X] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? ----- Removed a TODO for support snapshots for `lxc.copy()`. The lxc launch mechanism will not use snapshots, so this no longer needs to be implemented. (CRAFT-1340)

Evaluation history

Date	Model	Scores	Action	Summary
2026-10-03 03:30 PM UTC	stealth/space-bunny-alpha	—	closed resolved	Merged #181 via commit 8a8e93d, removing the lxc.copy() snapshot TODO because the LXC launch path no longer uses snapshots (CRAFT-1340).
2026-09-19 12:37 PM UTC	qwen/qwen3.6-35b-a3b	—	closed resolved	Merged PR removing a TODO for lxc.copy() snapshot support, as the lxc launch mechanism does not use snapshots.

Update history

Date	Change
2022-12-02 04:40 PM UTC	created

Related work

Related To: canonical/craft-providers#179 (confidence 98%)
PR #179 introduced lxc.copy(), where the obsolete snapshot TODO was located.

Related issues

Issue	Project	State	Summary	Similarity
#179 lxd: add lxc.copy() method to copy instances	craft-providers	merged	Merged PR #179, adding LXC instance-copy support via commit c87101a; it addressed CRAFT-1479.	72%