test: Fix vm.install for non-LVM cloud images
Some of our images, like centos-8-stream, are already built from the official cloud images instead of virt-install with LVM. More images are going to do that soon [1][2], so fix vm.install to only do the LVM grow steps if the image actually uses LVM. Also adjust the comment, as commit6ddaa5e0dd
fixed this for RHEL images. [1] https://github.com/cockpit-project/bots/pull/1518 [2] https://github.com/cockpit-project/bots/pull/1527 (cherry-picked from commit04ce221a65
)
This commit is contained in:
parent
e5b517add6
commit
4149d8d810
@ -35,12 +35,15 @@ if ! rpm -q qemu-kvm; then
|
|||||||
yum install -y qemu-kvm
|
yum install -y qemu-kvm
|
||||||
fi
|
fi
|
||||||
|
|
||||||
# Grow root partition to make room for images. This only works on Fedora right now.
|
# Grow root partition on LVM test images to make room for built images
|
||||||
parted --script /dev/vda resizepart 2 100%
|
PVS=$(pvs --noheadings -opv_name)
|
||||||
partprobe
|
if [ -n "$PVS" ]; then
|
||||||
pvs --noheadings -opv_name | xargs pvresize
|
echo ", +" | sfdisk -N 2 -f /dev/vda
|
||||||
rootlv=$(findmnt --noheadings -oSOURCE /)
|
partprobe
|
||||||
lvresize $rootlv -l+100%FREE -r
|
echo "$PVS" | xargs pvresize
|
||||||
|
rootlv=$(findmnt --noheadings -oSOURCE /)
|
||||||
|
lvresize $rootlv -l+100%FREE -r
|
||||||
|
fi
|
||||||
|
|
||||||
rm -rf build-results
|
rm -rf build-results
|
||||||
su builder -c "/usr/bin/mock --no-clean --resultdir build-results --rebuild $SRPM"
|
su builder -c "/usr/bin/mock --no-clean --resultdir build-results --rebuild $SRPM"
|
||||||
|
Loading…
Reference in New Issue
Block a user