| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-04 | |||
| 10:37:54 | ildikov | kashyap: thanks for looking into it | |
| 10:38:52 | kashyap | ildikov: No worries; Matt already did the sluething last night (I was away early yesterday). W.r.t versioning of the 'shareable' lock | |
| 10:39:23 | ArchiFleKs | Hi, this may sound like a trivial question but do you have more information about how nova and neutron dns worked together, I'm using neutron as a dns server, which work fine, I can have instances with .openstacklocal domain or a custom domain, but on ths instances itself, the hostname always get sets to .novalocal anyway, is that a behavior that will change ultimatly ? If I set dhcp_domain to an | |
| 10:39:24 | ArchiFleKs | empty string in nova, will it solve this ? | |
| 10:39:38 | ildikov | kashyap: yeah, I gave up around midnight my time too... :/ :) | |
| 10:39:55 | kashyap | Midnight...Sounds a bit too late :-) | |
| 10:40:22 | kashyap | lyarwood: No action needed; you were prompted because you were the current bug asignee | |
| 10:40:30 | lyarwood | hmmm I don't see what the qemu-img change has to do with this | |
| 10:41:02 | ildikov | kashyap: it's just such a bummer as I remember chatting with Daniel Berrange back at the time about the 'shareable' flag and the support for that is in for ages and kinda thought that the virt layer is a done deal and we "only" need to figure out Cinder and Nova itself... | |
| 10:41:36 | kashyap | lyarwood: It's due to the the versioning in Gate; just have to have some conditional logic in Nova | |
| 10:42:29 | kashyap | ildikov: BTW, some of the image locking stuff was a bit more recent (as recent as last month -- https://bugzilla.redhat.com/show_bug.cgi?id=1378242#c18) | |
| 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 | |