multipass: introduce ensure_multipass_is_ready() (CRAFT-438)
Metadata
Current evaluation
Merged PR #75 added ensure_multipass_is_ready() to verify Multipass installation and version support; merge commit 1f1ddce.
Resolution: closed resolved
Reason: PR #75 was approved and merged in commit 1f1ddce. Implementing commit b8c4fe0 added readiness checks and invoked them before provider use.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Add checks to ensure Multipass is ready for use similar to LXD's.
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 #75 added ensure_multipass_is_ready() to verify Multipass installation and version support; merge commit 1f1ddce. | |
| qwen/qwen3.6-35b-a3b | — | closed resolved | Merged PR introducing `ensure_multipass_is_ready()` to validate Multipass availability, mirroring LXD checks. Resolves CRAFT-438. |
Update history
| Date | Change |
|---|---|
| created |
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #16 multipass: add version() and wait_until_ready() to Multipass | craft-providers | merged | Merged PR #16 added Multipass version() and wait_until_ready(), with the readiness helper returning the detected version. | |
| #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. | |
| #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. | |
| #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. | |
| #628 Check Multipass version for 24.04 VMs | craft-providers | closed | Implemented a Multipass >=1.14.1 requirement for Noble/24.04 launches in commit a977369 and released it in craft-providers 1.24.2 and 2.0.1. | |
| #1796303 Check version of multipass | snapcraft (launchpad) | closed | Added a Multipass minimum-version check via canonical/craft-providers#42, requiring Multipass 1.7+; merged as ce6356d. | |
| #42 multipass: is_supported_version() interface and require >=1.7 | craft-providers | merged | Merged in craft-providers#42 via ce6356d; commit 1166096 added Multipass.is_supported_version() and raised the minimum supported version to 1.7. | |
| #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. | |
| #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. |