| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-04-10 | |||
| 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 | |
| 21:34:40 | melwitt | unless we do the thing you suggested earlier right, just prescribe the zuul version | |
| 21:35:03 | 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:08 | gmann | yeah | |
| 21:35:18 | melwitt | oh, so that's the snag. I see | |
| 21:36:55 | 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:37:24 | gmann | which should be min cases ? | |
| 21:37:44 | melwitt | well, I think it's pretty normal to run them locally for a stable backport. I do | |
| 21:38:35 | gmann | ok | |
| 21:39:11 | 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:40:20 | gmann | true. and having unused placement installed for unit test is not so bad | |
| 21:40:51 | gmann | i prefer that for simplicity of maintenance | |
| 21:40:53 | 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:41:02 | gmann | hummm | |
| 21:41:39 | gmann | in tox we need to update our-self on every branch but test-requirements.txt is automatic | |
| 21:41:40 | melwitt | I dunno, I just don't want to make that decision with just a couple of us I guess | |
| 21:42:39 | melwitt | oh, so you're saying we wouldn't have to do a version range in test-requirements? | |
| 21:44:09 | gmann | yeah, that will be handled by upper-constraint file. - https://github.com/openstack/nova/blob/stable/train/tox.ini#L18 | |
| 21:44:29 | melwitt | and clark said in tox.ini we get tox-siblings, but that doesn't do the same thing? | |
| 21:47:21 | gmann | ah you mean just have lower cap and then tox-siblings take zuul version to use | |
| 21:47:38 | gmann | let me try that. | |
| 21:47:42 | melwitt | "I" don't mean it, just trying to repeat what clarkb said | |
| 21:49:10 | melwitt | I don't understand this stuff hardly at all | |
| 21:49:47 | gmann | that should work i think. let me push that also in train gate and see | |
| 21:52:55 | gmann | melwitt: btw, keypair policy series is ready - https://review.opendev.org/#/c/718621/4 | |