| *** mlavalle has quit IRC | 00:02 | |
| *** brinzhang has quit IRC | 00:10 | |
| *** dlbewley has quit IRC | 00:11 | |
| *** brinzhang has joined #openstack-nova | 00:13 | |
| openstackgerrit | Merged openstack/nova master: Introduce scope_types in shelve server https://review.opendev.org/717571 | 00:18 |
|---|---|---|
| openstackgerrit | Merged openstack/nova master: Add new default roles in shelve server policies https://review.opendev.org/717581 | 00:18 |
| openstackgerrit | Merged openstack/nova master: Drop concept of '?validation' parameter https://review.opendev.org/717789 | 00:19 |
| *** gary_perkins has quit IRC | 00:24 | |
| *** gary_perkins has joined #openstack-nova | 00:24 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix servers policy for admin_or_owner https://review.opendev.org/718501 | 00:30 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix servers policy for admin_or_owner https://review.opendev.org/718501 | 00:31 |
| *** brinzhang_ has joined #openstack-nova | 00:39 | |
| *** brinzhang has quit IRC | 00:42 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing server policies https://review.opendev.org/718348 | 00:42 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in servers Policies https://review.opendev.org/662968 | 00:42 |
| openstackgerrit | Brin Zhang proposed openstack/python-novaclient master: FUP: Add volume-update CLI pre V285 tests https://review.opendev.org/718841 | 00:43 |
| brinzhang_ | gibi: the bp/destroy-instance-with-datavolume FUP patch see below: | 00:45 |
| brinzhang_ | gibi: for novalient https://review.opendev.org/#/c/718841/1 | 00:45 |
| brinzhang_ | gibi, gamnn, stephenfin: for nova https://review.opendev.org/#/c/718589/1 | 00:46 |
| gmann | brinzhang_: checking | 00:48 |
| brinzhang_ | gmann: thanks | 00:49 |
| melwitt | gmann: this looks like a legit failure, right? https://review.opendev.org/718604 | 00:55 |
| gmann | melwitt: right. working on this. | 00:56 |
| melwitt | gmann: k cool. just wanted to make sure | 00:56 |
| melwitt | I'll keep a watch on the freshly updated ones | 00:57 |
| *** brinzhang has joined #openstack-nova | 00:59 | |
| *** brinzhang_ has quit IRC | 01:02 | |
| *** gyee has quit IRC | 01:09 | |
| *** eharney has quit IRC | 01:12 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing keypairs policies https://review.opendev.org/718604 | 01:17 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in keypairs https://review.opendev.org/718609 | 01:17 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in keypairs policies https://review.opendev.org/718619 | 01:18 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Pass the actual target in keypairs policy https://review.opendev.org/718621 | 01:18 |
| gmann | melwitt: updated ^^. it should pass now. | 01:18 |
| melwitt | thanks | 01:19 |
| *** spatel has joined #openstack-nova | 01:29 | |
| *** lbragstad_ has joined #openstack-nova | 01:59 | |
| *** lbragstad has quit IRC | 02:02 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Pass the actual target in server group policy https://review.opendev.org/717176 | 02:02 |
| *** spatel has quit IRC | 02:11 | |
| openstackgerrit | Yingji Sun proposed openstack/nova master: Set different VirtualDevice.key https://review.opendev.org/713565 | 02:56 |
| *** brinzhang_ has joined #openstack-nova | 03:01 | |
| *** brinzhang has quit IRC | 03:04 | |
| *** ociuhandu has joined #openstack-nova | 03:04 | |
| *** mgariepy has quit IRC | 03:07 | |
| *** ociuhandu has quit IRC | 03:09 | |
| *** jdillaman has quit IRC | 03:13 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Pass the actual target in server group policy https://review.opendev.org/717176 | 03:45 |
| openstackgerrit | Brin Zhang proposed openstack/nova master: [Trivial] FUP: addressed comments in support non-admin filter instances https://review.opendev.org/718878 | 03:46 |
| *** rcernin has joined #openstack-nova | 03:46 | |
| openstackgerrit | Brin Zhang proposed openstack/python-novaclient master: [Trivial] FUP: Enhanced description for 'server list --config-drive' help https://review.opendev.org/718879 | 03:54 |
| *** evrardjp has quit IRC | 04:37 | |
| *** evrardjp has joined #openstack-nova | 04:37 | |
| *** rcernin has quit IRC | 05:03 | |
| openstackgerrit | Merged openstack/nova master: FUP: add missing test for PUT volume attachments API https://review.opendev.org/718589 | 05:21 |
| openstackgerrit | Merged openstack/nova master: Add new default roles in os-console-auth-tokens policies https://review.opendev.org/706689 | 05:21 |
| openstackgerrit | Merged openstack/nova master: Pass the actual target in os-console-auth-tokens policy https://review.opendev.org/706690 | 05:21 |
| openstackgerrit | Merged openstack/nova master: api: Add microversion for extra spec validation https://review.opendev.org/708436 | 05:22 |
| openstackgerrit | Merged openstack/nova master: docs: Add documentation for flavor extra specs https://review.opendev.org/710037 | 05:22 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in server group policies https://review.opendev.org/717175 | 05:24 |
| openstackgerrit | Merged openstack/python-novaclient master: FUP: Add volume-update CLI pre V285 tests https://review.opendev.org/718841 | 06:10 |
| openstackgerrit | Merged openstack/python-novaclient master: [Trivial] FUP: Enhanced description for 'server list --config-drive' help https://review.opendev.org/718879 | 06:10 |
| *** dpawlik has joined #openstack-nova | 06:20 | |
| *** ociuhandu has joined #openstack-nova | 06:35 | |
| *** ociuhandu has quit IRC | 06:40 | |
| *** slaweq has joined #openstack-nova | 07:00 | |
| openstackgerrit | Rui Zang proposed openstack/nova master: TEST: DO NOT REVIEW https://review.opendev.org/718895 | 07:02 |
| *** ociuhandu has joined #openstack-nova | 07:04 | |
| openstackgerrit | Rui Zang proposed openstack/nova master: TEST: DO NOT REVIEW https://review.opendev.org/718895 | 07:06 |
| *** kevinbenton has quit IRC | 07:08 | |
| *** tesseract has joined #openstack-nova | 07:08 | |
| *** kevinbenton has joined #openstack-nova | 07:08 | |
| *** ociuhandu has quit IRC | 07:09 | |
| *** yedongcan has joined #openstack-nova | 07:10 | |
| *** ociuhandu has joined #openstack-nova | 07:14 | |
| *** ociuhandu has quit IRC | 07:15 | |
| *** dklyle has quit IRC | 07:16 | |
| *** alex_xu has quit IRC | 07:16 | |
| *** ociuhandu has joined #openstack-nova | 07:16 | |
| openstackgerrit | Merged openstack/nova master: virt: Provide block_device_info during rescue https://review.opendev.org/700811 | 07:17 |
| *** ociuhandu has quit IRC | 07:19 | |
| *** ociuhandu has joined #openstack-nova | 07:19 | |
| openstackgerrit | Brin Zhang proposed openstack/nova master: [Trivial] FUP: addressed comments in support non-admin filter instances https://review.opendev.org/718878 | 07:26 |
| *** maciejjozefczyk has joined #openstack-nova | 07:29 | |
| *** tosky has joined #openstack-nova | 07:32 | |
| *** rpittau|afk is now known as rpittau | 07:36 | |
| *** slaweq has quit IRC | 07:51 | |
| *** links has joined #openstack-nova | 08:04 | |
| openstackgerrit | melanie witt proposed openstack/nova stable/train: Reset the cell cache for database access in Service https://review.opendev.org/718934 | 08:14 |
| brinzhang_ | gibi: this can be change completed https://blueprints.launchpad.net/nova/+spec/action-event-fault-details | 08:21 |
| brinzhang_ | gibi: https://blueprints.launchpad.net/nova/+spec/destroy-instance-with-datavolume | 08:21 |
| *** ociuhandu has quit IRC | 08:21 | |
| brinzhang_ | gibi: https://blueprints.launchpad.net/nova/+spec/non-admin-filter-instance-by-az | 08:22 |
| *** ociuhandu has joined #openstack-nova | 08:23 | |
| brinzhang_ | above all have commleted | 08:23 |
| *** tkajinam has quit IRC | 08:25 | |
| *** ociuhandu has quit IRC | 08:28 | |
| *** xek has joined #openstack-nova | 08:32 | |
| bauzas | brinzhang_: gibi is on holiday today (Easter Friday) | 08:51 |
| bauzas | brinzhang_: how can I help ? | 08:52 |
| bauzas | this being said, gibi thanks for having rechecked all the changes | 08:54 |
| brinzhang_ | bauzas: thanks, I completed these feature's need to be done by follow up. | 08:58 |
| bauzas | you mean the blueprint status ? | 08:58 |
| brinzhang_ | yes | 08:58 |
| bauzas | no worries, it'll be done :) | 08:58 |
| brinzhang_ | of course, there is the nova-cyborg-interaction https://blueprints.launchpad.net/nova/+spec/nova-cyborg-interaction | 08:58 |
| brinzhang_ | bauzas: this is the end follow up patch https://review.opendev.org/#/c/718878/, can you check it? | 09:00 |
| brinzhang_ | the end done by me :) | 09:00 |
| bauzas | brinzhang_: FWIW, you can help reviewers by mentioning the original Change-Id in your commit msg | 09:01 |
| bauzas | this way, when search thru Gerrit with a change-id, both will appear | 09:01 |
| bauzas | also, this FUP doesn't appear as related to the original patch | 09:02 |
| brinzhang_ | bauzas: in the future I will do by that way ^ | 09:02 |
| bauzas | brinzhang_: by making it dependent on the original patch, you create a gerrit UI visual link between both | 09:02 |
| bauzas | yeah, no worries, just explaining | 09:02 |
| 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:03 |
| 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 |
| bauzas | cool | 09:04 |
| 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:05 |
| openstackgerrit | Merged openstack/python-novaclient master: Microversion 2.86 - Extra spec validation https://review.opendev.org/718057 | 09:06 |
| brinzhang_ | bauzas: thanks, good morning, hope you have a good day :P | 09:06 |
| brinzhang_ | s/busy/because | 09:08 |
| bauzas | brinzhang_: you too (g'night) | 09:14 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: [Trivial] FUP: addressed comments in support non-admin filter instances https://review.opendev.org/718878 | 09:18 |
| bauzas | brinzhang_: just updated the commit msg to mention the change-id ^ | 09:24 |
| brinzhang_ | I have saw that change, thanks bauzas ^^ | 09:24 |
| bauzas | brinzhang_: thank you for rechecking nova patches that got trampled by the gate | 09:29 |
| brinzhang_ | bauzas: np, We should speed up patch merge, zuul does not rest, we can do it when you rest. | 09:30 |
| brinzhang_ | that we can speed up the nova release plan. | 09:31 |
| *** ociuhandu has joined #openstack-nova | 09:32 | |
| *** ociuhandu has quit IRC | 09:36 | |
| bauzas | brinzhang_: we're in European time, the gate is always less busy | 09:37 |
| bauzas | brinzhang_: you know this page, right ? http://zuul.openstack.org/status | 09:38 |
| bauzas | filter by 'nova' and you'll see all patches on different pipelines | 09:38 |
| brinzhang_ | bauzas: yes, I know, I just know how to search one patch by id | 09:38 |
| bauzas | you can also filter by project | 09:38 |
| brinzhang_ | yeah, I have tried, it can list more | 09:40 |
| brinzhang_ | it seems like not many patch running in zuul, I think I was wrong of above words | 09:40 |
| 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:56 |
| brinzhang_ | lywarwood: I will asked Qiu Fossen respin this bug, and paste the ERROR log in the bug | 09:57 |
| brinzhang_ | lyarwood: That happened in Rocky release, but we were compared in master, that also exist. | 09:59 |
| *** ociuhandu has joined #openstack-nova | 10:04 | |
| bauzas | brinzhang_: lyarwood is off as well | 10:05 |
| brinzhang_ | bauzas: yeah, I just left comment for him, we are in same bug fix ^ | 10:06 |
| *** rpittau is now known as rpittau|bbl | 10:19 | |
| *** liuyulong has quit IRC | 10:20 | |
| *** ivve has quit IRC | 11:28 | |
| *** damien_r has quit IRC | 11:47 | |
| *** ociuhandu has quit IRC | 11:50 | |
| *** ociuhandu has joined #openstack-nova | 11:51 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/train: libvirt: Calculate disk_over_committed for raw instances https://review.opendev.org/718964 | 11:55 |
| *** ociuhandu has quit IRC | 11:56 | |
| *** rpittau|bbl is now known as rpittau | 11:58 | |
| *** jraju__ has joined #openstack-nova | 12:01 | |
| *** links has quit IRC | 12:01 | |
| *** mdbooth has quit IRC | 12:26 | |
| *** martinkennelly has joined #openstack-nova | 12:38 | |
| *** yedongcan has left #openstack-nova | 13:07 | |
| *** ociuhandu has joined #openstack-nova | 13:09 | |
| *** mkrai has joined #openstack-nova | 13:10 | |
| *** zigo has quit IRC | 13:16 | |
| openstackgerrit | Merged openstack/nova master: [Trivial] FUP: addressed comments in support non-admin filter instances https://review.opendev.org/718878 | 13:18 |
| 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:18 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Pass the actual target in server group policy https://review.opendev.org/717176 | 13:31 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in server group policies https://review.opendev.org/717175 | 13:32 |
| *** mkrai has quit IRC | 13:32 | |
| *** mriedem has joined #openstack-nova | 13:32 | |
| bauzas | gmann: stephenfin is on holiday today | 13:34 |
| gmann | bauzas: ack, thanks. | 13:35 |
| bauzas | the gate is on a horrible state :/ | 13:37 |
| *** psachin has joined #openstack-nova | 13:45 | |
| *** alex_xu has joined #openstack-nova | 13:48 | |
| openstackgerrit | Alexandre arents proposed openstack/nova master: libvirt: ensure disk_over_commit is not negative https://review.opendev.org/719008 | 13:54 |
| *** dklyle has joined #openstack-nova | 13:55 | |
| *** zigo has joined #openstack-nova | 14:03 | |
| *** slaweq has joined #openstack-nova | 14:18 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in server group policies https://review.opendev.org/717175 | 14:23 |
| *** psachin has quit IRC | 14:24 | |
| *** slaweq has quit IRC | 14:27 | |
| *** ociuhandu has quit IRC | 14:28 | |
| *** ociuhandu has joined #openstack-nova | 14:28 | |
| *** slaweq has joined #openstack-nova | 14:29 | |
| *** ociuhandu has quit IRC | 14:33 | |
| *** slaweq has quit IRC | 14:33 | |
| *** ociuhandu has joined #openstack-nova | 14:34 | |
| *** ociuhandu has quit IRC | 14:38 | |
| openstackgerrit | Merged openstack/nova master: Follow-up for flavor-extra-spec-validators series https://review.opendev.org/718357 | 14:46 |
| openstackgerrit | Merged openstack/nova master: libvirt: Add support for stable device rescue https://review.opendev.org/700812 | 14:56 |
| bauzas | yay, 2 changes to go before the 2.87 microversion \o/ | 14:56 |
| *** rpittau is now known as rpittau|afk | 14:57 | |
| bauzas | oh shit, Gerrit will be restarted in less than 2 hours :( | 15:17 |
| 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 |
| bauzas | dammit. | 15:18 |
| *** JamesBenson has joined #openstack-nova | 15:24 | |
| *** ociuhandu has joined #openstack-nova | 15:24 | |
| gmann | bauzas: yeah, gate pipeline is not so heavy but its difficult to finish both pipeline before restart | 15:26 |
| bauzas | yeah, it's just we got a ton of rechecks due to spurious issues | 15:27 |
| bauzas | and this extra issue is killing me | 15:27 |
| bauzas | I was at least expecting the above change to merge before EOB | 15:27 |
| bauzas | (my own EOB which should be next hours normally) | 15:28 |
| bauzas | next hour* even | 15:28 |
| gmann | ohk. I will be online for another 6-7 hrs. i will keep eyes on that | 15:28 |
| *** tesseract has quit IRC | 15:29 | |
| bauzas | thanks | 15:30 |
| bauzas | I'll try to be around tonight | 15:30 |
| *** gyee has joined #openstack-nova | 15:31 | |
| *** ociuhandu has quit IRC | 15:35 | |
| *** ociuhandu has joined #openstack-nova | 15:35 | |
| *** ociuhandu has quit IRC | 15:40 | |
| -openstackstatus- NOTICE: review.opendev.org is being restarted for scheduled maintenance; see http://lists.opendev.org/pipermail/service-announce/2020-April/000003.html | 16:06 | |
| *** mlavalle has joined #openstack-nova | 16:09 | |
| *** slaweq has joined #openstack-nova | 16:12 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in server group policies https://review.opendev.org/717175 | 16:13 |
| *** artom has quit IRC | 16:17 | |
| openstackgerrit | Ivaylo Mitev proposed openstack/nova master: Implementing graceful shutdown. https://review.opendev.org/666245 | 16:29 |
| *** ociuhandu has joined #openstack-nova | 16:29 | |
| *** evrardjp has quit IRC | 16:37 | |
| *** evrardjp has joined #openstack-nova | 16:37 | |
| *** slaweq has quit IRC | 16:40 | |
| *** dpawlik has quit IRC | 16:53 | |
| -openstackstatus- NOTICE: etherpad.openstack.org will be offline for about 30 minutes while it is migrated to a new server with a new hostname; see http://lists.opendev.org/pipermail/service-announce/2020-April/000003.html | 17:02 | |
| *** jraju__ has quit IRC | 17:03 | |
| *** martinkennelly has quit IRC | 17:05 | |
| *** ociuhandu has quit IRC | 17:52 | |
| -openstackstatus- NOTICE: The etherpad migration is still in progress; revised estimated time of completion 18:30 UTC | 17:53 | |
| *** melwitt has quit IRC | 17:59 | |
| *** mugsie has quit IRC | 17:59 | |
| *** cmurphy has quit IRC | 17:59 | |
| *** melwitt has joined #openstack-nova | 18:03 | |
| *** mugsie has joined #openstack-nova | 18:03 | |
| *** cmurphy has joined #openstack-nova | 18:03 | |
| *** maciejjozefczyk has quit IRC | 18:16 | |
| *** slaweq has joined #openstack-nova | 18:30 | |
| *** N3l1x has quit IRC | 18:35 | |
| * bauzas crosses fingers for https://review.opendev.org/701430 which is in the gate pipeline | 19:21 | |
| *** ttsiouts has joined #openstack-nova | 19:41 | |
| *** markmcclain has quit IRC | 19:45 | |
| *** markmcclain has joined #openstack-nova | 19:46 | |
| *** artom has joined #openstack-nova | 19:55 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing quota class policies https://review.opendev.org/719095 | 20:02 |
| gmann | bauzas: be relax :) all jobs seems green on 701430. | 20:02 |
| bauzas | indeed | 20:03 |
| bauzas | 6 mins to go | 20:03 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in quota class Policies https://review.opendev.org/719096 | 20:09 |
| -openstackstatus- NOTICE: Due to a database migration error, etherpad.opendev.org is offline until further notice. | 20:09 | |
| openstackgerrit | Merged openstack/nova master: compute: Report COMPUTE_RESCUE_BFV and check during rescue https://review.opendev.org/701429 | 20:11 |
| openstackgerrit | Merged openstack/nova master: api: Introduce microverion 2.87 allowing boot from volume rescue https://review.opendev.org/701430 | 20:12 |
| bauzas | woooot ^ | 20:12 |
| openstackgerrit | Sylvain Bauza proposed openstack/python-novaclient master: Microversion 2.87 - Stable device boot from volume rescue https://review.opendev.org/714956 | 20:15 |
| bauzas | smcginnis: I just sent to the gate https://review.opendev.org/#/c/714956/10 which was holding the novaclient release https://review.opendev.org/#/c/718260/ | 20:17 |
| bauzas | smcginnis: once this one is merged, feel free to produce a new revision for the latter based on the right hash | 20:17 |
| bauzas | it's late here so I won't probably be around when it's done | 20:18 |
| bauzas | smcginnis: whooops, actually, I forgot this is Good Friday, nvm this, will see whether I can patch it if I'm around | 20:18 |
| smcginnis | bauzas: OK, if I'm around and see it, I can see about updating it. Otherwise, please update the patch when you're ready. | 20:19 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in quota class policies https://review.opendev.org/719100 | 20:20 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in servers policies https://review.opendev.org/662971 | 20:20 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix servers policy for admin_or_owner https://review.opendev.org/718501 | 20:23 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing server policies https://review.opendev.org/718348 | 20:23 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in servers Policies https://review.opendev.org/662968 | 20:23 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in servers policies https://review.opendev.org/662971 | 20:24 |
| melwitt | gmann: do you know if it's a known gate issue on stable/train this error in func test job? ERROR: Package 'openstack-placement' requires a different Python: 2.7.17 not in '>=3.6' | 20:29 |
| gmann | melwitt: is it py2 job ? | 20:29 |
| melwitt | I dunno yet :/ https://zuul.opendev.org/t/openstack/build/b460a8c59ad64e57b871aa3c00638b01 | 20:30 |
| gmann | one is tempest issue happening for tempest py2 jobs on stable/trian and stable stein - http://lists.openstack.org/pipermail/openstack-discuss/2020-April/014010.html | 20:30 |
| gmann | checking.. | 20:30 |
| melwitt | yeah this isn't tempest. first time I've seen it | 20:31 |
| gmann | most probably similar where stable py2 jobs end up using the master version which need >py3.6 | 20:32 |
| melwitt | ah I see | 20:33 |
| gmann | yeah, it is trying to clone master - https://zuul.opendev.org/t/openstack/build/b460a8c59ad64e57b871aa3c00638b01/log/job-output.txt#781 | 20:34 |
| melwitt | ok. how to fix? | 20:35 |
| gmann | checking why it does not use already cloned stable/train version of placement | 20:36 |
| melwitt | yeah ... you'd think it's all automatic to use stable/train on ... stable/train | 20:46 |
| gmann | yeah but seems this causing it https://github.com/openstack/nova/blob/28324e6aa7f1286e92c4e3292c57b9df24309cc8/.zuul.yaml#L57 | 20:46 |
| melwitt | fyi the gate bug on "timed out waiting for response from cell" will still hit some failures until we merge the stable/train fix. because it can currently fail on the old side of grenade | 20:47 |
| melwitt | oh hm ... | 20:48 |
| gmann | ohk, is it happening all time i mean timeout ? | 20:48 |
| gmann | i can try override-checkout to fix it fast and then debug why stable/train not used for required-projects | 20:48 |
| melwitt | no it is only sometimes. it's just I was confused at first when I saw the logstash still was having new hits. and I realized we would have to merge fix on stable/train too in order to see all hits stop | 20:49 |
| melwitt | since it is grenade running both stable/train and then upgrade, and then run master | 20:49 |
| melwitt | ok, cool. well, besides that I don't know why the functional test job should need openstack/placement. oh, maybe for the openstack/placement test fixtures? | 20:50 |
| gmann | ok. | 20:51 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova stable/train: Use placement stable branch version for functional py2 job https://review.opendev.org/719105 | 20:56 |
| gmann | melwitt: ^^ let's see if that fix | 20:56 |
| melwitt | cool thank you! I will keep 👀 on it | 20:57 |
| melwitt | also noting that if this is the recommended way to do it (by -infra) then we will need to fix all of our stable branches I would think | 20:58 |
| melwitt | actually I will ask them to look and confirm | 20:58 |
| gmann | yeah, required-project on stable branch should use stable version | 20:59 |
| *** ociuhandu has joined #openstack-nova | 21:01 | |
| artom | Have we considered just removing placement from required projects? The error is from cloning it, not using it. Why are we using placement in func jobs? | 21:04 |
| artom | I guess 787bb336065 is why... | 21:05 |
| *** ociuhandu has quit IRC | 21:06 | |
| melwitt | artom: there's more info in -infra, but this is some explanation https://github.com/openstack/nova/blob/stable/train/tox.ini#L76 | 21:17 |
| dansmith | artom: we use it for the fixture in func tests, IIRC | 21:19 |
| melwitt | yeah, tl;dr is we use it for the fixtures, and we don't want to install it for literally every test env, only func test envs, so we have to handle it in tox.ini | 21:29 |
| gmann | melwitt: dansmith is it too bad to hav placement installed for unit test case - https://github.com/openstack/nova/blob/stable/train/tox.ini#L83 | 21:30 |
| gmann | if no then having it in test-requirement.txt is much better solution | 21:30 |
| melwitt | yeah, I have no idea really | 21:32 |
| gmann | because we cannot solve stable branch and local run things without having constrain in tox.ini which need update on every placement new version | 21:34 |
| melwitt | unless we do the thing you suggested earlier right, just prescribe the zuul version | 21:34 |
| gmann | we can solve it for stable branch in generic way without need of update evrytime but local env run require manual handling of right version | 21:35 |
| gmann | yeah | 21:35 |
| melwitt | oh, so that's the snag. I see | 21:35 |
| gmann | that work if we leave local run case which will be only case where we want to run functional tests for stable fix locally. | 21:36 |
| gmann | which should be min cases ? | 21:37 |
| melwitt | well, I think it's pretty normal to run them locally for a stable backport. I do | 21:37 |
| gmann | ok | 21:38 |
| melwitt | I guess so far I'm thinking,if test-requirement.txt would be the idea solution, we would have to maintain version ranges for that per stable branch anyway. and if we're doing that anyway, doing it in tox.ini isn't any worse, like clarkb said | 21:39 |
| gmann | true. and having unused placement installed for unit test is not so bad | 21:40 |
| gmann | i prefer that for simplicity of maintenance | 21:40 |
| melwitt | well, I was thinking we do it in tox.ini at first and have a wider talk about putting into test-requirements.txt | 21:40 |
| gmann | hummm | 21:41 |
| gmann | in tox we need to update our-self on every branch but test-requirements.txt is automatic | 21:41 |
| melwitt | I dunno, I just don't want to make that decision with just a couple of us I guess | 21:41 |
| melwitt | oh, so you're saying we wouldn't have to do a version range in test-requirements? | 21:42 |
| *** xek has quit IRC | 21:43 | |
| gmann | yeah, that will be handled by upper-constraint file. - https://github.com/openstack/nova/blob/stable/train/tox.ini#L18 | 21:44 |
| melwitt | and clark said in tox.ini we get tox-siblings, but that doesn't do the same thing? | 21:44 |
| gmann | ah you mean just have lower cap and then tox-siblings take zuul version to use | 21:47 |
| gmann | let me try that. | 21:47 |
| melwitt | "I" don't mean it, just trying to repeat what clarkb said | 21:47 |
| melwitt | I don't understand this stuff hardly at all | 21:49 |
| gmann | that should work i think. let me push that also in train gate and see | 21:49 |
| gmann | melwitt: btw, keypair policy series is ready - https://review.opendev.org/#/c/718621/4 | 21:52 |
| gmann | and quota class -https://review.opendev.org/#/c/719100/ | 21:53 |
| melwitt | ok. if it gets too ugly, then I guess we just put into test-requirements | 21:55 |
| melwitt | cool, thanks | 21:55 |
| *** mriedem has left #openstack-nova | 22:00 | |
| *** ttsiouts has quit IRC | 22:01 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova stable/train: Use placement stable branch version for functional job https://review.opendev.org/719105 | 22:04 |
| gmann | melwitt: let's see if that work fine. I will propose it from master then backport | 22:05 |
| gmann | ^^ | 22:05 |
| melwitt | ack | 22:05 |
| eandersson | Before I forget. I know someone mentioned that they had a custom patch to expose numa pinning over api. | 22:08 |
| eandersson | Is that something someone can share? :D | 22:09 |
| eandersson | Also, what is the status of moving some of that to the placement api? | 22:09 |
| melwitt | nothing in the placement api yet, nothing in progress | 22:10 |
| melwitt | let me see if I can find the numa topo over api proposal | 22:10 |
| eandersson | I remember someone... probably years ago mentioned that they changed the api to expose it in a custom patch :D | 22:10 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in servers policies https://review.opendev.org/662971 | 22:10 |
| eandersson | thanks melwitt | 22:10 |
| eandersson | We have a use case where we might need it, and thinking either a custom patch and then maybe (no promise) try to help upstream for placement | 22:11 |
| eandersson | or a custom api outside of nova | 22:11 |
| melwitt | ok, looks like that landed https://review.opendev.org/#/q/topic:bp/show-server-numa-topology+(status:open+OR+status:merged) in train with microversion 2.78 | 22:12 |
| melwitt | https://docs.openstack.org/nova/latest/reference/api-microversion-history.html#id71 | 22:12 |
| eandersson | Awesome! | 22:12 |
| eandersson | We can just back-port that until we can upgrade to Train. | 22:13 |
| *** ociuhandu has joined #openstack-nova | 22:13 | |
| *** jamesdenton has joined #openstack-nova | 22:15 | |
| *** jamesden_ has quit IRC | 22:16 | |
| *** ttsiouts has joined #openstack-nova | 22:22 | |
| -openstackstatus- NOTICE: Maintenance on etherpad.opendev.org is complete and the service is available again | 22:25 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova stable/train: Use placement stable branch version for functional job https://review.opendev.org/719105 | 22:26 |
| *** ttsiouts has quit IRC | 22:27 | |
| *** ociuhandu has quit IRC | 22:29 | |
| *** ociuhandu has joined #openstack-nova | 22:30 | |
| openstackgerrit | melanie witt proposed openstack/nova stable/train: Reset the cell cache for database access in Service https://review.opendev.org/718934 | 22:34 |
| *** ociuhandu has quit IRC | 22:35 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in os-evacuate policies https://review.opendev.org/708237 | 22:40 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Pass the target in os-services APIs policy https://review.opendev.org/676688 | 22:46 |
| openstackgerrit | Merged openstack/python-novaclient master: Microversion 2.87 - Stable device boot from volume rescue https://review.opendev.org/714956 | 22:53 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Use placement stable version for functional job https://review.opendev.org/719115 | 22:55 |
| *** jamesden_ has joined #openstack-nova | 23:00 | |
| *** ttsiouts has joined #openstack-nova | 23:02 | |
| *** jamesdenton has quit IRC | 23:04 | |
| gmann | melwitt: it worked - https://review.opendev.org/#/c/719105/ | 23:09 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Pass the target in os-services APIs policy https://review.opendev.org/676688 | 23:12 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Pass the actual target in server group policy https://review.opendev.org/717176 | 23:13 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in server group policies https://review.opendev.org/717175 | 23:13 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in server group policies https://review.opendev.org/717175 | 23:14 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix servers policy for admin_or_owner https://review.opendev.org/718501 | 23:14 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing server policies https://review.opendev.org/718348 | 23:14 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in servers Policies https://review.opendev.org/662968 | 23:14 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in servers policies https://review.opendev.org/662971 | 23:15 |
| *** tosky has quit IRC | 23:20 | |
| *** rcernin has joined #openstack-nova | 23:20 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Use placement stable version for functional job https://review.opendev.org/719115 | 23:25 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Use placement stable version for functional job https://review.opendev.org/719119 | 23:29 |
| *** ttsiouts has quit IRC | 23:36 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova stable/train: Use placement stable version for functional job https://review.opendev.org/719120 | 23:38 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova stable/stein: Use placement stable version for functional job https://review.opendev.org/719121 | 23:40 |
| gmann | melwitt: dansmith artom : pushed the functional tox fix changes from master till stein - https://review.opendev.org/#/q/I96bb9fcb55f2a4fc3aefa01db70b45740db166c8 | 23:42 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in server external events policies https://review.opendev.org/717168 | 23:44 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Pass the actual target in server external events policy https://review.opendev.org/717169 | 23:44 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!