Posted	Nick	Remark
#openstack-nova - 2020-03-25
02:05:12	brinzhang_	gmann: done, thanks
02:05:51	brinzhang_	because of the policy name changes, I have tested in my local, it works fine.
02:08:38	openstackgerrit	Brin Zhang proposed openstack/nova master: Expose instance action event details out of the API https://review.opendev.org/694430
02:08:59	openstackgerrit	Brin Zhang proposed openstack/nova master: Add instance actions v283 samples test https://review.opendev.org/706251
02:11:39	openstackgerrit	melanie witt proposed openstack/nova master: DNM: try to get some debug info for bug 1844929 https://review.opendev.org/701478
02:11:41	openstack	bug 1844929 in OpenStack Compute (nova) "grenade jobs failing due to "Timed out waiting for response from cell" in scheduler" [High,Confirmed] https://launchpad.net/bugs/1844929
02:20:40	gmann	brinzhang_: after reading the 2.62 changes, i think we do not need the new policy. we already have existing policy which can control the info from non-admin. commented on spec also
02:20:56	gmann	let's wait for sean-k-mooney reply. https://review.opendev.org/#/c/694430/13/nova/policies/instance_actions.py@52
02:22:28	gmann	i think sean-k-mooney concern was we should not pass these info to non-admin which is controlled via existing policy also, like host info in 'events' - https://github.com/openstack/nova/blob/f454e1dec9580abf4605e071bdd678a40f492a49/nova/api/openstack/compute/instance_actions.py#L170
02:35:06	brinzhang_	gmann: I cannot open github, you mean the https://review.opendev.org/#/c/694430/13/nova/api/openstack/compute/instance_actions.py@172
02:35:35	brinzhang_	gmann: the show_host = api_version_request.is_supported(req, '2.62')?
02:35:37	gmann	brinzhang_: yeah, show_host with 2.62
02:37:25	gmann	brinzhang_: let's wait for sean-k-mooney reply before changes in case something i missed
02:37:50	brinzhang_	gmann: the .BASE_POLICY_NAME % 'events' is a rule_admin policy
02:37:54	openstackgerrit	Merged openstack/nova stable/train: libvirt: Provide the backing file format when creating qcow2 disks https://review.opendev.org/710788
02:38:34	brinzhang_	gmann: ok, let sean-k-mooney check again
02:40:41	gmann	yeah event policy is admin by default
02:42:07	brinzhang_	so show_host can be shown for an admin user, I think the details policy (SYSTEM_READER) is suitable for now.
02:44:46	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing flavor_manage policies https://review.opendev.org/714814
02:47:00	openstackgerrit	Brin Zhang proposed openstack/nova-specs master: [Trivial] Remove note for the implementation https://review.opendev.org/714817
03:00:54	openstackgerrit	Luyao Zhong proposed openstack/nova master: bug-fix: set do_cleanup always True for libvirt driver https://review.opendev.org/714593
03:00:54	openstackgerrit	Luyao Zhong proposed openstack/nova master: support live migration with vpmems https://review.opendev.org/687856
03:00:55	openstackgerrit	Luyao Zhong proposed openstack/nova master: Track orphan instances and error migrations in resource tracker https://review.opendev.org/714653
03:02:26	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in os-flavor-manage https://review.opendev.org/714818
03:18:11	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in os-flavor_manage policies https://review.opendev.org/714819
03:25:06	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Pass the actual target in os-flavor-manage policy https://review.opendev.org/714822
04:51:49	openstackgerrit	melanie witt proposed openstack/nova master: DNM: try to get some debug info for bug 1844929 https://review.opendev.org/701478
04:51:50	openstack	bug 1844929 in OpenStack Compute (nova) "grenade jobs failing due to "Timed out waiting for response from cell" in scheduler" [High,Confirmed] https://launchpad.net/bugs/1844929
05:26:15	openstackgerrit	Luyao Zhong proposed openstack/nova stable/train: bug-fix: Reject live migration with vpmem https://review.opendev.org/714064
06:12:55	openstackgerrit	Qiu Fossen proposed openstack/nova master: The instance is volume backed and power state is PAUSED，shelve the instance failed https://review.opendev.org/711609
06:12:55	openstackgerrit	Qiu Fossen proposed openstack/nova master: The instance is volume backed and power state is PAUSED，shelve the instance failed https://review.opendev.org/711609
06:40:15	openstackgerrit	Elod Illes proposed openstack/nova stable/pike: Mask the token used to allow access to consoles https://review.opendev.org/708876
06:40:31	openstackgerrit	Elod Illes proposed openstack/nova stable/pike: Avoid circular reference during serialization https://review.opendev.org/714148
06:48:21	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Delete ARQs for an instance when the instance is deleted. https://review.opendev.org/673735
06:48:22	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Enable hard/soft reboot with accelerators. https://review.opendev.org/697940
06:48:22	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Enable start/stop of instances with accelerators. https://review.opendev.org/699553
06:48:22	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Enable and use COMPUTE_ACCELERATORS trait. https://review.opendev.org/699554
06:48:23	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Bump compute rpcapi version and reduce Cyborg calls. https://review.opendev.org/704227
06:48:23	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Block unsupported instance operations with accelerators. https://review.opendev.org/674726
06:48:24	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Add cyborg tempest job. https://review.opendev.org/670999
08:13:02	openstackgerrit	Luyao Zhong proposed openstack/nova master: Track orphan instances and error migrations in resource tracker https://review.opendev.org/714653
08:26:20	nightmare_unreal	openstackgerrit:
08:44:33	luyao	brinzhang_: https://review.opendev.org/#/c/678451/ was moved to https://review.opendev.org/#/c/714653, I add more testcases, thanks for your review
08:55:32	lyarwood	elod: morning, https://review.opendev.org/#/c/713961/ if you have time, almost finished with these now.
09:00:32	brinzhang_	luyao: Thansk, got it, I will check later(I have some docs need to compete.)
09:02:08	luyao	brinzhang_: thanks
09:02:49	brinzhang_	luyao:np ^^
09:04:21	luyao	lyarwood: I have a seperate patch to address the 'do_cleanup' flag issue, could you look at it again? https://review.opendev.org/#/c/714593
09:05:14	lyarwood	yup can try today
09:05:45	luyao	lyarwood: thanks :)
09:07:28	luyao	elod: thanks for review, comments addressed https://review.opendev.org/#/c/714064/
09:18:37	brinzhang_	sean-k-mooney: https://review.opendev.org/#/c/694430/ and https://review.opendev.org/#/c/699669/ need your check, gmann have a question -1 for the new policy for show events:details, and I think it's necessary, and adopt my case, I left my comment in the spec.
09:28:43	openstackgerrit	jayaditya gupta proposed openstack/nova master: Support for nova-manage placement heal_allocations --cell https://review.opendev.org/714459
09:36:58	openstackgerrit	Huaqiang Wang proposed openstack/nova master: Refactor the code in checking available host CPUs https://review.opendev.org/714657
09:36:58	openstackgerrit	Huaqiang Wang proposed openstack/nova master: Introduce 'MIXED' CPU allocation policy for instance https://review.opendev.org/713354
09:36:59	openstackgerrit	Huaqiang Wang proposed openstack/nova master: Introduce the interface of creating 'MIXED' policy instance through 'PCPU' and 'VCPU' https://review.opendev.org/713355
09:36:59	openstackgerrit	Huaqiang Wang proposed openstack/nova master: metadata: export the vCPU IDs that are pinning on the host CPUs https://review.opendev.org/688936
09:37:29	huaqiang	stephenfin: nice! you quickly fixed so much for mixed-instance bp!
09:37:55	huaqiang	I'd like to follow your code and also contributes
09:38:11	stephenfin	huaqiang: Feel free to take ownership of the whole lot of them, if they're helpful
09:38:33	huaqiang	so I also updated the code to address the comments already made
09:39:04	huaqiang	You are the gurantee of the code
09:39:28	huaqiang	If you like to let me do some thing I'd like to do
09:39:35	huaqiang	maybe from testing your code?
09:39:42	huaqiang	I see not all test passed
09:40:49	stephenfin	Oh, I hadn't checked that yet. Let me respin things to fix those
09:40:57	stephenfin	Then we can figure out if any of them are useful
09:43:08	huaqiang	I'll spend about two hours in testing your patches that not marked with 'WIP' if you haven't test by yourself. or you can tell me which patch need more test
09:44:11	stephenfin	I think everything not marked in WIP is potentially useful
09:45:05	stephenfin	The WIP patches duplicate your work so I'll probably abandon my ones. I wrote those WIP patches last week before you submitted the new revision
09:45:12	huaqiang	I'd like to know if you will continue these 'WIP' patches?
09:45:22	huaqiang	ok
09:45:39	huaqiang	a lot of them are simular
09:45:40	stephenfin	I won't. You've already done that work
09:45:57	huaqiang	got.
09:46:32	huaqiang	I'd like the take the reposibility
10:00:55	elod	lyarwood: hi, +W'd
10:01:12	lyarwood	elod: many thanks :)
10:01:53	elod	luyao: thanks, looks good to me, +2
10:04:58	elod	lyarwood: thanks, too :) Now there are a bunch of patches in the gate queue in pike, hope we won't hit too many error_extending failures :S
10:12:05	lyarwood	elod: I noticed the tempest job had failed a few times, was that the issue?
10:12:14	lyarwood	elod: I haven't had time to look into it yet but wanted to later today
10:12:24	lyarwood	elod: really want to flush the stable/pike queue once and for all :)
10:14:09	elod	lyarwood: yes, most of them are that failure :S yes, stable/pike will look nice if everything will be merged in the queue \o/
10:19:49	lyarwood	elod: I'll take a look later today
10:26:39	elod	lyarwood: thanks! AFAIK that issue is not a new one, but strange that we hit now in that number... maybe because there were not so many check/gate runs towards pike in the last months
10:44:01	lyarwood	elod: https://review.opendev.org/#/c/697523/ - I don't think we can workaround the issue on stable/pike within c-vol, thoughts on blacklisting the specific test in our compute jobs?
10:44:38	openstackgerrit	John Garbutt proposed openstack/nova master: Prevent compute manager freeze when greenpool is full https://review.opendev.org/575034
10:44:43	openstackgerrit	John Garbutt proposed openstack/nova master: Prevent compute manager freeze when greenpool is full https://review.opendev.org/575034
10:47:29	johnthetubaguy	gibi: stephenfin: I noticed we were all talking about this patch, it seem very like one of the big ironic pain points, so I fixed up my worries: https://review.opendev.org/#/c/575034
10:49:18	johnthetubaguy	belmoreira: I am wondering if you have seen this patch, and if it would help your powersync issues at all: https://review.opendev.org/#/c/575034
10:51:54	belmoreira	johnthetubaguy: no, let me have a look
10:52:15	johnthetubaguy	its from the vmware folks, which I guess see related issues
10:57:44	openstackgerrit	Lee Yarwood proposed openstack/nova stable/pike: tempest: Avoid bug #1796708 on slower stable/pike CI hosts https://review.opendev.org/714915
10:57:45	openstack	bug 1796708 in Cinder "VolumesExtendTest.test_volume_extend_when_volume_has_snapshot intermittently fails with "Extend volume failed.: VolumeNotDeactivated: Volume volume-5514a6ad-abbb-46b3-a464-d73cc67e55af was not deactivated in time."" [Medium,Confirmed] https://launchpad.net/bugs/1796708
10:57:47	lyarwood	elod: ^ lets skip that test for now.
11:03:32	elod	lyarwood: usually i don't like disabling tests, but maybe now that is OK for that test on pike, especially for that bug is open since 2018 and has no fix. Let's see the regex works well in your patch :)
11:12:06	openstackgerrit	John Garbutt proposed openstack/nova master: Prevent compute manager freeze when greenpool is full https://review.opendev.org/575034
11:13:27	openstackgerrit	John Garbutt proposed openstack/nova master: Prevent compute manager freeze when greenpool is full https://review.opendev.org/575034
11:33:45	openstackgerrit	John Garbutt proposed openstack/nova master: WIP: Enforce resource limits using oslo.limit https://review.opendev.org/615180

Earlier   Later