| *** sanjayu__ has quit IRC | 01:45 | |
| *** Goneri has quit IRC | 03:44 | |
| *** ysandeep|off is now known as ysandeep|rover | 04:07 | |
| *** holser has joined #oooq | 04:12 | |
| *** holser has quit IRC | 04:16 | |
| *** ykarel|away is now known as ykarel | 04:58 | |
| *** sanjayu__ has joined #oooq | 05:05 | |
| *** jfrancoa has joined #oooq | 05:07 | |
| *** skramaja has joined #oooq | 05:07 | |
| ysandeep|rover | chandankumar, ykarel Good morning o/ fyi.. compute component pipeline affected - just opened https://bugs.launchpad.net/tripleo/+bug/1872399 | 05:09 |
|---|---|---|
| openstack | Launchpad 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 |
| ykarel | ysandeep|rover, ack | 05:12 |
| ykarel | looks like nova-api have issue | 05:13 |
| ykarel | https://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.gz | 05:13 |
| ykarel | u can add this to bug description and also try to find what patch has caused this and what's needed to clear it | 05:14 |
| *** jfrancoa has quit IRC | 05:15 | |
| *** jfrancoa has joined #oooq | 05:19 | |
| *** ratailor has joined #oooq | 05:26 | |
| *** udesale has joined #oooq | 05:27 | |
| *** marios has joined #oooq | 05:37 | |
| *** holser has joined #oooq | 05:42 | |
| ysandeep|rover | ykarel, 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 |
| ykarel | ysandeep|rover, nova_api_cron is expected | 06:06 |
| ykarel | as it runs after wait_service container | 06:06 |
| ykarel | https://opendev.org/openstack/tripleo-heat-templates/src/branch/master/deployment/nova/nova-api-container-puppet.yaml#L449 | 06:06 |
| ysandeep|rover | ykarel, ack thank you | 06:07 |
| *** jfrancoa has quit IRC | 06:18 | |
| ysandeep|rover | ykarel, okay so this patch introduced dataclassed 3 days back - https://review.opendev.org/#/c/704643/23/nova/api/validation/extra_specs/base.py | 06:24 |
| ykarel | ysandeep|rover, yeap | 06:26 |
| ysandeep|rover | patch 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 |
| akahat | Good afternoon!! | 06:32 |
| ykarel | ysandeep|rover, actually dataclasses is a new dependency https://github.com/openstack/requirements/commit/e7c7dbfc8e09f07ba19cb4474b13f98470ae16b7 | 06:36 |
| ykarel | and nova is it's first user | 06:36 |
| ykarel | so now this new dep needs to be added in RDO, once added it needs to be added in nova rpm spec file | 06:38 |
| ykarel | https://www.rdoproject.org/documentation/requirements/#adding-a-new-requirement-to-rdo | 06:38 |
| ysandeep|rover | ykarel, reading ^^ thank you | 06:39 |
| ykarel | this is special case(only needed for python3.6) so need to handle differently | 06:40 |
| ysandeep|rover | ykarel, ack.. I will try to prepare patches.. Will take your help if i got stuck somewhere.. reading docs | 06:43 |
| ykarel | ysandeep|rover, as said this is special, so direct patches will not work | 06:44 |
| ykarel | will need to check how to handle it, also need to coordinate with nova guys | 06:45 |
| ysandeep|rover | ykarel, ohhh | 06:45 |
| *** jfrancoa has joined #oooq | 06:46 | |
| ykarel | with python3.7 it's inbuilt in python3 | 06:46 |
| ykarel | so only needed for python3.6 | 06:46 |
| marios | chandankumar: akahat: o/ so merge ? https://review.opendev.org/#/c/718091/8 | 07:11 |
| marios | lgtm | 07:11 |
| marios | also | 07:11 |
| marios | see comment might be worth preping the voting stuff so we can merge when ready | 07:11 |
| chandankumar | marios, yes good to go | 07:12 |
| chandankumar | akahat, ping | 07:12 |
| chandankumar | akahat, please clean this job also https://github.com/rdo-infra/rdo-jobs/blob/master/zuul.d/standalone-jobs.yaml#L430 | 07:14 |
| akahat | chandankumar, ack | 07:19 |
| *** holser has quit IRC | 07:20 | |
| akahat | chandankumar, you want me to port this job to os_tempest? | 07:20 |
| chandankumar | akahat, just remove the os_tempest stuff from there, it will pull stuff from fs060 | 07:25 |
| akahat | chandankumar, ack | 07:25 |
| akahat | marios, chandankumar thank you: https://review.opendev.org/#/c/718091/8 :) | 07:26 |
| chandankumar | akahat, you also need to propose a patch to make sc12 voting | 07:29 |
| chandankumar | marios, commented back https://review.opendev.org/717043 | 07:31 |
| chandankumar | ysandeep|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 packages | 07:31 |
| marios | chandankumar: i think we need it for train though replied there i had similar issue in containers build work | 07:32 |
| marios | akahat: ack | 07:32 |
| marios | akahat: see comment on patch re voting 10:29 < chandankumar> akahat, you also need to propose a patch to make sc12 voting | 07:32 |
| chandankumar | ysandeep|rover, you can also try this tool https://github.com/rdo-infra/ci-config/tree/master/jcomparison | 07:33 |
| ysandeep|rover | chandankumar, 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 file | 07:33 |
| chandankumar | ysandeep|rover, ah ok, thanks! | 07:34 |
| *** ccamacho has joined #oooq | 07:34 | |
| chandankumar | ysandeep|rover, ah correct | 07:34 |
| chandankumar | marios, yes correct | 07:35 |
| *** amoralej|off is now known as amoralej | 07:36 | |
| ysandeep|rover | chandankumar, 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 in | 07:36 |
| chandankumar | aye | 07:37 |
| *** jpena|off is now known as jpena | 07:44 | |
| akahat | marios, ack | 07:51 |
| *** ysandeep|rover is now known as ysandeep|lunch | 08:07 | |
| *** jbadiapa has joined #oooq | 08:10 | |
| *** pojadhav is now known as pojadhav|lunch | 08:25 | |
| *** ykarel is now known as ykarel|lunch | 08:52 | |
| *** ysandeep|lunch is now known as ysandeep|rover | 09:05 | |
| *** jbadiapa has quit IRC | 09:05 | |
| *** ccamacho has quit IRC | 09:23 | |
| *** tosky has joined #oooq | 09:25 | |
| *** jbadiapa has joined #oooq | 09:31 | |
| *** ykarel|lunch is now known as ykarel | 09:38 | |
| *** pojadhav|lunch is now known as pojadhav | 09:45 | |
| *** jbadiapa has quit IRC | 10:07 | |
| *** ccamacho has joined #oooq | 10:09 | |
| *** ccamacho has quit IRC | 10:22 | |
| *** arxcruz is now known as arxcruz|off | 10:23 | |
| *** ccamacho has joined #oooq | 10:39 | |
| *** ysandeep|rover is now known as ysandeep|coffee | 11:02 | |
| *** jtomasek has joined #oooq | 11:24 | |
| *** jtomasek has quit IRC | 11:35 | |
| *** ysandeep|coffee is now known as ysandeep|rover | 11:35 | |
| *** jpena is now known as jpena|lunch | 11:38 | |
| *** rlandy has joined #oooq | 12:07 | |
| chandankumar | rlandy, welcome back :-) | 12:09 |
| rlandy | chandankumar: thanks | 12:09 |
| chandankumar | rlandy, need eyes on this https://review.opendev.org/717043 and https://code.engineering.redhat.com/gerrit/#/c/196373/ thanks :-) | 12:10 |
| rlandy | marios: hi - do you want to sync before the scrum? | 12:11 |
| marios | rlandy: o/ | 12:12 |
| marios | rlandy: hey welcome back ... wasn't sure if you were out today too | 12:12 |
| marios | rlandy: yes ... in 15 ok for you? | 12:13 |
| rlandy | marios: sure | 12:13 |
| weshay|ruck | ysandeep|rover, we need to test project periodic-tripleo-ci-centos-7-standalone-upgrade-stein | 12:19 |
| weshay|ruck | https://review.rdoproject.org/zuul/builds?job_name=periodic-tripleo-ci-centos-7-standalone-upgrade-stein | 12:19 |
| ysandeep|rover | weshay|ruck, something is wrong with stein branch for all tripleo projects.. please check RR notes | 12:20 |
| weshay|ruck | ysandeep|rover, what line? | 12:20 |
| ysandeep|rover | weshay|ruck, 123 | 12:20 |
| ysandeep|rover | weshay|ruck, https://review.opendev.org/#/c/718468/ | 12:21 |
| weshay|ruck | ysandeep|rover, oh.. we probably have to remove tripleo-ansible from required projects | 12:23 |
| weshay|ruck | in the job config | 12:23 |
| ysandeep|rover | weshay|ruck hey o/ please check chatter with smcginnis on openstack-release | 12:26 |
| chandankumar | weshay|ruck, when free please have a look on this https://review.opendev.org/#/c/718946/ | 12:28 |
| *** amoralej is now known as amoralej|lunch | 12:30 | |
| *** jpena|lunch is now known as jpena | 12:32 | |
| weshay|ruck | ysandeep|rover, cix | 12:34 |
| ysandeep|rover | weshay|ruck, o/ coming | 12:34 |
| *** ratailor has quit IRC | 12:37 | |
| *** udesale_ has joined #oooq | 12:38 | |
| *** udesale has quit IRC | 12:40 | |
| pojadhav | rlandy, hello | 12:42 |
| rlandy | pojadhav: hi - in meeting | 12:43 |
| pojadhav | rlandy, np | 12:43 |
| soniya29 | weshay|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/1da826d3 | 12:47 |
| soniya29 | weshay|ruck, chandankumar, kopecmartin, error specified here - https://review.opendev.org/#/c/718066/7/doc/source/validate/validate.rst | 12:50 |
| *** rfolco has joined #oooq | 12:52 | |
| rfolco | I won't be able to run the scrum today | 12:53 |
| rfolco | someone please cover me | 12:53 |
| *** rfolco is now known as rfolco|bbl | 12:53 | |
| *** jbadiapa has joined #oooq | 12:54 | |
| marios | still not using openstack- & will need to edit the containers.j2.yaml | 13:18 |
| marios | weshay|ruck: ^^^ | 13:19 |
| marios | https://review.opendev.org/#/c/718701/1/roles/standalone/tasks/main.yml | 13:19 |
| marios | weshay|ruck: ^^^ | 13:19 |
| *** amoralej|lunch is now known as amoralej | 13:23 | |
| chandankumar | rlandy, https://review.rdoproject.org/r/#/c/26272/ | 13:33 |
| *** ykarel is now known as ykarel|afk | 13:39 | |
| chandankumar | ysandeep|rover, add the failing test here https://opendev.org/openstack/tripleo-quickstart-extras/src/branch/master/roles/validate-tempest/vars/tempest_skip_master.yml | 13:49 |
| chandankumar | ysandeep|rover, then it will get skipped | 13:49 |
| chandankumar | ysandeep|rover, till then ci will pass and ovn team will work on fixing that | 13:50 |
| ysandeep|rover | chandankumar, ack o/ will try that tomorrow morning | 13:51 |
| *** rfolco|bbl is now known as rfolco | 13:52 | |
| *** ykarel|afk is now known as ykarel | 14:04 | |
| *** ysandeep|rover is now known as ysandeep|away | 14:12 | |
| weshay|ruck | marios, I'm in | 14:28 |
| chandankumar | rlandy, https://review.opendev.org/#/c/717043/ | 14:34 |
| rlandy | looking | 14:34 |
| rlandy | chandankumar: should we remove tripleo-buildimage-overcloud-hardened-full-centos-8? | 14:37 |
| rlandy | has never passed | 14:37 |
| rlandy | and I don't know if this introduced any new failures | 14:37 |
| chandankumar | rlandy, yes, I will remove in a seperate patch | 14:37 |
| rlandy | chandankumar: ok - voting | 14:37 |
| chandankumar | rlandy, we never built that | 14:37 |
| rlandy | chandankumar: yea - let's just ditch it then | 14:38 |
| rlandy | chandankumar: +2'ed https://review.opendev.org/#/c/717043 | 14:39 |
| rlandy | go ahead and merged when ready | 14:39 |
| *** ykarel is now known as ykarel|away | 14:41 | |
| rlandy | marios: ah ... see rhos_release_args: "-P 17 -p {{ latest_compose_id.content }} -r 8.2" | 14:43 |
| rlandy | from chandankumar's review | 14:43 |
| rlandy | we need that to get 8.2 | 14:43 |
| chandankumar | rlandy, it needs to change as rhos-17 is based on 8.3 | 14:44 |
| chandankumar | we need to find an automated way to get these lists | 14:44 |
| rlandy | chandankumar: https://code.engineering.redhat.com/gerrit/#/c/196373 | 14:45 |
| rlandy | one comment on projects | 14:45 |
| chandankumar | rlandy, checking | 14:45 |
| rlandy | chandankumar: I assume next sprint we add them to the pipeline | 14:46 |
| chandankumar | rlandy, yes | 14:46 |
| rlandy | so they will have a place to run shortly | 14:46 |
| chandankumar | rlandy, done | 14:49 |
| rlandy | chandankumar: ok - +2 - can merge when your upstream review does | 14:53 |
| *** skramaja has quit IRC | 15:01 | |
| marios | rlandy: ack will do | 15:13 |
| weshay|ruck | rlandy, you getting notes on this? | 15:31 |
| marios | rlandy: posted v2 https://code.engineering.redhat.com/gerrit/197197 & v19 @ https://code.engineering.redhat.com/gerrit/196858 | 15:55 |
| marios | rlandy: feel free to take it ... will pickup whatever you leavein my morning | 15:55 |
| marios | rlandy: around few mins if you want to check/clarify something but its the stuff we discussed earlier | 15:55 |
| rlandy | marios: ack - just in meeting - will pick up | 15:56 |
| weshay|ruck | marios, rlandy fyi.. fbts issues w/ osp-17 look to be clearing up.. we should be able to switch to dlrn/dlrn-deps next sprint | 15:59 |
| rlandy | weshay|ruck: ack | 16:00 |
| rlandy | marios: ok - out of meeting - looking at reviews | 16:00 |
| rlandy | marios: looks good - one piece missing | 16:13 |
| marios | rlandy: ack .. feel free to take and iterate and i will check for updates tomorrow before continuing | 16:16 |
| marios | rlandy: you can see it running on the patch & https://sf.hosted.upshift.rdu2.redhat.com/zuul/t/tripleo-ci-internal/status | 16:16 |
| rlandy | marios: ack - looking really good though | 16:16 |
| rlandy | making two edits | 16:17 |
| marios | rlandy: yeah the ansiblification is slow progress cos of the nature of the hacks but... sure ack thanks ttyl have a good one | 16:17 |
| rlandy | nice work! | 16:17 |
| marios | rlandy: team work | 16:17 |
| rlandy | ansiblification :) | 16:17 |
| marios | ;) | 16:17 |
| rlandy | hmmm ... looking at https://sf.hosted.upshift.rdu2.redhat.com/logs/58/196858/19/check/periodic-tripleo-build-containers-rhel-8-master/e8850f5/logs/build.log | 16:18 |
| rlandy | it's still looking for openstack- | 16:18 |
| rlandy | it doesn't pick up the image_prefix | 16:19 |
| rlandy | maybe another hack :( | 16:19 |
| rlandy | marios: you set that | 16:20 |
| rlandy | but it's not picked up | 16:20 |
| rlandy | image_prefix: rhel-binary- | 16:22 |
| marios | rlandy: yes i noticed that too not sure whats going on | 16:23 |
| rlandy | marios: I saw that in my manual install | 16:23 |
| marios | rlandy: was trying to see if the containers under docker/ were correct don't know yet | 16:23 |
| marios | rlandy: i.e. if the things i copies work etc | 16:23 |
| marios | anwyay | 16:23 |
| * marios off | 16:23 | |
| rlandy | marios: I landed up doing a manula replace | 16:23 |
| rlandy | ok - will fix | 16:24 |
| marios | rlandy: thx ttyl | 16:24 |
| *** marios is now known as marios|off | 16:26 | |
| baha | rfolco: 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 |
| rfolco | baha, let me refresh on this | 16:48 |
| rfolco | baha, you use upstream role right? | 16:48 |
| * rlandy needs to go on exciting trip to grocery store ... biab | 16:52 | |
| *** rlandy is now known as rlandy|biab | 16:52 | |
| baha | I believe so. | 16:52 |
| baha | Locally we pull down the upstream playbooks to run | 16:53 |
| *** jpena is now known as jpena|off | 17:01 | |
| *** marios|off has quit IRC | 17:01 | |
| rfolco | baha, do you have a link to the logs ? | 17:08 |
| rfolco | baha, I am pulling centos:8 just fine in my ppc box, need to look your kolla conf | 17:09 |
| *** udesale_ has quit IRC | 17:11 | |
| baha | rfolco: We don't automatically upload local build logs anywhere (they're all manual runs), but I can go grab my kolla conf for you | 17:12 |
| baha | Just want kolla-build.conf? | 17:12 |
| rfolco | baha, and if using buildah | 17:12 |
| baha | We're using buildah | 17:13 |
| rfolco | ok, so for now kolla conf, let me check something | 17:13 |
| baha | http://paste.openstack.org/show/792051/ this is what the kolla-build.conf output looks like | 17:14 |
| rfolco | k thx | 17:15 |
| baha | let me know if you need anything else! | 17:15 |
| rfolco | baha, are you sure your base container 'centos' is amd64 ? or centos-binary ? | 17:17 |
| rfolco | baha, its the base built or the base pulled ? | 17:17 |
| baha | The centos8 image that's pulled, we're failing on building the base image | 17:18 |
| baha | When running a podman image inspect on the centos8 image, we get "Architecture": "amd64" | 17:19 |
| rfolco | what does happen if you run docker pull centos:8 or just centos in your ppc box ? | 17:19 |
| rfolco | hmm 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 |
| baha | For reference, the node I'm running on was just a base centos 8 cloud image with the playbooks run on it | 17:21 |
| rfolco | baha, do you have access ? | 17:22 |
| baha | I 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/centos | 17:23 |
| baha | That does indeed correspond with /etc/registries.conf | 17:24 |
| baha | docker.io would've been the next and final stop | 17:24 |
| weshay|ruck | rfolco, rlandy|biab let's chat about get_hash today.. when we're all here please | 17:27 |
| rfolco | weshay|ruck, ok just ping me | 17:27 |
| rfolco | baha, my ppc box is centos7 so not a good env to see whats going on | 17:30 |
| baha | Gotcha. It definitely sounds like some disparity between podman/docker or centos7/8 | 17:32 |
| baha | I'd give you direct access but the box is behind firewall | 17:32 |
| rfolco | baha, I am requesting a new centos8 ppc box but it would take a while... | 17:34 |
| rfolco | baha, all centos8 ppc images are pulled just fine using docker* from centos7 ppc box | 17:35 |
| rfolco | baha, I would look at podman pull first | 17:35 |
| baha | rfolco: http://paste.openstack.org/show/792053/ so this strikes me as odd | 17:43 |
| baha | And leads me to believe it's not something as simple as podman not setting the correct architecture | 17:47 |
| mjturek | baha rfolco: FWIW same thing happens with centos:7 | 17:51 |
| mjturek | this is super strange | 17:51 |
| *** rlandy|biab is now known as rlandy | 17:51 | |
| baha | going to step AFK, back in a few | 17:51 |
| *** baha is now known as baha|brb | 17:51 | |
| rfolco | mjturek, baha, yes I just tested w/ docker-ce on a centos7 ppc box, and it pulls ppc image | 17:52 |
| mjturek | np baha | 17:52 |
| rfolco | just don't forget your mask | 17:52 |
| *** rlandy is now known as rlandy|lunch | 17:57 | |
| mjturek | :) | 17:58 |
| mjturek | rfolco http://paste.openstack.org/show/792054/ config file for podman fyi | 17:58 |
| mjturek | baha|brb maybe we need to open a podman bug? | 18:02 |
| *** amoralej is now known as amoralej|off | 18:03 | |
| rfolco | mjturek, if a simple podman pull brings amd64 image, yes, definitely its podman bug | 18:05 |
| mjturek | any place to ask podman questions? | 18:15 |
| *** rlandy|lunch is now known as rlandy | 18:19 | |
| rlandy | weshay|ruck: rfolco: ready to talk about get-hash whenever you are | 18:19 |
| weshay|ruck | aye.. same | 18:20 |
| weshay|ruck | rfolco, ? | 18:21 |
| rfolco | ok let me change room | 18:22 |
| rfolco | one min | 18:22 |
| weshay|ruck | https://meet.google.com/udw-kgxc-rhy?authuser=1 | 18:25 |
| rfolco | rlandy, ^ | 18:27 |
| rfolco | you joining ? | 18:27 |
| weshay|ruck | https://review.rdoproject.org/r/#/c/26346/ | 18:28 |
| weshay|ruck | https://logserver.rdoproject.org/46/26346/3/check/rdo-tox-molecule/789427e/tox/reports.html | 18:28 |
| rlandy | weshay|ruck: rfolco: https://code.engineering.redhat.com/gerrit/#/c/195738/3/zuul.d/dlrn-jobs.yaml | 18:31 |
| weshay|ruck | #2 https://review.rdoproject.org/r/#/c/26369/ | 18:55 |
| *** Goneri has joined #oooq | 19:03 | |
| weshay|ruck | rlandy, can you help me find required projects for 3rd party | 19:07 |
| weshay|ruck | what file is that? | 19:07 |
| weshay|ruck | rdo | 19:07 |
| rlandy | weshay|ruck: getting | 19:07 |
| rlandy | weshay|ruck: looking for https://github.com/rdo-infra/review.rdoproject.org-config/blob/master/zuul/rdo.yaml? | 19:08 |
| weshay|ruck | rlandy, ya.. think so | 19:09 |
| weshay|ruck | https://github.com/rdo-infra/review.rdoproject.org-config/blob/master/zuul/rdo.yaml#L685 | 19:09 |
| weshay|ruck | so.. the stein branch was removed from tripleo-ansible | 19:09 |
| weshay|ruck | tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001ERROR Unable to find role in /var/lib/zuul/builds/6c753679129442bcb43a50f3f0b3253e/ansible/pre_playbook_7/role_1/tripleo-ansible | 19:10 |
| weshay|ruck | https://review.opendev.org/#/c/718757/ | 19:10 |
| weshay|ruck | not sure how to resolve that for just the branch | 19:10 |
| * weshay|ruck looks | 19:10 | |
| rlandy | oh gee | 19:10 |
| rlandy | branch_override | 19:11 |
| rlandy | weshay|ruck: ^^ those are all the projects zuul could clone | 19:12 |
| rlandy | doesn't mean it does clone them all | 19:12 |
| rlandy | in the job | 19:12 |
| weshay|ruck | k | 19:12 |
| * weshay|ruck looking at ovb base | 19:12 | |
| rlandy | you can override required_projects | 19:12 |
| rlandy | weshay|ruck: or set the branch on the one repo | 19:12 |
| rlandy | see the example of the OVB repo | 19:12 |
| weshay|ruck | name: tripleo-ci-base-ovb-notebroker | 19:13 |
| *** baha|brb is now known as baha | 19:14 | |
| weshay|ruck | think I understand now | 19:14 |
| weshay|ruck | thanks rlandy | 19:14 |
| *** sanjayu__ has quit IRC | 19:39 | |
| rlandy | weshay|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 |
| rlandy | https://sf.hosted.upshift.rdu2.redhat.com/logs/58/196858/20/check/periodic-tripleo-build-containers-rhel-8-master/c6e6cdb/logs/build.log | 19:50 |
| rlandy | actually writing the dockerfile | 19:50 |
| rlandy | for base | 19:50 |
| *** irclogbot_1 has quit IRC | 19:51 | |
| *** irclogbot_2 has joined #oooq | 19:51 | |
| weshay|ruck | rlandy, /me looks | 19:54 |
| weshay|ruck | rlandy, that's for barbican not base afaict.. barbican-worker | 19:55 |
| weshay|ruck | https://opendev.org/openstack/kolla/src/branch/master/docker/macros.j2#L98 | 19:55 |
| rlandy | weshay|ruck: passes that | 19:55 |
| rlandy | it's on base | 19:55 |
| rlandy | 2020-04-13 15:18:14 | File "/tmp/kolla-builds/f69542cf-9ff7-49fd-9e0c-e67f41c042f4/docker/base/Dockerfile.j2", line 269, in top-level template code | 19:55 |
| weshay|ruck | rlandy, oh. | 19:56 |
| weshay|ruck | {{ macros.disable_extra_repos('epel') }} | 19:56 |
| weshay|ruck | rlandy, do you have a fake epel.repo/ | 19:56 |
| weshay|ruck | ? | 19:56 |
| weshay|ruck | it may be required by some of these docker files | 19:56 |
| rlandy | umm ... probably not - looking | 19:56 |
| rlandy | marios made a downstream pre | 19:57 |
| rlandy | we probably need to add that | 19:57 |
| weshay|ruck | maybe we did that manually once and forgot | 19:57 |
| rlandy | 2020-04-13 19:13:03.641018 | LOOP [build-containers : Add dummy epel and epel-module repository] | 20:01 |
| rlandy | 2020-04-13 19:13:04.217289 | primary | changed: "epel" | 20:01 |
| rlandy | 2020-04-13 19:13:04.447003 | primary | changed: "epel-modular" | 20:01 |
| rlandy | weshay|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|ruck | hrm.. | 20:04 |
| weshay|ruck | rlandy, is it on the base container though? | 20:04 |
| weshay|ruck | rlandy, may need to play w/ this manually | 20:04 |
| rlandy | checking | 20:08 |
| rlandy | weshay|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.log | 20:09 |
| rlandy | search for epel | 20:09 |
| rlandy | weshay|ruck: Oh ... | 20:10 |
| rlandy | FROM ubi8:8.2-ondeck | 20:11 |
| rlandy | it's on rhel base | 20:11 |
| rlandy | not on ondeck | 20:12 |
| rlandy | FROM {{ base_image }}:{{ base_distro_tag }} | 20:15 |
| rlandy | base_image=rhel | 20:17 |
| rlandy | no idea why it's going to ondeck | 20:17 |
| weshay|ruck | rlandy, ya.. perhaps we need two copies of the base | 20:18 |
| weshay|ruck | the docker files :( | 20:18 |
| weshay|ruck | I remember seeing ondeck | 20:18 |
| rlandy | why? | 20:18 |
| rlandy | ondesk is on the renedred file | 20:18 |
| rlandy | rendered | 20:18 |
| rlandy | weshay|ruck: is there is setting to overwrite the rendered file | 20:18 |
| rlandy | if it's already there? | 20:18 |
| weshay|ruck | rlandy, let's look at barbican-worker dockerfile | 20:19 |
| weshay|ruck | maybe that has it in the jinja | 20:19 |
| weshay|ruck | as ondeck | 20:19 |
| rlandy | 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/barbican-worker/Dockerfile | 20:19 |
| rlandy | that is correct | 20:19 |
| rlandy | 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/Dockerfile.j2 | 20:19 |
| weshay|ruck | ya | 20:19 |
| rlandy | ^^ is also correct | 20:19 |
| rlandy | it should have taken rhel:8 | 20:20 |
| weshay|ruck | 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/Dockerfile | 20:20 |
| weshay|ruck | has ondeck | 20:20 |
| rlandy | per https://sf.hosted.upshift.rdu2.redhat.com/logs/58/196858/20/check/periodic-tripleo-build-containers-rhel-8-master/c6e6cdb/conf/kolla-build.conf | 20:20 |
| rlandy | it should not use the rendered file | 20:20 |
| weshay|ruck | rlandy, 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/Dockerfile | 20:20 |
| weshay|ruck | not what is checked in | 20:20 |
| rlandy | weshay|ruck: I don;t think it rendered that file yet | 20:22 |
| rlandy | it failed doing that | 20:22 |
| rlandy | check the next container down | 20:22 |
| rlandy | http://pkgs.devel.redhat.com/cgit/containers/openstack-base/tree/Dockerfile.j2?h=rhos-17.0-rhel-8#n269 | 20:25 |
| *** jbadiapa has quit IRC | 20:25 | |
| rlandy | ^^ error | 20:25 |
| weshay|ruck | ya.. agree it looks like it failed IN the template | 20:26 |
| weshay|ruck | rlandy, we may need to.. do a find | exec rm -f DockerFile | 20:26 |
| rlandy | {% macro disable_extra_repos(repos) %} | 20:27 |
| weshay|ruck | to remove all the prerendered dockerfiles as to not confuse ourselves w/ the logs | 20:27 |
| *** jfrancoa has quit IRC | 20:34 | |
| *** ccamacho has quit IRC | 20:40 | |
| rlandy | weshay|ruck: ha ... http://git.app.eng.bos.redhat.com/git/kolla.git/commit/kolla/template/methods.py?h=rhos-17.0-trunk-patches&id=5e7979f31d8e30980a6e31490d00345e51a14006 | 20:58 |
| * weshay|ruck looks | 20:58 | |
| weshay|ruck | rlandy, ah.. is internal kolla missing a change? | 20:58 |
| weshay|ruck | rlandy, I wonder if we could use upstream kolla w/ downstream dockerfiles | 20:59 |
| weshay|ruck | to save grief | 20:59 |
| rlandy | weshay|ruck: we were patching kolla in the manual | 20:59 |
| rlandy | and probably didn;t get that latest change | 20:59 |
| rlandy | it's an error in the donwstream now | 20:59 |
| weshay|ruck | ah.. | 20:59 |
| weshay|ruck | so does that fix it? | 20:59 |
| rlandy | I'm guess it should be (['epel']) | 21:00 |
| rlandy | no - that is what raises the error | 21:00 |
| rlandy | befor it was just ignored | 21:00 |
| weshay|ruck | ah | 21:00 |
| weshay|ruck | ++ | 21:00 |
| rlandy | let me see if I can sed replace | 21:00 |
| rlandy | and if that works, I'll have to propose a patch | 21:01 |
| weshay|ruck | rlandy, so.. not there yet.. but I think this will simplify it https://review.rdoproject.org/r/26346 | 21:11 |
| rlandy | looking | 21:13 |
| weshay|ruck | rlandy, also .. I think that ovb change worked.. https://review.rdoproject.org/zuul/stream/4b5a8e8bb87341ac96abe81b675cbb9d?logfile=console.log | 21:14 |
| rlandy | nice | 21:14 |
| rlandy | weshay|ruck: get-hash makes my hair hurt | 21:14 |
| * rlandy will have to think about this | 21:15 | |
| rlandy | but the general idea looks right | 21:15 |
| weshay|ruck | rlandy, what I don't know is if we can use hyphens as a key for a var | 21:18 |
| weshay|ruck | osp-17: foo | 21:18 |
| rlandy | weshay|ruck: split the var | 21:19 |
| weshay|ruck | k.. we'll keep chatting | 21:19 |
| rlandy | weshay|ruck: like osp-{{osp-release}} | 21:20 |
| weshay|ruck | rlandy, ya.. I guess we can pass whatever in zuul | 21:21 |
| weshay|ruck | k | 21:21 |
| weshay|ruck | rlandy, can you please vote on https://review.rdoproject.org/r/#/c/26414/ | 21:21 |
| rlandy | done | 21:22 |
| weshay|ruck | thanks | 21:23 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!