Posted	Nick	Remark
#openstack-nova - 2019-06-17
16:51:28	mriedem	er >0 to 0
17:24:18	openstackgerrit	Matt Riedemann proposed openstack/nova master: Remove file-backed memory live migration compat check https://review.opendev.org/665728
18:14:40	openstackgerrit	Adam Spiers proposed openstack/nova master: Use fake flavor instead of empty dict in test https://review.opendev.org/662555
18:14:41	openstackgerrit	Adam Spiers proposed openstack/nova master: Pass extra_specs to flavor in vif tests https://review.opendev.org/662556
18:14:41	openstackgerrit	Adam Spiers proposed openstack/nova master: Extract SEV-specific bits on host detection https://review.opendev.org/636334
18:14:42	openstackgerrit	Adam Spiers proposed openstack/nova master: Add element to guest config for AMD SEV https://review.opendev.org/636318
18:14:42	openstackgerrit	Adam Spiers proposed openstack/nova master: Allow guest devices to include https://review.opendev.org/644564
18:14:43	openstackgerrit	Adam Spiers proposed openstack/nova master: Detect that SEV is required and enable iommu for devices https://review.opendev.org/644565
18:14:43	openstackgerrit	Adam Spiers proposed openstack/nova master: Use element when SEV is required https://review.opendev.org/662557
18:14:44	openstackgerrit	Adam Spiers proposed openstack/nova master: Enable memory locking if SEV is requested https://review.opendev.org/662558
18:21:57	openstackgerrit	Harald Jensås proposed openstack/nova master: cell_mapping - wrap IPv6 address in format_url https://review.opendev.org/664559
18:25:41	sean-k-mooney	stephenfin: just an fyi the kolla-ansibel change for removing nova-consoleauth just merged https://review.opendev.org/#/c/661427/
18:26:13	sean-k-mooney	Jeffrey4l: thanks for completing ^
18:29:39	hjensas	dansmith: Thanks for the review on ^^, you where right and I want to apologize for the 'did you look at the bug' comment. wdyt about the new revision?
18:30:10	dansmith	hjensas: you're still changing the existing format I think, but I'm in the middle of something else.. will comment in a few
18:30:42	hjensas	dansmith: thanks, appriciated.
18:41:10	mriedem	efried: would you agree with this? https://review.opendev.org/#/c/661237/9/nova/compute/api.py@1000
18:41:22	efried	mriedem: looking
18:41:24	mriedem	also, there is something really strange about the mock assertions he's making in this change which are passing but i'm not sure why
18:41:26	mriedem	you might have ideas
18:43:27	efried	mriedem: I agree with your comment on PlacementAPIConnectError. Commenting...
18:44:40	openstackgerrit	Harald Jensås proposed openstack/nova master: cell_mapping - wrap IPv6 address in format_url https://review.opendev.org/664559
18:48:14	efried	mriedem: I can play with that mock later on if you want me to.
18:48:47	mriedem	i just did http://paste.openstack.org/show/753112/ and it's failing if i don't use the yielded cctxt
18:48:51	mriedem	so i left a note about just doing that
18:49:06	efried	Been in The Worst Chair in Taylor all day so far - power has been out at my house and this coffee shop clearly does not want to encourage long stays.
18:49:20	efried	Apparently the power is back on, so I'll be moving shortly :)
18:49:38	mriedem	but how have all of the overly loud overly personal conversations around you been going?
18:49:45	efried	okay, I posted my comment agreeing with you about the exception
18:49:53	efried	heh, quite entertaining.
18:50:29	efried	Old folks talking about how things ain't how they used to be.
18:50:29	efried	A job interview.
18:50:29	efried	A married couple fighting.
18:50:29	efried	Kids with a baby (was it theirs?)
19:09:37	sean-k-mooney	gibi: is https://review.opendev.org/#/c/665715/1 for testing or do you intend it to actuly be a feature in the future?
19:26:43	openstackgerrit	Matt Riedemann proposed openstack/nova master: Deprecate RetryFilter https://review.opendev.org/663953
19:26:43	openstackgerrit	Matt Riedemann proposed openstack/nova master: Fix enabled_filters default value in admin config docs https://review.opendev.org/665751
19:27:41	openstackgerrit	Matt Riedemann proposed openstack/nova master: Deprecate RetryFilter https://review.opendev.org/663953
19:47:19	openstackgerrit	Dustin Cowles proposed openstack/nova master: Introduces SDK to IronicDriver and uses for node.get https://review.opendev.org/642899
19:47:20	openstackgerrit	Dustin Cowles proposed openstack/nova master: Use SDK instead of ironicclient for node.list https://review.opendev.org/656027
19:47:20	openstackgerrit	Dustin Cowles proposed openstack/nova master: Use SDK instead of ironicclient for validating instance and node https://review.opendev.org/656028
19:47:21	openstackgerrit	Dustin Cowles proposed openstack/nova master: Use SDK instead of ironicclient for setting instance id https://review.opendev.org/659690
19:47:21	openstackgerrit	Dustin Cowles proposed openstack/nova master: WIP: Use SDK instead of ironicclient for add/remove instance info from node https://review.opendev.org/659691
20:42:20	openstackgerrit	Merged openstack/nova-specs master: Update spec: filtering of alloc candidates by forbidden aggregates https://review.opendev.org/665605
21:24:28	openstackgerrit	melanie witt proposed openstack/nova master: Log quota legacy method warning only if counting from placement https://review.opendev.org/665765
21:26:43	melwitt	mriedem: I realized I accidentally made the log warning happen in cases it shouldn't in the counting quotas follow up patch https://review.opendev.org/#/c/662056/5/nova/quota.py@1315 fix is ^
21:30:32	mriedem	melwitt: small thing in there
21:31:20	melwitt	heh, thanks. I am just constantly doing that, adding the else: argh
21:32:30	openstackgerrit	melanie witt proposed openstack/nova master: Log quota legacy method warning only if counting from placement https://review.opendev.org/665765
21:32:35	mriedem	it tooks years of https://pmd.github.io/ forced on me
21:33:19	melwitt	oh, neat
21:34:54	sean-k-mooney	huh i didnt know there was an opensouce tool like that
21:35:27	sean-k-mooney	i guess it makes sense but the only cross lanaguage cross plathform static analiser i knew of were closed souce
21:35:48	openstackgerrit	Matt Riedemann proposed openstack/nova master: Clean up NumInstancesFilter related docs https://review.opendev.org/665768
22:39:41	Nick_A	Trying to inject a password into a kvm guest using libvirt. nova.conf is set up with inject_partition = -1, inject_password = True, inject_key = True. Keys are working but not passwords. What might we be missing?
22:47:02	melwitt	Nick_A: in addition to what's described in https://docs.openstack.org/nova/latest/admin/admin-password-injection.html I think you have to have the qemu guest agent package installed on the guest. not sure why that's not in the doc tho
22:52:04	melwitt	and also hw_qemu_guest_agent=yes image property https://docs.openstack.org/nova/latest/admin/configuration/hypervisor-kvm.html#guest-agent-support
22:53:17	Nick_A	melwitt we actually tried that already - it allows for updating of the password once the instance exists, but we can't seem to set it upon creation/rebuild
22:55:36	melwitt	Nick_A: that seems unexpected. there might be a bug, or maybe I'm just not aware there's something additional needed for it to work for create/rebuild
22:59:37	openstackgerrit	sean mooney proposed openstack/nova master: create initial image metadata prefilter https://review.opendev.org/665775
23:14:00	Nick_A	Even the random password generated when we manually openstack server create doesn't work...
23:18:28	openstackgerrit	Sam Morrison proposed openstack/nova stable/rocky: Fix double word hacking test https://review.opendev.org/665788
23:19:07	openstackgerrit	Sam Morrison proposed openstack/nova stable/queens: Fix double word hacking test https://review.opendev.org/665790
23:45:45	openstackgerrit	Merged openstack/nova master: Don't rely on SQLAlchemy collections magically initializing __dict__ https://review.opendev.org/661472
23:45:52	openstackgerrit	Merged openstack/nova master: docs: remove the RamFilter from example https://review.opendev.org/665546
23:51:29	openstackgerrit	sean mooney proposed openstack/os-traits master: add libvirt image metadata traits https://review.opendev.org/652996
23:51:32	openstackgerrit	sean mooney proposed openstack/os-traits master: add GOP, NONE and VIRTIO grapics models https://review.opendev.org/665830
#openstack-nova - 2019-06-18
01:03:44	openstackgerrit	Adam Spiers proposed openstack/nova master: Add extra spec parameter and image property for memory encryption https://review.opendev.org/664420
01:03:45	openstackgerrit	Adam Spiers proposed openstack/nova master: Use fake flavor instead of empty dict in test https://review.opendev.org/662555
01:03:45	openstackgerrit	Adam Spiers proposed openstack/nova master: Pass extra_specs to flavor in vif tests https://review.opendev.org/662556
01:03:46	openstackgerrit	Adam Spiers proposed openstack/nova master: Extract SEV-specific bits on host detection https://review.opendev.org/636334
01:03:46	openstackgerrit	Adam Spiers proposed openstack/nova master: Add element to guest config for AMD SEV https://review.opendev.org/636318
01:03:47	openstackgerrit	Adam Spiers proposed openstack/nova master: Allow guest devices to include https://review.opendev.org/644564
01:03:47	openstackgerrit	Adam Spiers proposed openstack/nova master: Detect that SEV is required and enable iommu for devices https://review.opendev.org/644565
01:03:48	openstackgerrit	Adam Spiers proposed openstack/nova master: Use element when SEV is required https://review.opendev.org/662557
01:03:48	openstackgerrit	Adam Spiers proposed openstack/nova master: Enable memory locking if SEV is requested https://review.opendev.org/662558
01:52:16	openstackgerrit	Merged openstack/os-vif stable/queens: Prevent "qbr" Linux Bridge from replying to ARP messages https://review.opendev.org/655694
02:12:31	openstackgerrit	sean mooney proposed openstack/nova master: add StorageTransform class to prefilter https://review.opendev.org/665844
06:39:37	gibi	sean-k-mooney: regarding os-vif and osprofiler. The idea come up in neutron to profile the timing of vif plug. To be able to do that we need to make sure the osprofiler info is transferred to neutron during the vif plug. So far the patch is a PoC which is missing the neutron part. In the long run we might want to have support for osprofiling vif plug in some way
06:45:56	gibi	sean-k-mooney: it was also discussed yesterday on the neutron performance meeting http://eavesdrop.openstack.org/meetings/neutron_performance/2019/neutron_performance.2019-06-17-16.00.log.html#l-20
07:45:09	openstackgerrit	Yongli He proposed openstack/nova-specs master: add spec "show-server-numa-topology" https://review.opendev.org/658716
07:49:15	openstackgerrit	Takashi NATSUME proposed openstack/nova master: Fix wrong assert methods https://review.opendev.org/665897
07:58:30	openstackgerrit	Yongli He proposed openstack/nova master: Add server sub-resource topology API https://review.opendev.org/621476
08:21:20	openstackgerrit	Hamdy Khader proposed openstack/nova master: OVS DPDK port representors support https://review.opendev.org/658785
09:21:10	bhagyashris__	stephenfin, sean-k-mooney, alex_xu: Hi, I have checked the replies on mailing list and encounter in issue so replied to mailing thread http://lists.openstack.org/pipermail/openstack-discuss/2019-June/007190.html I would like to hear your opinion.
09:26:04	openstackgerrit	Boxiang Zhu proposed openstack/nova master: Validate requested host/node during servers create https://review.opendev.org/661237
09:26:04	openstackgerrit	Boxiang Zhu proposed openstack/nova master: Add host and hypervisor_hostname flag to create server https://review.opendev.org/645520
09:30:12	openstackgerrit	Brin Zhang proposed openstack/python-novaclient master: Microversion 2.74: Support Specifying AZ to unshelve https://review.opendev.org/665136
09:33:50	stephenfin	bhagyashris__: Cool, I'll reply now. What email client are you using, btw? It's not quoting replies properly, whatever it is /o\
09:35:37	bhagyashris__	stephenfin: actaully I am using outlook in browser so it's not quoting properly, sorry for inconvenience :(
09:36:24	stephenfin	bhagyashris__: All good. I was going to point this out (hard learned lesson from my Intel days) but it only applies to the desktop version https://that.guru/blog/sane-outlook/
09:42:15	bhagyashris__	stephenfin: ohh ok. Thank you !
09:57:33	bhagyashris__	stephenfin: Got your reply thank you! Just one question if operator set only ``cpu_shared_set`` then in that case what should happen? Will it be reported as VCPU inventory or it should raise error?
09:58:05	stephenfin	It should be ignored for VCPU calculation and a warning raised
09:58:05	bhagyashris__	On Train
09:58:37	stephenfin	If cpu_dedicated_set is not defined, then cpu_shared_set should only be used for emulator threads
09:59:46	stephenfin	However, I think I addressed this in a previous email by saying it was extremely unlikely that cpu_shared_set would be defined without vcpu_pin_set, since it wouldn't offer any advantage (the emulator thread host cores would overlap with the pinned CPU host cores)
10:01:04	bhagyashris__	stephenfin: no I am talking about if operator sets only ``cpu_shared_set`` in Train on fresh compute node then it should report the VCPU inventory as per the specs
10:01:24	bhagyashris__	Right?
10:01:54	stephenfin	Then the spec needs a modification. They also need to set cpu_dedicated_set, even if it's to an empty string

Earlier   Later