Merge lp:~paride/utah/ppc64el-s390x-subiquity into lp:utah

Proposed by Paride Legovini
Status: Merged
Approved by: Joshua Powers
Approved revision: 1147
Merged at revision: 1143
Proposed branch: lp:~paride/utah/ppc64el-s390x-subiquity
Merge into: lp:utah
Diff against target: 102 lines (+31/-4)
4 files modified
conf/utah/default-vm-s390x.xml (+9/-1)
debian/changelog (+10/-0)
utah/isotest/iso_static_validation.py (+1/-1)
utah/provisioning/live_server.py (+11/-2)
To merge this branch: bzr merge lp:~paride/utah/ppc64el-s390x-subiquity
Reviewer Review Type Date Requested Status
Canonical CI Engineering Pending
Review via email: mp+366200@code.launchpad.net

Commit message

Enable live-server testing on s390x and ppc64el.

 * Various fixes for live-server testing on ppc64 and s390x
 * Add workaround for LP: #1822096.
 * live-server testing:
   - Use a newer way of specifying the public key that will be used to log in
   - Add a way to force a subiquity refresh to a specific channel via the
     command line.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'conf/utah/default-vm-s390x.xml'
2--- conf/utah/default-vm-s390x.xml 2019-03-25 12:22:16 +0000
3+++ conf/utah/default-vm-s390x.xml 2019-04-17 14:44:32 +0000
4@@ -25,6 +25,7 @@
5 <vcpu placement='static'>2</vcpu>
6 <os>
7 <type arch='s390x' machine='s390-ccw-virtio'>hvm</type>
8+ <boot dev='hd'/>
9 </os>
10 <clock offset='utc'/>
11 <on_poweroff>destroy</on_poweroff>
12@@ -36,6 +37,13 @@
13 <mac address='00:00:00:00:00:00'/>
14 <source network='default'/>
15 <model type='virtio'/>
16+ <address type='ccw' cssid='0xfe' ssid='0x0' devno='0x0003'/>
17 </interface>
18- </devices>
19+ <disk type='file' device='cdrom'>
20+ <driver name='qemu' type='raw'/>
21+ <backingStore/>
22+ <target dev='sdc' bus='scsi'/>
23+ <readonly/>
24+ </disk>
25+ </devices>
26 </domain>
27
28=== modified file 'debian/changelog'
29--- debian/changelog 2019-03-26 13:57:32 +0000
30+++ debian/changelog 2019-04-17 14:44:32 +0000
31@@ -1,3 +1,13 @@
32+utah (0.19~bzr201904171433) UNRELEASED; urgency=medium
33+ * Various fixes for live-server testing on ppc64 and s390x
34+ * Add workaround for LP: #1822096.
35+ * live-server testing:
36+ - Use a newer way of specifying the public key that will be used to log in
37+ - Add a way to force a subiquity refresh to a specific channel via the
38+ command line.
39+
40+ -- Paride Legovini <paride.legovini@canonical.com> Wed, 17 Apr 2019 16:34:51 +0200
41+
42 utah (0.19~bzr201903261308) UNRELEASED; urgency=medium
43 * ISO static validation support for s390x
44 * d/changelog provisional version string: date --utc '+%Y%m%d%H%M'
45
46=== modified file 'utah/isotest/iso_static_validation.py'
47--- utah/isotest/iso_static_validation.py 2019-04-03 18:18:58 +0000
48+++ utah/isotest/iso_static_validation.py 2019-04-17 14:44:32 +0000
49@@ -386,7 +386,7 @@
50 if self.st_arch == 'ppc64el':
51 files_list = open(os.path.join(DATA_PATH,
52 'file_list_subiquity_powerpc'))
53- if self.st_arch == 's390x':
54+ elif self.st_arch == 's390x':
55 files_list = open(os.path.join(DATA_PATH,
56 'file_list_subiquity_s390x'))
57 else:
58
59=== modified file 'utah/provisioning/live_server.py'
60--- utah/provisioning/live_server.py 2019-04-12 04:02:44 +0000
61+++ utah/provisioning/live_server.py 2019-04-17 14:44:32 +0000
62@@ -117,6 +117,7 @@
63 with open(config.sshpublickey) as fp:
64 keydata = fp.read()
65 answers["SSH"] = {
66+ 'install': True, # workaround for LP: #1825165
67 'install_server': True,
68 'authorized_keys': [keydata],
69 }
70@@ -270,7 +271,7 @@
71 target.set('port', '0')
72 console.append(target)
73 devices.append(console)
74- else:
75+ elif self.image.arch != 'ppc64el':
76 serial = ElementTree.Element('serial')
77 serial.set('type', 'file')
78 source = ElementTree.Element('source')
79@@ -334,6 +335,10 @@
80 except libvirt.libvirtError:
81 pass
82 finally:
83+ # Workaround for LP: #1822096
84+ # To be removed once fixed in libvirtd.
85+ time.sleep(5)
86+
87 vm.undefine()
88
89 after = time.time()
90@@ -352,7 +357,11 @@
91 devices = xml.find('devices')
92 for disk in list(devices.iterfind('disk')):
93 if disk.get('device') == 'cdrom':
94- disk.remove(disk.find('source'))
95+ if disk.find('target').get('bus') == 'virtio':
96+ devices.remove(disk)
97+ else:
98+ disk.remove(disk.find('source'))
99+ continue
100 source = disk.find('source')
101 if source is not None and source.get('file') == self.answers_image_path:
102 devices.remove(disk)

Subscribers

People subscribed via source and target branches