| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-04-10 | |||
| 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 | |
| 15:27:49 | bauzas | I was at least expecting the above change to merge before EOB | |
| 15:28:16 | bauzas | (my own EOB which should be next hours normally) | |
| 15:28:27 | bauzas | next hour* even | |
| 15:28:55 | gmann | ohk. I will be online for another 6-7 hrs. i will keep eyes on that | |
| 15:30:34 | bauzas | thanks | |
| 15:30:48 | bauzas | I'll try to be around tonight | |
| 16:13:45 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in server group policies https://review.opendev.org/717175 | |
| 16:29:23 | openstackgerrit | Ivaylo Mitev proposed openstack/nova master: Implementing graceful shutdown. https://review.opendev.org/666245 | |
| 20:02:06 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing quota class policies https://review.opendev.org/719095 | |
| 20:02:55 | gmann | bauzas: be relax :) all jobs seems green on 701430. | |
| 20:03:08 | bauzas | indeed | |
| 20:03:11 | bauzas | 6 mins to go | |
| 20:09:33 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in quota class Policies https://review.opendev.org/719096 | |
| 20:11:59 | openstackgerrit | Merged openstack/nova master: compute: Report COMPUTE_RESCUE_BFV and check during rescue https://review.opendev.org/701429 | |
| 20:12:06 | openstackgerrit | Merged openstack/nova master: api: Introduce microverion 2.87 allowing boot from volume rescue https://review.opendev.org/701430 | |
| 20:12:53 | bauzas | woooot ^ | |
| 20:15:15 | openstackgerrit | Sylvain Bauza proposed openstack/python-novaclient master: Microversion 2.87 - Stable device boot from volume rescue https://review.opendev.org/714956 | |
| 20:17:20 | 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:51 | bauzas | smcginnis: once this one is merged, feel free to produce a new revision for the latter based on the right hash | |
| 20:18:23 | bauzas | it's late here so I won't probably be around when it's done | |
| 20:18:53 | bauzas | smcginnis: whooops, actually, I forgot this is Good Friday, nvm this, will see whether I can patch it if I'm around | |
| 20:19:17 | 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:20:33 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in quota class policies https://review.opendev.org/719100 | |
| 20:20:45 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in servers policies https://review.opendev.org/662971 | |
| 20:23:12 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix servers policy for admin_or_owner https://review.opendev.org/718501 | |
| 20:23:34 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing server policies https://review.opendev.org/718348 | |
| 20:23:54 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in servers Policies https://review.opendev.org/662968 | |
| 20:24:09 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in servers policies https://review.opendev.org/662971 | |
| 20:29:01 | 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:44 | gmann | melwitt: is it py2 job ? | |
| 20:30:08 | melwitt | I dunno yet :/ https://zuul.opendev.org/t/openstack/build/b460a8c59ad64e57b871aa3c00638b01 | |
| 20:30:40 | 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:48 | gmann | checking.. | |
| 20:31:14 | melwitt | yeah this isn't tempest. first time I've seen it | |
| 20:32:44 | gmann | most probably similar where stable py2 jobs end up using the master version which need >py3.6 | |
| 20:33:19 | melwitt | ah I see | |
| 20:34:41 | gmann | yeah, it is trying to clone master - https://zuul.opendev.org/t/openstack/build/b460a8c59ad64e57b871aa3c00638b01/log/job-output.txt#781 | |
| 20:35:23 | melwitt | ok. how to fix? | |
| 20:36:43 | gmann | checking why it does not use already cloned stable/train version of placement | |
| 20:46:00 | melwitt | yeah ... you'd think it's all automatic to use stable/train on ... stable/train | |
| 20:46:43 | gmann | yeah but seems this causing it https://github.com/openstack/nova/blob/28324e6aa7f1286e92c4e3292c57b9df24309cc8/.zuul.yaml#L57 | |
| 20:47:16 | 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:48:01 | melwitt | oh hm ... | |
| 20:48:12 | gmann | ohk, is it happening all time i mean timeout ? | |
| 20:48:43 | gmann | i can try override-checkout to fix it fast and then debug why stable/train not used for required-projects | |
| 20:49:02 | 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:33 | melwitt | since it is grenade running both stable/train and then upgrade, and then run master | |
| 20:50:33 | 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:51:23 | gmann | ok. | |
| 20:56:16 | openstackgerrit | Ghanshyam Mann proposed openstack/nova stable/train: Use placement stable branch version for functional py2 job https://review.opendev.org/719105 | |
| 20:56:18 | gmann | melwitt: ^^ let's see if that fix | |
| 20:57:07 | melwitt | cool thank you! I will keep 👀 on it | |
| 20:57:07 | melwitt | cool thank you! I will keep 👀 on it | |
| 20:58:00 | 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:21 | melwitt | actually I will ask them to look and confirm | |
| 20:59:56 | gmann | yeah, required-project on stable branch should use stable version | |
| 21:04:29 | 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:05:36 | artom | I guess 787bb336065 is why... | |
| 21:17:02 | 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:19:28 | dansmith | artom: we use it for the fixture in func tests, IIRC | |
| 21:29:26 | 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:30:32 | 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:54 | gmann | if no then having it in test-requirement.txt is much better solution | |
| 21:32:30 | melwitt | yeah, I have no idea really | |
| 21:34:14 | 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 | |