Posted	Nick	Remark
#openstack-nova - 2020-04-09
19:06:07	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Fix servers policy for admin_or_owner https://review.opendev.org/718501
19:07:02	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing server policies https://review.opendev.org/718348
19:07:18	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in servers Policies https://review.opendev.org/662968
20:01:49	openstackgerrit	Merged openstack/nova master: Provide the parent pGPU when creating a new vGPU https://review.opendev.org/715489
20:01:58	openstackgerrit	Merged openstack/nova master: libvirt: Calculate disk_over_committed for raw instances https://review.opendev.org/717037
20:11:23	larsks	Howdy. Has any encountered a situation where 'server list' shows a server, but any attempt to interact with that (server show <uuid>, etc) results in "No server with a name or ID of <uuid> exists"? In the database, the server has vm_state "error" and task_state "NULL".
20:11:32	larsks	This is queens.
20:13:43	melwitt	yes, let me look for a link
20:15:06	melwitt	I think that's likely https://bugs.launchpad.net/nova/+bug/1784093 which was fixed in queens 17.0.11 upstream
20:15:07	openstack	Launchpad bug 1784093 in OpenStack Compute (nova) stein "Build requests can be orphaned without instance mappings" [Medium,Fix committed] - Assigned to melanie witt (melwitt)
20:21:43	larsks	melwitt: thanks, perfect!
20:29:06	dansmith	that looks more like a missing instance mapping to me,
20:29:44	dansmith	although potentially missing BR too I guess
20:29:45	dansmith	or maybe that bug is about BRs, but the fix is instance mappings?
20:36:06	melwitt	yeah, it is. it's a bit different. I think the fix is the same (atomic update of BR, IM, RS)
20:36:18	dansmith	ack
20:36:24	melwitt	but yes this is a missing IM
21:19:23	melwitt	dansmith: ideas welcome on what to do to recover/cleanup when an instance mapping update fails due to DBError https://bugs.launchpad.net/nova/+bug/1871925
21:19:24	openstack	Launchpad bug 1871925 in OpenStack Compute (nova) "Server appears in "openstack server list" but "openstack server (show|delete|etc)" insists it doesn't exist" [Medium,Confirmed]
21:20:41	melwitt	the "easy" solution is retry a few times and hope that does it
21:24:39	bauzas	gibi: good call on rechecking changes for 2.86 and 2.87, will follow up on this
21:33:40	melwitt	I think I'm gonna propose a retry first
22:24:50	mriedem	there was a bug like that in novaclient once, related to markers and how novaclient did it's marker stuff when listing servers. since osc uses novaclient, depending on when that was fixed and the version of osc being used here it could be related.
22:25:41	mriedem	e.g. the marker was for a build request or something, but i can't remember the details. there have been a few weird things like that with markers.
22:30:00	melwitt	noted
23:29:15	openstackgerrit	Merged openstack/nova master: Support different vGPU types per pGPU https://review.opendev.org/715490
23:29:22	openstackgerrit	Merged openstack/nova master: Functional test with pGPUs https://review.opendev.org/717975
23:29:29	openstackgerrit	Merged openstack/nova master: Add test coverage of existing shelve policies https://review.opendev.org/717539
23:30:13	openstackgerrit	Merged openstack/nova master: Reset the cell cache for database access in Service https://review.opendev.org/717662
#openstack-nova - 2020-04-10
00:18:43	openstackgerrit	Merged openstack/nova master: Introduce scope_types in shelve server https://review.opendev.org/717571
00:18:51	openstackgerrit	Merged openstack/nova master: Add new default roles in shelve server policies https://review.opendev.org/717581
00:19:05	openstackgerrit	Merged openstack/nova master: Drop concept of '?validation' parameter https://review.opendev.org/717789
00:30:46	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Fix servers policy for admin_or_owner https://review.opendev.org/718501
00:31:36	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Fix servers policy for admin_or_owner https://review.opendev.org/718501
00:42:34	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing server policies https://review.opendev.org/718348
00:42:48	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in servers Policies https://review.opendev.org/662968
00:43:22	openstackgerrit	Brin Zhang proposed openstack/python-novaclient master: FUP: Add volume-update CLI pre V285 tests https://review.opendev.org/718841
00:45:23	brinzhang_	gibi: the bp/destroy-instance-with-datavolume FUP patch see below:
00:45:37	brinzhang_	gibi: for novalient https://review.opendev.org/#/c/718841/1
00:46:07	brinzhang_	gibi, gamnn, stephenfin: for nova https://review.opendev.org/#/c/718589/1
00:48:59	gmann	brinzhang_: checking
00:49:59	brinzhang_	gmann: thanks
00:55:57	melwitt	gmann: this looks like a legit failure, right? https://review.opendev.org/718604
00:56:28	gmann	melwitt: right. working on this.
00:56:44	melwitt	gmann: k cool. just wanted to make sure
00:57:47	melwitt	I'll keep a watch on the freshly updated ones
01:17:28	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing keypairs policies https://review.opendev.org/718604
01:17:51	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in keypairs https://review.opendev.org/718609
01:18:01	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in keypairs policies https://review.opendev.org/718619
01:18:12	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Pass the actual target in keypairs policy https://review.opendev.org/718621
01:18:50	gmann	melwitt: updated ^^. it should pass now.
01:19:34	melwitt	thanks
02:02:42	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Pass the actual target in server group policy https://review.opendev.org/717176
02:56:34	openstackgerrit	Yingji Sun proposed openstack/nova master: Set different VirtualDevice.key https://review.opendev.org/713565
03:45:35	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Pass the actual target in server group policy https://review.opendev.org/717176
03:46:03	openstackgerrit	Brin Zhang proposed openstack/nova master: [Trivial] FUP: addressed comments in support non-admin filter instances https://review.opendev.org/718878
03:54:55	openstackgerrit	Brin Zhang proposed openstack/python-novaclient master: [Trivial] FUP: Enhanced description for 'server list --config-drive' help https://review.opendev.org/718879
05:21:17	openstackgerrit	Merged openstack/nova master: FUP: add missing test for PUT volume attachments API https://review.opendev.org/718589
05:21:29	openstackgerrit	Merged openstack/nova master: Add new default roles in os-console-auth-tokens policies https://review.opendev.org/706689
05:21:45	openstackgerrit	Merged openstack/nova master: Pass the actual target in os-console-auth-tokens policy https://review.opendev.org/706690
05:22:06	openstackgerrit	Merged openstack/nova master: api: Add microversion for extra spec validation https://review.opendev.org/708436
05:22:24	openstackgerrit	Merged openstack/nova master: docs: Add documentation for flavor extra specs https://review.opendev.org/710037
05:24:33	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in server group policies https://review.opendev.org/717175
06:10:06	openstackgerrit	Merged openstack/python-novaclient master: FUP: Add volume-update CLI pre V285 tests https://review.opendev.org/718841
06:10:07	openstackgerrit	Merged openstack/python-novaclient master: [Trivial] FUP: Enhanced description for 'server list --config-drive' help https://review.opendev.org/718879
07:02:53	openstackgerrit	Rui Zang proposed openstack/nova master: TEST: DO NOT REVIEW https://review.opendev.org/718895
07:06:39	openstackgerrit	Rui Zang proposed openstack/nova master: TEST: DO NOT REVIEW https://review.opendev.org/718895
07:17:29	openstackgerrit	Merged openstack/nova master: virt: Provide block_device_info during rescue https://review.opendev.org/700811
07:26:08	openstackgerrit	Brin Zhang proposed openstack/nova master: [Trivial] FUP: addressed comments in support non-admin filter instances https://review.opendev.org/718878
08:14:21	openstackgerrit	melanie witt proposed openstack/nova stable/train: Reset the cell cache for database access in Service https://review.opendev.org/718934
08:21:42	brinzhang_	gibi: this can be change completed https://blueprints.launchpad.net/nova/+spec/action-event-fault-details
08:21:48	brinzhang_	gibi: https://blueprints.launchpad.net/nova/+spec/destroy-instance-with-datavolume
08:22:14	brinzhang_	gibi: https://blueprints.launchpad.net/nova/+spec/non-admin-filter-instance-by-az
08:23:36	brinzhang_	above all have commleted
08:51:58	bauzas	brinzhang_: gibi is on holiday today (Easter Friday)
08:52:17	bauzas	brinzhang_: how can I help ?
08:54:30	bauzas	this being said, gibi thanks for having rechecked all the changes
08:58:18	brinzhang_	bauzas: thanks, I completed these feature's need to be done by follow up.
08:58:30	bauzas	you mean the blueprint status ?
08:58:36	brinzhang_	yes
08:58:45	bauzas	no worries, it'll be done :)
08:58:58	brinzhang_	of course, there is the nova-cyborg-interaction https://blueprints.launchpad.net/nova/+spec/nova-cyborg-interaction
09:00:12	brinzhang_	bauzas: this is the end follow up patch https://review.opendev.org/#/c/718878/, can you check it?
09:00:28	brinzhang_	the end done by me :)
09:01:29	bauzas	brinzhang_: FWIW, you can help reviewers by mentioning the original Change-Id in your commit msg
09:01:44	bauzas	this way, when search thru Gerrit with a change-id, both will appear
09:02:10	bauzas	also, this FUP doesn't appear as related to the original patch
09:02:37	brinzhang_	bauzas: in the future I will do by that way ^
09:02:45	bauzas	brinzhang_: by making it dependent on the original patch, you create a gerrit UI visual link between both
09:02:55	bauzas	yeah, no worries, just explaining
09:03:34	bauzas	take the local branch you work on, just add another commit on top of it before uploading to Gerrit (thru git review or by hand)
09:04:01	brinzhang_	bauzas: I know you mean, I ussualy use patch liknk to another patch, I will remember you remind, and do that in my future patch
09:04:07	bauzas	cool
09:05:58	brinzhang_	bauzas: yes, busy some works need to do, some times no much time to push the follow up patch, I know the way what you said :)
09:06:26	openstackgerrit	Merged openstack/python-novaclient master: Microversion 2.86 - Extra spec validation https://review.opendev.org/718057
09:06:52	brinzhang_	bauzas: thanks, good morning, hope you have a good day :P
09:08:25	brinzhang_	s/busy/because
09:14:12	bauzas	brinzhang_: you too (g'night)
09:18:22	openstackgerrit	Sylvain Bauza proposed openstack/nova master: [Trivial] FUP: addressed comments in support non-admin filter instances https://review.opendev.org/718878
09:24:10	bauzas	brinzhang_: just updated the commit msg to mention the change-id ^

Earlier   Later