Merge ~mreed8855/plainbox-provider-certification-server:remove_uvt into plainbox-provider-certification-server:master

Proposed by Michael Reed
Status: Merged
Approved by: Michael Reed
Approved revision: 52474ae9d9750d691e14ba17424d8fed087702ed
Merged at revision: f509c69b28fc2c9f9a54b1575d7f1f06ffffaa6f
Proposed branch: ~mreed8855/plainbox-provider-certification-server:remove_uvt
Merge into: plainbox-provider-certification-server:master
Diff against target: 132 lines (+17/-50)
4 files modified
configs/canonical-certification.conf (+15/-38)
units/packaging.pxu (+0/-10)
units/server-functional-18.04.pxu (+1/-1)
units/server-soc-cert-18.04.pxu (+1/-1)
Reviewer Review Type Date Requested Status
Jeff Lane  Approve
Review via email: mp+423557@code.launchpad.net

Commit message

The kvm virtualization test using the uvtool has been deprecated. I have removed the remaining code. This is being submitted in conjunction with the following MP:

https://code.launchpad.net/~mreed8855/plainbox-provider-checkbox/+git/plainbox-provider-checkbox/+merge/423556

To post a comment you must log in.
Revision history for this message
Jeff Lane  (bladernr) wrote :

LGTM, thanks

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/configs/canonical-certification.conf b/configs/canonical-certification.conf
2index 4ea41ea..34e9320 100644
3--- a/configs/canonical-certification.conf
4+++ b/configs/canonical-certification.conf
5@@ -26,59 +26,36 @@ DISK_MDADM_READ_PERF = 150
6
7 # For virtualization testing:
8 #
9-# The KVM_IMAGE variable is now used in the lxd virtual machine test.
10+# The KVM_IMAGE and LXD_TEMPLATES variables are now used in the lxd
11+# virtual machine test.
12 #
13-# The KVM_TIMEOUT and variable will soon be deprecated
14-# and will no longer work
15+# The LXD_TEMPLATES is the tarball that contains the LXD templates and YAML
16+# necessary for defining containers and virtual machines. This variable
17+# should be set to the same value that is used for lxd container testing.
18 #
19-# KVM_TIMEOUT sets the timeout to wait for the cloudimg to boot before exiting
20-# test. It is an integer in seconds (the default is 3000 seconds)
21 # KVM_IMAGE the location to find the cloud image to use for testing KVM. This
22 # will allow you to specify a variety of location options depending on your
23 # needs.
24 # Options available:
25-# - http://myserver.com/imagedir/ (note the trailing / is required!!)
26-# - This will tell the script to look into /imagedir/ on the remote server
27-# for files in the standard ubuntu Cloud filename format
28-# RELEASE-server-cloudimg-ARCH-disk1.img
29 # - http://someotherserver.net/imagedir/ubuntu/mycloudimg.img
30 # - This will tell the script to download the file mycloudimg.img from the
31 # remote server.
32 # - file:///local/path/to/cloud.img
33 # - This will tell the script to look at the path specified on the local
34 # filesystem for a copy of the cloud image to use.
35-# - /local/path/to/cloud.img
36-# - This will work the same as the file:// example above and should be a
37-# full path to the cloud image.
38-#
39-# Note that a MAAS server set up via the MANIACS guide may optionally house
40-# copies of the KVM images under http://{ip-address}/cloud/.
41-#
42-# UNCOMMENT EITHER OR BOTH OF THE FOLLOWING 2 LINES and set the proper
43-# parameters as necessary:
44-#KVM_TIMEOUT = 300
45-#KVM_IMAGE = /path/to/cloudimage-filename.img
46-
47-# The UVT_IMAGE_OR_SOURCE will soon be deprecated
48-# For virtualization testing using the uvtool:
49-# UVT_IMAGE_OR_SOURCE - the location of of the cloud image or the source
50-# for testing kvm using the uvtool
51-# Options:
52-#
53-# Specify a location for uvt-simplestreams-libvirt to retrive an image
54 #
55 # Example:
56-# UVT_IMAGE_OR_SOURCE = http://cloud-images.ubuntu.com/daily/server/daily/server/{release}
57-#
58-# Specify a cloud image for uvt-kvm create to create a vm
59-# - file:///local/path/to/cloud.img
60-# - This will tell the script to look at the path specified on the local
61-# filesystem for a copy of the cloud image to use.
62+# KVM_IMAGE = http://cloud-images.ubuntu.com/daily/server/daily/server/{release}
63 #
64 # Example:
65-# UVT_IMAGE_OR_SOURCE = file:///home/ubuntu/focal-server-cloudimg-amd64.img
66+# KVM_IMAGE = file:///home/ubuntu/jammy-server-cloudimg-amd64.img
67 #
68-#UVT_IMAGE_OR_SOURCE =
69+# Note that a MAAS server set up via the MANIACS guide may optionally house
70+# copies of the KVM images under http://{ip-address}/cloud/.
71+#
72+# UNCOMMENT THE FOLLOWING LINE and set the proper
73+# parameters as necessary:
74+# KVM_IMAGE = /path/to/cloudimage-filename.img
75
76 # For LXD testing:
77 #
78@@ -102,8 +79,8 @@ DISK_MDADM_READ_PERF = 150
79 # UNCOMMENT BOTH OF THE FOLLOWING 2 LINES nad set the proper URLs as necessary
80 # OR leave them commented out and the LXD test will obtain the needed images
81 # from cloud-images.ubuntu.com.
82-#LXD_ROOTFS = http://server_url/path/to/bionic-server-cloudimg-amd64.squashfs
83-#LXD_TEMPLATE = http://server_url/path/to/bionic-server-cloudimg-amd64-lxd.tar.xz
84+#LXD_ROOTFS = http://server_url/path/to/jammy-server-cloudimg-amd64.squashfs
85+#LXD_TEMPLATE = http://server_url/path/to/jammy-server-cloudimg-amd64-lxd.tar.xz
86
87 # The STRESS_NG_MIN_SWAP_SIZE environment variable sets the minimum swap
88 # space, in gibibytes (GiB), that the memory/memory_stress_ng test requires.
89diff --git a/units/packaging.pxu b/units/packaging.pxu
90index 70b6430..930cc28 100644
91--- a/units/packaging.pxu
92+++ b/units/packaging.pxu
93@@ -1,13 +1,3 @@
94-# This is for 'virtualization/kvm_check_vm'
95-unit: packaging meta-data
96-os-id: debian
97-Depends: uvtool
98-
99-# This is for 'virtualization/kvm_check_vm'
100-unit: packaging meta-data
101-os-id: debian
102-Depends: uvtool-libvirt
103-
104 # The 'memory/memory_stress_ng' job requires numactl
105 unit: packaging meta-data
106 os-id: debian
107diff --git a/units/server-functional-18.04.pxu b/units/server-functional-18.04.pxu
108index 083a8fb..74f3df3 100644
109--- a/units/server-functional-18.04.pxu
110+++ b/units/server-functional-18.04.pxu
111@@ -29,7 +29,7 @@ include:
112 memory/info certification-status=blocker
113 power-management/rtc certification-status=blocker
114 virtualization/verify_lxd certification-status=blocker
115- virtualization/kvm_check_vm certification-status=blocker
116+ virtualization/verify_lxd_vm certification-status=blocker
117 info/kvm_output certification-status=non-blocker
118 miscellanea/oops certification-status=non-blocker
119 miscellanea/oops_results.log certification-status=non-blocker
120diff --git a/units/server-soc-cert-18.04.pxu b/units/server-soc-cert-18.04.pxu
121index 05ef64e..b8b4bd3 100644
122--- a/units/server-soc-cert-18.04.pxu
123+++ b/units/server-soc-cert-18.04.pxu
124@@ -29,7 +29,7 @@ include:
125 power-management/rtc certification-status=blocker
126 stress/cpu_stress_ng_test certification-status=blocker
127 virtualization/verify_lxd certification-status=blocker
128- virtualization/kvm_check_vm certification-status=blocker
129+ virtualization/verify_lxd_vm certification-status=blocker
130 info/kvm_output certification-status=non-blocker
131 miscellanea/oops certification-status=non-blocker
132 miscellanea/oops_results.log

Subscribers

People subscribed via source and target branches