| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-04-17 | |||
| 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' | |
| 13:18:46 | mriedem | gibi: granted, the legacy notification is a bit dumb in that it sends the start notification with the updates but it also sends the end notification with the same updates, but not the full aggregate meta, so the legacy notification is also arguably incomplete | |
| 13:19:20 | mriedem | since i don't know how these are consumed, i don't want to rathole on it too much and that's why i didn't -1 | |
| 13:19:43 | Shilpa | stephenfin: i am still finding root cause of tempst failure, and it was failed for for patch set 1 too, but missing logs there since got over written because of patch set 2 | |
| 13:19:44 | mriedem | if the consumer wants/needs to know the delta, i guess they can diff the metadata from the start/end notifications | |
| 13:20:13 | stephenfin | Shilpa: That configuration value definitely works if you test manually? | |
| 13:20:47 | stephenfin | Shilpa: Hmm, looks like we're not passing the token correctly http://logs.openstack.org/72/550172/2/check/tempest-full/f6945b6/controller/logs/screen-n-novnc.txt.gz | |
| 13:22:12 | gibi | mriedem: I would go for the solution where any update.start notification sends information about the state _before_ the update and the update.end sends the new state in the payload. In some cases we deviated from that and added the diff directly to the payload instance.update old_task_sate new_task_state but there we don't have start and end notification to use | |
| 13:22:17 | Shilpa | stephenfin: i have checked these logs and here seems token value is empty | |
| 13:24:23 | Shilpa | stephenfin: from where we can confirm, vnc 1.0.0 get downloaded successfully? i checked http://logs.openstack.org/72/550172/2/check/tempest-full/f6945b6/job-output.json.gz, but how once can confirm downloading of latest VNC version is successful? | |
| 13:24:55 | mriedem | gibi: yeah, +W | |
| 13:25:21 | gibi | mriedem: I think the proposed versioned aggregate.update_metadata is a good start. If somebody needs the explicity diff in the payload we can add that later | |
| 13:25:24 | gibi | mriedem: thanks | |
| 13:25:59 | gibi | mriedem: btw, I will have to go offline for a while this afternoon but I will be back for the notification subteam meeting | |
| 13:26:02 | openstackgerrit | Matthew Booth proposed openstack/nova master: Rename recreate to evacuate in driver signatures https://review.openstack.org/560900 | |
| 13:26:15 | stephenfin | Shilpa: Not a clue, I'm afriad | |
| 13:32:52 | sahid | mriedem: still not convinced about https://review.openstack.org/#/c/553072/ ? | |
| 13:33:54 | Shilpa | stephenfin: one more findings, http://logs.openstack.org/72/550172/2/check/tempest-full/f6945b6/controller/logs/screen-n-cond-cell1.txt.gz, here novncproxy_base_url is not reflected what is expected, still referring vvnc_auto.html | |
| 13:34:09 | mriedem | sahid: just forgot about it | |
| 13:34:42 | sahid | mriedem: oh ok no worries | |
| 13:37:29 | Shilpa | stephenfin: i have seen workflow -1, what fix in nova you are expecting here? | |
| 13:38:30 | stephenfin | Shilpa: I noticed all the existing examples used 'path=websockify/' (i.e. added the '/') | |
| 13:38:47 | stephenfin | Just curious to see if that affects things. Perhaps the urlparse module needs it | |
| 13:38:55 | stephenfin | I'm just poking stuff | |
| 13:41:14 | Shilpa | stephenfin: ok, will wiat for Zuul , thank you. | |
| 13:44:48 | bauzas | mriedem: any help would be appreciated for https://review.openstack.org/#/c/561923/ | |
| 13:44:57 | bauzas | mriedem: looks like it's blocking some internal CI | |
| 13:50:07 | mriedem | bauzas: want to remove the note from the commit message? https://review.openstack.org/#/c/561925 | |
| 13:50:41 | dansmith | sahid: should these be vhost-user in the config help string as well? https://review.openstack.org/#/c/484997/19/nova/conf/libvirt.py | |