Skip to content

Invoke virt-v2v-in-place on host#510

Merged
tomponline merged 2 commits intocanonical:latest-edgefrom
MusicDin:feat/v2v
Jul 25, 2024
Merged

Invoke virt-v2v-in-place on host#510
tomponline merged 2 commits intocanonical:latest-edgefrom
MusicDin:feat/v2v

Conversation

@MusicDin
Copy link
Member

@MusicDin MusicDin commented Jul 12, 2024

This is required for virt-v2v-in-place to be invoked on the host.

Requires: canonical/lxd#13748

MusicDin added 2 commits July 12, 2024 12:04
Signed-off-by: Din Music <din.music@canonical.com>
Signed-off-by: Din Music <din.music@canonical.com>
@MusicDin MusicDin marked this pull request as ready for review July 24, 2024 13:34
@MusicDin MusicDin requested a review from tomponline as a code owner July 24, 2024 13:34
@tomponline tomponline merged commit a8a1ab6 into canonical:latest-edge Jul 25, 2024
@MusicDin MusicDin deleted the feat/v2v branch July 25, 2024 15:00
tomponline added a commit to canonical/lxd that referenced this pull request Jul 26, 2024
This PR adds new conversion option `virtio` (disabled by default). When
enabled, LXD invokes `virt-v2v-in-place` tool on the host to inject
virtio drivers into the image. For Windows images, additional drivers
need to be present in `/usr/share/virt-tools`.

This requires:
- For rebase: #13385
- Custom command in LXD snap to invoke virt-v2v-in-place on the host:
canonical/lxd-pkg-snap#510
@simondeziel
Copy link
Member

simondeziel commented Aug 7, 2024

It's a bit late now (I was in vacation) but could we have used the existing run-host wrapper mechanism? Are the unset XDG_RUNTIME_DIR and PYTHONPATH the reason why it was forked into another wrapper?

@tomponline
Copy link
Member

If we can use run-host that would be useful.

tomponline added a commit that referenced this pull request Aug 29, 2024
From #510

This is required for conversion option `virtio` which failed the tests
for 5.21-edge in canonical/lxd-ci#274.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants