multipass: introduce MultipassProvider
Metadata
Current evaluation
Closed unmerged; MultipassProvider was later added by merged PR #163 in commit fa5a5e7, superseding this abandoned implementation.
Resolution: closed superseded
Reason: PR #17 was closed without merging. PR #163 subsequently introduced the Provider, LXDProvider, and MultipassProvider classes and was merged in commit fa5a5e7.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Note that this PR does not incorporate minimum version checks.
Depends on #13 #14 #15 #16
TODO: unit tests after initial review
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed superseded | Closed unmerged; MultipassProvider was later added by merged PR #163 in commit fa5a5e7, superseding this abandoned implementation. | |
| qwen/qwen3.6-35b-a3b | — | closed superseded | PR #21 was closed without merging. The MultipassProvider feature was later introduced in PR #163, which added Provider, LXDProvider, and MultipassProvider classes adapted from rockcraft. |
Update history
| Date | Change |
|---|---|
| created |
Related work
-
Superseded By:
canonical/craft-providers#163
(confidence 99%)
Merged PR #163 added MultipassProvider in commit fa5a5e7 using an implementation adapted from rockcraft.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #21 multipass: introduce MultipassProvider | craft-providers | closed | Closed unmerged after prolonged inactivity, with four unresolved review comments. MultipassProvider later landed via #163 (commit fa5a5e7). | |
| #11 multipass: introduce MultipassInstance | craft-providers | merged | Merged PR #11 introduced MultipassInstance via commit 615064d (merge ba6cd15); Windows support remained a noted follow-up. | |
| #308 [WIP] providers: experimental craft-provider support | charmcraft | closed | Closed unmerged as WIP; Multipass provider support was later delivered by merged #491 (implementation 84132dc1, merge 1b855093). | |
| #9 multipass: introduce Multipass and MultipassError | craft-providers | merged | PR #9 by cjp256 merged into craft-providers (commit e418497), adding a Multipass class wrapping the multipass CLI plus MultipassError, unit/integration tests (test_todo dropped), and pytest-subprocess in requirements-dev. | |
| #491 providers: introduce multipass support (CRAFT-409) | charmcraft | merged | Multipass support was merged in canonical/charmcraft#491 via merge commit 1b855093, with implementation commit 84132dc1 adding provider selection and tests. | |
| #2100 build_providers: new build provider using multipass | snapcraft | merged | Merged #2100 in commit c44a4ece9, adding the build_providers package and an MVP Multipass build provider. | |
| #254 feat(Provider): default to the multipass provider | imagecraft | merged | Merged PR #254 (commit fdad50c) by lengau: Imagecraft now overrides ProviderService.get_provider to default to the Multipass provider, a temporary workaround for open issue #253. | |
| #22 multipass: introduce installation support | craft-providers | merged | Merged PR #22 via commit e163f50, adding Linux and macOS Multipass installation support, error handling, tests, and CI uninstall support. | |
| #120 multipass: do not leak start command | craft-providers | merged | Merged PR #120 in commit 79d7089, fixing Multipass so its start command is no longer leaked. | |
| #23 multipass: introduce launch to create and configure instances | craft-providers | merged | Merged PR #23 added Multipass launch support for creating and configuring instances in commit 01457c7, merged as df00fcc after installation support from #22. |