*** irclogbot_0 has quit IRC | 00:16 | |
*** rlandy has joined #oooq | 00:18 | |
*** rlandy has quit IRC | 00:18 | |
*** irclogbot_1 has joined #oooq | 00:19 | |
*** rfolco has quit IRC | 00:33 | |
*** saneax has joined #oooq | 02:21 | |
*** skramaja has joined #oooq | 03:39 | |
*** pojadhav has joined #oooq | 03:40 | |
*** saneax has quit IRC | 03:58 | |
*** raukadah is now known as chandankumar | 04:04 | |
*** ratailor has joined #oooq | 04:39 | |
*** ykarel has joined #oooq | 04:43 | |
*** jfrancoa has joined #oooq | 04:50 | |
*** skramaja_ has joined #oooq | 04:50 | |
*** skramaja has quit IRC | 04:51 | |
*** ysandeep|away is now known as ysandeep | 04:59 | |
*** ysandeep is now known as ysandeep|rover | 04:59 | |
*** udesale has joined #oooq | 05:48 | |
*** saneax has joined #oooq | 05:49 | |
*** soniya29 has joined #oooq | 06:09 | |
*** jfrancoa has quit IRC | 06:23 | |
*** marios has joined #oooq | 06:27 | |
*** ykarel_ has joined #oooq | 06:31 | |
*** ykarel has quit IRC | 06:33 | |
*** jfrancoa has joined #oooq | 06:38 | |
*** holser has joined #oooq | 06:53 | |
*** bogdando has joined #oooq | 07:02 | |
*** ykarel_ is now known as ykarel | 07:08 | |
*** amoralej|off is now known as amoralej | 07:16 | |
*** dtantsur|afk is now known as dtantsur | 07:27 | |
*** yolanda has joined #oooq | 07:31 | |
*** tosky has joined #oooq | 07:34 | |
*** derekh has joined #oooq | 07:39 | |
*** jpena|off is now known as jpena | 07:46 | |
*** soniya29 has quit IRC | 07:49 | |
*** jbadiapa has joined #oooq | 07:51 | |
*** soniya29 has joined #oooq | 07:51 | |
chandankumar | arxcruz: Hello | 07:55 |
---|---|---|
chandankumar | arxcruz: I think i got the issue why skip list is not working | 07:56 |
chandankumar | arxcruz: Here is https://opendev.org/openstack/tripleo-quickstart-extras/src/branch/master/playbooks/tempest.yml#L100 -> skiplist is getting generated | 07:56 |
chandankumar | arxcruz: but in fs052 https://opendev.org/openstack/tripleo-quickstart/src/branch/master/config/general_config/featureset052.yml#L49 is passed which is used in validate tempest | 07:57 |
chandankumar | that's why it is not working | 07:58 |
arxcruz | chandankumar: yup, i think we need to move everything to tempest.yml | 07:59 |
arxcruz | chandankumar: all the logic | 07:59 |
chandankumar | I am working on a patch | 07:59 |
chandankumar | arxcruz: can we get one more feature in tempest skip list, once skip lsit tests get generted, can we dump the output in a file also? | 08:02 |
arxcruz | chandankumar we can do it in the task on tempest | 08:02 |
arxcruz | chandankumar: https://github.com/openstack/tripleo-quickstart-extras/blob/master/playbooks/tempest.yml#L100 | 08:03 |
arxcruz | chandankumar: just need to write a file with the content of tempest_skip_register.skipped_tests | 08:03 |
chandankumar | arxcruz: yes | 08:10 |
arxcruz | chandankumar: "[]" as list on your patch is that valid ? | 08:17 |
chandankumar | arxcruz: nice catch, done | 08:19 |
*** derekh has quit IRC | 08:34 | |
*** ysandeep|rover is now known as ysandeep|lunch | 08:48 | |
akahat | chandankumar, marios, ysandeep|lunch, zbr|ruck, please take a when you are free. https://review.rdoproject.org/r/#/c/28588/ | 08:57 |
marios | akahat: ack | 09:00 |
*** pojadhav is now known as pojadhav|lunch | 09:03 | |
arxcruz | zbr|ruck: is it okay to have inner blocks in ansible ? | 09:13 |
marios | arxcruz: afaik yes | 09:19 |
arxcruz | thanks | 09:20 |
*** sshnaidm|off is now known as sshnaidm | 09:27 | |
*** ysandeep|lunch is now known as ysandeep | 09:30 | |
ysandeep | akahat, already reviewed on friday, lgtm +1 | 09:30 |
*** ysandeep is now known as ysandeep|rover | 09:31 | |
*** pojadhav|lunch is now known as pojadhav | 09:42 | |
arxcruz | chandankumar: i think we should land my patch first before yours https://review.opendev.org/#/c/741905/ | 09:50 |
arxcruz | chandankumar: this will consolidate validate-tempest and os_tempest in one single place, making easy to transition all the jobs to os_tempest | 09:51 |
chandankumar | arxcruz: sure | 09:51 |
chandankumar | arxcruz: I am still testing the stuff | 09:51 |
arxcruz | chandankumar: in this way, i can use this playbook to write the clouds.yaml for the old jobs | 09:51 |
arxcruz | chandankumar: yup, mine still testing as well | 09:51 |
chandankumar | arxcruz: :-) | 09:52 |
*** ratailor has quit IRC | 09:58 | |
*** ratailor_ has joined #oooq | 09:58 | |
*** tosky has quit IRC | 10:08 | |
*** holser has quit IRC | 10:08 | |
*** irclogbot_1 has quit IRC | 10:08 | |
*** fuzzball81 has quit IRC | 10:08 | |
pojadhav | chandankumar, marios : Please add in your review list https://review.rdoproject.org/r/#/c/28603/ | 10:09 |
*** pojadhav is now known as pojadhav|afk | 10:10 | |
*** tosky has joined #oooq | 10:10 | |
*** holser has joined #oooq | 10:10 | |
*** irclogbot_1 has joined #oooq | 10:10 | |
*** fuzzball81 has joined #oooq | 10:10 | |
marios | ack pojadhav|afk | 10:10 |
*** derekh has joined #oooq | 10:20 | |
zbr|ruck | chandankumar: basic: https://review.rdoproject.org/r/#/c/28587/ | 10:21 |
zbr|ruck | and https://review.rdoproject.org/r/#/c/28607/ | 10:22 |
chandankumar | pojadhav|afk: zbr|ruck commented back | 10:26 |
zbr|ruck | chandankumar: not sure why not +2 on the one with tasks | 10:41 |
chandankumar | zbr|ruck: https://review.rdoproject.org/r/#/c/28607/ I think we can delete this https://review.rdoproject.org/r/#/c/28607/1/molecule/promoter/functional_tests.yml file | 10:42 |
chandankumar | since we moved the tasks to verify | 10:42 |
chandankumar | then I can give +2 | 10:42 |
chandankumar | otherwise we need to submit another review to remove that | 10:43 |
zbr|ruck | i think that there is a confusion, same tasks run, is only that we no longer keep them in outside tasks file. | 10:47 |
zbr|ruck | check zuul result https://review.rdoproject.org/r/#/c/28607/ | 10:47 |
chandankumar | zbr|ruck: quick call? | 10:53 |
*** ysandeep|rover is now known as ysandeep|afk | 11:04 | |
*** pojadhav|afk is now known as pojadhav | 11:07 | |
pojadhav | chandankumar, worked on review comment.. please have a look https://review.rdoproject.org/r/28603 | 11:09 |
*** ysandeep|afk is now known as ysandeep|rover | 11:31 | |
*** pojadhav is now known as pojadhav|brb | 11:33 | |
*** weshay_pto is now known as weshay_ | 11:34 | |
weshay_ | 0/ | 11:35 |
ysandeep|rover | weshay_, o/ welcome back | 11:35 |
zbr|ruck | weshay_: welcome back! and thanks for doing the CIX. i am going for lunch now. | 11:35 |
weshay_ | thank you :) | 11:35 |
weshay_ | k | 11:36 |
*** jpena is now known as jpena|lunch | 11:45 | |
*** rfolco has joined #oooq | 11:59 | |
*** yolanda has quit IRC | 12:14 | |
*** yolanda has joined #oooq | 12:15 | |
*** Goneri has joined #oooq | 12:17 | |
rfolco | zbr|ruck, can you tell what you tried on https://review.rdoproject.org/r/#/c/28597/ | 12:22 |
rfolco | zbr|ruck, I mean, those are all good valid, but it broke the promotion :( | 12:26 |
*** jpena|lunch is now known as jpena | 12:32 | |
*** pojadhav|brb is now known as pojadhav | 12:33 | |
*** amoralej is now known as amoralej|lunch | 12:40 | |
*** rlandy has joined #oooq | 12:52 | |
*** rlandy is now known as rlandy|ruck | 12:54 | |
rlandy|ruck | zbr|ruck: ysandeep|rover: hello | 12:54 |
rlandy|ruck | how are things? | 12:54 |
rlandy|ruck | weshay_: welcome back | 12:54 |
weshay_ | thanks | 12:55 |
weshay_ | thanks all for the cover while I was PTO | 13:00 |
weshay_ | https://photos.app.goo.gl/31xgaN1tetTVWn2t8 | 13:00 |
weshay_ | is the results of the week | 13:00 |
ysandeep|rover | rlandy|ruck, things are cool, no new issues today, working to fix existing onces | 13:01 |
rlandy|ruck | weshay_: wow - a whole professional building crew there | 13:01 |
rlandy|ruck | ysandeep|rover: lol - well we got enough from last weel | 13:03 |
rlandy|ruck | see my sob story notes | 13:03 |
*** udesale_ has joined #oooq | 13:03 | |
rlandy|ruck | I'm just going to run the train/stein/rocky/queens last few jobs on rdocloud to clear promotions | 13:03 |
rlandy|ruck | this is crazy | 13:03 |
ysandeep|rover | rlandy|ruck, ack | 13:04 |
rlandy|ruck | we keep missing the upload job | 13:04 |
rlandy|ruck | ysandeep|rover: master is a mess | 13:04 |
rlandy|ruck | will deal with that afterwards | 13:05 |
rlandy|ruck | weshay_: remember the whole component pipeline discussion about one component blocking the other??? | 13:06 |
rlandy|ruck | well we have two such situations now | 13:06 |
*** udesale has quit IRC | 13:06 | |
rlandy|ruck | time to deal with the future | 13:06 |
ysandeep|rover | rlandy|ruck, fyi.. regarding https://bugs.launchpad.net/tripleo/+bug/1887856 - to fix issues we hit in integratrion line, Yatin proposed https://review.rdoproject.org/r/#/c/28604/ which should hopefully clear the issue, we retriggered master main integration line a while back to confirm it works. | 13:09 |
openstack | Launchpad bug 1887856 in tripleo "centos-8 master tripleo component tests are failing with "ModuleNotFoundError: No module named 'blazarclient'"" [Critical,In progress] - Assigned to Ronelle Landy (rlandy) | 13:09 |
*** rascasoft has joined #oooq | 13:10 | |
rlandy|ruck | ykarel++ thanks | 13:10 |
ykarel | rlandy|ruck, ysandeep|rover jobs are started resulting into green, for osp17 u will need similar fixes | 13:17 |
*** skramaja_ has quit IRC | 13:17 | |
rlandy|ruck | ykarel: awesome thanks | 13:18 |
*** ratailor_ has quit IRC | 13:19 | |
*** saneax has quit IRC | 13:34 | |
*** derekh has quit IRC | 13:34 | |
*** saneax has joined #oooq | 13:35 | |
*** amoralej|lunch is now known as amoralej | 13:35 | |
*** TrevorV has joined #oooq | 13:35 | |
sshnaidm | rlandy|ruck, https://review.rdoproject.org/r/#/c/28387/ | 13:37 |
rlandy|ruck | ykarel: https://review.opendev.org/#/c/741633/ is very red in the gates ... | 13:46 |
rlandy|ruck | dlrn build | 13:47 |
*** derekh has joined #oooq | 14:06 | |
ykarel | rlandy|ruck, looking | 14:18 |
ykarel | rlandy|ruck, looks related to patch, will abort for now and revisit | 14:24 |
rlandy|ruck | ykarel: ack - thanks | 14:25 |
weshay_ | marios, https://hackmd.io/HQ5hyGAOSuG44Le2x6YzUw | 14:26 |
zbr|ruck | rfolco: i would like to talk regarding our own ensure-docker and what we can do to avoid recreating roles that we have on zuul. | 14:35 |
zbr|ruck | can we have a quick meeting, others welcomed | 14:35 |
rfolco | zbr|ruck, chat | 14:36 |
rfolco | https://meet.google.com/oiv-geho-mai | 14:36 |
zbr|ruck | i am inclined to try to re-use roles from zuul-jobs because *others* maintain them | 14:36 |
zbr|ruck | and they are quire well tested | 14:36 |
rfolco | zbr|ruck, ok lets chat, I have some special requirements | 14:37 |
zbr|ruck | at this moment these roles are not published to galaxy but this may chance in the future | 14:37 |
zbr|ruck | but we could just install them using sideloading, like we already did so far with other ansible repos. | 14:38 |
rfolco | zbr|ruck, you come? | 14:38 |
zbr|ruck | yep | 14:38 |
*** ysandeep|rover is now known as ysandeep|away | 14:47 | |
*** ykarel is now known as ykarel|away | 14:50 | |
*** ysandeep|away is now known as ysandeep | 14:54 | |
*** Tengu has quit IRC | 15:04 | |
*** Tengu has joined #oooq | 15:05 | |
*** ykarel|away has quit IRC | 15:11 | |
*** weshay_ is now known as weshay|ruck | 15:12 | |
weshay|ruck | zbr|ruck, rlandy|ruck /me checking in as ruck/rover | 15:12 |
rlandy|ruck | weshay|ruck: k - want me to check out then? | 15:13 |
*** zbr|ruck is now known as zbr|rover | 15:13 | |
weshay|ruck | zbr|ruck, rlandy|ruck sandeep has a medical issue he needs to attend to | 15:13 |
rlandy|ruck | weshay|ruck: ack - np | 15:14 |
zbr|rover | ack | 15:14 |
weshay|ruck | rlandy|ruck, may still need ur help.. we'll see how it goes | 15:14 |
rlandy|ruck | sure | 15:14 |
rlandy|ruck | weshay|ruck: arxcruz: we have an error in ci.cents wrt skip list ... | 15:15 |
rlandy|ruck | https://ci.centos.org/job/tripleo-quickstart-promote-master-current-tripleo-delorean-minimal/772/console | 15:15 |
rlandy|ruck | legit? | 15:16 |
rlandy|ruck | 10:47:01 The task includes an option with an undefined variable. The error was: 'zuul' is undefined | 15:16 |
weshay|ruck | arxcruz, that looks legit | 15:16 |
*** ysandeep is now known as ysandeep|away | 15:17 | |
rlandy|ruck | arxcruz: same in ussuri | 15:17 |
weshay|ruck | you need to handle non zuul workflows :) | 15:17 |
rlandy|ruck | ok ... looking into it | 15:17 |
weshay|ruck | thanks | 15:17 |
rlandy|ruck | weshay|ruck: pls pencil me in for this afternoon - want to review all the promotion criteria with you | 15:20 |
weshay|ruck | aye | 15:22 |
rlandy|ruck | arxcruz: fyi ... https://bugs.launchpad.net/tripleo/+bug/1888244 - putting in a patch for your review | 15:25 |
openstack | Launchpad bug 1888244 in tripleo "jobs are failing in ci.centos when installing tempest-skiplist - The task includes an option with an undefined variable. The error was: 'zuul' is undefined" [Critical,Triaged] | 15:25 |
pojadhav | weshay|ruck rlandy|ruck, help to merge this : https://review.rdoproject.org/r/#/c/28603/ | 15:33 |
chandankumar | arxcruz: https://review.rdoproject.org/r/#/c/28543/ is working fine with my changes | 15:37 |
*** marios is now known as marios|out | 15:48 | |
*** marios|out has quit IRC | 15:49 | |
*** pojadhav is now known as pojadhav|afk | 15:55 | |
chandankumar | arxcruz: this one https://review.opendev.org/#/c/741905/ is good to go? | 15:55 |
*** sshnaidm is now known as sshnaidm|afk | 15:57 | |
arxcruz | rlandy|ruck: weshay|ruck checking, sorry, i was in a doc. appointment | 15:59 |
*** bogdando has quit IRC | 15:59 | |
rlandy|ruck | arxcruz: np - patch put in for your review | 16:00 |
rlandy|ruck | arxcruz: https://review.opendev.org/741977 Support non-zuul workflow for list_skipped_yaml_file | 16:00 |
arxcruz | rlandy|ruck: i think the problem is https://opendev.org/openstack/tripleo-quickstart-extras/src/branch/master/playbooks/tempest.yml#L87 | 16:03 |
rlandy|ruck | arxcruz: ok feel free to edit | 16:03 |
* rlandy|ruck takes a guess at this | 16:03 | |
*** holser has quit IRC | 16:07 | |
*** holser has joined #oooq | 16:08 | |
*** rascasoft has quit IRC | 16:08 | |
arxcruz | rlandy|ruck: Support non-zuul workflow for list_skipped_yaml_file | 16:10 |
arxcruz | Currently, the step to install the tempest-skiplist | 16:10 |
arxcruz | fails on non-zuuul platforms. | 16:10 |
arxcruz | This review allows the test to source the skiplist | 16:10 |
arxcruz | file from the local workspace if needed. | 16:10 |
arxcruz | ops | 16:10 |
arxcruz | lol | 16:10 |
arxcruz | rlandy|ruck: https://review.opendev.org/#/c/741979/ | 16:10 |
rlandy|ruck | arxcruz: do I abandon https://review.opendev.org/#/c/741977/? | 16:11 |
rlandy|ruck | still needed? | 16:11 |
*** holser_ has joined #oooq | 16:11 | |
*** amoralej is now known as amoralej|off | 16:11 | |
arxcruz | rlandy|ruck: no, i think this is also required if people want to invoke the playbook directly | 16:12 |
arxcruz | rlandy|ruck: i need to think the best way to do it... maybe adding in the defaults | 16:12 |
rlandy|ruck | hmm .. failed openstack-tempest-skiplist-validateFAILURE in 2m 34s | 16:12 |
rlandy|ruck | FileNotFoundError: [Errno 2] No such file or directory: '/home/zuul/"src/opendev.org/openstack/openstack-tempest-skiplist"' | 16:14 |
*** holser has quit IRC | 16:14 | |
rlandy|ruck | arxcruz: ok - so clearly got that one wrong :( | 16:14 |
arxcruz | rlandy|ruck: that's weird, why is getting " " | 16:15 |
rlandy|ruck | who knows? | 16:15 |
rlandy|ruck | lunch - brb | 16:15 |
weshay|ruck | ci.centos is always good at reminding us there are other drivers than zuul | 16:16 |
*** udesale_ has quit IRC | 16:28 | |
arxcruz | rlandy|ruck: let's see if with ternary it will work | 16:28 |
*** dtantsur is now known as dtantsur|afk | 16:29 | |
rlandy|ruck | k | 16:30 |
chandankumar | arxcruz: can we merge this https://review.opendev.org/#/c/741905/ then? | 16:40 |
weshay|ruck | rlandy|ruck, ready when ever to chat https://code.engineering.redhat.com/gerrit/#/c/206448/ | 16:42 |
arxcruz | chandankumar: hold on, i need to update it | 16:44 |
chandankumar | ok | 16:44 |
*** jpena is now known as jpena|off | 16:45 | |
weshay|ruck | rfolco, https://review.opendev.org/#/c/741868/ | 16:59 |
weshay|ruck | Ian is suggesting this as an alternative.. | 16:59 |
rfolco | weshay|ruck, this is not my understanding | 16:59 |
rfolco | oh I see what you mean | 17:00 |
rfolco | yes | 17:00 |
*** derekh has quit IRC | 17:00 | |
weshay|ruck | seems fine | 17:00 |
rfolco | I committed the last fix if you see in that change | 17:00 |
rfolco | just working on tripleo-ci changes to not break the centos8 builds, removing py2 code.... https://review.opendev.org/#/c/733659/ | 17:01 |
weshay|ruck | k | 17:01 |
rfolco | weshay|ruck, I want to get this done by eod so he can review/merge | 17:01 |
weshay|ruck | k | 17:03 |
weshay|ruck | [11:01:36] <weshay|ruck> what do folks think of py3 on centos7 https://review.opendev.org/#/c/741868/ | 17:04 |
weshay|ruck | [11:02:56] <clarkb> weshay|ruck: I don't think we should preinstall it via dib in the yum element, but I think jobs can install it if they want it | 17:04 |
weshay|ruck | [11:03:23] <clarkb> we've finally gotten to a point where we don't preinstall a bunch of extra stuff which causes confusion and problems and I think adding python3 to centos7 in dib would get us back into that situation for some things | 17:04 |
weshay|ruck | [11:03:31] <clarkb> but in the jobs you can definitely install it | 17:04 |
weshay|ruck | [11:03:34] <weshay|ruck> ah.. I see | 17:04 |
weshay|ruck | [11:03:40] <weshay|ruck> fair point | 17:04 |
rfolco | weshay|ruck, the problem is that in the jobs you can't install inside the chroot (base cloud image) | 17:05 |
rfolco | weshay|ruck, unless we mock an alternative image that comes with python3 plus pyyaml pre-installed | 17:06 |
weshay|ruck | the chroot is only in play w/ DIB ya? | 17:07 |
rlandy|ruck | weshay|ruck: k - ready to chat | 17:07 |
weshay|ruck | rfolco, will pick this up in a bit | 17:07 |
weshay|ruck | rlandy|ruck, https://meet.google.com/sug-srsq-vvy | 17:07 |
rlandy|ruck | joining | 17:08 |
rfolco | weshay|ruck, yes, dib downloads the base image, construct the chroot and add elements to it. But for dib7 it gets a cloud image without py3 installed | 17:08 |
rlandy|ruck | weshay|ruck: https://bugs.launchpad.net/tripleo/+bug/1885602 | 17:48 |
openstack | Launchpad bug 1885602 in tripleo "Octavia component: failing tempest, Details: {'faultcode': 'Client', 'faultstring': 'Policy does not allow this request to be performed.'," [Critical,Triaged] - Assigned to Brent Eagles (beagles) | 17:48 |
rlandy|ruck | https://review.opendev.org/#/c/738488/ | 17:48 |
rlandy|ruck | https://review.opendev.org/#/c/739981/ | 17:49 |
rlandy|ruck | https://trunk.rdoproject.org/centos8-master/report.html | 17:51 |
rlandy|ruck | weshay|ruck: https://bugs.launchpad.net/tripleo/+bug/1887856 | 17:52 |
openstack | Launchpad bug 1887856 in tripleo "centos-8 master tripleo component tests are failing with "ModuleNotFoundError: No module named 'blazarclient'"" [Critical,In progress] - Assigned to Ronelle Landy (rlandy) | 17:52 |
akahat | chandankumar, we can merge this: https://review.opendev.org/#/c/736149/ | 17:55 |
weshay|ruck | rfolco, so I think infra wants us not to use dib to get py3 on centos-7 | 18:10 |
weshay|ruck | rfolco, and install it live | 18:10 |
weshay|ruck | [11:02:56] <clarkb> weshay|ruck: I don't think we should preinstall it via dib in the yum element, but I think jobs can install it if they want it | 18:10 |
weshay|ruck | [11:03:23] <clarkb> we've finally gotten to a point where we don't preinstall a bunch of extra stuff which causes confusion and problems and I think adding python3 to centos7 in dib would get us back into that situation for some things | 18:10 |
weshay|ruck | [11:03:31] <clarkb> but in the jobs you can definitely install it | 18:10 |
weshay|ruck | [11:03:34] <weshay|ruck> ah.. I see | 18:10 |
weshay|ruck | [11:03:40] <weshay|ruck> fair point | 18:10 |
rfolco | weshay|ruck, I got their point. | 18:10 |
weshay|ruck | k | 18:11 |
rfolco | BUT, that's a problem | 18:11 |
weshay|ruck | k. let's chat | 18:11 |
weshay|ruck | https://meet.google.com/ocn-ampa-kba | 18:11 |
*** pojadhav|afk is now known as pojadhav | 18:18 | |
weshay|ruck | rfolco, <pre>/home/whayutin/git/TRIPLEO/tripleo-image-elements/elements/delorean-repo/pre-install.d</pre> | 18:22 |
rlandy|ruck | chandankumar: you still around? | 18:32 |
rlandy|ruck | weshay|ruck: pojadhav pointed out that we are missing containers in centos8 train ... | 18:37 |
weshay|ruck | hrm | 18:37 |
rlandy|ruck | the fix, I think is to switch the list to use https://github.com/openstack/tripleo-common/blob/stable/train/container-images/overcloud_containers.yaml | 18:37 |
rlandy|ruck | we will have to make the change in production ie: on the server | 18:37 |
weshay|ruck | rlandy|ruck, yes.. it should be | 18:37 |
weshay|ruck | did the promotion server not change that in a branchful way? | 18:37 |
rlandy|ruck | but I'll propose the change in ci-config first | 18:38 |
rlandy|ruck | weshay|ruck: we need to overwrite the source of list to push containers | 18:38 |
rlandy|ruck | see the master file | 18:38 |
weshay|ruck | rlandy|ruck, wait.. can we chat again | 18:39 |
weshay|ruck | https://meet.google.com/wdu-kyvk-qie | 18:40 |
rlandy|ruck | joining | 18:40 |
rlandy|ruck | weshay|ruck: https://github.com/rdo-infra/ci-config/commit/ea18db9a2decfa7c797606db37595c07a6af9d0b | 18:47 |
rlandy|ruck | weshay|ruck: http://38.145.34.55/centos8_train.log | 18:48 |
*** soniya29 has quit IRC | 18:50 | |
rlandy|ruck | https://github.com/openstack/tripleo-common/blob/master/container-images/tripleo_containers.yaml | 18:51 |
*** pojadhav is now known as pojadhav|away | 18:51 | |
*** tosky has quit IRC | 18:57 | |
weshay|ruck | rlandy|ruck, https://github.com/openstack/tripleo-common/blob/stable/train/container-images/ | 19:16 |
weshay|ruck | rlandy|ruck, https://opendev.org/openstack/tripleo-common/raw/branch/stable/train/container-images | 19:17 |
weshay|ruck | rlandy|ruck, https://opendev.org/openstack/tripleo-common/raw/commit/da384ef49866fe87f3d22c1a0202fa81bd62213b/container-images/overcloud_containers.yaml | 19:18 |
weshay|ruck | rlandy|ruck, https://opendev.org/openstack/tripleo-common/src/commit/da384ef49866fe87f3d22c1a0202fa81bd62213b/container-images/overcloud_containers.yaml | 19:26 |
weshay|ruck | rlandy|ruck, failed on https://opendev.org/openstack/tripleo-common/src/commit/da384ef49866fe87f3d22c1a0202fa81bd62213b/container-images/overcloud_containers.yaml | 19:27 |
arxcruz | rlandy|ruck: https://review.opendev.org/#/c/741977/ will fix | 19:31 |
arxcruz | chandankumar: kopecmartin https://review.opendev.org/#/c/741977/ | 19:31 |
arxcruz | if you guys have time | 19:31 |
weshay|ruck | arxcruz, thanks | 19:33 |
*** dsneddon has joined #oooq | 19:35 | |
arxcruz | weshay|ruck: https://review.opendev.org/#/c/741905/ is also required | 19:35 |
arxcruz | at least now, the variables are all in one single place | 19:35 |
weshay|ruck | rlandy|ruck, 2020-07-20 19:34:06,817 9877 ERROR promoter Unable to download containers template at "https://opendev.org/openstack/tripleo-common/src/commit/"/da384ef49866fe87f3d22c1a0202fa81bd62213b/co | 19:35 |
arxcruz | so it's easier to manage | 19:36 |
rlandy|ruck | promoter No containers name found in https://opendev.org/openstack/tripleo-common/raw/commit/da384ef49866fe87f3d22c1a0202fa81bd62213b/container-images/overcloud_containers.yaml | 19:46 |
rlandy|ruck | weshay|ruck: I think I know why the list return empty | 20:14 |
rlandy|ruck | if 'container_images_template' in container_list: | 20:14 |
rlandy|ruck | vs | 20:14 |
rlandy|ruck | elif 'container_images' in container_list: | 20:14 |
rlandy|ruck | weshay|ruck: it's making the choice based on what we pass | 20:16 |
rlandy|ruck | ^^ that is why it's not picking up | 20:17 |
weshay|ruck | rlandy|ruck, ha.. I just saw the same thing | 20:20 |
weshay|ruck | ya | 20:20 |
rlandy|ruck | weshay|ruck: the choice shoudl be made on whether the self.containers_list_path contains j2 | 20:21 |
* rlandy|ruck puts up patch | 20:21 | |
weshay|ruck | rlandy|ruck, no | 20:21 |
weshay|ruck | it's right | 20:21 |
weshay|ruck | https://opendev.org/openstack/tripleo-common/src/branch/stable/train/container-images/overcloud_containers.yaml#L1 | 20:21 |
weshay|ruck | https://opendev.org/openstack/tripleo-common/src/branch/stable/train/container-images/overcloud_containers.yaml.j2#L1 | 20:21 |
rlandy|ruck | weshay|ruck: image source being kolla or tripleo is base on new vs old container build | 20:22 |
weshay|ruck | yes | 20:22 |
rlandy|ruck | so ... | 20:22 |
rlandy|ruck | if we used the rendered file ... | 20:22 |
weshay|ruck | rlandy|ruck, let's chat again | 20:22 |
rlandy|ruck | ok | 20:22 |
rlandy|ruck | https://meet.google.com/wdu-kyvk-qie? | 20:23 |
rlandy|ruck | weshay|ruck: ^^ old chat | 20:23 |
weshay|ruck | rlandy|ruck, https://sourcegraph.com/github.com/rdo-infra/ci-config/-/commit/59f68a1509f1768bcd7d10a764bb9511e9aab8da?visible=2 | 20:31 |
*** jfrancoa has quit IRC | 20:37 | |
*** jbadiapa has quit IRC | 21:24 | |
*** holser has joined #oooq | 21:48 | |
*** holser_ has quit IRC | 21:50 | |
*** TrevorV has quit IRC | 22:19 | |
rlandy|ruck | arxcruz: thanks | 23:10 |
*** holser has quit IRC | 23:12 | |
*** holser has joined #oooq | 23:27 | |
*** rlandy|ruck has quit IRC | 23:33 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!