Page MenuHomeSoftware Heritage
Differential D4137

packer: Add template to build qemu/libvirt images
ClosedPublic
Actions

Authored by ardumont on Oct 3 2020, 8:56 PM.

Details

Reviewers
vsellier
Group Reviewers
Reviewers
Maniphest Tasks
T2606: Test puppet configuration in a local vagrant environment
Commits
rSENV9198d86e3909: packer: Add template to build qemu/libvirt images
Summary

Build ok (cf. P800)

Goal is to try and build vm with libvirt/qemu possibly instead of virtualbox [1]

[1] https://github.com/vagrant-libvirt/vagrant-libvirt#create-vagrantfilen

Related to T2606

Test Plan

packer build debian_buster.qemu.json runs fine

Diff Detail

Repository
rSENV Puppet Environment
Branch
master
Lint
Lint Skipped
Unit
Unit Tests Skipped
Build Status
Buildable 15868
Build 24418: arc lint + arc unit

Event Timeline

ardumont created this revision.Oct 3 2020, 8:56 PM
Herald added a reviewer: Reviewers. · View Herald TranscriptOct 3 2020, 8:56 PM
Harbormaster completed remote builds in B15863: Diff 14582.Oct 3 2020, 8:56 PM
ardumont added inline comments.Oct 3 2020, 8:57 PM
packer/debian_buster.qemu.json
33

grub-install step failed due to this naming which is different with the other template (in there sda)...

ardumont planned changes to this revision.Oct 3 2020, 8:57 PM
ardumont updated this revision to Diff 14583.Oct 3 2020, 8:59 PM

Drop the specific vbox vbguest install step (no need here ¯\_(ツ)_/¯)

Harbormaster completed remote builds in B15864: Diff 14583.Oct 3 2020, 8:59 PM
ardumont planned changes to this revision.Oct 3 2020, 8:59 PM
ardumont edited the summary of this revision. (Show Details)
ardumont requested review of this revision.Oct 3 2020, 9:09 PM
ardumont edited the summary of this revision. (Show Details)
ardumont added subscribers: vsellier, vlorentz, olasd.Oct 3 2020, 9:11 PM

heads up @vlorentz (since you raise the license remark) @olasd (since you raised the question about vbox) @vsellier (since you are the original author ;)

template build process worked, that's a start, i guess.

ardumont added inline comments.Oct 3 2020, 9:13 PM
packer/debian_buster.qemu.json
16

up to line 9, that's the main differing part from the original template from D4136

(well that and the /dev/vda bootloader install instruction below and the qcow2 format at the end.)

33

explicitely /dev/sda is replaced by /dev/vda

68

.qcow2 format is different than the .box

ardumont added a child revision: D4138: Vagrantfile: Allow libvirt vm build.Oct 4 2020, 1:21 PM
ardumont mentioned this in D4136: packer: Move specific vbox configuration closer to the top.Oct 4 2020, 1:27 PM
ardumont updated this revision to Diff 14587.Oct 5 2020, 9:11 AM

Rebase

Harbormaster completed remote builds in B15868: Diff 14587.Oct 5 2020, 9:11 AM
ardumont edited the summary of this revision. (Show Details)Oct 6 2020, 11:45 AM
ardumont added a task: T2606: Test puppet configuration in a local vagrant environment.
ardumont edited the test plan for this revision. (Show Details)Oct 6 2020, 2:17 PM
olasd added inline comments.Oct 6 2020, 2:18 PM
packer/debian_buster.qemu.json
3

That's huge. I'm guessing that does thin provisioning (i.e. it's not allocating 20 GB worth of zeroes)?

ardumont added inline comments.Oct 6 2020, 2:20 PM
packer/debian_buster.qemu.json
3

No idea, really. I started from the other vbox template which defined it as well.
I'll check.

ardumont added inline comments.Oct 10 2020, 12:34 PM
packer/debian_buster.qemu.json
3

[1] does not convey much information

I'll decrease to say 5G, hoping that's sufficient for the template build to pass.

[1] https://www.packer.io/docs/builders/qemu.html#disk_size

ardumont updated this revision to Diff 14911.Oct 10 2020, 1:12 PM
  • Decrease disk_size to 5G
  • template build still ok (P800 updated)
Harbormaster completed remote builds in B16144: Diff 14911.Oct 10 2020, 1:12 PM
ardumont added inline comments.Oct 12 2020, 10:59 AM
packer/debian_buster.qemu.json
3

so i should have realized earlier, as @vsellier made me notice the finalized template is about ~500Mib.
So it's not indeed allocating the 20Gb worth of zeroes.

And my tryout of a deposit-staging with a 5Gb disk is not enough space.
So i'll readapt this.
(Most possibly in another diff though, to avoid the rebase dance a bit, if you don't mind ;).

vsellier accepted this revision as: vsellier.Oct 12 2020, 3:36 PM

Thanks, it's really great.
I have tested locally the qemutest vm and converted the staging-webapp and staging-deposit vms, everything looks good.
The virtualbox and libvirt networks (with the same ip range) can't cohabit together but after a cleanup on the virtualbox side, everything works as expected.

The vm management itself (up / destroy / status) is really smooth compared to the virtualbox provider.

As discuss, I'm ok to completely remove the virtualbox usage to focus on only one provider.

This revision is now accepted and ready to land.Oct 12 2020, 3:36 PM
Closed by commit rSENV9198d86e3909: packer: Add template to build qemu/libvirt images (authored by ardumont). · Explain WhyOct 12 2020, 3:50 PM
This revision was automatically updated to reflect the committed changes.
ardumont added a commit: rSENV9198d86e3909: packer: Add template to build qemu/libvirt images.

Revision Contents
Changeset List

PathSize
packer/
72 lines

Diff 14587

View Options

packer/debian_buster.qemu.json

Loading...
About · Developer information · Legal