Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-04-10
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 ^
09:24:59 brinzhang_ I have saw that change, thanks bauzas ^^
09:29:08 bauzas brinzhang_: thank you for rechecking nova patches that got trampled by the gate
09:30:41 brinzhang_ bauzas: np, We should speed up patch merge, zuul does not rest, we can do it when you rest.
09:31:04 brinzhang_ that we can speed up the nova release plan.
09:37:13 bauzas brinzhang_: we're in European time, the gate is always less busy
09:38:05 bauzas brinzhang_: you know this page, right ? http://zuul.openstack.org/status
09:38:21 bauzas filter by 'nova' and you'll see all patches on different pipelines
09:38:39 brinzhang_ bauzas: yes, I know, I just know how to search one patch by id
09:38:55 bauzas you can also filter by project
09:40:04 brinzhang_ yeah, I have tried, it can list more
09:40:57 brinzhang_ it seems like not many patch running in zuul, I think I was wrong of above words
09:56:28 brinzhang_ lyarwood: for this patch https://review.opendev.org/#/c/711609, mainly to solve the PAUSED instance shelve operation, if the instance in PAUSED status, we cannot shelve it if we are not clean_shutdown, this error comes form our product
09:57:43 brinzhang_ lywarwood: I will asked Qiu Fossen respin this bug, and paste the ERROR log in the bug
09:59:28 brinzhang_ lyarwood: That happened in Rocky release, but we were compared in master, that also exist.
10:05:43 bauzas brinzhang_: lyarwood is off as well
10:06:50 brinzhang_ bauzas: yeah, I just left comment for him, we are in same bug fix ^
11:55:26 openstackgerrit Lee Yarwood proposed openstack/nova stable/train: libvirt: Calculate disk_over_committed for raw instances https://review.opendev.org/718964
13:18:46 openstackgerrit Merged openstack/nova master: [Trivial] FUP: addressed comments in support non-admin filter instances https://review.opendev.org/718878
13:18:52 gmann stephenfin: if you are online, would you like to re-vote on this. failure is fixed in base patch and now gate is green - https://review.opendev.org/#/c/718619/4
13:31:56 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Pass the actual target in server group policy https://review.opendev.org/717176
13:32:28 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add new default roles in server group policies https://review.opendev.org/717175
13:34:20 bauzas gmann: stephenfin is on holiday today
13:35:06 gmann bauzas: ack, thanks.
13:37:38 bauzas the gate is on a horrible state :/
13:54:43 openstackgerrit Alexandre arents proposed openstack/nova master: libvirt: ensure disk_over_commit is not negative https://review.opendev.org/719008
14:23:08 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add new default roles in server group policies https://review.opendev.org/717175
14:46:40 openstackgerrit Merged openstack/nova master: Follow-up for flavor-extra-spec-validators series https://review.opendev.org/718357
14:56:14 openstackgerrit Merged openstack/nova master: libvirt: Add support for stable device rescue https://review.opendev.org/700812
14:56:51 bauzas yay, 2 changes to go before the 2.87 microversion \o/
15:17:01 bauzas oh shit, Gerrit will be restarted in less than 2 hours :(
15:18:50 bauzas https://review.opendev.org/#/c/701430/ is still in the check pipeline (again) for 1h 20 mins left, so that means it would be in the gate pipeline when Gerrit will restart
15:18:52 bauzas dammit.
15:26:23 gmann bauzas: yeah, gate pipeline is not so heavy but its difficult to finish both pipeline before restart
15:27:27 bauzas yeah, it's just we got a ton of rechecks due to spurious issues
15:27:37 bauzas and this extra issue is killing me

Earlier   Later