Earlier  
Posted Nick Remark
#openstack-nova - 2020-04-09
18:17:31 stephenfin gmann: You're going to address the issues with the policy patches that are failing at the gate, I assume?
18:17:56 stephenfin I started to fix some of them today but got distracted and probably won't have a chance to close it off now
18:18:03 gmann stephenfin: yeah. working on those
18:19:04 gmann stephenfin: np!.
18:19:18 gmann let me check all review if any query for you.
18:19:56 openstackgerrit Stephen Finucane proposed openstack/nova master: Add test coverage of existing shelve policies https://review.opendev.org/717539
18:20:04 openstackgerrit Stephen Finucane proposed openstack/nova master: Introduce scope_types in shelve server https://review.opendev.org/717571
18:20:11 openstackgerrit Stephen Finucane proposed openstack/nova master: Add new default roles in shelve server policies https://review.opendev.org/717581
18:22:43 gmann stephenfin: replied on this. is ok for you ? -https://review.opendev.org/#/c/718619/2/nova/policies/keypairs.py@26
18:30:05 openstackgerrit sean mooney proposed openstack/nova master: Delete ARQs by UUID if Cyborg ARQ bind fails. https://review.opendev.org/716186
18:35:51 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing keypairs policies https://review.opendev.org/718604
18:36:10 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in keypairs https://review.opendev.org/718609
18:36:24 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add new default roles in keypairs policies https://review.opendev.org/718619
18:36:36 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Pass the actual target in keypairs policy https://review.opendev.org/718621
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

Earlier   Later