Posted	Nick	Remark
#openstack-nova - 2020-04-03
01:18:27	alex_xu	brinzhang_: you are welcome
01:20:17	brinzhang_	gmann: I reviewed you and dansmith's chat record, I am sorry not to participate in your discussion. But I have not got more clear thing what I need to do, I tend to wait for dansmith rebase and than update.
01:26:26	openstackgerrit	sean mooney proposed openstack/nova master: cyborg evacuate support https://review.opendev.org/715326
02:14:36	openstackgerrit	Merged openstack/nova master: Ensure we pass a target in admin actions https://review.opendev.org/663095
02:18:35	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in security group policies https://review.opendev.org/716793
02:20:49	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing server external events policies https://review.opendev.org/717155
02:21:28	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in limits policy https://review.opendev.org/715680
02:22:06	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in limits policies https://review.opendev.org/715760
02:22:29	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Pass the actual target in limits policy https://review.opendev.org/715761
02:38:43	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in server external events https://review.opendev.org/717167
02:44:38	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in server external events policies https://review.opendev.org/717168
02:47:48	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Pass the actual target in server external events policy https://review.opendev.org/717169
03:52:50	openstackgerrit	Merged openstack/nova master: Pass the actual target in migrations policy https://review.opendev.org/716147
04:28:25	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Add release notes for Cyborg-Nova integration. https://review.opendev.org/716185
04:28:26	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Delete ARQs by UUID if Cyborg ARQ bind fails. https://review.opendev.org/716186
04:41:45	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing server group policies https://review.opendev.org/717173
04:48:15	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in server external events policies https://review.opendev.org/717168
04:49:25	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Pass the actual target in server external events policy https://review.opendev.org/717169
04:58:36	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in server group policy https://review.opendev.org/717174
05:21:36	openstackgerrit	Merged openstack/nova master: Allow PUT volume attachments API to modify delete_on_termination https://review.opendev.org/693828
05:33:25	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in server group policies https://review.opendev.org/717175
05:38:01	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Pass the actual target in server group policy https://review.opendev.org/717176
06:25:35	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing server metadata policies https://review.opendev.org/717182
06:43:57	bauzas	_erlon_: apologies, I was done for the day
06:50:35	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Fix server metadata policy to be admin_or_owner https://review.opendev.org/717189
07:01:49	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Add release notes for Cyborg-Nova integration. https://review.opendev.org/716185
07:01:50	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Delete ARQs by UUID if Cyborg ARQ bind fails. https://review.opendev.org/716186
07:19:44	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing server password policies https://review.opendev.org/717204
07:30:32	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing server metadata policies https://review.opendev.org/717182
07:33:46	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Add release notes for Cyborg-Nova integration. https://review.opendev.org/716185
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?:)

Earlier   Later