Posted	Nick	Remark
#openstack-nova - 2020-12-17
17:11:05	gibi	if it doesn't really help then I can try to look at serializing them
17:11:32	gibi	but it would be better to merge stephen's series instead
17:12:20	gibi	anyhow this is the end of my day see you all tomorrow
17:12:26	stephenfin	o/
17:12:55	gibi	stephenfin: o/
17:57:09	gmann	gibi: yeah, we can check in advance specially if those new test needs to skip in multi-cell job
18:21:51	sean-k-mooney	artom: i see you added your self to https://review.opendev.org/c/openstack/nova/+/767368
18:22:19	sean-k-mooney	that is required before https://review.opendev.org/c/openstack/neutron/+/766277 can merge on the neutron side
18:22:31	sean-k-mooney	and we need to backport it to at least train
18:23:00	sean-k-mooney	its related to https://review.opendev.org/c/openstack/nova/+/602432 also
18:23:32	sean-k-mooney	we are currenly up to 6 pathces we need to backport to fix live migration
18:23:39	sean-k-mooney	3 on nova and 3 on neutron
18:24:06	sean-k-mooney	2 are merged on the neutron side already and 1 on the nova side
18:24:28	sean-k-mooney	if we need to go before train we will need to backport thing in os-vif also
18:25:25	sean-k-mooney	anyway im going to be calling it a day soon then going on PTO till january so ill respond it comments when im back if there are any
18:27:47	sean-k-mooney	gibi: just saw this in the conflicts list https://review.opendev.org/c/openstack/nova/+/709658 we proably should arcive that just to not distract peopel with it
18:29:41	sean-k-mooney	oh enjoy the break all o/ i may or may not be around over the next 2 weeks but im on pto until the 4th of january
18:55:16	lyarwood	\o enjoy either way, catch you in the new year
19:10:14	artom	sean-k-mooney, yeah, seemed like a thing I need to look at
19:10:43	artom	(Sorry for the lag, was doing some errands and getting a haircut, before Shutdown 2 Electric Boogalloo after Christmas)
19:11:04	sean-k-mooney	no worries
19:11:38	sean-k-mooney	tl;dr it only every worked because of buggy behavior in neutron that when fix means we need to actully wait properly in nova
19:12:02	sean-k-mooney	basically we were migration before because fo a network-vif-plugged event form the dhcp agent
19:12:13	sean-k-mooney	not form the l2 agent that we were waiting for
19:12:30	artom	sean-k-mooney, I suspect this'll be another "Artom and Sean spent 2 hours on GMeet" kind of situation :)
19:12:54	sean-k-mooney	about an hour and with radolfo but ya
19:13:23	sean-k-mooney	i had a suption this was true for a while but we found the atcual code that was at fault friday last week
19:14:16	sean-k-mooney	we have had a much of unxepcect event in the logs for a few releases and this is why many of them happened
19:14:42	sean-k-mooney	its all related to the multipe port binding work in rocky
22:12:21	openstackgerrit	Ghanshyam proposed openstack/placement master: Remove deprecated [placement]/policy_file config option https://review.opendev.org/c/openstack/placement/+/764409
22:12:45	openstackgerrit	Ghanshyam proposed openstack/placement master: [goal] Deprecate the JSON formatted policy file https://review.opendev.org/c/openstack/placement/+/764394
22:38:24	openstackgerrit	Ghanshyam proposed openstack/nova stable/rocky: DNM: testing gate https://review.opendev.org/c/openstack/nova/+/767027
22:38:57	openstackgerrit	melanie witt proposed openstack/nova master: Add functional regression test for bug 1853009 https://review.opendev.org/c/openstack/nova/+/695012
22:38:57	openstack	bug 1853009 in OpenStack Compute (nova) ussuri "Ironic node rebalance race can lead to missing compute nodes in DB" [High,In progress] https://launchpad.net/bugs/1853009 - Assigned to Mark Goddard (mgoddard)
22:38:57	openstackgerrit	melanie witt proposed openstack/nova master: Clear rebalanced compute nodes from resource tracker https://review.opendev.org/c/openstack/nova/+/695187
22:38:57	openstackgerrit	melanie witt proposed openstack/nova master: Invalidate provider tree when compute node disappears https://review.opendev.org/c/openstack/nova/+/695188
22:38:57	openstackgerrit	melanie witt proposed openstack/nova master: Prevent deletion of a compute node belonging to another host https://review.opendev.org/c/openstack/nova/+/694802
22:38:58	openstackgerrit	melanie witt proposed openstack/nova master: Fix inactive session error in compute node creation https://review.opendev.org/c/openstack/nova/+/695189
23:07:55	openstackgerrit	Lance Bragstad proposed openstack/placement master: Bump oslo.log version to 4.3.0 https://review.opendev.org/c/openstack/placement/+/760229
23:07:56	openstackgerrit	Lance Bragstad proposed openstack/placement master: Pass context objects to oslo.policy directly https://review.opendev.org/c/openstack/placement/+/766785
23:07:57	openstackgerrit	Lance Bragstad proposed openstack/placement master: Implement secure RBAC for resource providers https://review.opendev.org/c/openstack/placement/+/760240
#openstack-nova - 2020-12-18
01:13:35	openstackgerrit	Ghanshyam proposed openstack/placement master: [goal] Deprecate the JSON formatted policy file https://review.opendev.org/c/openstack/placement/+/764394
06:36:52	openstackgerrit	zhulipeng proposed openstack/nova master: Fixed multipath residue when evacuating the VM https://review.opendev.org/c/openstack/nova/+/767468
07:49:20	openstackgerrit	Wenping Song proposed openstack/nova-specs master: Libvirt supports composing cyborg owned vGPU accelerator into domain XML https://review.opendev.org/c/openstack/nova-specs/+/750116
08:26:19	gibi	sean-k-mooney: thanks for the headsup, I abandoned https://review.opendev.org/c/openstack/nova/+/709658 now
10:10:35	openstackgerrit	Elod Illes proposed openstack/nova stable/rocky: [stable-only] Cap bandit to 1.6.2 https://review.opendev.org/c/openstack/nova/+/766492
10:14:49	songwenping__	gibi, lyarwood: please merge the trival change patch https://review.opendev.org/c/openstack/nova/+/767449 replaces the old patch https://review.opendev.org/c/openstack/nova/+/759577. thanks.
10:15:46	lyarwood	songwenping__: what was wrong with the original?
10:17:17	songwenping__	the test cannot pass always.
10:20:09	lyarwood	songwenping__: it's the same change, all you need to do is rebase it.
10:21:03	songwenping__	ok, i try to fix the original again.
10:29:49	openstackgerrit	Wenping Song proposed openstack/nova master: Libvirt vGPU support https://review.opendev.org/c/openstack/nova/+/767694
10:42:57	openstackgerrit	Wenping Song proposed openstack/nova-specs master: Libvirt supports composing cyborg owned vGPU accelerator into domain XML https://review.opendev.org/c/openstack/nova-specs/+/750116
11:40:12	openstackgerrit	Wenping Song proposed openstack/nova master: Trival Change: Remove redundant code in instance delete https://review.opendev.org/c/openstack/nova/+/759577
11:59:50	openstackgerrit	Brin Zhang proposed openstack/nova master: WIP:Replaces tenant_id with project_id from Flavor Access APIs https://review.opendev.org/c/openstack/nova/+/767704
12:25:26	openstackgerrit	Elod Illes proposed openstack/nova stable/rocky: [stable-only] Cap bandit to 1.6.2 https://review.opendev.org/c/openstack/nova/+/766492
12:46:34	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Run the db migration tests in the same test worker https://review.opendev.org/c/openstack/nova/+/767590
12:47:01	gibi	stephenfin, sean-k-mooney: another try as increasing the timeout did not help ^^
14:13:44	openstackgerrit	Lee Yarwood proposed openstack/nova stable/rocky: [stable-only] Cap bandit to 1.6.2 https://review.opendev.org/c/openstack/nova/+/766492
15:24:20	spatel	ubuntu lxc container doesn't have vi/vim/pico/nano any kind of editor.
15:47:55	gibi	stephenfin: left questions in https://review.opendev.org/c/openstack/nova/+/758395
15:53:17	spatel	Seriously ubuntu 20.04 make rc.local in systemd :(
16:29:41	openstackgerrit	Ghanshyam proposed openstack/placement master: [goal] Deprecate the JSON formatted policy file https://review.opendev.org/c/openstack/placement/+/764394
16:35:07	gibi	I'm leaving for PTO now. I will be back on 4th of Jan
17:13:54	gmann	o/
#openstack-nova - 2020-12-19
02:21:44	openstackgerrit	Ghanshyam proposed openstack/placement master: [goal] Deprecate the JSON formatted policy file https://review.opendev.org/c/openstack/placement/+/764394
04:47:25	openstackgerrit	Brin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from Flavor Access APIs https://review.opendev.org/c/openstack/nova/+/767704
05:35:27	openstackgerrit	Ghanshyam proposed openstack/placement master: [goal] Deprecate the JSON formatted policy file https://review.opendev.org/c/openstack/placement/+/764394
06:28:05	openstackgerrit	Merged openstack/nova stable/ussuri: [stable-only] fix lower-constraints and disable qos resize https://review.opendev.org/c/openstack/nova/+/766738
08:18:22	openstackgerrit	Merged openstack/nova stable/ussuri: Fix a hacking test https://review.opendev.org/c/openstack/nova/+/765082
08:21:07	openstackgerrit	Takashi Natsume proposed openstack/nova stable/train: Fix a hacking test https://review.opendev.org/c/openstack/nova/+/767793
16:20:44	openstackgerrit	Merged openstack/nova stable/victoria: Add missing exception https://review.opendev.org/c/openstack/nova/+/763389
16:39:20	openstackgerrit	Merged openstack/nova stable/ussuri: Handle disabled CPU features to fix live migration failures https://review.opendev.org/c/openstack/nova/+/758761
#openstack-nova - 2020-12-20
12:59:13	openstackgerrit	Lee Yarwood proposed openstack/nova master: WIP run-evacuate-hook: Use osc to evacuate instances https://review.opendev.org/c/openstack/nova/+/767984
#openstack-nova - 2020-12-21
10:34:22	stephenfin	lyarwood: Morning. Think we could get this remove-six series closed out this week while things are quiet? https://review.opendev.org/q/topic:%2522bp/six-removal%2522+status:open+project:openstack/nova
11:14:40	lyarwood	stephenfin: ack sorry ssh dropped, I'll look at that shortly
12:22:33	stephenfin	lyarwood: Thanks :)
14:37:42	gmann	stephenfin: can you check these two placement's APi policy work https://review.opendev.org/c/openstack/placement/+/764409 https://review.opendev.org/c/openstack/placement/+/764394/13
14:45:29	stephenfin	gmann: sure, will do
14:45:40	gmann	stephenfin: thanks
14:55:57	openstackgerrit	Merged openstack/nova master: Reproduce bug 1907522 in functional test https://review.opendev.org/c/openstack/nova/+/766791
14:55:57	openstack	bug 1907522 in OpenStack Compute (nova) "test_*_with_qos_min_bw_allocation fails in the nova-multi-cell job with: nova.exception.MigrationPreCheckError: Migration pre-check error: Failed to create port bindings for host " [High,In progress] https://launchpad.net/bugs/1907522 - Assigned to Balazs Gibizer (balazs-gibizer)
14:58:45	openstackgerrit	Merged openstack/nova master: Fallback to same-cell resize with qos ports https://review.opendev.org/c/openstack/nova/+/766925
15:44:18	ikla	is there anything special to overcommit ram/cpu on a hosts?
15:44:30	ikla	is this all handled in the flavor?
16:23:09	openstackgerrit	Lee Yarwood proposed openstack/nova master: api-ref: Add notes about volume attach and detach being async https://review.opendev.org/c/openstack/nova/+/768104
16:23:45	lyarwood	stephenfin: ^ no idea how I should flag these APIs as async, would appreciate a review if you have time this week.
16:57:50	openstackgerrit	Merged openstack/nova master: Bump the lowest eventlet version to 0.26.1 https://review.opendev.org/c/openstack/nova/+/761427
17:12:58	openstackgerrit	Artom Lifshitz proposed openstack/nova master: WIP: objects: Add `socket` to NUMACell and PciDevice https://review.opendev.org/c/openstack/nova/+/766795
17:12:59	openstackgerrit	Artom Lifshitz proposed openstack/nova master: WIP: Start populating `socket_id` in host NUMA topology https://review.opendev.org/c/openstack/nova/+/766816
17:26:06	stephenfin	ikla: Depends on what version of nova you're using and whether we're talking new or existing compute nodes. If it's pre-Newton, you'll simply want to set 'ram_allocation_ratio' and 'cpu_allocation_ratio' on the compute nodes
17:28:11	stephenfin	If it's any later, you can set 'initial_ram_allocation_ratio' and 'initial_cpu_allocation_ratio' to seed new compute nodes but that won't override what's in placement for existing compute nodes. You can either update placement manually (e.g. using osc-placement) or override that by setting the old 'ram_allocation_ratio' and 'cpu_allocation_ratio' values
18:08:30	stephenfin	gmann: Left a -1 on the first one for mistake in the doc. Easily fixed though. +2 on the other
18:09:04	stephenfin	and +2 on that doc patch, lyarwood. Makes sense to me. We should probably make the HTTP 202 = async link more obvious in general but I have no clever ideas there either
18:11:02	gmann	stephenfin: that is stein upgrade doc so did not change to oslo.policy.policy_file - https://review.opendev.org/c/openstack/placement/+/764409/4/doc/source/admin/upgrade-to-stein.rst#125
18:11:44	gmann	I had to change it with non config option format as this config option is removed now and lead to config option not found error if kept as it is
18:11:55	stephenfin	gmann: Ah, in that case I think you meant ``[placement] policy_file``
18:12:05	stephenfin	not ``[oslo.config].placement.policy_file``
18:12:14	gmann	sure, I can update that
18:13:19	openstackgerrit	Ghanshyam proposed openstack/placement master: Remove deprecated [placement]/policy_file config option https://review.opendev.org/c/openstack/placement/+/764409

Earlier   Later