| *** dviroel|rover|afk is now known as dviroel|out | 00:00 | |
| opendevreview | Martin Kopec proposed openstack/tempest master: Fix test_rebuild_server by waiting for floating ip https://review.opendev.org/c/openstack/tempest/+/814085 | 00:07 |
|---|---|---|
| *** jpena|off is now known as jpena | 07:30 | |
| *** jpena is now known as jpena|lunch | 11:31 | |
| *** dviroel|out is now known as dviroel|rover | 11:41 | |
| *** jpena|lunch is now known as jpena | 12:22 | |
| *** dviroel|rover is now known as dviroel|afk | 12:54 | |
| opendevreview | Dr. Jens Harbott proposed openstack/devstack master: Create clouds.yaml early enough https://review.opendev.org/c/openstack/devstack/+/814142 | 13:07 |
| frickler | dansmith: ^^ I wanted to move the write_clouds_yaml earlier anyway in https://review.opendev.org/c/openstack/devstack/+/780417 | 14:39 |
| frickler | slaweq: the tobiko periodic jobs are in an awful state, could we stop running them until they get fixed? | 14:45 |
| dansmith | frickler: ah yeah that makes sense | 14:53 |
| sophieh20201 | Please review: https://review.opendev.org/c/openstack/tempest/+/813676 Thanks! | 15:27 |
| tosky | sophieh20201: does this set just the compute microversion in volume calls or also the volume microversion?> | 15:33 |
| opendevreview | Ade Lee proposed openstack/tempest master: Add support for ecdsa keys https://review.opendev.org/c/openstack/tempest/+/807465 | 15:39 |
| sophieh20201 | Tosky, just the compute microversion in volume calls. However, in the tempest/lib/common/api_version_utils.py, the compute microversions are defined as min_microversion, max_microversion and the volume microversions are defined as volume_min_microversion, volume_max_microversion. | 15:39 |
| sophieh20201 | So I have to change all the volume microversion names | 15:39 |
| sophieh20201 | under api/volume | 15:40 |
| opendevreview | Ade Lee proposed openstack/tempest master: Add skip for tests that wont work under fips https://review.opendev.org/c/openstack/tempest/+/810808 | 15:40 |
| opendevreview | Ade Lee proposed openstack/tempest master: DNM - Add temporary link to patched paramiko https://review.opendev.org/c/openstack/tempest/+/810809 | 15:42 |
| opendevreview | Gorka Eguileor proposed openstack/devstack master: Add LVM NVMe support https://review.opendev.org/c/openstack/devstack/+/814193 | 15:51 |
| *** dviroel|afk is now known as dviroel|rover | 15:52 | |
| gmann | sophieh20201: one, this lgtm. please update the commit msg as you are changing volume test to use lib.common.api_microversion_fixture | 16:22 |
| gmann | *done | 16:22 |
| sophieh20201 | Thanks gmann for the comments! | 16:28 |
| opendevreview | Carlos Eduardo proposed openstack/devstack-plugin-ceph master: Fix Ganesha repository broken urls https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/813446 | 16:33 |
| *** jpena is now known as jpena|off | 16:41 | |
| opendevreview | Ade Lee proposed openstack/devstack master: Add option to set chap algoorithms for iscsid for FIPS https://review.opendev.org/c/openstack/devstack/+/803800 | 16:43 |
| opendevreview | Ade Lee proposed openstack/devstack master: Add option to set chap algoorithms for iscsid for FIPS https://review.opendev.org/c/openstack/devstack/+/803800 | 16:44 |
| opendevreview | Ade Lee proposed openstack/devstack master: Add option to set chap algorithms for iscsid for FIPS https://review.opendev.org/c/openstack/devstack/+/803800 | 16:45 |
| opendevreview | Sophie Huang proposed openstack/tempest master: Use lib.commom.api_microversion_fixture to set volume, compute API microversion in volume API testing https://review.opendev.org/c/openstack/tempest/+/813676 | 17:10 |
| opendevreview | Sophie Huang proposed openstack/tempest master: Use lib.commom.api_microversion_fixture to set volume, compute API microversion in volume API testing https://review.opendev.org/c/openstack/tempest/+/813676 | 17:13 |
| opendevreview | Sophie Huang proposed openstack/tempest master: Use lib.commom.api_microversion_fixture to set volume, compute API microversion in volume API testing https://review.opendev.org/c/openstack/tempest/+/813676 | 17:13 |
| gmann | sophieh20201: thanks for update, +A | 17:38 |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org will be offline starting in 5 minutes, at 18:00 UTC, for scheduled project rename maintenance, which should last no more than an hour (but will likely be much shorter): http://lists.opendev.org/pipermail/service-announce/2021-October/000024.html | 17:59 | |
| *** dviroel|rover is now known as dviroel|rover|afk | 18:01 | |
| *** dviroel|rover|afk is now known as dviroel|rover | 18:09 | |
| melwitt | hi, does anyone know whether it's possible to define a required-projects list for all jobs in a project in a .zuul.yaml? from the documentation it looked like required-projects can't go under - project:? | 19:39 |
| melwitt | https://zuul-ci.org/docs/zuul/reference/project_def.html | 19:41 |
| gmann | melwitt: afaik it has to be per job. not sure but i think that can't be defined in template also, only in job definition. | 20:12 |
| melwitt | gmann: it's this https://review.opendev.org/c/openstack/nova/+/812236/8/.zuul.yaml which I depends-on https://review.opendev.org/c/openstack/devstack/+/812179 | 20:12 |
| melwitt | gmann: basically in the unmerged oslo.limit changes, I added new methods to interface, so need to depend on it for all jobs else get AttributeError. this is a DNM patch I put in the middle of the nova patch series | 20:14 |
| melwitt | gmann: the next thing I was thinking to try is setting OSLOLIMIT_REPO and OSLOLIMIT_BRANCH to my gerrit change | 20:16 |
| gmann | melwitt: i can see LIBS_FROM_GIT="oslo.limit" in controller localrc there then it should be added in required_projects in write-devstack-local-conf role | 20:19 |
| melwitt | gmann: that's what I thought too :( I'm really confused about what is happening. https://zuul.opendev.org/t/openstack/build/b8eb7fbcc33e46c58124cf7c87cc99e8/log/job-output.txt#5584 | 20:20 |
| melwitt | gmann: and it is in there, see https://zuul.opendev.org/t/openstack/build/b8eb7fbcc33e46c58124cf7c87cc99e8/log/controller/logs/_.localrc_auto.txt | 20:20 |
| melwitt | and while I could copy paste it for all jobs in .zuul.yaml instead of putting it under "- project:", how would I get the inherited jobs to use it? (the ones that come from the templates) | 20:21 |
| gmann | yeah, i have not tired it in template before. | 20:27 |
| gmann | write-devstack-local-conf role has it in localrc file during LIBS_FROM_GIT to required-projects conversion but seems that is not happening. checking.. | 20:29 |
| melwitt | gmann: is it because devstack-gate is no longer used? | 20:32 |
| gmann | melwitt: no, devstack-gate LIBS_FROM_GIT is all handled in devstack via role and git clone | 20:35 |
| melwitt | ack. I'm looking for where/how does $git_dest get created | 20:35 |
| gmann | it is recognized as use lib from git too https://zuul.opendev.org/t/openstack/build/b8eb7fbcc33e46c58124cf7c87cc99e8/log/job-output.txt#5562 | 20:38 |
| melwitt | yeah.. but somehow the /opt/stack/oslo.limit directory is not getting created. I don't yet find where that is supposed to be created (like for other repos too) | 20:39 |
| melwitt | oh, sorry, nvm. I think I misread the error message | 20:41 |
| melwitt | somehow $ERROR_ON_CLONE is True | 20:43 |
| gmann | melwitt: with git from lib case, either we need ERROR_ON_CLONE=False | 20:43 |
| gmann | yeah | 20:43 |
| melwitt | that's odd, I never had to do that before but I see now that it should help. thank you | 20:43 |
| gmann | it is true in devstack-base job | 20:43 |
| melwitt | huh, interesting stackrc:815:ERROR_ON_CLONE=$(trueorfalse False ERROR_ON_CLONE) | 20:44 |
| gmann | I think your devstack patch should make it work but need to check https://review.opendev.org/c/openstack/devstack/+/812179 | 20:44 |
| gmann | melwitt: https://github.com/openstack/devstack/blob/d603c2a459a211ca29662796d4939424dbbac079/.zuul.yaml#L314 | 20:44 |
| melwitt | so it defaults to False in devstack but the base job sets it to True... I wonder why? | 20:44 |
| melwitt | maybe these jobs are intended to never take libs from git. I never knew that before | 20:47 |
| melwitt | gmann++ thank you for your help, I think I have a way forward now | 20:47 |
| gmann | melwitt: yeah by default devstack based job are installing from pypi unless these libs are in required-projects or in LIBS_FROM_GIT | 20:48 |
| gmann | melwitt: if you want to add for all jobs one defined in tempest also then better is to just add required-project in devstack-base and then tests | 20:49 |
| gmann | in this itself 812179 | 20:50 |
| gmann | melwitt: also we should better add the ERROR_ON_CLONE value in error message to tell this is why devstack not cloning the repo and using from git | 20:51 |
| melwitt | gmann: ok thanks for the advice, I will try that | 20:56 |
| melwitt | re: the error message, +1 I think that could be helpful. I probably still wouldn't have realized what was happening but maybe others would heh | 20:58 |
| opendevreview | Ghanshyam proposed openstack/devstack master: Clarify error message for ERROR_ON_CLONE=True https://review.opendev.org/c/openstack/devstack/+/814233 | 21:00 |
| gmann | melwitt: ^^ please add if more values in error message can be helpful. I remember I debugged the same issue before and again forget the code about ERROR_ON_CLONE and RECLONE things | 21:01 |
| melwitt | kk | 21:03 |
| *** dviroel|rover is now known as dviroel|rover|afk | 21:08 | |
| opendevreview | Merged openstack/tempest master: Use lib.commom.api_microversion_fixture to set volume, compute API microversion in volume API testing https://review.opendev.org/c/openstack/tempest/+/813676 | 21:27 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!