Posted	Nick	Remark
#openstack-nova - 2020-10-09
16:44:18	sean-k-mooney	ok we might want to update the downstream bz to say the input device cant be removed but we are goign to do this instead
16:44:53	stephenfin	I don't think the PS2 device is a massive issue. It's the USB bus that was doing most of the damage
16:45:01	stephenfin	But I should check again to be sure
16:45:10	stephenfin	Monday! For now, it's sunny outside so o/ :)
16:45:51	sean-k-mooney	i guess thre expect result was actully "no USB and no graphics"
16:46:09	sean-k-mooney	so ps2 device or virtio might be fine actully
16:46:18	sean-k-mooney	based on https://bugzilla.redhat.com/show_bug.cgi?id=1787442
16:46:18	openstack	bugzilla.redhat.com bug 1787442 in documentation ""Configuring Real-Time Compute" lacks guest device configuration tuning" [Unspecified,Closed: wontfix] - Assigned to stephenfin
16:50:53	admin0	sean-k-mooney, i made it blank and it still gives me a .novalocal
16:51:00	admin0	is that something from neutron in dhcp ?
16:51:47	sean-k-mooney	no i think since novalocal is the default of that coning option you have to do =""
16:51:55	sean-k-mooney	i dont think blank will actully work
16:52:40	sean-k-mooney	melwitt: do you recal if ye set dhcp_domain= or dhcp_domain=""
16:52:56	melwitt	dhcp_domain=""
16:53:07	melwitt	set to empty string
16:53:30	melwitt	sorry, that's what I meant when I said "blank" earlier
16:55:21	sean-k-mooney	ya ok that makes sense
16:55:39	sean-k-mooney	dhcp_domain= i think will be treated like its unset and the default will be used
16:56:06	sean-k-mooney	admin0: this might only take effect for new vms too
16:56:35	admin0	unless i can modify the dhcp config manually
16:56:38	sean-k-mooney	for existing vm i think we store there hostname in the db
16:57:18	sean-k-mooney	you would need to update the instnace.hostname filed in the db
16:57:21	sean-k-mooney	https://github.com/openstack/nova/blob/master/nova/objects/instance.py#L130
18:00:08	openstackgerrit	melanie witt proposed openstack/nova stable/queens: [stable-only] Add functional test for bug 1731668 https://review.opendev.org/756636
18:00:08	openstack	bug 1731668 in OpenStack Compute (nova) queens "placement: claim allocations fails with IndexError in _ensure_lookup_table_entry" [Low,In progress] https://launchpad.net/bugs/1731668 - Assigned to melanie witt (melwitt)
18:00:08	openstackgerrit	melanie witt proposed openstack/nova stable/queens: [stable-only] Use a separate transaction for reading after race https://review.opendev.org/756637
18:03:20	openstackgerrit	melanie witt proposed openstack/nova stable/queens: [stable-only] Add functional test for bug 1731668 https://review.opendev.org/756636
18:03:20	openstack	bug 1731668 in OpenStack Compute (nova) queens "placement: claim allocations fails with IndexError in _ensure_lookup_table_entry" [Low,In progress] https://launchpad.net/bugs/1731668 - Assigned to melanie witt (melwitt)
18:03:21	openstackgerrit	melanie witt proposed openstack/nova stable/queens: [stable-only] Use a separate transaction for reading after race https://review.opendev.org/756637
18:03:43	melwitt	omg uuughh
18:05:27	openstackgerrit	melanie witt proposed openstack/nova stable/queens: [stable-only] Add functional test for bug 1731668 https://review.opendev.org/756636
18:05:27	openstack	bug 1731668 in OpenStack Compute (nova) queens "placement: claim allocations fails with IndexError in _ensure_lookup_table_entry" [Low,In progress] https://launchpad.net/bugs/1731668 - Assigned to melanie witt (melwitt)
18:05:28	openstackgerrit	melanie witt proposed openstack/nova stable/queens: [stable-only] Use a separate transaction for reading after race https://review.opendev.org/756637
18:06:19	melwitt	sorry for the spam
19:05:31	openstackgerrit	melanie witt proposed openstack/nova stable/train: Add a workaround config toggle to refuse ceph image upload https://review.opendev.org/757177
19:47:44	melwitt	mnaser: is ^ something you would find useful/appropriate for sta
19:47:55	melwitt	*stable/train or nah?
#openstack-nova - 2020-10-12
01:00:45	openstackgerrit	Jorhson Deng proposed openstack/nova master: add task state for attach volume and detach volume https://review.opendev.org/754258
05:56:04	openstackgerrit	MaAoyu proposed openstack/os-traits master: bump py37 to py38 in tox.ini https://review.opendev.org/757432
06:27:41	openstackgerrit	zhanghao proposed openstack/nova master: Fix virsh domifstat to get vhostuser vif statistics https://review.opendev.org/757448
07:00:36	gibi	good morning nova
07:26:51	openstackgerrit	Merged openstack/nova master: Follow up for cherry-pick check for merge patch https://review.opendev.org/756639
08:23:34	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Revert "Temporarily disable parts of heal port allocation test" https://review.opendev.org/755995
08:25:25	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Use cell targeted context to query BDMs for metadata https://review.opendev.org/752459
08:25:33	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Clean up the DynamicVendorData constructor https://review.opendev.org/752718
08:26:53	gibi	stephenfin: hi! if you want to start your week with a simple patch then I suggest https://review.opendev.org/#/c/754124/ :)
08:36:11	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Move query param generation to RequestGroup o.vo https://review.opendev.org/756894
08:36:12	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Refactor _claim_pci_device_for_interface_attach to prepare for qos https://review.opendev.org/756895
08:36:12	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Allow extending an existing instance allocation https://review.opendev.org/757109
08:37:36	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Make remove allocation symmetric with add allocation https://review.opendev.org/757110
08:38:59	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Factor out port resource deallocation https://review.opendev.org/757111
08:43:08	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Support interface attach with qos ports https://review.opendev.org/756530
09:00:16	stephenfin	gibi: sure, done
09:09:55	openstackgerrit	Merged openstack/nova-specs master: Move implemented specs https://review.opendev.org/754124
09:16:18	gibi	stephenfin: thanks
09:29:03	openstackgerrit	Wenping Song proposed openstack/python-novaclient master: Cleanup py27 support https://review.opendev.org/757513
09:40:22	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: Fix issue in exception type https://review.opendev.org/756069
09:40:22	openstackgerrit	Stephen Finucane proposed openstack/nova master: functional: Add tests for mixed CPU policy https://review.opendev.org/755852
09:40:23	openstackgerrit	Stephen Finucane proposed openstack/nova master: functional: Add test for #1898272 https://review.opendev.org/756100
09:40:23	openstackgerrit	Stephen Finucane proposed openstack/nova master: hardware: Check inventory of shared CPUs for 'mixed' policy https://review.opendev.org/756101
09:40:24	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: Add 'InstanceNUMACell.min_threads' https://review.opendev.org/756070
09:42:35	stephenfin	lyarwood, gibi: What do I do about https://review.opendev.org/#/c/631038/ now? The issue is still there for libvirt < 4.3.0, but our min supported version on master is now 5.0.0
09:43:07	stephenfin	lyarwood, gibi: i.e. should I propose as stable only or simply drop it?
09:44:53	openstackgerrit	Stephen Finucane proposed openstack/nova master: Make overcommit check for pinned instance pagesize aware https://review.opendev.org/631053
09:46:45	stephenfin	lyarwood, gibi: Also, could I get that related fix merged? ^ I'm going to rework that function (again) but want to get that and https://review.opendev.org/#/c/756101/ in first so I can backport
09:47:19	gibi	stephenfin: https://review.opendev.org/#/c/631038/ is a bugfix that is not effecting master. It think it is totally OK to propose it to stable branches as stable-only fix
09:48:10	stephenfin	great; coming right up
09:58:02	openstackgerrit	Stephen Finucane proposed openstack/nova stable/ussuri: [Stable Only] Handle libvirt reporting incorrect 4k page quantities https://review.opendev.org/757524
09:58:19	stephenfin	gibi, lyarwood: One stable-only patch awaits ^ :)
09:58:32	gibi	stephenfin: thanks
10:38:08	gibi	stephenfin: I have question in https://review.opendev.org/#/c/631053/9//COMMIT_MSG@18
10:38:25	stephenfin	looking
10:43:29	lyarwood	stephenfin: ack sorry stuck downstream with something today
10:48:35	stephenfin	gibi: replied; tl;dr: on x86 hosts with no explicit mempage requests in the guest, nova will always use standard (4k) pages
10:48:54	stephenfin	i.e. it'll never consume 2M or 1G pages unless the guest explicitly requests them
10:49:11	sean-k-mooney	also replied
10:49:42	sean-k-mooney	we do have a hw:mem_page_size=any that was ment to allow nova to use any page size but we did nto fully implement it
10:50:38	sean-k-mooney	not specifying hw:mem_page_size is different then setting hw:mem_page_size=small or hw:mem_page_size=4k explcitly
10:50:51	gibi	OK, thanks this helps
10:51:00	sean-k-mooney	when unset nova will ignore numa affintiy and allow over commit
10:51:21	sean-k-mooney	if set to any value including 4k or small over commit is disabeld and numa affinity is enforced
10:52:12	sean-k-mooney	hw:mem_page_size=any today means use 4k small pages unless the image asks for a page size then do what the image says
10:52:45	sean-k-mooney	the orginal intent of hw:mem_page_size=any was to use the smallest page size that would fit
10:53:04	sean-k-mooney	unless the image requested one
10:53:16	sean-k-mooney	thats the bit we never implemented
10:55:01	sean-k-mooney	stephenfin: by the way this brings me back to the topic of defualting to hw:mem_page_size=small when ever we have a numa toplogoy and it is not set
10:55:39	sean-k-mooney	we actully proably would want to default to hw:mem_page_size=any
10:56:12	sean-k-mooney	but ya i need to go work on that stable abi spec
11:27:09	openstackgerrit	zhanghao proposed openstack/nova master: Fix virsh domifstat to get vhostuser vif statistics https://review.opendev.org/757448
11:42:24	openstackgerrit	Wenping Song proposed openstack/python-novaclient master: Remove Babel requirement https://review.opendev.org/757541
12:33:44	openstackgerrit	Merged openstack/python-novaclient master: Cleanup py27 support https://review.opendev.org/757513
12:50:23	openstackgerrit	Wenping Song proposed openstack/python-novaclient master: Remove Babel from lower-constraints.txt https://review.opendev.org/757541
13:20:32	gibi	dansmith, melwitt, gmann: responded in https://review.opendev.org/#/c/731396
14:01:52	openstackgerrit	Andrew Bonney proposed openstack/nova master: Handle disabled CPU features to fix live migration failures https://review.opendev.org/757577
14:02:23	sean-k-mooney	does anyone remember why we use psycopg2 and not psycopg2-binary in our test requirements
14:03:06	sean-k-mooney	its come up a few times before but it would be nice if we could swap to the binary wheel instead of having to instlal postgress dev libs to build the python bindingsx
15:46:18	openstackgerrit	Merged openstack/python-novaclient master: Remove Babel from lower-constraints.txt https://review.opendev.org/757541
15:53:52	openstackgerrit	Mark Goddard proposed openstack/nova master: Fix PCI passthrough race on reschedule (claims) https://review.opendev.org/710847
16:13:26	openstackgerrit	melanie witt proposed openstack/nova stable/victoria: Follow up for cherry-pick check for merge patch https://review.opendev.org/757613
16:14:06	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Refactor _claim_pci_device_for_interface_attach to prepare for qos https://review.opendev.org/756895

Earlier   Later