Posted	Nick	Remark
#openstack-nova - 2020-10-09
16:41:58	stephenfin	because it's useless otherwise
16:42:20	stephenfin	and the fourth adds support for virtio devices because they're supposed to be far more performant and it was an easy win
16:43:00	stephenfin	Agreed. However, I can't remove it until you add that system metadata thing for Q35
16:43:37	sean-k-mooney	:( ya
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

Earlier   Later