| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-05-18 | |||
| 04:45:37 | openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add snapshot function https://review.openstack.org/534240 | |
| 04:45:38 | openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340 | |
| 04:45:39 | openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add get console output https://review.openstack.org/543344 | |
| 06:01:30 | openstackgerrit | Tsuyoshi Nagata proposed openstack/nova master: nova improvement of maximum attach volumes more than 26 vols https://review.openstack.org/567472 | |
| 06:54:58 | openstackgerrit | Robin Naundorf proposed openstack/nova master: Fix inconsistency in docs https://review.openstack.org/569348 | |
| 07:32:31 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Microversion 2.63 - Use new format policy in server group https://review.openstack.org/567534 | |
| 07:37:25 | openstackgerrit | Robin Naundorf proposed openstack/nova master: Fix inconsistency in docs https://review.openstack.org/569348 | |
| 08:10:20 | kashyap | Hey folks, anyone got a Windows Nova guest XML? If so, mind posting it somewhere? | |
| 08:32:18 | openstackgerrit | sahid proposed openstack/nova master: pci: don't consider case when match tags specs https://review.openstack.org/565808 | |
| 08:32:19 | openstackgerrit | sahid proposed openstack/nova master: network: update pci request spec to handle trusted tags https://review.openstack.org/458820 | |
| 08:32:20 | openstackgerrit | sahid proposed openstack/nova master: libvirt: configure trust mode for vfs https://review.openstack.org/458514 | |
| 08:32:21 | openstackgerrit | sahid proposed openstack/nova master: libvirt: add vf_trusted field for network metadata https://review.openstack.org/566343 | |
| 08:32:22 | openstackgerrit | sahid proposed openstack/nova master: metadata: add vf_trusted field to device metadata https://review.openstack.org/566344 | |
| 08:52:25 | openstackgerrit | sahid proposed openstack/nova master: libvirt: place emulator threads on CONF.compute.cpu_shared_set https://review.openstack.org/510897 | |
| 09:01:17 | openstackgerrit | Naichuan Sun proposed openstack/nova master: WIP: xenapi: drop deprecated vif_driver config option https://review.openstack.org/567877 | |
| 09:08:02 | openstackgerrit | Naichuan Sun proposed openstack/nova master: WIP: xenapi: drop deprecated vif_driver config option https://review.openstack.org/567877 | |
| 09:11:07 | openstackgerrit | Naichuan Sun proposed openstack/nova master: WIP: xenapi: drop deprecated vif_driver config option https://review.openstack.org/567877 | |
| 09:22:21 | lyarwood | mdbooth: random question, do you recall if we have a fake block_device_info creating method somewhere in test_driver.py? I was sure we did but I can't seem to find the damn thing. | |
| 09:26:48 | openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313 | |
| 09:26:49 | openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations` https://review.openstack.org/521717 | |
| 09:26:50 | openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): support compute node resource provider update https://review.openstack.org/521041 | |
| 09:29:00 | mdbooth | lyarwood: There are a ton of tests which create block_device_info | |
| 09:29:31 | mdbooth | lyarwood: IIRC they usually create a BDM list and pass it to the relevant function in driver | |
| 09:30:14 | mdbooth | lyarwood: You can crib a BDM list from somewhere. | |
| 09:31:01 | lyarwood | mdbooth: ah right, that makes sense | |
| 09:33:09 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: libvirt: Configure multiple vGPU types https://review.openstack.org/564214 | |
| 09:33:10 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: libvirt: mdevs returning parent and vendor PCI info https://review.openstack.org/562304 | |
| 09:33:11 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: libvirt: Get inventories per vGPU type https://review.openstack.org/564215 | |
| 10:13:58 | mdbooth | lyarwood: Did you find something to crib, btw? | |
| 10:14:49 | lyarwood | mdbooth: yeah, trying to do a few things at once here but I'll push it in a mind. | |
| 10:14:51 | lyarwood | min* | |
| 10:15:25 | mdbooth | lyarwood: kk. Was going to dig out a specific if you hadn't found anything. | |
| 10:16:34 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Skip fetching the virtual size of block devices https://review.openstack.org/569062 | |
| 10:16:41 | lyarwood | mdbooth: ^ for your reviewing pleasure if you have time | |
| 10:17:34 | mdbooth | lyarwood: hehe | |
| 10:17:59 | lyarwood | pep8-- it passed but missed something there | |
| 10:18:02 | lyarwood | quick respin | |
| 10:18:32 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Skip fetching the virtual size of block devices https://review.openstack.org/569062 | |
| 10:19:22 | mdbooth | lyarwood: https://network-data-cabling.co.uk/wp-content/uploads/cable-spaghetti.jpg What do you mean you unplugged the wrong thing? | |
| 10:19:41 | lyarwood | ^_^ | |
| 10:21:26 | mdbooth | I like to compare changing the libvirt driver to playing Mikado. | |
| 10:31:06 | kashyap | lyarwood: Oops: https://review.openstack.org/#/c/569062/ | |
| 10:37:02 | lyarwood | kashyap: yeah, happens to us all :) | |
| 10:38:30 | kashyap | lyarwood: Of course; was just teasing :-) | |
| 12:33:44 | bauzas | efried: when you're up, got a sec ? | |
| 12:33:52 | bauzas | it's about https://review.openstack.org/#/c/560444/ | |
| 13:37:13 | Shilpa | andreykurilin: Hi | |
| 13:37:21 | andreykurilin | hi | |
| 13:38:19 | Shilpa | andreykurilin: i am working on https://bugs.launchpad.net/python-novaclient/+bug/1744118 | |
| 13:38:21 | openstack | Launchpad bug 1744118 in python-novaclient "Arguments not initialized if unsupported by keystoneauth plugin" [Medium,Confirmed] | |
| 13:39:15 | Shilpa | updated novaclient/shell.py, and further facing issue of 404 | |
| 13:40:16 | Shilpa | ERROR (NotFound): (http://10.232.48.204/identity/tokens): The resource could not be found. (HTTP 404) | |
| 13:40:53 | andreykurilin | Shilpa: link to a patch? so I can see the modified code | |
| 13:41:50 | Shilpa | as of now not uploaded any patch | |
| 13:42:00 | Shilpa | will create paste docuemnt for you, wait | |
| 13:43:53 | Shilpa | http://paste.openstack.org/show/721234/, here diff is added | |
| 13:51:28 | gibi | mriedem: hi!, you marked https://bugs.launchpad.net/placement-osc-plugin/+bug/1771761 as affecting stable/queens but it seems that the trait tests does not existing there | |
| 13:51:29 | openstack | Launchpad bug 1771761 in placement-osc-plugin "trait functional tests fail due to changed error message from placement" [Medium,In progress] - Assigned to Balazs Gibizer (balazs-gibizer) | |
| 13:51:50 | mriedem | oh right, good point | |
| 13:51:54 | mriedem | removed | |
| 13:53:55 | kashyap | melwitt: Sorry, was occupied yesterday, I see you've sorted out the disk limit discussion with DanPB | |
| 13:54:03 | kashyap | 17:55 < kashyap> fetch = +refs/heads/*:refs/remotes/cgit/* | |
| 13:54:03 | kashyap | 17:55 < kashyap> url = https://git.openstack.org/openstack/nova | |
| 13:54:03 | kashyap | 17:55 < kashyap> [remote "cgit"] | |
| 13:54:13 | kashyap | Urgh, wrong copy / paste | |
| 13:54:27 | kashyap | I meant this: https://rwmj.wordpress.com/2017/04/25/how-many-disks-can-you-add-to-a-virtual-linux-machine/ | |
| 13:58:10 | andreykurilin | Shilpa: I never worked with custome auth plugins, so I can just guess what is happening. Possibly, you have keystone v2 disabled (it sounds ok), but your checks for auth_url contain error and instead of calling v3 endpoint, it tries to use v2. Another possible option is that https://github.com/openstack/python-novaclient/blob/master/novaclient/shell.py#L603-L604 returns the default auth plugin for some reason and not your custom one | |
| 13:58:40 | dansmith | kashyap: yeah, I think it's well-understood that you can technically attach a bunch | |
| 13:59:07 | kashyap | But I see we were looking for a sensible upper limit in Nova's context | |
| 13:59:25 | dansmith | kashyap: I'm kinda dubious on the reasoning for nova, and the current silly limitation _does_ prevent you from having a large tenant with a lot of volume quota attaching a large number to a single guest and completely killing the host machine | |
| 14:00:15 | kashyap | dansmith: Current limitation is 26, right? | |
| 14:00:18 | Shilpa | andreykurilin: Thank you for explaination here, will go through again and get back to you next week | |
| 14:00:33 | dansmith | mriedem: that skipping of the extra weigh/filter step is breaking a regression test in a strange way, because of the server group filter | |
| 14:00:36 | kashyap | dansmith: Yeah, I do agree with that; and I had a chuckle when I first saw a version of the upstream patch adding 2 billion as the limit! | |
| 14:00:40 | dansmith | mriedem: when I'm off my next call I'd like to discuss | |
| 14:02:24 | mriedem | ok, i'm on a call now too | |
| 14:14:21 | openstackgerrit | Eric Berglund proposed openstack/nova stable/pike: Stringify instance UUID https://review.openstack.org/569421 | |
| 14:24:27 | gibi | stephenfin: would it be possible to do the refactoring in https://review.openstack.org/#/c/564444/ separately from the main implementation series? I would need to do something similar to resolve the uglyness of https://review.openstack.org/#/c/567268/5/nova/compute/api.py@811 | |
| 14:26:46 | mriedem | dansmith: oh on https://review.openstack.org/#/c/569127/ i gotcha | |
| 14:27:07 | dansmith | yah | |
| 14:56:14 | dansmith | mriedem: yeah, so I removed the index>0 check and replaced it with a num_alts check, | |
| 14:56:32 | dansmith | which makes sense I think.. even if we have num_instances=3, if we aren't getting alts, we don't need to re-sort even if index=2 | |
| 14:56:54 | dansmith | but if I do that, it breaks your reschedule regression test for server groups, only because the reschedule filters those out | |
| 14:57:08 | mriedem | s/your/mel's/ | |
| 14:57:12 | dansmith | which I think means we had a bug already, but didn't notice it because we wouldn't re-sort the host list for that test where we only had one | |
| 14:57:13 | mriedem | but yeah, i just left some comments | |
| 14:57:16 | mriedem | and was looking at the failing test | |
| 14:57:18 | dansmith | oh, heh, I expected it was you | |
| 14:57:35 | bauzas | gibi: stephenfin is already traveling AFAIK | |
| 14:57:38 | mriedem | so that test doesn't do multi-create, and it doesn't mess with max attempts, | |
| 14:57:45 | mriedem | only triggers a reschedule | |
| 14:57:46 | bauzas | and me is about to disappear now | |
| 14:57:50 | gibi | bauzas: thanks for the info | |
| 14:58:03 | mriedem | dansmith: is this 100% fail or intermittent | |
| 14:58:18 | bauzas | folks visiting YVR, see you next week | |
| 14:58:41 | dansmith | mriedem: 100% | |
| 14:58:47 | mriedem | dansmith: ok, reading back on https://review.openstack.org/#/c/513931/ | |
| 14:58:53 | bauzas | gibi: that's why I bail out earlier | |
| 14:58:58 | gibi | bauzas: safe travell | |
| 14:59:07 | gibi | bauzas: see you in YVR | |