Posted	Nick	Remark
#openstack-nova - 2020-04-03
07:33:47	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Delete ARQs by UUID if Cyborg ARQ bind fails. https://review.opendev.org/716186
07:48:43	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing server password policies https://review.opendev.org/717204
08:00:01	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Fix server metadata policy to be admin_or_owner https://review.opendev.org/717189
08:05:18	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Fix server password policy to be admin_or_owner https://review.opendev.org/717212
08:07:06	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing server metadata policies https://review.opendev.org/717182
08:10:45	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing server metadata policies https://review.opendev.org/717182
08:18:13	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in server password https://review.opendev.org/717215
08:19:04	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in server metadata https://review.opendev.org/717215
08:20:07	luyao	stephenfin: Hi are you about?
08:36:19	brinzhang	gibi: https://review.opendev.org/#/c/716185/5 Sundar added the accelerator docs in api-guide, but there is an issue for api-ref, and releasenotes, I use "tox -e api-ref" and "tox -e releasenotes" tested in my local env, that run successed, but failed by zuul check :(
08:40:30	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in server metadata policies https://review.opendev.org/717255
08:41:53	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing server metadata policies https://review.opendev.org/717182
08:42:12	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in server metadata https://review.opendev.org/717215
08:42:30	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in server metadata policies https://review.opendev.org/717255
08:43:33	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing server password policies https://review.opendev.org/717204
08:46:20	gibi	brinzhang: reno is generated with tox -e docs and the other job that is failed is not api-ref bug api-guide
08:47:08	gibi	s/bug/but/
08:54:05	brinzhang	gibi: got it, but also not get why failed
08:54:41	brinzhang	gibi: did you looked into ?
08:54:49	gibi	brinzhang: yes, just left comment on Sunder patch
08:55:21	brinzhang	yes, saw you comment
08:57:40	brinzhang	gibi: it's true, I will update his patch, we are talked before, thanks
09:11:52	openstackgerrit	Stephen Finucane proposed openstack/nova master: api: Add framework for extra spec validation https://review.opendev.org/704643
09:11:52	openstackgerrit	Stephen Finucane proposed openstack/nova master: api: Add support for new cyborg extra specs https://review.opendev.org/716222
09:11:53	openstackgerrit	Stephen Finucane proposed openstack/nova master: api: Add microversion 2.85, extra spec validation https://review.opendev.org/708436
09:11:53	openstackgerrit	Stephen Finucane proposed openstack/nova master: docs: Add documentation for flavor extra specs https://review.opendev.org/710037
09:12:24	openstackgerrit	Stephen Finucane proposed openstack/nova master: api: Add microversion for extra spec validation https://review.opendev.org/708436
09:12:31	openstackgerrit	Stephen Finucane proposed openstack/nova master: docs: Add documentation for flavor extra specs https://review.opendev.org/710037
09:12:34	stephenfin	gibi: Hopefully the last rebase necessary for that, assuming bauzas delivers /o\ ^
09:13:14	bauzas	stephenfin: you're on my 'next' queue, once I'm done with triaging bugs (unfortunately, downstream only... :( :( :( )
09:13:32	bauzas	children are not hitting me a lot this morning \o/
09:13:33	gibi	stephenfin: ack, looking
09:14:01	brinzhang	stenphenfin: right, indeed, many old URLs were named using undercore and may need to be updated in bulk.
09:14:11	openstackgerrit	Brin Zhang proposed openstack/nova master: Add release notes for Cyborg-Nova integration. https://review.opendev.org/716185
09:14:50	stephenfin	brinzhang: you don't need to rename the release note - just the doc in 'api-guide/'
09:15:02	stephenfin	the release note names are never exposed via a URL
09:15:25	brinzhang	stephenfin: done
09:15:59	brinzhang	gibi, stephenfin: I just resoulved your comments
09:18:24	gibi	stephenfin: extra_spec bump looks good
09:18:37	gibi	brinzhang: cyborg doc looks good
09:18:50	brinzhang	gibi: looks good to me
09:20:17	brinzhang	stephenfin: about you patch for use unittest.mock instead of mock https://review.opendev.org/#/c/714676/, does every project to upgrade (must)?
09:21:50	brinzhang	https://stackoverflow.com/questions/55807259/python-mock-vs-unittest-mock-patch says pyhton-mock and unittest.mock are the same.
09:29:26	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing server password policies https://review.opendev.org/717204
09:34:21	luyao	stephenfin: Morning, not sure if you saw my messages before, Thanks for reviewing first patch for vpmem live migration, https://review.opendev.org/#/c/715362/, gibi double checked the versioned notification body change. I just would like to make sure you also have the second patch in your review list. https://review.opendev.org/#/c/687856 .
09:34:52	stephenfin	luyao: Yup, I do. Focusing on the mixed CPU series at the moment but I'll jump back to that early next week
09:36:10	luyao	stephenfin: I saw your hard work on mixed CPU, understand. Thanks in advance. :)
09:38:37	stephenfin	dansmith: Could you take a look at https://review.opendev.org/#/c/537414/ again today, please?
09:46:44	brinzhang	stephenfin: I would like to do in Cyborg, because Cyborg also removed python2, so I want to know does it necessary to in Cyborg
09:47:53	stephenfin	brinzhang: Apparently mock 4.0 breaks things and I think that was the reason for switching. However, I'm seeing the same issue when running tests with Python 3.8 so I think the issue is there
09:48:17	stephenfin	so I'm not sure whether switching from mock to unittest.mock is such a good idea right now
09:50:23	brinzhang	So the biggest reason for the exchange is the problem you will encounter when switching pyhton3.8 in the future, as you described in the message in the patch.
09:52:15	stephenfin	To the best of my knowledge, yes
09:52:21	stephenfin	I need to check with smcginnis later
09:52:58	brinzhang	stephenfin: ok, if you get more details, pls repin me, I would like to know ^^
09:53:07	brinzhang	stephenfin: thanks
10:09:58	bauzas	sean-k-mooney: thanks for your comments on https://review.opendev.org/#/c/715490/ will provide a new revision based on your thoughts
10:49:28	openstackgerrit	Merged openstack/nova stable/stein: Functional test for UnexpectedDeletingTaskStateError https://review.opendev.org/715399
10:56:05	sean-k-mooney	bauzas: glad they were useful
12:08:56	openstackgerrit	Merged openstack/nova stable/stein: Unplug VIFs as part of cleanup of networks https://review.opendev.org/715400
12:39:20	alex_xu	stephenfin: are you going to update https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/use-pcpu-and-vcpu-in-one-instance, or you need help, I'm feeling the time tight now
13:18:13	stephenfin	alex_xu: I was waiting to hash out the top few patches before rebasing the whole thing. I can address those things today though, if you'd like
13:18:17	stephenfin	In the next hour, in fact
13:46:41	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Stabilize functional tests https://review.opendev.org/717070
15:13:01	openstackgerrit	Merged openstack/nova master: Support for nova-manage placement heal_allocations --cell https://review.opendev.org/714459
15:13:07	openstackgerrit	Merged openstack/nova stable/rocky: Functional test for UnexpectedDeletingTaskStateError https://review.opendev.org/715403
15:18:18	dansmith	stephenfin: this isn't really critical, but maybe you can answer this and make it moot: https://review.opendev.org/#/c/537414/23/nova/objects/compute_node.py@552
15:24:34	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Delete ARQs by UUID if Cyborg ARQ bind fails. https://review.opendev.org/716186
15:55:00	amodi	okay
16:05:03	spotz	Hey all odd question, does anyone know of a pre-Folsom diagram of Nova so how Nova looked before cells was first introduced?:)
16:09:39	melwitt	spotz: you can dig up old docs like this https://github.com/openstack/nova/blob/essex-eol/doc/source/images/NOVA_ARCH.svg
16:10:33	spotz	Thanks melwitt! I knew they were archived off o.o
16:10:42	melwitt	spotz: but fwiw, the current diagram doesn't show cells anyway https://docs.openstack.org/nova/latest/user/architecture.html cells are meant to be transparent to end users
16:11:48	spotz	melwitt: I'm trying to show how cells have evolved to help clarify v2 questions. So was thinking a pre-folsom architecture, cells v1 and then cells v2
16:13:25	sean-k-mooney	spotz: cellsv2 is basiclly not an evolcution fo cellsv1
16:13:35	sean-k-mooney	spotz: its a very different approch
16:13:54	spotz	sean-k-mooney: Yeah and I'm going over that as well based off the manifesto
16:14:32	sean-k-mooney	spotz: dansmith did a pretty good summit presintation on this too
16:15:25	spotz	I'll look that up, thanks sean-k-mooney
16:15:29	melwitt	spotz: oh, ok. I dunno if you've seen this but it has some info on v1 vs v2 https://docs.openstack.org/nova/mitaka/cells.html
16:16:01	spotz	melwitt: Yep got that:)
16:17:39	spotz	I'm just a visual learner so like to include visuals to help explain things
16:17:56	sean-k-mooney	spotz: this is the one i was thinking of https://www.youtube.com/watch?v=XQjABSLjk-A but there are quite a few oter presination too https://www.youtube.com/user/OpenStackFoundation/search?query=cell
16:18:29	spotz	Thanks sean-k-mooney:)
16:18:44	spotz	Appreciate all the help and quick responses!
16:24:08	openstackgerrit	Mark Goddard proposed openstack/nova master: Add functional regression test for bug 1853009 https://review.opendev.org/695012
16:24:10	openstack	bug 1853009 in OpenStack Compute (nova) "Ironic node rebalance race can lead to missing compute nodes in DB" [Undecided,In progress] https://launchpad.net/bugs/1853009 - Assigned to Mark Goddard (mgoddard)
16:24:10	openstackgerrit	Mark Goddard proposed openstack/nova master: Clear rebalanced compute nodes from resource tracker https://review.opendev.org/695187
16:24:10	openstackgerrit	Mark Goddard proposed openstack/nova master: Invalidate provider tree when compute node disappears https://review.opendev.org/695188
16:24:10	openstackgerrit	Mark Goddard proposed openstack/nova master: Prevent deletion of a compute node belonging to another host https://review.opendev.org/694802
16:24:10	openstackgerrit	Mark Goddard proposed openstack/nova master: Fix inactive session error in compute node creation https://review.opendev.org/695189
16:51:35	openstackgerrit	Stephen Finucane proposed openstack/nova master: objects: Add online migration for legacy NUMA objects https://review.opendev.org/537414
19:11:38	openstackgerrit	Sylvain Bauza proposed openstack/nova master: Provide the parent pGPU when creating a new vGPU https://review.opendev.org/715489
19:11:39	openstackgerrit	Sylvain Bauza proposed openstack/nova master: Support different vGPU types per pGPU https://review.opendev.org/715490
20:12:32	openstackgerrit	Merged openstack/nova master: Add test coverage of existing limits policies https://review.opendev.org/715674
20:12:41	openstackgerrit	Merged openstack/nova master: Correct security groups policy check_str https://review.opendev.org/716782
20:12:50	openstackgerrit	Merged openstack/nova master: Add test coverage of existing security groups policies https://review.opendev.org/716779
21:25:14	openstackgerrit	melanie witt proposed openstack/nova master: Repro bug 1845530: versioned discovery is authed https://review.opendev.org/685180
21:25:16	openstack	bug 1845530 in OpenStack Compute (nova) "Versioned discovery endpoint should not require authentication" [Undecided,In progress] https://launchpad.net/bugs/1845530 - Assigned to Eric Fried (efried)
21:25:16	openstackgerrit	melanie witt proposed openstack/nova master: Allow versioned discovery unauthenticated https://review.opendev.org/685181

Earlier   Later