← Back to issue list

[multipass] use stdio to get data in/out of Multipass

View original Github issue

Metadata

Project
snapcraft
Number
#2784
Type
pull request
State
merged
Author
Saviq
Labels
Created
Updated
Closed

Current evaluation

Switched Multipass data transfer to stdio for reliable operation under strict confinement. Refactored into push_file and pull_file functions, removed --cloud-init support. Approved and merged after passing CI checks.

Suggested action:

No scores available.

Issue body

With Multipass going strict, that's the reliable way. - [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] Have you successfully run `./runtests.sh static`? - [x] Have you successfully run `./runtests.sh tests/unit`? I've not found a better way around [LP#1849753](https://bugs.launchpad.net/ubuntu/+source/snapd/+bug/1849753). This does build the Multipass snap on strict Multipass from `snap refresh --channel edge/pr1074` fine.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Switched Multipass data transfer to stdio for reliable operation under strict confinement. Refactored into push_file and pull_file functions, removed --cloud-init support. Approved and merged after passing CI checks.
qwen3.6-35b-a3b-mtp-q6 Merged after refactoring into push_file and pull_file functions. Switches Multipass data transfer to stdio for reliable operation under strict confinement, resolving LP#1849753 and removing cloud-init support.
qwen3.6-35b-a3b-mtp-q6 Merged. Switched Multipass data transfer to stdio for reliability under strict confinement. Refactored into push_file and pull_file functions and removed --cloud-init support.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#2408 multipass: avoid stdin where possible snapcraft merged Merged. Fixes multipass metrics UI conflicts by redirecting unused stdin to subprocess.DEVNULL. Approved by two reviewers, passed CI, and integrated with 41 additions and 29 deletions across two files.
71%