| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-04 | |||
| 10:42:30 | openstack | bugzilla.redhat.com bug 1378242 in libvirt "QEMU image file locking (libvirt)" [Unspecified,On_qa] - Assigned to pkrempa | |
| 10:42:50 | kashyap | But yeah, the 'shareable' flag has been around for ages | |
| 10:42:54 | ildikov | kashyap: yeah, I saw that one this morning | |
| 10:43:36 | lyarwood | kashyap: I'm not sure I agree with Matt's point in https://review.openstack.org/#/c/267587/75/nova/virt/libvirt/guest.py re the versions | |
| 10:44:21 | lyarwood | kashyap: https://github.com/libvirt/libvirt/commit/28907b0043fbf71085a798372ab9c816ba043b93 doesn't suggest we need to do anything different here, libvirt and QEMU should work out when to provide the share-rw=on flag | |
| 10:46:46 | kashyap | lyarwood: I think it's too late for that | |
| 10:47:08 | lyarwood | kashyap: for what? | |
| 10:47:17 | openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: libvirt: use 'host-passthrough' as default on AArch64 https://review.openstack.org/530965 | |
| 10:47:22 | kashyap | For libvirt / QEMU to do the "right thing" | |
| 10:47:37 | kashyap | lyarwood: At the moment, Nova simply needs the conditional logic, going by the errors Matt and ildikov are reporting | |
| 10:48:07 | lyarwood | I'm obviously missing something here because what you're saying makes no sense | |
| 10:48:36 | lyarwood | the unsupported flag error is a docs bug | |
| 10:48:51 | hrw | now the worst part - reading nova/libvirt tests to find where it needs to be changed for https://review.openstack.org/530965 patch | |
| 10:48:55 | kashyap | Did you also look at this change: https://review.openstack.org/#/c/505673/ | |
| 10:49:32 | kashyap | hrw: Heya, just saw your change, note that 'host-passthrough' means, migration will completely break *unless* all the CPUs involved are all *identical* | |
| 10:49:39 | lyarwood | kashyap: that's totally unrelated to this isn't it? | |
| 10:49:43 | kashyap | hrw: I'll add some comments | |
| 10:49:56 | kashyap | lyarwood: No, it is related. | |
| 10:49:57 | hrw | kashyap: thanks | |
| 10:50:10 | lyarwood | kashyap: that just allows us to run qemu-img info against a live instance | |
| 10:50:18 | hrw | kashyap: I am aware of that issue. | |
| 10:51:05 | hrw | kashyap: getting that fixed layer after layer. | |
| 10:51:31 | kashyap | lyarwood: That's true; the two aspects here are - file locking from the standalone tool, 'qemu-img', and the locking semantics when running qemu-system-x86_64 | |
| 10:51:39 | kashyap | Give me a few minutes before I coment more | |
| 10:52:03 | lyarwood | ildikov: are we adding <shareable/> to the disk XML of multi-attached disks? | |
| 10:52:35 | hrw | kashyap: Kolla deploys Nova now with cpu_mode set. But not everyone uses Kolla-ansible to deploy. Then libvirt needs to be changed probably. I hope to meet some virt-arm people face-to-face later this month and want to discuss that | |
| 10:53:15 | ildikov | lyarwood: theoretically yes, proposed code change is here: https://review.openstack.org/#/c/267587/78/nova/virt/libvirt/volume/volume.py | |
| 10:53:46 | lyarwood | ildikov: thanks | |
| 10:54:31 | lyarwood | ildikov: hmmm shouldn't that be data.get('multiattach', False) ? | |
| 10:54:42 | lyarwood | good old connection_info, confusing everyone yet again | |
| 10:55:14 | kashyap | hrw: What does the 'Kolla' tool set it to? | |
| 10:55:24 | hrw | kashyap: host-passthrough | |
| 10:55:44 | kashyap | hrw: Hmm, that's wrong... | |
| 10:56:04 | ildikov | lyarwood: nope, it's not coming from Cinder, I add it here: https://review.openstack.org/#/c/267587/78/nova/virt/block_device.py | |
| 10:56:32 | ildikov | lyarwood: I mean it's in the volume details and I chose connection_info to pass it with the other stuff | |
| 10:56:48 | lyarwood | ildikov: ah, I assumed the backend was writing this in, sorry | |
| 10:57:10 | ildikov | can add it to the data part of it, we actually wanted to remove the nested dict structure of lovely connection_info, but seemed like a too huge headache | |
| 10:57:22 | ildikov | don't get me started on it... :) | |
| 10:58:25 | ildikov | lyarwood: no worries, would be nice, but it's a less direct interaction in the sense, that you can only create a multi-attach volume if the back end supports it, but then the info is stored in the volume object | |
| 11:06:38 | lyarwood | ildikov: yeah true | |
| 11:10:15 | yasemin_ | hi, i want to add to hyper-v compute node to ocata openstack system, i installed it, but it gives an error "Filter ImagePropertiesFilter returned 0 hosts" ? Do you have any idea? Could you help me ? | |
| 11:10:55 | openstackgerrit | Stephen Finucane proposed openstack/nova master: console: introduce framework for RFB authentication https://review.openstack.org/345397 | |
| 11:10:56 | openstackgerrit | Stephen Finucane proposed openstack/nova master: console: introduce the VeNCrypt RFB authentication scheme https://review.openstack.org/345398 | |
| 11:10:56 | openstackgerrit | Stephen Finucane proposed openstack/nova master: console: Provide an RFB security proxy implementation https://review.openstack.org/345399 | |
| 11:10:57 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Document TLS security setup for noVNC proxy https://review.openstack.org/500544 | |
| 11:13:58 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: test https://review.openstack.org/529519 | |
| 11:13:59 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add index(updated_at) on migrations table. https://review.openstack.org/531132 | |
| 11:14:01 | hrw | kashyap: https://bugs.launchpad.net/ubuntu/+source/libvirt/+bug/1673467 | |
| 11:14:02 | 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) | |
| 11:14:38 | hrw | 11:14 (228s) linaro@cb-r1-m1-c1n1:~$ uname -m;virsh domcapabilities --emulatorbin /usr/bin/qemu-system-aarch64 | grep host-model | |
| 11:14:41 | hrw | aarch64 | |
| 11:14:44 | hrw | <mode name='host-model' supported='no'/> | |
| 11:14:57 | openstackgerrit | Stephen Finucane proposed openstack/nova master: console: introduce framework for RFB authentication https://review.openstack.org/345397 | |
| 11:14:57 | openstackgerrit | Stephen Finucane proposed openstack/nova master: console: introduce the VeNCrypt RFB authentication scheme https://review.openstack.org/345398 | |
| 11:14:58 | openstackgerrit | Stephen Finucane proposed openstack/nova master: console: Provide an RFB security proxy implementation https://review.openstack.org/345399 | |
| 11:14:58 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Document TLS security setup for noVNC proxy https://review.openstack.org/500544 | |
| 11:16:22 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add index(updated_at) on migrations table. https://review.openstack.org/531132 | |
| 11:22:41 | kashyap | hrw: Thanks for the link | |
| 13:18:27 | yasemin_ | hi, i want to add to hyper-v compute node to ocata openstack system, i installed it, but it gives an error "Filter ImagePropertiesFilter returned 0 hosts" ? Do you have any idea? Could you help me ? | |
| 13:35:35 | alex_xu | cdent: the patch LGTM, just a small thing https://review.openstack.org/#/c/513526/17/nova/api/openstack/placement/schemas/allocation_candidate.py, but I won't block on it | |
| 13:38:02 | jaypipes | yasemin_: please see /topic. better to ask your question on the openstack@ mailing list with the [nova][hyper-v] subject markers. | |
| 13:39:14 | hrw | kashyap: custom/cortex-a53 fails :( | |
| 13:39:51 | yasemin_ | <jaypipes> okey thank you | |
| 13:40:46 | kashyap | hrw: Oops | |
| 13:42:24 | kashyap | (Okay; let's move this to OFTC, Virt channel.) | |
| 13:42:36 | cdent | alex_xu: I borrowed that get schema from nova's positive_integer in parameter_types, which has the same redundancies (presumably to protect against the query string parsing library changing from underneath it) | |
| 14:07:01 | alex_xu | cdent: the positive_integer is used for the request body, not just the query string | |
| 14:07:44 | efried | jaypipes: I was planning to answer mriedem's call for more details on nrp (http://lists.openstack.org/pipermail/openstack-dev/2018-January/125953.html). Unless you were already doing that? | |
| 14:08:00 | jaypipes | efried: no. please go ahead. | |
| 14:08:26 | jaypipes | efried: in the middle of trying to get the aggregate affinity series straightened up. should take me another few hours. | |
| 14:09:02 | efried | jaypipes: Roger. FYI, the main thing I'd be throwing you under the bus for is accomodating nrps in GET /allocation_candidates | |
| 14:09:21 | efried | (throwing under bus - was that the right analogy? mebbe not) | |
| 14:10:04 | jaypipes | efried: that's cool. | |
| 14:14:10 | stephenfin | 😃 | |
| 14:14:14 | stephenfin | Whoops | |
| 14:27:13 | openstackgerrit | Claudiu Belu proposed openstack/nova master: hyperv: Cleans up live migration Planned VM https://review.openstack.org/478943 | |
| 14:30:55 | openstackgerrit | Lajos Katona proposed openstack/nova master: Deduplicate aggregate notification samples https://review.openstack.org/531162 | |
| 14:37:15 | openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: libvirt: use 'host-passthrough' as default on AArch64 https://review.openstack.org/530965 | |
| 14:55:08 | openstackgerrit | Matthew Booth proposed openstack/nova master: Fix fake libvirt XML generation for disks https://review.openstack.org/531165 | |
| 14:58:05 | mriedem | kashyap: lyarwood: thanks for looking at https://review.openstack.org/#/c/267587/ - i think we're on the same page, | |
| 14:58:15 | mriedem | libvirt 3.10 has a fix for a regression introduced in qemu 2.10 | |
| 14:58:22 | kashyap | mriedem: I noticed your ping too last evening | |
| 14:58:32 | kashyap | I saw your sleuthing on the review | |
| 14:58:36 | mriedem | so i added the version check on startup such that multiattach isn't supported unless qemu<2.10 or libvirt>=3.10 | |
| 14:58:37 | kashyap | Yep | |
| 14:58:45 | mriedem | which sucks because we don't have those versions in our CI env | |
| 14:59:39 | kashyap | (Maybe we should finally get that utopic gate job that will run on newer versions to allow us to test at least) | |
| 14:59:56 | mriedem | we are using the pike UCA which is about as new as it gets for ubuntu | |
| 15:00:30 | mriedem | jamespage: coreycb: what do you think are the chances of getting these libvirt patches backported from 3.10 to 3.6 in the pike UCA? https://bugzilla.redhat.com/show_bug.cgi?id=1378242#c10 | |
| 15:00:31 | openstack | bugzilla.redhat.com bug 1378242 in libvirt "QEMU image file locking (libvirt)" [Unspecified,On_qa] - Assigned to pkrempa | |
| 15:00:56 | kashyap | Yep, I recall; mriedem -- I wonder if you've ever seen this (for DevStack, though) -- https://review.openstack.org/#/c/108714/10 | |
| 15:01:05 | mriedem | alternatively, i'm going to have to probably look at using a special job which doesn't use the pike UCA so we just use the xenial package versions which would have qemu<2.10 | |
| 15:01:41 | mriedem | kashyap: so i thought markus_z and tonyb already had a job that does something like this | |
| 15:02:00 | kashyap | mriedem: I vaguely recall talking to them too, at Barcelona, but it didn't go anywhere | |
| 15:02:17 | mriedem | i have the job name handy, sec | |
| 15:02:28 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/newton: Raise MarkerNotFound if BuildRequestList.get_by_filters doesn't find marker https://review.openstack.org/530982 | |
| 15:03:38 | kashyap | http://lists.openstack.org/pipermail/openstack-dev/2016-October/105552.html | |
| 15:03:50 | mriedem | kashyap: legacy-tempest-dsvm-nova-libvirt-kvm-apr | |
| 15:04:10 | kashyap | Thanks; let me look up the job definition | |
| 15:04:30 | mriedem | http://git.openstack.org/cgit/openstack-infra/openstack-zuul-jobs/tree/playbooks/legacy/tempest-dsvm-nova-libvirt-kvm-apr/run.yaml#n2 | |