Earlier  
Posted Nick Remark
#openstack-nova - 2018-01-04
15:05:23 kashyap Oh, didn't know of that thing
15:05:50 kashyap Why is it called 'legacy'?
15:06:01 mriedem because it existed before zuulv3
15:06:07 coreycb jamespage: mriedem: do those patches enable sharable disks with qemu 2.10? 2.10 is what we have for pike/queens atm.
15:06:15 mriedem coreycb: yes
15:06:27 mriedem coreycb: and yeah, qemu 2.10 is precisely the problem
15:06:40 kashyap Okido, makes sense.
15:07:02 mriedem kashyap: i don't know if that apr job works, but i can propose that we run it on devstack experimental queue so i can see
15:07:28 kashyap mriedem: Yeah, I'm first looking at what's in there - git://git.openstack.org/openstack/devstack-plugin-additional-pkg-repos
15:08:55 kashyap Last updated 01-Apr-2016.
15:09:08 kashyap So at least it needs updates to the file: devstack/lib/libvirt
15:09:36 kashyap (To reflect correct versions.)
15:10:04 hrw uf. one patch, one bug. and then 3 other bugs for the same. ouch
15:10:25 kashyap hrw: What are the three other bugs? It's the same one we were talking about earlier, right
15:10:46 mriedem kashyap: oh hmm https://git.openstack.org/cgit/openstack/devstack-plugin-additional-pkg-repos/tree/devstack/lib/libvirt#n21
15:10:50 kashyap hrw: I'm even surprised that OpenStack even works on AArch64
15:10:54 mriedem pointing at liberty still yeah...
15:11:12 kashyap Yeah, the last Git commit should gave it away
15:11:26 mriedem kashyap: and i guess these versions for libvirt and qemu https://git.openstack.org/cgit/openstack/devstack-plugin-additional-pkg-repos/tree/devstack/lib/libvirt#n24
15:11:45 hrw kashyap: https://bugs.launchpad.net/nova/+bug/1741230 was opened by me for my patch. Then found https://bugs.launchpad.net/ubuntu/+source/libvirt/+bug/1673467 with some extra info. Then reported bug at upstream libvirt: https://bugzilla.redhat.com/show_bug.cgi?id=1531076 and got pointed to discussion in https://bugzilla.redhat.com/show_bug.cgi?id=1430987 one where again good info is provided.
15:11:47 openstack Launchpad bug 1741230 in OpenStack Compute (nova) "libvirt: use 'host-passthrough' as default on AArch64" [Undecided,In progress] - Assigned to Marcin Juszkiewicz (hrw)
15:11:48 openstack Launchpad bug 1673467 in OpenStack nova-compute charm "[ocata] unsupported configuration: CPU mode 'host-model' for aarch64 kvm domain on aarch64 host is not supported by hypervisor" [High,Fix released] - Assigned to James Page (james-page)
15:11:49 openstack bugzilla.redhat.com bug 1531076 in libvirt "Support 'host-model' on aarch64" [Unspecified,New] - Assigned to libvirt-maint
15:11:50 openstack bugzilla.redhat.com bug 1430987 in libvirt "No cpu model and feature in capabilities" [High,Assigned] - Assigned to abologna
15:11:59 hrw kashyap: we use OpenStack on AArch64 since Liberty ;D
15:12:16 kashyap hrw: Yeah, at this point, your getting traction on the upstream Bugzilla filed is the best bet.
15:12:48 mriedem looking at https://packages.ubuntu.com/search?suite=all§ion=all&arch=any&keywords=libvirt-bin&searchon=names
15:12:52 hrw kashyap: first edition of Linaro Developer Cloud was Liberty based. and then ~200 users/projects/companies used it.
15:12:53 mriedem ubuntu doesn't have a libvirt 3.10 yet
15:13:04 kashyap Even the UCA?
15:13:17 hrw kashyap: when Mitaka got released we migrated as it gave us UEFI support which simplified booting A LOT.
15:13:22 mriedem i assume Pike UCA is the latest, and that has libvirt 3.6
15:13:26 mriedem same as bionic
15:13:34 hrw kashyap: then we migrated to Newton.
15:13:51 hrw kashyap: there are some testing deployments of Pike but we target Queens
15:13:54 mriedem same issue for qemu https://packages.ubuntu.com/search?suite=all§ion=all&arch=any&keywords=qemu-kvm&searchon=names
15:13:58 mriedem 2.10 is the latest in bionic
15:13:59 mriedem and pike UCA
15:13:59 hrw mriedem: there is Queens UCA
15:14:03 mriedem hrw: orly
15:14:15 hrw iirc
15:14:24 hrw http://ubuntu-cloud.archive.canonical.com/ubuntu/dists/xenial-updates/queens/
15:14:57 hrw mriedem: but still libvirt 3.6.0 ;(
15:15:00 kashyap hrw: Okay. But meanwhile, my Aarch64 Mustang is remote, and I haven't accessed it in a while. If you get sometime, can you power your machine on (the one you talked about earlier in the day) & try?
15:15:23 hrw kashyap: I use xgene1 based HPe Moonshots
15:15:42 hrw kashyap: remotely as we have a bunch of those at Linaro lab
15:15:45 mriedem yup still 3.6 http://ubuntu-cloud.archive.canonical.com/ubuntu/dists/xenial-updates/queens/main/binary-amd64/Packages
15:15:49 mriedem so that doesn't help
15:16:07 hrw mriedem: make a PPA and grab libvirt from buster?
15:16:20 kashyap hrw: Are there any real users using OpenStack on AArch64, besides test / devel at Linaro?
15:16:29 hrw mriedem: that's what I did for Debian stable at Linaro
15:16:35 hrw kashyap: yes, they are
15:16:53 hrw kashyap: can not list names
15:17:11 openstackgerrit Eric Berglund proposed openstack/nova master: WIP: PowerVM Driver: vSCSI https://review.openstack.org/526094
15:17:13 hrw kashyap: we provide VM instances as build slaves for several CI systems for example
15:17:13 kashyap No prob
15:17:46 hrw kashyap: also provided set of VMs as porter boxes for several projects
15:17:47 kashyap Nod
15:17:50 mriedem alternatively, i can push a devstack DNM patch which doesn't use the Pike UCA and we'll get qemu 2.5
15:18:06 kashyap mriedem: You mean, just to exercise the relevant code for tests?
15:18:18 mriedem yes
15:18:56 mriedem even if jamespage / coreycb were able to backport those patches from 3.10 to 3.6 in the pike UCA, we'd have to put a workaround option in nova to bypass the version check and just run the multiattach code
15:18:59 hrw kashyap: the plan is to migrate all our cloud setups from our Newton to clean Queens. from set of venvs to containers
15:19:08 mriedem i.e. 'i've patched my packages, so don't care about the versions you think are required'
15:19:37 hrw mriedem: can not libvirt report that multiattach feature in domcapabilities or sth?
15:20:01 kashyap Yeah, I saw your version check code in that patch series
15:20:10 hrw mriedem: so instead "if libvirt >= x.y" you can use "if libvirt.capabilities.contains('multiattach')"
15:20:31 mriedem hrw: maybe? https://github.com/libvirt/libvirt/commit/860a3c4bea1d24773d8a495f213d5de3ac48a462
15:20:40 hrw version check suxx when features are not enabled on all archs
15:20:53 mriedem is that how this would work with ^ and "disk-share-rw"?
15:21:49 hrw kashyap: so, can I get +2 from you on patch? ;D
15:21:59 kashyap hrw: I can't +2
15:22:11 kashyap hrw: But I'd like someone else's opinion, too, like mriedem
15:22:18 kashyap Or mdrabe
15:22:21 hrw kashyap: or w8... I was supposed to do something with tests... no idea how to do that part
15:22:22 kashyap Err, mdbooth I mean
15:22:50 mriedem kashyap: can you tell me if https://github.com/libvirt/libvirt/commit/860a3c4bea1d24773d8a495f213d5de3ac48a462 is used to expose a hypervisor capability?
15:23:00 mdbooth kashyap: ?
15:23:01 kashyap hrw: I have one more comment on the change
15:23:08 kashyap Almost about to hit send
15:23:10 mdbooth Which change?
15:23:12 mriedem like in https://github.com/openstack/nova/blob/master/nova/virt/libvirt/host.py#L615
15:23:12 kashyap mriedem: 1 sec, let me look
15:23:13 hrw mdbooth: https://review.openstack.org/#/c/530965/
15:23:15 kashyap mdbooth: https://review.openstack.org/#/c/530965/4
15:23:33 kashyap mdbooth: It's arch-specific, but migration-related, too. I noted the concern there
15:24:18 mdbooth kashyap: I don't have a useful opinion on that unfortunately without doing my own doc diving.
15:24:32 kashyap mdbooth: Okido, I have enough context there. Disregard
15:25:16 hrw mdbooth: imho https://bugzilla.redhat.com/show_bug.cgi?id=1430987 is best part of info on subject
15:25:17 openstack bugzilla.redhat.com bug 1430987 in libvirt "No cpu model and feature in capabilities" [High,Assigned] - Assigned to abologna
15:25:50 kashyap mriedem: Still checking
15:25:57 hrw kashyap: thx
15:29:27 kashyap mriedem: Hmm, I just built the newest libvirt-python bindings, and don't see it any of the capabilities; let me ask one of the libvirt folks
15:30:16 kashyap mriedem: So no -- it isn't exposed via any capabilities currently (like `virsh (dom)capablities`)
15:30:39 kashyap mriedem: So Peter (who wrote that commit says):
15:31:16 kashyap 16:29 < pkrempa> kashyap: it is not exposed currently, since it's supposed to be transparent for the users
15:31:19 kashyap 16:30 < pkrempa> and if it's not transparent I'd suggest to complain to qemu
15:31:38 mriedem kashyap: ok so we're stuck with version checks
15:31:40 mriedem thanks for asking
15:31:56 mriedem we could, as noted, add a workaround config option to bypass the version checks if you've patched your packages

Earlier   Later