Posted	Nick	Remark
#openstack-nova - 2020-03-10
15:13:03	mriedem	oh, well i'll +2 then
15:55:23	openstackgerrit	Stephen Finucane proposed openstack/nova stable/train: Fix os-keypairs pagination links https://review.opendev.org/711895
15:56:52	openstackgerrit	Stephen Finucane proposed openstack/nova stable/stein: Fix os-keypairs pagination links https://review.opendev.org/711896
15:58:13	openstackgerrit	Stephen Finucane proposed openstack/nova stable/rocky: Fix os-keypairs pagination links https://review.opendev.org/711898
16:06:32	openstackgerrit	Sylvain Bauza proposed openstack/nova master: Pass allocations to virt drivers when resizing https://review.opendev.org/589085
16:06:33	openstackgerrit	Sylvain Bauza proposed openstack/nova master: Pass allocations to virt drivers when reverting resize https://review.opendev.org/712118
16:41:49	stephenfin	lyarwood: dead easy stable patch for you here https://review.opendev.org/711895
16:48:25	openstackgerrit	Stephen Finucane proposed openstack/nova master: Add transform_image_metadata request filter https://review.opendev.org/665775
16:58:51	lyarwood	stephenfin: are there many more of these?
16:59:00	stephenfin	I think that's the only other one
16:59:12	lyarwood	kk
16:59:20	stephenfin	it's the only one I could spot anyway
17:33:51	openstackgerrit	Monty Taylor proposed openstack/nova master: Run sdk functional tests on nova changes https://review.opendev.org/712092
17:45:25	openstackgerrit	John Garbutt proposed openstack/nova master: WIP: Enforce unified limits using oslo.limit https://review.opendev.org/615180
17:45:25	openstackgerrit	John Garbutt proposed openstack/nova master: Add unified limits configuration https://review.opendev.org/712137
17:45:26	openstackgerrit	John Garbutt proposed openstack/nova master: Add tests for the UnifiedLimits quota driver https://review.opendev.org/712138
17:45:26	openstackgerrit	John Garbutt proposed openstack/nova master: Add logic to enforce local api and db limits https://review.opendev.org/712139
17:45:27	openstackgerrit	John Garbutt proposed openstack/nova master: WIP: Add tests for API using unified limits driver https://review.opendev.org/712140
17:45:27	openstackgerrit	John Garbutt proposed openstack/nova master: Make unified limits APIs return reserved of 0 https://review.opendev.org/712141
17:45:28	openstackgerrit	John Garbutt proposed openstack/nova master: WIP: Enforce api and db limits https://review.opendev.org/712142
17:45:28	openstackgerrit	John Garbutt proposed openstack/nova master: WIP: Update quota_class APIs for db and api limits https://review.opendev.org/712143
19:54:16	donnyd	sean-k-mooney: can you kick off a multi-numa job if you have anything that can be - I was hoping to test to make sure it works
19:59:39	sean-k-mooney	donnyd: just rechecked https://review.opendev.org/#/c/708977/
20:07:21	sean-k-mooney	donnyd: seams to be working fine http://zuul.openstack.org/stream/a34b85c571ed47db8c4f1a4b2f1a0d82?logfile=console.log
20:50:24	donnyd	Woot woot
21:11:06	openstackgerrit	Monty Taylor proposed openstack/nova master: Run sdk functional tests on nova changes https://review.opendev.org/712092
21:54:42	openstackgerrit	Merged openstack/nova stable/stein: Fix hypervisors paginted collection_name. https://review.opendev.org/711669
#openstack-nova - 2020-03-11
00:28:23	openstackgerrit	Merged openstack/nova master: Don't overwrite greenthread-local context in host manager https://review.opendev.org/582332
00:29:12	melwitt	rm_work: if you're around, do you recall what version you tried your test with re: your comment on this bug? https://bugs.launchpad.net/nova/+bug/1863190
00:29:12	openstack	Launchpad bug 1863190 in OpenStack Compute (nova) "Server group anti-affinity no longer works" [Undecided,New]
01:22:03	openstackgerrit	Matt Riedemann proposed openstack/nova stable/queens: Add retry_on_deadlock to migration_update DB API https://review.opendev.org/712212
02:43:25	openstackgerrit	melanie witt proposed openstack/nova master: Add config option for neutron client retries https://review.opendev.org/712226
05:07:26	openstackgerrit	Merged openstack/nova stable/queens: libvirt: Ignore DiskNotFound during update_available_resource https://review.opendev.org/711279
05:36:21	rm_work	melwitt: so that was on rocky -- but we figured out that we had build retries set to 0 so it would never retry, thus breaking that logic
05:37:34	rm_work	And with soft antiaffinity, obviously it would just "work" and so given certain packing schemes, we're almost guaranteed to have them land on the same host, which seems kinda dumb
05:38:27	rm_work	The conclusion we came to is that relying on the retry behavior to fix schedule races did not seem like a good approach
05:38:59	rm_work	And that there should be better more up front ways of handling that kind of locking
05:39:33	rm_work	Octavia does similar locking using DB logic
06:19:35	rm_work	melwitt: johnsom could detail some of that more probably
10:49:39	openstackgerrit	Stephen Finucane proposed openstack/nova master: vif: Remove dead code https://review.opendev.org/703146
10:50:48	stephenfin	bauzas: Any chance you could blast these through this morning? https://review.opendev.org/#/c/703973/ https://review.opendev.org/#/c/703974/
10:51:11	stephenfin	Final two patches for nova-net removal
10:52:36	stephenfin	bauzas: and if you're doing that, there's another dead easy one here https://review.opendev.org/#/c/686997/
10:52:51	openstackgerrit	Lee Yarwood proposed openstack/nova master: images: Move qemu-img info calls into privsep https://review.opendev.org/706897
10:52:52	openstackgerrit	Lee Yarwood proposed openstack/nova master: images: Allow the output format of qemu-img info to be controlled https://review.opendev.org/706898
10:52:52	openstackgerrit	Lee Yarwood proposed openstack/nova master: virt: Pass request context to extend_volume https://review.opendev.org/706899
10:52:53	openstackgerrit	Lee Yarwood proposed openstack/nova master: libvirt: Correctly resize encrypted LUKSv1 volumes https://review.opendev.org/706900
10:55:40	lyarwood	sean-k-mooney: ^ removed run_as_root and made it clear that the new func is privileged
10:56:56	openstackgerrit	Lee Yarwood proposed openstack/nova master: DNM - Test TEMPEST_EXTEND_ATTACHED_ENCRYPTED_VOLUME https://review.opendev.org/707593
10:58:22	bauzas	stephenfin: will try
11:11:06	openstackgerrit	Brin Zhang proposed openstack/nova master: Add new default roles in os-instance-actions policies https://review.opendev.org/706470
11:56:42	sean-k-mooney	lyarwood: yep reviewing your series now
11:56:59	sean-k-mooney	and ya its clearer what you are doing
12:02:14	lyarwood	sean-k-mooney: ack thanks
12:11:26	lyarwood	stephenfin: ^ when you have time later could you hit https://review.opendev.org/#/q/topic:bug/1861071 again, I'll rebase the followups once this lands now to avoid extra churn in the gate.
12:12:02	stephenfin	sure. ping me again if I don't have it done by EOD :)
12:12:23	lyarwood	sure thing
12:12:33	lyarwood	and thanks obviously :)
12:19:27	sean-k-mooney	lyarwood: i think i reviewed them all
12:19:46	sean-k-mooney	it looks like the chain is broken into 3 seriese so they shoudl all be rebased back into one
12:20:30	sean-k-mooney	or at least be rebased so they are update with the bottom 4 patches
12:30:23	lyarwood	sean-k-mooney: yup, I'll rebase the rest later today, just switching to something else for a whil.e
12:46:39	brinzhang	johnthetubaguy: https://review.opendev.org/#/c/706470/ I used your suggestion, that looks like ok for me, I was update that you can review again if you have time ^^
12:48:20	brinzhang	gmann: you can review this patch, and move the problem from your list I asked before
13:06:38	luyao	gibi: are you around?
13:07:10	openstackgerrit	Brin Zhang proposed openstack/nova master: Granular GET os-instance-actions API policies https://review.opendev.org/711791
13:07:14	gibi	luyao: hi!
13:08:07	luyao	gibi: Would you like review the vpmem live migration support? I need someone help me confirm if the implementations is OK. :D
13:08:30	luyao	https://review.opendev.org/#/q/topic:support-live-migration-with-virtual-persistent-memory+(status:open+OR+status:merged)
13:08:59	gibi	luyao: sure. I can try. I have to finish the review of the cyborg integration first but then I can look at your series
13:11:42	luyao	gibi: Thank you. looking forward for your comments. FYI. I don't add enough testcases now, if the implementation is acceptable, I'll do that.
13:12:12	gibi	luyao: OK, I will keep that in mind
13:12:42	brinzhang	luyao: I will tyr to check your patch tomorrow ^^
13:13:12	luyao	brinzhang: Thanks, welcome. :D
13:13:37	brinzhang	luyao: I already add that to me review list, and will do later
13:14:40	brinzhang	We would like this feature
13:16:13	luyao	brinzhang: Great.
13:26:17	openstackgerrit	Balazs Gibizer proposed openstack/nova master: [Community goal] Update contributor documentation https://review.opendev.org/712420
13:27:20	gibi	stephenfin: ussuri community goal doc change ^^
13:27:30	gibi	stephenfin: I feel you will have comments about it
13:53:13	openstackgerrit	Sylvain Bauza proposed openstack/nova master: Pass allocations to virt drivers when reverting resize https://review.opendev.org/712118
14:16:55	dansmith	gibi: thoughts on the vmware deprecation patch? I'm glad it got some attention raised, but we've been a long time without vmware ci and despite some response, I feel like we probably need a deadline or something to make sure something happens
14:17:50	dansmith	I'm not even sure when the last time the vmware ci commented on a nova change, but I feel like it's been a *long* time
14:23:58	sean-k-mooney	dansmith: we could merge the patch propose a revert and hold it until m3. if the ci is running by m3 we merge the revert if not then leave it
14:24:31	dansmith	yep, we could also convert this to just a quality warning, merge that now, and hold the deprecation warning on the ci
14:24:35	dansmith	which i think is what we did for xen
14:25:47	gmann	brinzhang: i will check after coffee.
14:26:10	brinzhang	gmann: cool, thanks
14:30:22	mriedem	we merged the deprecation warning for xen in train, it wasn't contingent
14:33:36	dansmith	okay when I was looking for examples, I found a "merge quality warning" for one of the drivers before a deprecation
14:33:46	gibi	dansmith: we can merge a deprecation warning without causing extra problem to hemna and undeprecate if vmware ci magicly shows up
14:33:47	mriedem	looks like dan's patch has at least gotten the necessary attention,
14:33:53	mriedem	whether that holds or not i guess we'll see
14:34:15	dansmith	gibi: ack, well, that's what I've got up, so maybe go vote on that :)
14:34:16	gibi	the real break happens if we delete something
14:34:25	gibi	dansmith: will do
14:34:32	mriedem	yeah the people -1ing the change are saying "please don't delete it"
14:34:35	mriedem	which isn't what the patch does
14:34:52	dansmith	mriedem: I quote you: " After three months since the quality warning change merged [1]
14:34:52	dansmith	there has still been no progress in finding a maintainer for"

Earlier   Later