| *** rfolco has quit IRC | 00:32 | |
| *** dsneddon has quit IRC | 01:08 | |
| *** Goneri has quit IRC | 01:49 | |
| *** Goneri has joined #oooq | 01:50 | |
| *** Goneri has quit IRC | 01:50 | |
| *** Goneri has joined #oooq | 01:59 | |
| *** Goneri has quit IRC | 01:59 | |
| *** Goneri has joined #oooq | 02:02 | |
| *** Goneri has quit IRC | 02:02 | |
| *** Goneri has joined #oooq | 02:04 | |
| *** Goneri has quit IRC | 02:20 | |
| *** Goneri has joined #oooq | 02:23 | |
| *** saneax has joined #oooq | 02:27 | |
| *** Goneri has quit IRC | 02:31 | |
| *** saneax has quit IRC | 02:33 | |
| *** saneax has joined #oooq | 02:38 | |
| *** saneax has quit IRC | 02:57 | |
| *** saneax has joined #oooq | 03:10 | |
| *** skramaja has joined #oooq | 03:56 | |
| *** udesale has joined #oooq | 04:07 | |
| *** ratailor has joined #oooq | 04:07 | |
| *** ykarel has joined #oooq | 04:26 | |
| *** Goneri has joined #oooq | 04:52 | |
| *** Goneri has quit IRC | 05:03 | |
| *** Goneri has joined #oooq | 05:16 | |
| *** ysandeep|away is now known as ysandeep | 05:19 | |
| *** Goneri has quit IRC | 05:22 | |
| *** Goneri has joined #oooq | 05:23 | |
| *** Goneri has quit IRC | 05:30 | |
| *** Goneri has joined #oooq | 05:32 | |
| *** Goneri has quit IRC | 05:34 | |
| *** Goneri has joined #oooq | 05:37 | |
| *** dsneddon has joined #oooq | 05:42 | |
| *** Goneri has quit IRC | 05:43 | |
| *** Goneri has joined #oooq | 05:46 | |
| *** dsneddon has quit IRC | 05:47 | |
| *** marios has joined #oooq | 06:02 | |
| *** udesale has quit IRC | 06:08 | |
| *** udesale has joined #oooq | 06:10 | |
| *** jfrancoa has joined #oooq | 06:11 | |
| *** jfrancoa has quit IRC | 06:14 | |
| *** jfrancoa has joined #oooq | 06:15 | |
| *** jfrancoa has quit IRC | 06:26 | |
| *** jfrancoa has joined #oooq | 06:26 | |
| *** Goneri has quit IRC | 06:26 | |
| *** jfrancoa has quit IRC | 06:26 | |
| *** jfrancoa has joined #oooq | 06:27 | |
| *** tosky has joined #oooq | 06:42 | |
| *** ysandeep is now known as ysandeep|rover | 06:46 | |
| *** Goneri has joined #oooq | 06:59 | |
| akahat | marios, ysandeep|rover, chandankumar o/ ... I made periodic-tripleo-ci-centos-8-containers-undercloud-minion-train non voting. https://review.rdoproject.org/r/#/c/28588/ | 07:06 |
|---|---|---|
| akahat | and ysandeep|rover is working on adding same job upstream | 07:06 |
| akahat | if everything looks good. then can we merge it./ | 07:07 |
| marios | akahat: ack will check | 07:07 |
| akahat | marios, thank :) | 07:07 |
| *** amoralej|off is now known as amoralej | 07:08 | |
| *** jpena|off is now known as jpena | 07:21 | |
| *** dpawlik2 has joined #oooq | 07:22 | |
| *** holser has joined #oooq | 07:23 | |
| chandankumar | akahat: \o/ | 07:32 |
| chandankumar | akahat: I have taken a look at this patch https://review.opendev.org/#/c/723876/ | 07:32 |
| chandankumar | all depends on are working fine | 07:32 |
| chandankumar | akahat: it appears to be a legit issue from ironic side https://zuul.opendev.org/t/openstack/build/d602a64aab1145619f788a7ea31e9377/log/job-output.txt#4609 | 07:34 |
| *** Goneri has quit IRC | 07:35 | |
| chandankumar | akahat: better open a bug or check on #openstack-ironic channel, I am also hitting something different issue on OSA side | 07:35 |
| chandankumar | https://review.opendev.org/#/c/742381/ ironic patch does nothing with tripleo | 07:35 |
| *** jbadiapa has joined #oooq | 07:37 | |
| *** holser_ has joined #oooq | 07:39 | |
| *** holser__ has joined #oooq | 07:41 | |
| *** holser has quit IRC | 07:41 | |
| *** bogdando has joined #oooq | 07:41 | |
| chandankumar | marios: ysandeep|rover arxcruz https://review.rdoproject.org/zuul/builds?pipeline=openstack-periodic-weekend%20 | 07:43 |
| chandankumar | new set of skip list jobs landed, you can safely ignore these jobs while ruck rovering | 07:43 |
| *** holser has joined #oooq | 07:44 | |
| *** holser_ has quit IRC | 07:44 | |
| ysandeep|rover | chandankumar, ack thanks! | 07:44 |
| *** holser__ has quit IRC | 07:45 | |
| marios | ack chandankumar | 07:47 |
| zbr | morning everyone | 07:49 |
| zbr | panda: can you sort the -1 on https://review.rdoproject.org/r/#/c/27648/ ? | 07:49 |
| *** Tengu has quit IRC | 07:51 | |
| *** Tengu has joined #oooq | 07:58 | |
| *** tosky has quit IRC | 08:03 | |
| *** Tengu has quit IRC | 08:04 | |
| *** dtantsur|afk is now known as dtantsur | 08:05 | |
| *** Tengu has joined #oooq | 08:06 | |
| *** Tengu has quit IRC | 08:11 | |
| *** ysandeep|rover is now known as ysandeep|lunch | 08:15 | |
| akahat | chandankumar, ack. | 08:29 |
| akahat | chandankumar, it is throwing __init__ error because of this: https://review.opendev.org/#/c/739779 | 08:36 |
| akahat | https://review.opendev.org/#/c/739779/2/ironic/common/hash_ring.py | 08:36 |
| chandankumar | akahat: ah, May be we need some change on tripleo heat template side | 08:39 |
| *** holser_ has joined #oooq | 08:39 | |
| *** holser has quit IRC | 08:40 | |
| *** holser_ has quit IRC | 08:40 | |
| *** ysandeep|lunch is now known as ysandeep|rover | 08:42 | |
| *** holser has joined #oooq | 08:42 | |
| *** holser_ has joined #oooq | 08:47 | |
| *** derekh has joined #oooq | 08:48 | |
| chandankumar | ysandeep|rover: bhagyashris https://review.opendev.org/#/c/738375/5 please have a look, when free | 08:49 |
| *** holser has quit IRC | 08:50 | |
| *** holser_ has quit IRC | 08:52 | |
| *** holser has joined #oooq | 08:57 | |
| chandankumar | zbr: panda marios https://review.rdoproject.org/r/#/c/28624/ patch is ready now :-) | 08:57 |
| panda | chandankumar: will stable branch all switch to the new build method too ? | 08:59 |
| chandankumar | panda: yes | 08:59 |
| chandankumar | panda: may be this week itself from train onwards | 08:59 |
| chandankumar | queens and stein will follow the old method | 09:00 |
| marios | chandankumar: ack | 09:00 |
| panda | chandankumar:wait | 09:00 |
| panda | chandankumar: I'll ask again | 09:00 |
| panda | chandankumar: will *ALL* stable branches switch to the new build method too ? | 09:00 |
| ysandeep|rover | chandankumar, our plan was to remove c7 jobs in a different patch after porting c7 to c8 jobs in different projects(triple-common,tripleo-ansible..), but if you think its better to do it here.. we will do soo.. | 09:01 |
| *** holser has quit IRC | 09:01 | |
| *** holser has joined #oooq | 09:02 | |
| chandankumar | panda: sorry, Not all branches will switch to new build method, from train onwards, it will use new build method and queens, rocky and stein will still use the old method | 09:02 |
| panda | chandankumar: ok, you need to maintain backwards compatibility, or you will not be able to promote queen, rocky and stein anymore | 09:03 |
| panda | chandankumar: commented directly on the patch | 09:05 |
| chandankumar | panda: ack, will take a look, thanks! | 09:06 |
| chandankumar | ysandeep|rover: thanks, readded my votes, I will add c7 based on hackmd so that we donot forgot | 09:08 |
| *** holser has quit IRC | 09:09 | |
| ysandeep|rover | chandankumar, ack thanks.. bhagyashris fyi.. | 09:09 |
| akahat | chandankumar, we need to update tooz here: https://opendev.org/openstack/tripleo-heat-templates/src/branch/master/lower-constraints.txt#L149 | 09:09 |
| *** holser has joined #oooq | 09:09 | |
| *** Tengu has joined #oooq | 09:12 | |
| arxcruz | sshnaidm: regarding https://review.opendev.org/#/c/742906/ do you think only this is enough? i'm trying to install it via tripleo-quickstart requirements.txt but i'm getting this: | 09:17 |
| arxcruz | 2020-07-25 16:57:10.072236 | primary | ERROR: tripleo-quickstart 2.1.2.dev403 has requirement openstacksdk>=0.12, but you'll have openstacksdk 0.11.4 which is incompatible. | 09:18 |
| arxcruz | and i'm not seeing on tq use upper-constraints | 09:18 |
| sshnaidm | arxcruz, how do you install it? | 09:19 |
| arxcruz | sshnaidm: https://536f9c61966afc49eab1-3f411ee28045e7c41608c6d36a93ae2a.ssl.cf5.rackcdn.com/740662/6/check/tripleo-ci-centos-7-containers-multinode-queens/00374b9/job-output.txt | 09:19 |
| sshnaidm | arxcruz, yeah, but how to do you install it? | 09:20 |
| arxcruz | sshnaidm: just added in tripleo-quickstart requirements.txt | 09:20 |
| sshnaidm | arxcruz, link? | 09:21 |
| arxcruz | sshnaidm: https://review.opendev.org/#/c/740662/ | 09:22 |
| sshnaidm | arxcruz, openstacksdk-0.48.0 is installed | 09:23 |
| sshnaidm | arxcruz, oh, queens.. | 09:24 |
| arxcruz | sshnaidm: where? | 09:24 |
| arxcruz | yeah, the problem is with queens :/ | 09:24 |
| sshnaidm | arxcruz, yeah, so you need to ask amoralej or jpena to add openstacksdk higher versions to queens repos | 09:24 |
| sshnaidm | arxcruz, you don't have 0.12 in queens repo | 09:24 |
| amoralej | sshnaidm, arxcruz https://github.com/openstack/requirements/blob/stable/queens/upper-constraints.txt#L414 | 09:25 |
| amoralej | upper-constraints is 0.11.4 in queens | 09:25 |
| arxcruz | yeah, that's what's messing everything | 09:26 |
| arxcruz | openstack ansible modules requires at least 0.12 | 09:26 |
| arxcruz | queens latest was 0.11.4 | 09:26 |
| amoralej | but we can't bump it in queens | 09:26 |
| amoralej | or we may breack other stuff | 09:26 |
| amoralej | yeah, see comment from sean | 09:27 |
| sshnaidm | I still think it's not really related, openstacksdk form latest versions should support all older openstack versions | 09:29 |
| sshnaidm | arxcruz, but really, where in quickstart install we use upper constraint? | 09:29 |
| arxcruz | sshnaidm: i don't know, i said, i'm not seeing tq using upper-constraint | 09:29 |
| chandankumar | akahat: regarding tooz, which version is getting installed in the deployment? | 09:30 |
| *** jfrancoa has quit IRC | 09:31 | |
| *** tosky has joined #oooq | 09:33 | |
| chandankumar | akahat: you need to update tooz on rdo side may be in rdoinfo for victoria, check with ykarel or amoralej reg: https://review.opendev.org/#/c/739779 | 09:40 |
| amoralej | rdo has 2.7.0 which is the one in u-c | 09:41 |
| *** jfrancoa has joined #oooq | 09:44 | |
| chandankumar | amoralej: https://06d1312bb3be444c75a6-5c39bede91061630f9efbbd6c20220a5.ssl.cf1.rackcdn.com/723876/14/check/tripleo-ci-centos-8-scenario012-standalone/d602a64/logs/undercloud/var/log/extra/podman/containers/ironic_api/podman_info.log | 09:46 |
| chandankumar | amoralej: python3-tooz.noarch 2.6.0-0.20200630192101.80f379b.el8 @delorean-component-common | 09:46 |
| amoralej | mmm, let me check when we updated it | 09:46 |
| amoralej | i see 2.7.0 in victoria-uc tag | 09:47 |
| amoralej | it may be missing a promotion or something like that | 09:47 |
| chandankumar | let me check common component | 09:47 |
| chandankumar | amoralej: yes, we need a promotion https://trunk.rdoproject.org/centos8-master/component/common/consistent/ | 09:49 |
| chandankumar | 2.7.0 is here | 09:49 |
| chandankumar | amoralej: thanks! | 09:49 |
| amoralej | glad to help | 09:49 |
| sshnaidm | arxcruz, https://github.com/openstack/tripleo-ci/blob/master/roles/run-test/tasks/main.yaml#L142:L142 | 10:05 |
| arxcruz | sshnaidm: oh... | 10:06 |
| sshnaidm | arxcruz, https://github.com/openstack/tripleo-ci/blob/master/roles/run-test/templates/common_vars.bash.j2#L30:L30 | 10:06 |
| arxcruz | my bad, I search for uper-constraints.txt | 10:06 |
| sshnaidm | arxcruz, and finally https://github.com/openstack/tripleo-quickstart/blob/master/quickstart.sh#L136:L136 | 10:06 |
| arxcruz | sshnaidm: any sugestion how to workaround this problem ? | 10:07 |
| sshnaidm | arxcruz, queens uses python2, so this code works ^ | 10:07 |
| sshnaidm | arxcruz, need to exclude openstacksdk from there | 10:08 |
| arxcruz | sshnaidm: from there where? | 10:08 |
| sshnaidm | arxcruz, something like PIP_CONSTRAINT= pip install openstacksdk==0.12 ? | 10:12 |
| arxcruz | sshnaidm: so, is that okay to remove the upper-constraints.txt and just use openstacksdk ? | 10:12 |
| sshnaidm | arxcruz, or create a separate reqs file "frei_requirements.txt" and PIP_CONSTRAINT= pip install -r rei_requirements.txt where we'll have all non-limited reqs | 10:13 |
| sshnaidm | arxcruz, openstacksdk from master supports queens openstack, I even have job that proves this: https://review.opendev.org/#/c/742346/ | 10:15 |
| sshnaidm | arxcruz, it's not really part of openstack release, it's operators tool | 10:15 |
| sshnaidm | and this is hard to explain | 10:15 |
| sshnaidm | people are usual to tie everything to openstack release version | 10:16 |
| *** ysandeep|rover is now known as ysandeep|afk | 11:04 | |
| arxcruz | sshnaidm: not sure if https://github.com/openstack/tripleo-quickstart/blob/master/quickstart.sh#L136:L136 is being used in queens, otherwise, the upper_constraint would have openstacksdk 0.36 | 11:18 |
| arxcruz | sshnaidm: nevermind, train is the default | 11:20 |
| arxcruz | sshnaidm: would be okay to have a if release = queens ? | 11:20 |
| sshnaidm | arxcruz, what if release queens? | 11:21 |
| arxcruz | sshnaidm: if release is queens, unset the PIP_CONSTRAINT on https://github.com/openstack/tripleo-quickstart/blob/master/quickstart.sh#L136:L136 | 11:23 |
| sshnaidm | arxcruz, I think we want to use upper-constraints everywhere, except of openstacksdk case | 11:23 |
| arxcruz | sshnaidm: how to skip only the openstacksdk ? | 11:24 |
| sshnaidm | arxcruz, like I wrote above, for example | 11:24 |
| sshnaidm | or maybe to try to convince folks to merge change for upper-constraints, but I'd rely on that | 11:25 |
| sshnaidm | s/I'd/I would NOT/ | 11:25 |
| arxcruz | odyssey4me: hey | 11:26 |
| arxcruz | odyssey4me: regarding the openstacksdk i just saw your comment | 11:27 |
| odyssey4me | o/ arxcruz | 11:27 |
| arxcruz | odyssey4me: we want to use os_tempest on queens and stein job, however, in queens we get an error because the os_* requires openstacksdk>=0.12 and installed is 0.11.36 | 11:27 |
| arxcruz | is there a way to override it on os_tempest ? | 11:27 |
| odyssey4me | arxcruz: yes, OSA does layered overrides - it's been a while since I've looked, though, so I'll have to dig into it - give me a few mins | 11:29 |
| arxcruz | odyssey4me: sure | 11:29 |
| *** ysandeep|afk is now known as ysandeep|rover | 11:29 | |
| odyssey4me | arxcruz: I think the way to do it is to implement a constraint in the parameter venv_build_global_constraints | 11:32 |
| odyssey4me | arxcruz: https://opendev.org/openstack/ansible-role-python_venv_build/src/branch/master/defaults/main.yml#L69-L72 | 11:32 |
| odyssey4me | arxcruz: that gets built into a file: https://opendev.org/openstack/ansible-role-python_venv_build/src/branch/master/tasks/python_venv_install.yml#L76-L83 | 11:33 |
| arxcruz | odyssey4me: cool, let me see if I can make it work on tempest side | 11:34 |
| odyssey4me | arxcruz: then used ahead of the other constraints: https://opendev.org/openstack/ansible-role-python_venv_build/src/branch/master/tasks/python_venv_install.yml#L100-L157 | 11:34 |
| arxcruz | odyssey4me: probably just venv_build_global_constraints: - openstacksdk>=0.12 | 11:34 |
| arxcruz | in a set_fact in tempest.yml | 11:34 |
| odyssey4me | arxcruz: or just a var in the playbook/task or group_var... less tasks :) | 11:36 |
| arxcruz | odyssey4me: only queens is having this problem, i don't want to mess with other branches | 11:36 |
| arxcruz | but hey, won't hurt to try | 11:36 |
| arxcruz | :D | 11:36 |
| arxcruz | odyssey4me: shoud this be in the include_role os_tempest or a level up ? | 11:38 |
| *** jpena is now known as jpena|lunch | 11:38 | |
| arxcruz | odyssey4me: https://github.com/openstack/tripleo-quickstart-extras/blob/master/playbooks/tempest.yml#L123 | 11:38 |
| odyssey4me | arxcruz: it looks like the convention is to use set_fact, so perhaps do that... but also this is for queens only, right, so you'll need a conditional there | 11:42 |
| arxcruz | odyssey4me: ack | 11:42 |
| arxcruz | odyssey4me: https://review.opendev.org/#/c/742624/5/playbooks/tempest.yml should do the trick right | 11:46 |
| *** sanjayu_ has joined #oooq | 11:53 | |
| weshay|ruck | ysandeep|rover, zbr both centos-7/8 train are promoted or promoting right now | 11:55 |
| *** saneax has quit IRC | 11:56 | |
| ysandeep|rover | weshay|ruck, Hey o/ do we have any plans to include c8 train status here - http://rhos-release.virt.bos.redhat.com:3030/rhosp | 11:56 |
| *** amoralej is now known as amoralej|lunch | 11:57 | |
| weshay|ruck | ysandeep|rover, we'll probably flip it to c8 vs c7 when we're ready, however.. you can view http://dashboard-ci.tripleo.org/d/QrkFP-zGz/upstream-and-rdo-promotions?orgId=1 | 11:57 |
| ysandeep|rover | weshay|ruck, ack, thanks! | 11:58 |
| chandankumar | akahat: testing sc012 here https://review.rdoproject.org/r/#/c/28717/ here we will get latest tooz | 11:58 |
| ysandeep|rover | weshay|ruck, fyi.. we hit upstream jobs failure because of new validation component - https://bugs.launchpad.net/tripleo/+bug/1889045 , taken care of - rerunning jobs to confirm jobs are green now | 11:59 |
| openstack | Launchpad bug 1889045 in tripleo "Upstream Check/Gate jobs failing during python-tripleoclient rpm build with error - No matching package to install: 'validations-common'" [Critical,Triaged] | 11:59 |
| weshay|ruck | ah.. interesting /me looks | 11:59 |
| akahat | chandankumar, ack | 11:59 |
| weshay|ruck | ysandeep|rover, zbr either of you know what's up w/ sce001/002 on master periodic? | 11:59 |
| *** udesale_ has joined #oooq | 12:00 | |
| *** rfolco has joined #oooq | 12:00 | |
| weshay|ruck | ysandeep|rover, quick mtg | 12:01 |
| ysandeep|rover | weshay|ruck, ack | 12:02 |
| *** udesale has quit IRC | 12:03 | |
| *** rlandy has joined #oooq | 12:04 | |
| rlandy | ysandeep|rover: hey - nice to see you back! | 12:05 |
| ysandeep|rover | rlandy, hey o/ thank you! | 12:05 |
| rlandy | bhagyashris: pojadhav: hi - are there ovb jobs for train centos8? | 12:07 |
| rlandy | I didn;t see one foe the baremetal pipeline | 12:07 |
| rlandy | for | 12:07 |
| bhagyashris | rlandy, in meeting will ans soon | 12:11 |
| rlandy | ysandeep|rover: weshay|ruck: we have the python-importlib deps built in rhos-17 candidate repo now - I am rerunning the rhos-17 integration line | 12:17 |
| rlandy | chandankumar: hey - how are the train c8 container build jobs? | 12:17 |
| ysandeep|rover | rlandy, ack | 12:19 |
| *** derekh has quit IRC | 12:25 | |
| chandankumar | rlandy: \o/ | 12:30 |
| chandankumar | rlandy: https://review.rdoproject.org/r/#/c/28687/ is working fine | 12:32 |
| chandankumar | for train and ussuri | 12:32 |
| chandankumar | rlandy: patches up also https://review.rdoproject.org/r/#/q/topic:stablenewcontainerworkflow+(status:open+OR+status:merged) | 12:32 |
| rlandy | https://review.opendev.org/#/c/736752/ - cool - in gates | 12:32 |
| rlandy | chandankumar: very nice! | 12:33 |
| rlandy | we need that ahead of 16.2 | 12:33 |
| *** jpena|lunch is now known as jpena | 12:34 | |
| rlandy | chandankumar: is there any other playbook change we need to port downstream other than https://code.engineering.redhat.com/gerrit/#/c/207276/? | 12:36 |
| rlandy | downstream does not inherit playbooks | 12:36 |
| chandankumar | rlandy: I think we are all set there | 12:36 |
| rlandy | ok - great | 12:37 |
| rlandy | ugh - image build failure | 12:37 |
| rlandy | home/zuul/workspace/venv/lib/python3.6/site-packages/cliff/app.py:415: ResourceWarning: unclosed file <_io.BufferedReader name=4> | 12:38 |
| chandankumar | rlandy: I will take a look in sometime, sorry got into another meeting | 12:40 |
| rlandy | np - I'm rerunning | 12:40 |
| bhagyashris | rlandy, are you talking about upstream one or rdo one ---> ovb jobs for train centos8 | 12:46 |
| rlandy | bhagyashris: https://review.rdoproject.org/zuul/builds?pipeline=openstack-component-baremetal | 12:47 |
| rlandy | component pipeline | 12:47 |
| rlandy | so RDO | 12:47 |
| rlandy | just checking | 12:47 |
| rlandy | I can add it if need be | 12:47 |
| rlandy | like a train version of periodic-tripleo-ci-centos-8-ovb-3ctlr_1comp-featureset001-baremetal-ussuri | 12:48 |
| bhagyashris | rlandy, ohh component pipeline ... ysandeep|rover has added component jobs so may be he has good idea than me | 12:48 |
| bhagyashris | rlandy, checking ... | 12:48 |
| weshay|ruck | rlandy, FYI... javier just mentioned python-importlib-resources and python-importlib-metadata should be built now | 12:49 |
| rlandy | weshay|ruck: ack ... already rerunning rhos-17 integration line | 12:49 |
| weshay|ruck | perfect | 12:49 |
| ysandeep|rover | rlandy, I haven't added ovb job in c8 train components job yet, seeing the infra issues we had lately | 12:50 |
| rlandy | ysandeep|rover: ok - I'm going to add it | 12:50 |
| ysandeep|rover | rlandy, ack | 12:50 |
| rlandy | we can comment it out of criteria if needed | 12:50 |
| ysandeep|rover | rlandy, ack | 12:51 |
| akahat | rlandy, weshay|ruck ysandeep|rover chandankumar do we have ironic multinode job? | 12:55 |
| chandankumar | akahat: I think nope, we have nova less deployment with ironic is deployed via metalsmith | 12:57 |
| weshay|ruck | akahat, multinode as in upstream multinode ( the name ) or ovb... that has multiple nodes? | 12:58 |
| weshay|ruck | upstream jobs bypass ironic entirely because of nodepool controls the nodes | 12:58 |
| weshay|ruck | you can only setup the ironic service and use tempest upstream... you can't run deployment actions w/ ironic upstream | 12:59 |
| weshay|ruck | that's one of the primary reasons for ovb | 12:59 |
| *** amoralej|lunch is now known as amoralej | 12:59 | |
| rfolco | arxcruz, rfolco, zbr, sshnaidm, rlandy, marios, ysandeep, bhagyashris, soniya29, pojadhav, akahat, weshay, chandankumar | 13:00 |
| rfolco | ping scrum | 13:00 |
| chandankumar | rfolco: sorry skipping the scrum | 13:01 |
| rfolco | ack | 13:01 |
| akahat | weshay|ruck, chandankumar got it. :) | 13:01 |
| akahat | thanks :) | 13:01 |
| weshay|ruck | https://review.opendev.org/#/c/741868/ | 13:05 |
| bhagyashris | weshay|ruck, https://hackmd.io/oFX0m-T8SHWqqwzoHmC4mg?edit | 13:18 |
| weshay|ruck | bhagyashris++ | 13:19 |
| weshay|ruck | thanks | 13:19 |
| weshay|ruck | lots of moving pieces :) | 13:19 |
| marios | recheck awoo https://review.opendev.org/#/c/742182/3 | 13:21 |
| *** derekh has joined #oooq | 13:22 | |
| *** dtantsur is now known as dtantsur|brb | 13:22 | |
| *** ratailor has quit IRC | 13:22 | |
| chandankumar | akahat: https://logserver.rdoproject.org/17/28717/1/check/periodic-tripleo-ci-centos-8-scenario012-standalone-master/d40bc8b/logs/undercloud/var/log/tempest/tempest_run.log.txt.gz with latest tooz | 13:25 |
| akahat | chandankumar, Found network None available for tenant | 13:26 |
| akahat | chandankumar, like weshay|ruck mention: you can't run deployment actions w/ ironic upstream.. so is this test expected to run? | 13:27 |
| akahat | this test try to create one baremetal node... and there it is going to ERROR state. | 13:28 |
| chandankumar | akahat: Do we need to create a different node for that? | 13:34 |
| chandankumar | may be something bigger which is currently used in standalone | 13:35 |
| chandankumar | ? | 13:35 |
| akahat | chandankumar, nodes which we are creating.. i think those are no use. | 13:36 |
| akahat | like what? | 13:36 |
| weshay|ruck | arxcruz, https://ci.centos.org/view/rdo/view/promotion-pipeline/ | 13:36 |
| chandankumar | akahat: use a different nodeset which is bigger than current one | 13:36 |
| chandankumar | which has much space where test can create the node | 13:37 |
| akahat | chandankumar, we can try that.. | 13:37 |
| *** TrevorV has joined #oooq | 13:40 | |
| chandankumar | rlandy: where we are testing 16.2 image build? | 13:41 |
| rlandy | chandankumar: nowhere yet ... https://code.engineering.redhat.com/gerrit/#/q/topic:prep-16-2+(status:open+OR+status:merged) | 13:44 |
| rlandy | chandankumar: some debast about 16.2 vs 16_2 | 13:44 |
| rlandy | debate | 13:44 |
| chandankumar | ok | 13:45 |
| chandankumar | it is going to be a version so 16.2 sounds good | 13:45 |
| rlandy | weshay|ruck: https://review.opendev.org/#/c/742575/ ( fs010) and https://review.opendev.org/#/c/742677/ ( mass removal) | 13:54 |
| rlandy | chandankumar: my patches have a chicken/egg issue. I need to duplicate config and then merge jobs and then remove old config | 13:55 |
| chandankumar | rlandy: sorry you mean downstream? | 13:55 |
| rlandy | chandankumar: ack | 13:56 |
| chandankumar | rlandy: https://review.rdoproject.org/r/#/c/28624/ promoter patch, I need to do another iteration, got some suggestion from panda | 13:57 |
| rlandy | ok | 13:57 |
| akahat | sshnaidm, o/ | 13:58 |
| sshnaidm | akahat, hi | 13:58 |
| akahat | sshnaidm, we was talking about: ironic_tempest_plugin.tests.scenario.test_baremetal_multitenancy.BaremetalMultitenancy.test_baremetal_multitenancy tests. | 13:59 |
| akahat | only this test is failing: https://logserver.rdoproject.org/17/28717/1/check/periodic-tripleo-ci-centos-8-scenario012-standalone-master/d40bc8b/logs/undercloud/var/log/tempest/tempest_run.log.txt.gz | 13:59 |
| akahat | and this is the error: State transition "BUILD/scheduling" ==> "ERROR/None" after 2 second wait | 13:59 |
| *** pojadhav is now known as pojadhav|dinner | 14:00 | |
| ysandeep|rover | weshay|ruck, https://bugs.launchpad.net/tripleo/+bug/1889045 solved for now, whenever you get time, just want to confirm with you on what new jobs we need for this component. | 14:01 |
| openstack | Launchpad bug 1889045 in tripleo "Upstream Check/Gate jobs failing during python-tripleoclient rpm build with error - No matching package to install: 'validations-common'" [Critical,Fix released] | 14:01 |
| weshay|ruck | k.. will look shortly | 14:02 |
| sshnaidm | akahat, so this is scenario test | 14:03 |
| akahat | sshnaidm, yes. | 14:04 |
| sshnaidm | akahat, have we run it always? | 14:04 |
| akahat | sshnaidm, i'm not sure when it was run with standalone tripleo. | 14:04 |
| akahat | sshnaidm, this test is passing on ironic side: https://26ac4362fb28c9ae1745-4bfcd1a0cc7c3d4f9c4976468d7d6805.ssl.cf5.rackcdn.com/741445/12/check/ironic-tempest-ipa-wholedisk-direct-tinyipa-multinode/0c9adce/testr_results.html | 14:07 |
| sshnaidm | akahat, well, I suppose they prepare nodes while installing devstack | 14:09 |
| sshnaidm | akahat, not sure we need to test scenario tests tbh.. | 14:09 |
| sshnaidm | akahat, you can look how I create fake nodes for ironic here: https://github.com/sshnaidm/ironicdev | 14:10 |
| akahat | looking. | 14:11 |
| sshnaidm | akahat, and try to play with it, but I don't know if we need to go into this in our CI | 14:11 |
| *** ysandeep|rover is now known as ysandeep|away | 14:17 | |
| *** bhagyashris is now known as bhagyashris|away | 14:24 | |
| weshay|ruck | rlandy, marios I've communicated your concerns about debug to DF. Will follow up w/ what I mentioned in the scrum | 14:42 |
| *** skramaja has quit IRC | 14:43 | |
| rlandy | weshay|ruck: thanks | 14:45 |
| weshay|ruck | marios, we wanted to break this monitoring by release right? http://dashboard-ci.tripleo.org/d/g98Hf29Zz/upgrades-cockpit?orgId=1 | 14:57 |
| weshay|ruck | I'll do that on wednesday | 14:57 |
| weshay|ruck | marios, if you think of any other improvements let me know | 14:58 |
| weshay|ruck | not sure if the promtion blockers are working there.. will investigate that as well | 14:58 |
| *** dtantsur|brb is now known as dtantsur | 15:06 | |
| marios | weshay|ruck: looking | 15:08 |
| rlandy | pojadhav|dinner: bhagyashris|away: ysandeep|away: weshay|ruck: https://review.rdoproject.org/r/28719 Add OVB jobs to CentOS 8 Train component lines | 15:09 |
| marios | weshay|ruck: nice. yeah release would be good | 15:09 |
| weshay|ruck | aye.. will revamp and ping you later this week | 15:09 |
| weshay|ruck | rlandy, 039 on tripleo as well.. or too much? | 15:10 |
| weshay|ruck | tht is the problem.. w/ everything | 15:10 |
| * zbr feeling that he wasted almost a day trying to lear how to use react-redux | 15:12 | |
| * rlandy checks ussuri patter | 15:13 | |
| rlandy | weshay|ruck: ^^ not on ussuri but ipa-multinode | 15:14 |
| rlandy | is | 15:14 |
| rlandy | not working yet on traim | 15:14 |
| rlandy | ade is on it | 15:14 |
| marios | anyone else have issues pulling tripleo-heat-templates or just me | 15:27 |
| *** bogdando has quit IRC | 15:27 | |
| *** ykarel is now known as ykarel|away | 15:33 | |
| *** pojadhav|dinner is now known as pojadhav | 15:35 | |
| *** sanjayu__ has joined #oooq | 15:49 | |
| *** sanjayu_ has quit IRC | 15:52 | |
| *** marios has quit IRC | 16:02 | |
| weshay|ruck | rlandy, that go ok? | 16:07 |
| weshay|ruck | FYI folks.. I have relocate.. out for about 30 min | 16:07 |
| rlandy | weshay|ruck: yep - all fine | 16:08 |
| rlandy | weshay|ruck: just s request for an md5 with dockerfiles | 16:08 |
| rlandy | and naming suggestion was t stick with 16.2 | 16:08 |
| rlandy | only use 16-2 for dlrn and container namespace | 16:09 |
| weshay|ruck | k.. I'll buy it | 16:09 |
| rlandy | weshay|ruck: we may upset zuul | 16:09 |
| rlandy | in which case - I said all bets are off and everything gets named with an underscaore | 16:09 |
| weshay|ruck | we have time to play | 16:09 |
| rlandy | weshay|ruck: sending you one doc in pvt | 16:10 |
| weshay|ruck | k.. thanks | 16:10 |
| weshay|ruck | k.. relocating.. back in 30 | 16:11 |
| rlandy | k | 16:11 |
| *** ykarel|away has quit IRC | 16:15 | |
| *** udesale_ has quit IRC | 16:21 | |
| *** amoralej is now known as amoralej|off | 16:25 | |
| weshay|ruck | back | 16:35 |
| *** Goneri has joined #oooq | 16:38 | |
| *** jpena is now known as jpena|off | 16:41 | |
| *** sanjayu__ has quit IRC | 16:48 | |
| *** dtantsur is now known as dtantsur|afk | 16:59 | |
| *** derekh has quit IRC | 17:00 | |
| rlandy | weshay|ruck: did you create the tripleorhos-17 namespace on the internal registry? | 17:05 |
| rlandy | we need a tripleorhos-16-2 | 17:05 |
| weshay|ruck | 0/ | 17:05 |
| weshay|ruck | rlandy, I did but I don't remember if we just asked for it or not /me looks at infra doc | 17:06 |
| *** jbadiapa has quit IRC | 17:12 | |
| rlandy | weshay|ruck: question about downstream branches ... | 17:25 |
| rlandy | we only really have periodic jobs there | 17:25 |
| rlandy | and we left the branches as .* | 17:25 |
| rlandy | I guess 17 should be rhos-17.0-trunk-patches? | 17:26 |
| rlandy | and 16.2 rhos-16.2-trunk-patches? | 17:26 |
| weshay|ruck | rlandy, I think the trunk-patches can be implied | 17:26 |
| rlandy | meaning? | 17:26 |
| weshay|ruck | in the job name | 17:26 |
| rlandy | we have them defined on the repos we take | 17:26 |
| rlandy | so leave the jobs with branches: .*? | 17:27 |
| weshay|ruck | depending on our bandwidth, check jobs may be in the future.. but it's not something I'm hearing about so .. low priority | 17:28 |
| weshay|ruck | but like migi really wanted check jobs... right so .. it may be in our future... if I'm answering the right question | 17:28 |
| weshay|ruck | we need to use the right branches of course.. but the specific name does not need to be reflected in the job name | 17:29 |
| weshay|ruck | still not 100% sure if I'm speaking to the question though tbh | 17:29 |
| migi | rlandy: weshay|ruck: hmm I would double check with jschlueter or jjson as there are also non -trunk planned to be used if I am correct | 17:30 |
| migi | rlandy: weshay|ruck: currently for 15+ the -trunk is implied | 17:30 |
| rlandy | migi: weshay|ruck: right now it's periodic only ... | 17:30 |
| migi | also this is for compute and may be different for other | 17:30 |
| rlandy | https://code.engineering.redhat.com/gerrit/#/c/207282/2/zuul.d/tripleo-build-containers.yaml | 17:31 |
| rlandy | ^^ so that job | 17:31 |
| weshay|ruck | rlandy, ya.. that review looks fine to me | 17:32 |
| rlandy | ok | 17:32 |
| rlandy | we can change it going forward | 17:32 |
| weshay|ruck | if we did put up a check job... we should revisit the naming convention | 17:32 |
| weshay|ruck | for the check job only | 17:32 |
| weshay|ruck | naming sucks | 17:32 |
| chandankumar | weshay|ruck: https://review.opendev.org/#/c/743258/ | 17:34 |
| *** dsneddon has joined #oooq | 17:42 | |
| rlandy | weshay|ruck: one more - for periodic, do we need to define a override-checkout: "stable/train" and vars: branch_override: 'stable/train' | 17:44 |
| rlandy | or maybe that's branch_override trunk-*-patches | 17:45 |
| rlandy | https://github.com/rdo-infra/rdo-jobs/blob/master/zuul.d/integration-jobs-c8-train.yaml#L11 | 17:46 |
| rlandy | ^^ defined there | 17:46 |
| *** ykarel has joined #oooq | 17:53 | |
| *** ykarel has quit IRC | 18:01 | |
| weshay|ruck | that has to be the real branch name I think | 18:12 |
| migi | weshay|ruck: rlandy: ok but nothing stops to fix the name and in case other component or second branch name requires add it as separate job | 18:21 |
| rlandy | yeah - can change it later | 18:22 |
| *** tosky has quit IRC | 18:33 | |
| weshay|ruck | rlandy, ok.. have you ever had to change the container_prepare vars in a testproject, trying to override the registry server | 18:46 |
| weshay|ruck | rlandy, don't see it here https://opendev.org/openstack/tripleo-ci/src/commit/5a902b351f3728a95e4a989527178c66815bdc54/roles/run-test/tasks/main.yaml#L8 | 18:46 |
| rlandy | weshay|ruck: never - why do you need to override the server? | 18:47 |
| weshay|ruck | rlandy, I'm trying out Evilien's job that builds containers locally and deploys standalone in one run | 18:47 |
| rlandy | those vars get set elsewhere | 18:47 |
| weshay|ruck | https://review.rdoproject.org/r/#/c/28727/ | 18:47 |
| weshay|ruck | build_container_images: true | 18:47 |
| rlandy | weshay|ruck: yeah - let me dig that var out | 18:47 |
| weshay|ruck | and it works | 18:48 |
| weshay|ruck | but the deployment is trying to pull from the rdo registry | 18:48 |
| rlandy | internally it's a differ registry | 18:48 |
| weshay|ruck | ya | 18:48 |
| rlandy | getting | 18:48 |
| weshay|ruck | oh.. I need to create a namespace | 18:48 |
| * weshay|ruck looks for that as you look up var | 18:48 | |
| rlandy | http://git.app.eng.bos.redhat.com/git/tripleo-environments.git/tree/config/release/rhos-17.yml#n8 | 18:51 |
| weshay|ruck | tripleorhos-16 ? | 18:51 |
| weshay|ruck | is that good | 18:51 |
| rlandy | weshay|ruck: was expecting tripleorhos-16-2 | 18:52 |
| weshay|ruck | rlandy, can I set docker_registry_host in zuul? | 18:52 |
| rlandy | but that is fine | 18:52 |
| weshay|ruck | or I can just put up another change and depends on it | 18:52 |
| weshay|ruck | ok.. 16-2 | 18:52 |
| weshay|ruck | k | 18:52 |
| rlandy | weshay|ruck: so depends where it is used | 18:52 |
| rlandy | if within toci | 18:52 |
| rlandy | you need to featureset_override | 18:53 |
| rlandy | if outside toci, zuul is ok | 18:53 |
| rlandy | OR | 18:53 |
| rlandy | you can be fancy | 18:53 |
| rlandy | and allow for a job var override | 18:53 |
| rlandy | weshay|ruck: if it's just one test run, | 18:53 |
| rlandy | add a DNM change to the release file | 18:54 |
| * rlandy checks the review | 18:54 | |
| rlandy | weshay|ruck: in this case, I would add a featureset_override | 18:55 |
| rlandy | as we might want to do this check more often | 18:55 |
| rlandy | weshay|ruck: and you probably also want docker_registry_namespace | 18:56 |
| rlandy | weshay|ruck: is this the same load balancer error we saw upstream: https://sf.hosted.upshift.rdu2.redhat.com/logs/84/195884/23/check/periodic-tripleo-ci-rhel-8-scenario010-standalone-rhos-17/0cfef35/logs/undercloud/var/log/tempest/tempest_run.log | 19:15 |
| * rlandy wants to promote rhos-17 w/o scenario010 | 19:16 | |
| rlandy | think octavia component will clear it | 19:16 |
| cgoncalves | rlandy, yes, same. fixed merged in octavia master and stable/train | 19:18 |
| rlandy | cgoncalves: k - great - downstream just needs to run the same promotion up/down process | 19:21 |
| weshay|ruck | rlandy, sorry.. network went poof | 19:25 |
| weshay|ruck | back | 19:25 |
| rlandy | weshay|ruck: no worries | 19:26 |
| rlandy | this 16.2 16-2 is already making me crazy | 19:26 |
| weshay|ruck | heh.. k.. you have plenty of time :) | 19:26 |
| rlandy | weshay|ruck: also ... release: train (not stable/train)? | 19:28 |
| weshay|ruck | rlandy, in config or what? | 19:28 |
| rlandy | weshay|ruck: in job configs | 19:29 |
| rlandy | looks like we define release as train | 19:29 |
| rlandy | osp_release: rhos-16.2 | 19:29 |
| weshay|ruck | aye... k | 19:29 |
| weshay|ruck | rlandy, just FYI.. no vote required.. I think this will get it done for my needs https://review.opendev.org/#/c/743325/ | 19:39 |
| rlandy | weshay|ruck: reasonable - would put the same change on master.yaml | 19:40 |
| weshay|ruck | rlandy, it's already there :) | 19:42 |
| weshay|ruck | across all the non-periodic releases | 19:42 |
| * weshay|ruck learns something new every day | 19:42 | |
| rlandy | lol | 19:42 |
| rlandy | ok | 19:42 |
| *** jfrancoa has quit IRC | 20:13 | |
| rlandy | weshay|ruck: we're going to have to do some fake promotions | 20:40 |
| rlandy | to get to a point where we can test the integration line | 20:40 |
| weshay|ruck | rlandy, k.. I'll have to think about that later.. docker.io finally started to rate limit our container pulls | 20:41 |
| rlandy | weshay|ruck: no rush - will ping about that tomorrow | 20:41 |
| *** TrevorV has quit IRC | 21:00 | |
| weshay|ruck | rlandy, you have a min.. need to bounce an idea around | 21:08 |
| rlandy | weshay|ruck: what's up? | 21:13 |
| weshay|ruck | rlandy, hey.. let me move to quiet place | 21:18 |
| * weshay|ruck sends mtg | 21:18 | |
| weshay|ruck | https://meet.google.com/efr-bwub-pzw | 21:18 |
| weshay|ruck | rlandy, ^ | 21:19 |
| rlandy | joining | 21:19 |
| *** tosky has joined #oooq | 21:25 | |
| rlandy | weshay|ruck: how's the local test looking? | 22:08 |
| weshay|ruck | good.. | 22:09 |
| weshay|ruck | this works fine | 22:09 |
| weshay|ruck | when: | 22:09 |
| weshay|ruck | - "(job.build_container_images|default(false)|bool) or (build_container_images|default(false)|bool)" | 22:09 |
| weshay|ruck | almost have the patch.. adding a default in tqe/common | 22:09 |
| weshay|ruck | rlandy, not sure about | 22:14 |
| weshay|ruck | <pre><font color="#AFC4DB"><b>docker_registry_host</b></font><font color="#FFD7D7">:</font> <font color="#9B859D">"{{ job.build_container_images|default(false)|bool | ternary('127.0.0.1:5001', 'docker.io') }}"</font> | 22:14 |
| weshay|ruck | </pre> | 22:14 |
| weshay|ruck | sorry | 22:14 |
| weshay|ruck | docker_registry_host: "{{ job.build_container_images|default(false)|bool | ternary('127.0.0.1:5001', 'docker.io') }}" | 22:14 |
| rlandy | what's wrong with that? | 22:18 |
| rlandy | oh I see | 22:18 |
| weshay|ruck | that's fine, but I have to add to job.build_contianer_images .. so both job. and build_container work | 22:18 |
| rlandy | or within the definition | 22:19 |
| rlandy | docker_registry_host: >- | 22:19 |
| weshay|ruck | aye | 22:19 |
| weshay|ruck | will try >- now | 22:19 |
| rlandy | {% if job.build_container_images is defined %} | 22:19 |
| rlandy | {elif | 22:20 |
| rlandy | weshay|ruck: ^^ | 22:20 |
| rlandy | .build_container_images is defined | 22:20 |
| weshay|ruck | and if job is defined | 22:20 |
| rlandy | else | 22:20 |
| weshay|ruck | for non zuul | 22:20 |
| rlandy | if, elif and else | 22:20 |
| rlandy | make sense? | 22:21 |
| rlandy | oh dear internal promoter | 22:21 |
| rlandy | better spend some time there tomorrow | 22:22 |
| weshay|ruck | k.. so it goes | 22:22 |
| rlandy | doesn't work anymore /o\ | 22:23 |
| rlandy | lookng ofr the .j2 | 22:23 |
| rlandy | will have to reget all those patches | 22:23 |
| weshay|ruck | I mean I can remove that from the release files | 22:23 |
| rlandy | weshay|ruck: ok | 22:24 |
| rlandy | will look at patch when ready | 22:24 |
| weshay|ruck | docker_registry_host: >- | 22:29 |
| weshay|ruck | {% if job is defined and | 22:29 |
| weshay|ruck | job.build_container_images is defined and | 22:29 |
| weshay|ruck | job.build_container_images|bool or | 22:29 |
| weshay|ruck | build_container_images|bool -%} | 22:29 |
| weshay|ruck | '127.0.0.1:5001' | 22:29 |
| weshay|ruck | {%- else -%} | 22:29 |
| weshay|ruck | 'docker.io' | 22:29 |
| weshay|ruck | {%- endif -%} | 22:29 |
| weshay|ruck | seems to work w/ or w/o job. | 22:30 |
| rlandy | build_container_images|default(false) | 22:30 |
| rlandy | build_container_images|default(false)|bool | 22:30 |
| weshay|ruck | aye | 22:30 |
| weshay|ruck | will update | 22:31 |
| rlandy | isn't this fun | 22:32 |
| weshay|ruck | heh | 22:32 |
| weshay|ruck | rlandy, https://review.opendev.org/#/q/topic:dos_docker.io+(status:open+OR+status:merged) | 22:42 |
| rlandy | looking | 22:45 |
| rlandy | weshay|ruck: you only changed the master file but ... | 22:46 |
| rlandy | https://review.opendev.org/#/c/743432/2/roles/extras-common/defaults/main.yml | 22:46 |
| rlandy | would include ussuri | 22:46 |
| weshay|ruck | rlandy, ya.. it's turned on for ussuri in tqe | 22:46 |
| weshay|ruck | interested in seeing how we want to turn it on | 22:47 |
| weshay|ruck | from reviews | 22:47 |
| rlandy | https://review.opendev.org/#/c/743433/ | 22:47 |
| rlandy | where's the equivalent ussuri release change? | 22:47 |
| rlandy | am I being stupid? | 22:47 |
| rlandy | by default on in master and ussuri from tqe ... ok | 22:48 |
| rlandy | gets picked up in | 22:48 |
| rlandy | https://review.opendev.org/#/c/743432/2/roles/standalone/tasks/main.yml | 22:48 |
| rlandy | ok | 22:48 |
| weshay|ruck | aye | 22:49 |
| rlandy | weshay|ruck: chat? | 22:50 |
| weshay|ruck | sure.. sec | 22:50 |
| rlandy | https://github.com/openstack/tripleo-quickstart/blob/master/config/release/tripleo-ci/CentOS-8/ussuri.yml#L9 | 22:51 |
| weshay|ruck | rlandy, https://meet.google.com/jfm-mduk-juv | 22:51 |
| *** rlandy is now known as rlandy|bbl | 23:03 | |
| *** sanjayu__ has joined #oooq | 23:10 | |
| *** sanjayu__ has quit IRC | 23:10 | |
| *** tosky has quit IRC | 23:25 | |
| *** holser has quit IRC | 23:30 | |
| *** holser has joined #oooq | 23:34 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!