multipass: introduce installation support
Metadata
Current evaluation
Merged PR #22 via commit e163f50, adding Linux and macOS Multipass installation support, error handling, tests, and CI uninstall support.
Resolution: closed resolved
Reason: The pull request was approved and merged into main as e163f50. Commit ff36501 introduced Multipass installation support, MultipassInstallationError, and associated tests.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- Add MultipassInstallationError.
- Add unit tests for Multipass errors (formatting).
- Add unit & integration tests for installer bits.
- Update CI to allow MacOS integration tests to uninstall multipass.
Signed-off-by: Chris Patterson <chris.patterson@canonical.com>
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #22 via commit e163f50, adding Linux and macOS Multipass installation support, error handling, tests, and CI uninstall support. | |
| qwen/qwen3.6-35b-a3b | — | closed resolved | Merged PR introducing Multipass installation support, including MultipassInstallationError, unit/integration tests, and CI updates for macOS. |
Update history
| Date | Change |
|---|---|
| created |
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #11 multipass: introduce MultipassInstance | craft-providers | merged | Merged PR #11 introduced MultipassInstance via commit 615064d (merge ba6cd15); Windows support remained a noted follow-up. | |
| #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. | |
| #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. | |
| #75 multipass: introduce ensure_multipass_is_ready() (CRAFT-438) | craft-providers | merged | Merged PR #75 added ensure_multipass_is_ready() to verify Multipass installation and version support; merge commit 1f1ddce. | |
| #1792374 Instructions on how to install multipass missing | snapcraft (launchpad) | closed | Added platform-specific Multipass guidance: Linux offers interactive installation, while macOS links to Canonical's releases page. | |
| #307 tests: test latest interim release for multipass on macos | craft-providers | closed | Merged PR #308 (e0b2f24) updated MultipassProvider for Lunar, superseding #307's unmerged macOS test-matrix proposal. | |
| #28 multipass tests: cleanup after launch tests | craft-providers | merged | Merged PR #28 to ensure temporary Multipass instances are cleaned up after launch tests, adding explicit teardown handling in the multipass test fixture. | |
| #30 multipass: introduce auto_clean option for launch() | craft-providers | merged | Merged PR #30 added a Multipass launch() auto_clean option that removes incompatible existing instances and launches fresh ones (merge b13242a). | |
| #1807737 install multipass from the cask on osx | snapcraft (launchpad) | closed | Guided macOS Multipass install shipped: snapcore/snapcraft#2427 (commit e9ae8d29f) added Darwin detection of Multipass installed via Homebrew cask, and the Launchpad bug was closed. | |
| #27 multipass: doc fix and require check for Multipass._run() | craft-providers | merged | Merged PR #27 (2891d9d, merge 6c1b4fa), requiring check for Multipass._run() and correcting its documentation. |