| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-04-17 | |||
| 10:38:36 | openstackgerrit | sahid proposed openstack/nova master: libvirt: add support for virtio-net rx/tx queue sizes https://review.openstack.org/484997 | |
| 10:38:51 | sahid | sean-k-mooney: ^ I preferd to fix the test | |
| 10:39:30 | sean-k-mooney | sahid: cool :) as i said it was only a minor nit but it nice to use valid values in the tests | |
| 10:40:57 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608 | |
| 10:40:58 | sahid | sean-k-mooney: yes yes, just i think we should try to stay coherent even in tests :) | |
| 10:41:06 | sahid | it was a good point | |
| 10:41:10 | kashyap | stephenfin: Hi there. Is it too unreasonable to replace Mox with Mock, while fixing the fallout from libvirt / QEMU version bump: http://kashyapc.fedorapeople.org/0001-tests-unit-Fix-test_live_migration_update_graphics_x.patch | |
| 10:41:41 | kashyap | stephenfin: When you can, read the commit message, and the first point in the "main changes". | |
| 10:42:04 | sahid | sean-k-mooney: about https://review.openstack.org/#/c/553072/ | |
| 10:42:10 | stephenfin | kashyap: I personally think that's fine, given that the test was having issues as is | |
| 10:42:18 | kashyap | Yeah, thank you! | |
| 10:42:27 | sahid | matt had some questions, i tried to reply but perhaps you want to give more details | |
| 10:42:31 | kashyap | stephenfin: I spent 2 hours fixing two tests. I'm slow as molasses with this stuff | |
| 10:42:34 | kashyap | Thanks | |
| 10:43:18 | stephenfin | bauzas: Did I hear something about us deprecating AggregateCoreFilter ? | |
| 10:43:40 | bauzas | stephenfin: hola, sorry saw your late ping | |
| 10:43:46 | bauzas | stephenfin: but I was running :p | |
| 10:43:54 | stephenfin | bauzas: All good :) | |
| 10:44:10 | bauzas | stephenfin: the aggregate filters ? well, it's a long story :) | |
| 10:45:11 | bauzas | stephenfin: AFAIK, the aggregate filters aren't deprecated *yet* | |
| 10:45:27 | bauzas | but there was a discussion about providing that feature by the Placement API | |
| 10:47:40 | stephenfin | bauzas: OK, so not gone yet. I'll tackle this issue another way so | |
| 10:49:15 | openstackgerrit | Martin Midolesov proposed openstack/nova master: [vmwareapi] Expose cpu model on homogeneous cluster; Reformating code; https://review.openstack.org/561901 | |
| 10:53:39 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add host/hostId to instance action events API https://review.openstack.org/557145 | |
| 10:58:16 | openstackgerrit | Merged openstack/nova stable/queens: Skip placement on rebuild in same host https://review.openstack.org/561014 | |
| 11:21:17 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483 | |
| 11:23:55 | mdbooth | lyarwood: Done | |
| 11:25:04 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Improve performance when list instances with IP filter https://review.openstack.org/539469 | |
| 11:27:05 | lyarwood | mdbooth: thanks | |
| 11:27:24 | mdbooth | lyarwood: Took me a while to re-load the context :) | |
| 11:27:52 | openstackgerrit | Martin Midolesov proposed openstack/nova master: [vmwareapi] Expose cpu model on homogeneous cluster; Refactoring code; https://review.openstack.org/561588 | |
| 11:28:33 | lyarwood | mdbooth: yeah, I did come across https://bugs.launchpad.net/nova/+bug/1510328 from danpb btw, might try and follow up with that once this lands | |
| 11:28:33 | openstack | Launchpad bug 1510328 in OpenStack Compute (nova) "Nova pre-allocation of qcow2 is flawed" [Low,Confirmed] | |
| 11:29:26 | mdbooth | lyarwood: IIRC that's a pita to do without refactoring the imagebackend. | |
| 11:29:54 | lyarwood | mdbooth: oh joy, well maybe not then | |
| 11:30:32 | mdbooth | lyarwood: Look into it, though. I'm pretty jaded about most of that code, so I may not be looking at it fresh. | |
| 11:32:22 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/ocata: Modify incorrect debug meaasge in _inject_data https://review.openstack.org/519950 | |
| 11:36:31 | openstackgerrit | Hamdy Khader proposed openstack/nova master: Adding NVMEoF for libvirt driver https://review.openstack.org/482640 | |
| 11:40:01 | openstackgerrit | sahid proposed openstack/nova master: libvirt: place emulator threads on CONF.cpu_share_set https://review.openstack.org/510897 | |
| 11:40:02 | openstackgerrit | sahid proposed openstack/nova master: compute: introduce cpu_share_set https://review.openstack.org/561912 | |
| 11:40:03 | openstackgerrit | sahid proposed openstack/nova master: libvirt: pass emulator threads policy insteadwhether it's isolated https://review.openstack.org/561913 | |
| 11:41:09 | sahid | jaypipes: I have introduce the cpu_share_set by this serie ^ you might be interested. I tryed to limit the scope to my spec and I expect you to update the option for usage of cpu-resource spec | |
| 11:45:25 | openstackgerrit | sahid proposed openstack/nova-specs master: fix blueprint related to the spec https://review.openstack.org/561916 | |
| 11:55:32 | kashyap | mdbooth: Following up from our yesterday's discussion, I fixed the tests | |
| 11:55:46 | kashyap | mdbooth: And I had to use Mock, I'm afraid. The tests were already broken / failing | |
| 11:56:02 | kashyap | mdbooth: You can see in this small commit message: https://kashyapc.fedorapeople.org/0001-tests-unit-Fix-test_live_migration_update_graphics_x.patch | |
| 11:59:11 | jaypipes | sahid: rock on, thank you sir :) | |
| 12:19:17 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Deduplicate notification samples Rocky - 7 https://review.openstack.org/560748 | |
| 12:21:08 | openstackgerrit | Sylvain Bauza proposed openstack/nova stable/queens: ironic: Get correct inventory for deployed node https://review.openstack.org/561923 | |
| 12:21:54 | jaypipes | jianghuaw_: still up? | |
| 12:23:43 | openstackgerrit | Sylvain Bauza proposed openstack/nova stable/pike: ironic: Get correct inventory for deployed node https://review.openstack.org/561925 | |
| 12:26:25 | bauzas | stable cores, looks like we break some TripleO CI, and we need https://review.openstack.org/#/q/I6717ce19f6005c8ebb7af75437a72876c5a53f34 to be accepted for helping them | |
| 12:26:48 | bauzas | free beers offered | |
| 12:28:44 | openstackgerrit | Sylvain Bauza proposed openstack/nova stable/pike: ironic: Get correct inventory for deployed node https://review.openstack.org/561925 | |
| 12:38:44 | openstackgerrit | Merged openstack/nova stable/pike: Fix wrapping of neutron forbidden error https://review.openstack.org/560087 | |
| 12:41:30 | lyarwood | bauzas: free beer you say? | |
| 12:41:53 | bauzas | yup, free as in 'I pay' | |
| 12:42:05 | bauzas | Vancouver is in 1 month | |
| 12:42:25 | bauzas | and even if I have a gold fish mind, I will still remember :) | |
| 12:42:39 | mdbooth | lyarwood: As long as you meet in person at a specific bar in France, location to be revealed no more than 1 hour in advance. | |
| 12:43:26 | bauzas | well, if in France, no beer | |
| 12:43:33 | bauzas | Chartreuse instead | |
| 12:44:15 | mdbooth | bauzas: I've never had that... | |
| 12:44:53 | bauzas | again, https://en.wikipedia.org/wiki/Chartreuse_(liqueur) | |
| 12:45:51 | mdbooth | bauzas: Next time I'm in France :) | |
| 12:46:18 | bauzas | mdbooth: didn't you tried the Génépi last time you were there ? | |
| 12:47:15 | mdbooth | bauzas: Not that I specifically recall, although there were drinks I drank without necessarily knowing what they were :) | |
| 12:47:25 | bauzas | that's sad | |
| 12:47:36 | bauzas | you just had half the experience | |
| 12:47:46 | bauzas | skiing is one thing | |
| 12:48:00 | bauzas | after-skiing is another thing | |
| 12:48:39 | mdbooth | Apres-ski for me is mostly food and bed :( | |
| 12:48:58 | openstackgerrit | Merged openstack/nova stable/queens: only increment disk address unit for scsi devices https://review.openstack.org/561196 | |
| 12:49:49 | bauzas | mdbooth: tss tss, no excuse | |
| 12:49:53 | stephenfin | bauzas: Alternatively, go to 🇦🇹 where the beer is plentiful and the <something decidedly PC about the opposite sex> | |
| 12:50:07 | bauzas | I do ski with 70-yr old people | |
| 12:50:32 | bauzas | actally, skiing on Fridays out of vacation periods make you consider young on the slopes... | |
| 12:50:54 | mdbooth | bauzas: They've all been skiing since they were 2, though | |
| 12:51:25 | bauzas | nah | |
| 12:51:46 | bauzas | some decided to retire in the mountains | |
| 12:51:57 | bauzas | like new-age monks | |
| 12:52:27 | bauzas | one good way of praying : take a 60% slope | |
| 12:53:15 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Add skip_latest_microversion decorator https://review.openstack.org/433585 | |
| 12:53:16 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Run api sample tests against 2.latest https://review.openstack.org/430352 | |
| 12:57:36 | openstackgerrit | Konstantinos Samaras-Tsakiris proposed openstack/nova master: Add `hide_hypervisor_id` flavor extra_spec https://review.openstack.org/555861 | |
| 13:04:41 | mriedem | gibi: question in https://review.openstack.org/#/c/460625/ | |
| 13:10:15 | kosamara | mriedem I fixed the nitpicks and created a blueprint | |
| 13:12:55 | Shilpa | stephenfin: hi | |
| 13:13:11 | stephenfin | Shilpa: o/ | |
| 13:13:33 | Shilpa | stephenfin: i was going through temst failure logs for https://review.openstack.org/#/c/550172 | |
| 13:14:12 | Shilpa | stephenfin: Also checked at http://codesearch.openstack.org/?q=vnc_auto.html&i=nope&files=&repos= | |
| 13:14:49 | Shilpa | stephenfin: here there are multiple projects referring novncproxy_base_url | |
| 13:15:04 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Add CPUWeigher https://review.openstack.org/379525 | |
| 13:15:40 | Shilpa | stephenfin: do we need to inform respective projects to modify the same, or what action we need to take here? | |
| 13:15:46 | gibi | mriedem: the versioned notification works based on the values of the Aggregate ovo so if we want to send the update.start notification with the already updated data then we have to update the Aggregate ovo before the start notification. But that would mean the start notification is sent _after_ the actual update so it is not a start notification any more | |
| 13:16:12 | gibi | mriedem: a bit of chicken and egg | |
| 13:17:12 | stephenfin | Shilpa: We probably do, yes. However, why does that affect the Tempest job? | |
| 13:17:15 | mriedem | gibi: well, or add a 'new_metadata' field to the aggregate payload, something like that | |
| 13:17:21 | Shilpa | stephenfin: is because of any refrence we are missing and cauing temst failure for https://review.openstack.org/#/c/550172? | |
| 13:17:23 | mriedem | or 'metadata_updates' | |