Monday, 2020-04-13

*** sanjayu__ has quit IRC01:45
*** Goneri has quit IRC03:44
*** ysandeep|off is now known as ysandeep|rover04:07
*** holser has joined #oooq04:12
*** holser has quit IRC04:16
*** ykarel|away is now known as ykarel04:58
*** sanjayu__ has joined #oooq05:05
*** jfrancoa has joined #oooq05:07
*** skramaja has joined #oooq05:07
ysandeep|roverchandankumar, ykarel Good morning o/ fyi.. compute component pipeline affected - just opened https://bugs.launchpad.net/tripleo/+bug/187239905:09
openstackLaunchpad bug 1872399 in tripleo "Deployment failed because "nova_wait_for_api_service" container failed to start (INFO:nova_wait_for_api_service:Waiting for nova-api service)" [Critical,Triaged]05:09
ykarelysandeep|rover, ack05:12
ykarellooks like nova-api have issue05:13
ykarelhttps://logserver.rdoproject.org/openstack-component-compute/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-8-standalone-full-tempest-scenario-compute-master/b952891/logs/undercloud/var/log/containers/httpd/nova-api/nova_api_wsgi_error.log.txt.gz05:13
ykarelu can add this to bug description and also try to find what patch has caused this and what's needed to clear it05:14
*** jfrancoa has quit IRC05:15
*** jfrancoa has joined #oooq05:19
*** ratailor has joined #oooq05:26
*** udesale has joined #oooq05:27
*** marios has joined #oooq05:37
*** holser has joined #oooq05:42
ysandeep|roverykarel, Noticed one another thing.. Comparing good run and bad run - nova_compute and nova_api_cron containers are missing in bad run  - http://paste.openstack.org/show/792006/ , missing nova_api_cron container looks suspicious..06:02
ykarelysandeep|rover, nova_api_cron is expected06:06
ykarelas it runs after wait_service container06:06
ykarelhttps://opendev.org/openstack/tripleo-heat-templates/src/branch/master/deployment/nova/nova-api-container-puppet.yaml#L44906:06
ysandeep|roverykarel, ack thank you06:07
*** jfrancoa has quit IRC06:18
ysandeep|roverykarel, okay so this patch introduced dataclassed 3 days back - https://review.opendev.org/#/c/704643/23/nova/api/validation/extra_specs/base.py06:24
ykarelysandeep|rover, yeap06:26
ysandeep|roverpatch owner mentioned dataclasses in lower-constraints files - https://review.opendev.org/#/c/704643/23/lower-constraints.txt .. comparing other modules to confirm in which all files we need to mention and where we missed for this module.06:30
akahatGood afternoon!!06:32
ykarelysandeep|rover, actually dataclasses is a new dependency https://github.com/openstack/requirements/commit/e7c7dbfc8e09f07ba19cb4474b13f98470ae16b706:36
ykareland nova is it's first user06:36
ykarelso now this new dep needs to be added in RDO, once added it needs to be added in nova rpm spec file06:38
ykarelhttps://www.rdoproject.org/documentation/requirements/#adding-a-new-requirement-to-rdo06:38
ysandeep|roverykarel, reading ^^ thank you06:39
ykarelthis is special case(only needed for python3.6) so need to handle differently06:40
ysandeep|roverykarel, ack.. I will try to prepare patches.. Will take your help if i got stuck somewhere.. reading docs06:43
ykarelysandeep|rover, as said this is special, so direct patches will not work06:44
ykarelwill need to check how to handle it, also need to coordinate with nova guys06:45
ysandeep|roverykarel, ohhh06:45
*** jfrancoa has joined #oooq06:46
ykarelwith python3.7 it's inbuilt in python306:46
ykarelso only needed for python3.606:46
marioschandankumar: akahat: o/ so merge ? https://review.opendev.org/#/c/718091/807:11
marioslgtm07:11
mariosalso07:11
mariossee comment might be worth preping the voting stuff so we can merge when ready07:11
chandankumarmarios, yes good to go07:12
chandankumarakahat, ping07:12
chandankumarakahat, please clean this job also https://github.com/rdo-infra/rdo-jobs/blob/master/zuul.d/standalone-jobs.yaml#L43007:14
akahatchandankumar, ack07:19
*** holser has quit IRC07:20
akahatchandankumar, you want me to port this job to os_tempest?07:20
chandankumarakahat, just remove the os_tempest stuff from there, it will pull stuff from fs06007:25
akahatchandankumar, ack07:25
akahatmarios, chandankumar thank you: https://review.opendev.org/#/c/718091/8 :)07:26
chandankumarakahat, you also need to propose a patch to make sc12 voting07:29
chandankumarmarios, commented back https://review.opendev.org/71704307:31
chandankumarysandeep|rover, \o/ regarding above compute pipeline bug, it might be broken due to nova component changes for sure as content only got changed for nova related packages07:31
marioschandankumar: i think we need it for train though replied there i had similar issue in containers build work07:32
mariosakahat: ack07:32
mariosakahat: see comment on patch re voting 10:29 < chandankumar> akahat, you also need to propose a patch to make sc12 voting07:32
chandankumarysandeep|rover, you can also try this tool https://github.com/rdo-infra/ci-config/tree/master/jcomparison07:33
ysandeep|roverchandankumar, dataclasses recently added its a new dependency https://github.com/openstack/requirements/commit/e7c7dbfc8e09f07ba19cb4474b13f98470ae16b7 and nova is it's first user , so now this new dep needs to be added in RDO, once added it needs to be added in nova rpm spec file07:33
chandankumarysandeep|rover, ah ok, thanks!07:34
*** ccamacho has joined #oooq07:34
chandankumarysandeep|rover, ah correct07:34
chandankumarmarios, yes correct07:35
*** amoralej|off is now known as amoralej07:36
ysandeep|roverchandankumar, ykarel suggested we cannot direct patch it as this is special case(only needed for python3.6, with python3.7 it's inbuilt in python3)  so need to handle differently - we will need to check how to handle it, also need to coordinate with nova guys. I will ping sfinucan(nova engineer) for help once he is in07:36
chandankumaraye07:37
*** jpena|off is now known as jpena07:44
akahatmarios, ack07:51
*** ysandeep|rover is now known as ysandeep|lunch08:07
*** jbadiapa has joined #oooq08:10
*** pojadhav is now known as pojadhav|lunch08:25
*** ykarel is now known as ykarel|lunch08:52
*** ysandeep|lunch is now known as ysandeep|rover09:05
*** jbadiapa has quit IRC09:05
*** ccamacho has quit IRC09:23
*** tosky has joined #oooq09:25
*** jbadiapa has joined #oooq09:31
*** ykarel|lunch is now known as ykarel09:38
*** pojadhav|lunch is now known as pojadhav09:45
*** jbadiapa has quit IRC10:07
*** ccamacho has joined #oooq10:09
*** ccamacho has quit IRC10:22
*** arxcruz is now known as arxcruz|off10:23
*** ccamacho has joined #oooq10:39
*** ysandeep|rover is now known as ysandeep|coffee11:02
*** jtomasek has joined #oooq11:24
*** jtomasek has quit IRC11:35
*** ysandeep|coffee is now known as ysandeep|rover11:35
*** jpena is now known as jpena|lunch11:38
*** rlandy has joined #oooq12:07
chandankumarrlandy, welcome back :-)12:09
rlandychandankumar: thanks12:09
chandankumarrlandy, need eyes on this https://review.opendev.org/717043 and https://code.engineering.redhat.com/gerrit/#/c/196373/ thanks :-)12:10
rlandymarios: hi - do you want to sync before the scrum?12:11
mariosrlandy: o/12:12
mariosrlandy: hey welcome back ... wasn't sure if you were out today too12:12
mariosrlandy: yes ... in 15 ok for you?12:13
rlandymarios: sure12:13
weshay|ruckysandeep|rover, we need to test project periodic-tripleo-ci-centos-7-standalone-upgrade-stein12:19
weshay|ruckhttps://review.rdoproject.org/zuul/builds?job_name=periodic-tripleo-ci-centos-7-standalone-upgrade-stein12:19
ysandeep|roverweshay|ruck, something is wrong with stein branch for all tripleo projects.. please check RR notes12:20
weshay|ruckysandeep|rover, what line?12:20
ysandeep|roverweshay|ruck, 12312:20
ysandeep|roverweshay|ruck, https://review.opendev.org/#/c/718468/12:21
weshay|ruckysandeep|rover, oh.. we probably have to remove tripleo-ansible from required projects12:23
weshay|ruckin the job config12:23
ysandeep|roverweshay|ruck hey o/ please check chatter with smcginnis on openstack-release12:26
chandankumarweshay|ruck, when free please have a look on this https://review.opendev.org/#/c/718946/12:28
*** amoralej is now known as amoralej|lunch12:30
*** jpena|lunch is now known as jpena12:32
weshay|ruckysandeep|rover, cix12:34
ysandeep|roverweshay|ruck, o/ coming12:34
*** ratailor has quit IRC12:37
*** udesale_ has joined #oooq12:38
*** udesale has quit IRC12:40
pojadhavrlandy, hello12:42
rlandypojadhav: hi - in meeting12:43
pojadhavrlandy, np12:43
soniya29weshay|ruck, chandankumar, kopecmartin, What if the validate command returns error different then error specified here?. Here is the output when the schema added is tested against validate command:-https://paste.centos.org/view/1da826d312:47
soniya29weshay|ruck, chandankumar, kopecmartin, error specified here - https://review.opendev.org/#/c/718066/7/doc/source/validate/validate.rst12:50
*** rfolco has joined #oooq12:52
rfolcoI won't be able to run the scrum today12:53
rfolcosomeone please cover me12:53
*** rfolco is now known as rfolco|bbl12:53
*** jbadiapa has joined #oooq12:54
mariosstill not using openstack- & will need to edit the containers.j2.yaml13:18
mariosweshay|ruck: ^^^13:19
marioshttps://review.opendev.org/#/c/718701/1/roles/standalone/tasks/main.yml13:19
mariosweshay|ruck: ^^^13:19
*** amoralej|lunch is now known as amoralej13:23
chandankumarrlandy, https://review.rdoproject.org/r/#/c/26272/13:33
*** ykarel is now known as ykarel|afk13:39
chandankumarysandeep|rover, add the failing test here https://opendev.org/openstack/tripleo-quickstart-extras/src/branch/master/roles/validate-tempest/vars/tempest_skip_master.yml13:49
chandankumarysandeep|rover, then it will get skipped13:49
chandankumarysandeep|rover, till then ci will pass and ovn team will work on fixing that13:50
ysandeep|roverchandankumar, ack o/ will try that tomorrow morning13:51
*** rfolco|bbl is now known as rfolco13:52
*** ykarel|afk is now known as ykarel14:04
*** ysandeep|rover is now known as ysandeep|away14:12
weshay|ruckmarios, I'm in14:28
chandankumarrlandy, https://review.opendev.org/#/c/717043/14:34
rlandylooking14:34
rlandychandankumar: should we remove tripleo-buildimage-overcloud-hardened-full-centos-8?14:37
rlandyhas never passed14:37
rlandyand I don't know if this introduced any new failures14:37
chandankumarrlandy, yes, I will remove in a seperate patch14:37
rlandychandankumar: ok - voting14:37
chandankumarrlandy, we never built that14:37
rlandychandankumar: yea - let's just ditch it then14:38
rlandychandankumar: +2'ed https://review.opendev.org/#/c/71704314:39
rlandygo ahead and merged when ready14:39
*** ykarel is now known as ykarel|away14:41
rlandymarios: ah ... see    rhos_release_args: "-P 17 -p {{ latest_compose_id.content }} -r 8.2"14:43
rlandyfrom chandankumar's review14:43
rlandywe need that to get 8.214:43
chandankumarrlandy, it needs to change as rhos-17 is based on 8.314:44
chandankumarwe need to find an automated way to get these lists14:44
rlandychandankumar: https://code.engineering.redhat.com/gerrit/#/c/19637314:45
rlandyone comment on projects14:45
chandankumarrlandy, checking14:45
rlandychandankumar: I assume next sprint we add them to the pipeline14:46
chandankumarrlandy, yes14:46
rlandyso they will have a place to run shortly14:46
chandankumarrlandy, done14:49
rlandychandankumar: ok - +2 - can merge when your upstream review does14:53
*** skramaja has quit IRC15:01
mariosrlandy: ack will do15:13
weshay|ruckrlandy, you getting notes on this?15:31
mariosrlandy: posted v2 https://code.engineering.redhat.com/gerrit/197197 & v19 @  https://code.engineering.redhat.com/gerrit/19685815:55
mariosrlandy: feel free to take it ... will pickup whatever you leavein my morning15:55
mariosrlandy: around few mins if you want to check/clarify something but its the stuff we discussed earlier15:55
rlandymarios: ack - just in meeting - will pick up15:56
weshay|ruckmarios, rlandy fyi.. fbts issues w/ osp-17 look to be clearing up.. we should be able to switch to dlrn/dlrn-deps next sprint15:59
rlandyweshay|ruck: ack16:00
rlandymarios: ok - out of meeting  - looking at reviews16:00
rlandymarios: looks good - one piece missing16:13
mariosrlandy: ack .. feel free to take and iterate and i will check for updates tomorrow before continuing16:16
mariosrlandy: you can see it running on the patch  & https://sf.hosted.upshift.rdu2.redhat.com/zuul/t/tripleo-ci-internal/status16:16
rlandymarios: ack - looking really good though16:16
rlandymaking two edits16:17
mariosrlandy: yeah the ansiblification is slow progress cos of the nature of the hacks but... sure ack thanks ttyl have a good one16:17
rlandynice work!16:17
mariosrlandy: team work16:17
rlandyansiblification :)16:17
marios;)16:17
rlandyhmmm  ... looking at https://sf.hosted.upshift.rdu2.redhat.com/logs/58/196858/19/check/periodic-tripleo-build-containers-rhel-8-master/e8850f5/logs/build.log16:18
rlandyit's still looking for openstack-16:18
rlandyit doesn't pick up the image_prefix16:19
rlandymaybe another hack :(16:19
rlandymarios: you set that16:20
rlandybut it's not picked up16:20
rlandyimage_prefix: rhel-binary-16:22
mariosrlandy: yes i noticed that too not sure whats going on16:23
rlandymarios: I saw that in my manual install16:23
mariosrlandy: was trying to see if the containers under docker/ were correct don't know yet16:23
mariosrlandy: i.e. if the things i copies work etc16:23
mariosanwyay16:23
* marios off 16:23
rlandymarios: I landed up doing a manula replace16:23
rlandyok - will fix16:24
mariosrlandy: thx ttyl16:24
*** marios is now known as marios|off16:26
baharfolco: We're finding that the playbooks are pulling down amd64 centos images for container builds on local ppc64le nodes, and haven't been able to figure out where to address it. Do you know where in the playbooks that would be configured/changed?16:46
rfolcobaha, let me refresh on this16:48
rfolcobaha, you use upstream role right?16:48
* rlandy needs to go on exciting trip to grocery store ... biab16:52
*** rlandy is now known as rlandy|biab16:52
bahaI believe so.16:52
bahaLocally we pull down the upstream playbooks to run16:53
*** jpena is now known as jpena|off17:01
*** marios|off has quit IRC17:01
rfolcobaha, do you have a link to the logs ?17:08
rfolcobaha, I am pulling centos:8 just fine in my ppc box, need to look your kolla conf17:09
*** udesale_ has quit IRC17:11
baharfolco: We don't automatically upload local build logs anywhere (they're all manual runs), but I can go grab my kolla conf for you17:12
bahaJust want kolla-build.conf?17:12
rfolcobaha, and if using buildah17:12
bahaWe're using buildah17:13
rfolcook, so for now kolla conf, let me check something17:13
bahahttp://paste.openstack.org/show/792051/ this is what the kolla-build.conf output looks like17:14
rfolcok thx17:15
bahalet me know if you need anything else!17:15
rfolcobaha, are you sure your base container 'centos' is amd64 ? or centos-binary ?17:17
rfolcobaha, its the base built or the base pulled ?17:17
bahaThe centos8 image that's pulled, we're failing on building the base image17:18
bahaWhen running a podman image inspect on the centos8 image, we get "Architecture": "amd64"17:19
rfolcowhat does happen if you run docker pull centos:8 or just centos in your ppc box ?17:19
rfolcohmm maybe your registries.conf has been modified ? it would be good to debug and see its pulling from where ? rdoproject ? docker.io? quay.io ?17:20
bahaFor reference, the node I'm running on was just a base centos 8 cloud image with the playbooks run on it17:21
rfolcobaha, do you have access ?17:22
bahaI do. It looks like podman is attempting to pull from a few places (registry.access.redhat, then registry.fedoraproject, then it catches on registry.centos.org/centos17:23
bahaThat does indeed correspond with /etc/registries.conf17:24
bahadocker.io would've been the next and final stop17:24
weshay|ruckrfolco, rlandy|biab let's chat about get_hash today.. when we're all here please17:27
rfolcoweshay|ruck, ok just ping me17:27
rfolcobaha, my ppc box is centos7 so not a good env to see whats going on17:30
bahaGotcha. It definitely sounds like some disparity between podman/docker or centos7/817:32
bahaI'd give you direct access but the box is behind firewall17:32
rfolcobaha, I am requesting a new centos8 ppc box but it would take a while...17:34
rfolcobaha, all centos8 ppc images are pulled just fine using docker* from centos7 ppc box17:35
rfolcobaha, I would look at podman pull first17:35
baharfolco: http://paste.openstack.org/show/792053/ so this strikes me as odd17:43
bahaAnd leads me to believe it's not something as simple as podman not setting the correct architecture17:47
mjturekbaha rfolco: FWIW same thing happens with centos:717:51
mjturekthis is super strange17:51
*** rlandy|biab is now known as rlandy17:51
bahagoing to step AFK, back in a few17:51
*** baha is now known as baha|brb17:51
rfolcomjturek, baha, yes I just tested w/ docker-ce on a centos7 ppc box, and it pulls ppc image17:52
mjtureknp baha17:52
rfolcojust don't forget your mask17:52
*** rlandy is now known as rlandy|lunch17:57
mjturek:)17:58
mjturekrfolco http://paste.openstack.org/show/792054/ config file for podman fyi17:58
mjturekbaha|brb maybe we need to open a podman bug?18:02
*** amoralej is now known as amoralej|off18:03
rfolcomjturek, if a simple podman pull brings amd64 image, yes, definitely its podman bug18:05
mjturekany place to ask podman questions?18:15
*** rlandy|lunch is now known as rlandy18:19
rlandyweshay|ruck: rfolco: ready to talk about get-hash whenever you are18:19
weshay|ruckaye.. same18:20
weshay|ruckrfolco, ?18:21
rfolcook let me change room18:22
rfolcoone min18:22
weshay|ruckhttps://meet.google.com/udw-kgxc-rhy?authuser=118:25
rfolcorlandy, ^18:27
rfolcoyou joining ?18:27
weshay|ruckhttps://review.rdoproject.org/r/#/c/26346/18:28
weshay|ruckhttps://logserver.rdoproject.org/46/26346/3/check/rdo-tox-molecule/789427e/tox/reports.html18:28
rlandyweshay|ruck: rfolco: https://code.engineering.redhat.com/gerrit/#/c/195738/3/zuul.d/dlrn-jobs.yaml18:31
weshay|ruck#2 https://review.rdoproject.org/r/#/c/26369/18:55
*** Goneri has joined #oooq19:03
weshay|ruckrlandy, can you help me find required projects for 3rd party19:07
weshay|ruckwhat file is that?19:07
weshay|ruckrdo19:07
rlandyweshay|ruck: getting19:07
rlandyweshay|ruck: looking for https://github.com/rdo-infra/review.rdoproject.org-config/blob/master/zuul/rdo.yaml?19:08
weshay|ruckrlandy, ya.. think so19:09
weshay|ruckhttps://github.com/rdo-infra/review.rdoproject.org-config/blob/master/zuul/rdo.yaml#L68519:09
weshay|ruckso.. the stein branch was removed from tripleo-ansible19:09
weshay|rucktripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001ERROR Unable to find role in /var/lib/zuul/builds/6c753679129442bcb43a50f3f0b3253e/ansible/pre_playbook_7/role_1/tripleo-ansible19:10
weshay|ruckhttps://review.opendev.org/#/c/718757/19:10
weshay|rucknot sure how to resolve that for just the branch19:10
* weshay|ruck looks19:10
rlandyoh gee19:10
rlandybranch_override19:11
rlandyweshay|ruck: ^^ those are all the projects zuul could clone19:12
rlandydoesn't mean it does clone them all19:12
rlandyin the job19:12
weshay|ruckk19:12
* weshay|ruck looking at ovb base19:12
rlandyyou can override required_projects19:12
rlandyweshay|ruck: or set the branch on the one repo19:12
rlandysee the example of the OVB repo19:12
weshay|ruck    name: tripleo-ci-base-ovb-notebroker19:13
*** baha|brb is now known as baha19:14
weshay|ruckthink I understand now19:14
weshay|ruckthanks rlandy19:14
*** sanjayu__ has quit IRC19:39
rlandyweshay|ruck: ever seen this error in container builds? 2020-04-13 15:18:14 |     raise TypeError("First argument should be a list of repositories")19:50
rlandyhttps://sf.hosted.upshift.rdu2.redhat.com/logs/58/196858/20/check/periodic-tripleo-build-containers-rhel-8-master/c6e6cdb/logs/build.log19:50
rlandyactually writing the dockerfile19:50
rlandyfor base19:50
*** irclogbot_1 has quit IRC19:51
*** irclogbot_2 has joined #oooq19:51
weshay|ruckrlandy, /me looks19:54
weshay|ruckrlandy, that's for barbican not base afaict.. barbican-worker19:55
weshay|ruckhttps://opendev.org/openstack/kolla/src/branch/master/docker/macros.j2#L9819:55
rlandyweshay|ruck: passes that19:55
rlandyit's on base19:55
rlandy2020-04-13 15:18:14 |   File "/tmp/kolla-builds/f69542cf-9ff7-49fd-9e0c-e67f41c042f4/docker/base/Dockerfile.j2", line 269, in top-level template code19:55
weshay|ruckrlandy, oh.19:56
weshay|ruck {{ macros.disable_extra_repos('epel') }}19:56
weshay|ruckrlandy, do you have a fake epel.repo/19:56
weshay|ruck?19:56
weshay|ruckit may be required by some of these docker files19:56
rlandyumm ... probably not - looking19:56
rlandymarios made a downstream pre19:57
rlandywe probably need to add that19:57
weshay|ruckmaybe we did that manually once and forgot19:57
rlandy2020-04-13 19:13:03.641018 | LOOP [build-containers : Add dummy epel and epel-module repository]20:01
rlandy2020-04-13 19:13:04.217289 | primary | changed: "epel"20:01
rlandy2020-04-13 19:13:04.447003 | primary | changed: "epel-modular"20:01
rlandyweshay|ruck: yep we do ... https://sf.hosted.upshift.rdu2.redhat.com/logs/58/196858/20/check/periodic-tripleo-build-containers-rhel-8-master/c6e6cdb/etc/yum.repos.d/20:03
weshay|ruckhrm..20:04
weshay|ruckrlandy, is it on the base container though?20:04
weshay|ruckrlandy, may need to play w/ this manually20:04
rlandychecking20:08
rlandyweshay|ruck: yep - should be https://sf.hosted.upshift.rdu2.redhat.com/logs/58/196858/20/check/periodic-tripleo-build-containers-rhel-8-master/c6e6cdb/logs/build-rhel-base.log20:09
rlandysearch for epel20:09
rlandyweshay|ruck: Oh ...20:10
rlandyFROM ubi8:8.2-ondeck20:11
rlandyit's on rhel base20:11
rlandynot on ondeck20:12
rlandyFROM {{ base_image }}:{{ base_distro_tag }}20:15
rlandybase_image=rhel20:17
rlandyno idea why it's going to ondeck20:17
weshay|ruckrlandy, ya.. perhaps we need two copies of the base20:18
weshay|ruckthe docker files :(20:18
weshay|ruckI remember seeing ondeck20:18
rlandywhy?20:18
rlandyondesk is on the renedred file20:18
rlandyrendered20:18
rlandyweshay|ruck: is there is setting to overwrite the rendered file20:18
rlandyif it's already there?20:18
weshay|ruckrlandy, let's look at barbican-worker dockerfile20:19
weshay|ruckmaybe that has it in the jinja20:19
weshay|ruckas ondeck20:19
rlandyhttps://sf.hosted.upshift.rdu2.redhat.com/logs/58/196858/20/check/periodic-tripleo-build-containers-rhel-8-master/c6e6cdb/logs/buildah-builds/kolla-builds/f69542cf-9ff7-49fd-9e0c-e67f41c042f4/docker/barbican-worker/Dockerfile20:19
rlandythat is correct20:19
rlandyhttps://sf.hosted.upshift.rdu2.redhat.com/logs/58/196858/20/check/periodic-tripleo-build-containers-rhel-8-master/c6e6cdb/logs/buildah-builds/kolla-builds/f69542cf-9ff7-49fd-9e0c-e67f41c042f4/docker/base/Dockerfile.j220:19
weshay|ruckya20:19
rlandy^^ is also correct20:19
rlandyit should have taken rhel:820:20
weshay|ruckhttps://sf.hosted.upshift.rdu2.redhat.com/logs/58/196858/20/check/periodic-tripleo-build-containers-rhel-8-master/c6e6cdb/logs/buildah-builds/kolla-builds/f69542cf-9ff7-49fd-9e0c-e67f41c042f4/docker/base/Dockerfile20:20
weshay|ruckhas ondeck20:20
rlandyper https://sf.hosted.upshift.rdu2.redhat.com/logs/58/196858/20/check/periodic-tripleo-build-containers-rhel-8-master/c6e6cdb/conf/kolla-build.conf20:20
rlandyit should not use the rendered file20:20
weshay|ruckrlandy, this is what kolla rendered https://sf.hosted.upshift.rdu2.redhat.com/logs/58/196858/20/check/periodic-tripleo-build-containers-rhel-8-master/c6e6cdb/logs/buildah-builds/kolla-builds/f69542cf-9ff7-49fd-9e0c-e67f41c042f4/docker/base/Dockerfile20:20
weshay|rucknot what is checked in20:20
rlandyweshay|ruck: I don;t think it rendered that file yet20:22
rlandyit failed doing that20:22
rlandycheck the next container down20:22
rlandyhttp://pkgs.devel.redhat.com/cgit/containers/openstack-base/tree/Dockerfile.j2?h=rhos-17.0-rhel-8#n26920:25
*** jbadiapa has quit IRC20:25
rlandy^^ error20:25
weshay|ruckya.. agree it looks like it failed IN the template20:26
weshay|ruckrlandy, we may need to.. do a find | exec rm -f DockerFile20:26
rlandy{% macro disable_extra_repos(repos) %}20:27
weshay|ruckto remove all the prerendered dockerfiles as to not confuse ourselves w/ the logs20:27
*** jfrancoa has quit IRC20:34
*** ccamacho has quit IRC20:40
rlandyweshay|ruck: ha ... http://git.app.eng.bos.redhat.com/git/kolla.git/commit/kolla/template/methods.py?h=rhos-17.0-trunk-patches&id=5e7979f31d8e30980a6e31490d00345e51a1400620:58
* weshay|ruck looks20:58
weshay|ruckrlandy, ah.. is internal kolla missing a change?20:58
weshay|ruckrlandy, I wonder if we could use upstream kolla w/ downstream dockerfiles20:59
weshay|ruckto save grief20:59
rlandyweshay|ruck: we were patching kolla in the manual20:59
rlandyand probably didn;t get that latest change20:59
rlandyit's an error in the donwstream now20:59
weshay|ruckah..20:59
weshay|ruckso does that fix it?20:59
rlandyI'm guess it should be (['epel'])21:00
rlandyno - that is what raises the error21:00
rlandybefor it was just ignored21:00
weshay|ruckah21:00
weshay|ruck++21:00
rlandylet me see if I can sed replace21:00
rlandyand if that works, I'll have to propose a patch21:01
weshay|ruckrlandy, so.. not there yet.. but I think this will simplify it https://review.rdoproject.org/r/2634621:11
rlandylooking21:13
weshay|ruckrlandy, also .. I think that ovb change worked.. https://review.rdoproject.org/zuul/stream/4b5a8e8bb87341ac96abe81b675cbb9d?logfile=console.log21:14
rlandynice21:14
rlandyweshay|ruck: get-hash makes my hair hurt21:14
* rlandy will have to think about this21:15
rlandybut the general idea looks right21:15
weshay|ruckrlandy, what I don't know is if we can use hyphens as a key for a var21:18
weshay|ruckosp-17: foo21:18
rlandyweshay|ruck: split the var21:19
weshay|ruckk.. we'll keep chatting21:19
rlandyweshay|ruck: like osp-{{osp-release}}21:20
weshay|ruckrlandy, ya.. I guess we can pass whatever in zuul21:21
weshay|ruckk21:21
weshay|ruckrlandy, can you please vote on https://review.rdoproject.org/r/#/c/26414/21:21
rlandydone21:22
weshay|ruckthanks21:23

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!