Wednesday, 2020-02-05

*** Goneri has joined #oooq00:05
*** jmasud has quit IRC00:11
*** jmasud has joined #oooq00:14
*** ysandeep|away is now known as ysandeep00:17
*** dsneddon has joined #oooq01:35
*** dsneddon has quit IRC01:41
*** Goneri has quit IRC01:47
*** rfolco|bbl has quit IRC02:40
*** rlandy has quit IRC03:03
*** apetrich has quit IRC03:12
*** ykarel|away is now known as ykarel03:30
ykarelweshay|ruck, wrt <weshay|ruck> the logs are broken again, wrong setting are applied there as we rely on provider, https://review.opendev.org/#/c/705379/ should fix it03:33
ykarelcan bring that setting in seperate patch though if combined one takes time03:35
weshay|ruckykarel, isn't this crazy :)03:38
ykareltoo much :)03:38
weshay|ruckhrm.. https://review.opendev.org/#/c/705378/03:41
weshay|ruckI'd make a joke.. but03:42
ykarelhmm ^^ is blocked as zbr wanted to do dynamic generation of logs03:42
ykarelreadme03:42
weshay|ruckya.. while we bleed03:42
weshay|ruckykarel, who hooked up the footer in 3rd party? it's awesome03:46
ykarelme no idea03:55
ykarelweshay|ruck, ^03:56
weshay|ruckykarel, well.. high five that person if we find out :)03:56
weshay|ruckykarel, and thanks to you as well for the logging patches :)03:56
weshay|ruckykarel, didn't think I had to coordinate "fix the logging" but apparently we do03:57
weshay|ruckI'll round folks up this week03:57
ykarelweshay|ruck, ack03:57
*** jmasud has quit IRC04:13
*** jmasud has joined #oooq04:13
*** skramaja has joined #oooq04:15
*** raukadah is now known as chkumar|rover04:22
ykarelweshay|ruck, so i understand u only fixed the ReadME to footer, thats nice Thanks04:34
ykarelwith https://review.opendev.org/#/c/702286/2/tasks/publish.yml04:35
*** udesale has joined #oooq04:53
*** soniya29 has joined #oooq05:29
*** ratailor has joined #oooq05:40
*** marios has joined #oooq06:02
*** jbadiapa has joined #oooq06:51
*** jfrancoa has joined #oooq06:52
* chkumar|rover headed to wework07:18
*** dtantsur has joined #oooq07:25
*** jaosorior has joined #oooq07:38
*** ykarel is now known as ykarel|lunch07:42
*** ysandeep is now known as ysandeep|lunch07:46
mariospanda: o/ hey see email when you get a chance re "ppc issues again" ... afaics it isn't fatal but maybe I missed something. I am hoping its just the (expected) error pulling ppc containers and then it continue, but weshay|ruck saw the error and assumed it was fatal. not sure but i can't see something more in logs07:46
*** jaosorior has quit IRC07:53
*** amoralej|off is now known as amoralej07:58
*** jaosorior has joined #oooq08:06
*** holser has joined #oooq08:08
*** jmasud has quit IRC08:12
*** jtomasek has quit IRC08:12
*** jmasud has joined #oooq08:14
*** tesseract has joined #oooq08:16
zbrykarel|lunch: re readme, i am afraid that the progress is blocked waiting for your review on https://review.opendev.org/#/c/705454/08:17
chkumar|roverykarel|lunch, http://zuul.openstack.org/builds?job_name=tripleo-ci-centos-7-standalone&branch=stable%2Ftrain&pipeline=gate&branch=stable%2Ftrain it appears to be healthy08:20
*** jtomasek has joined #oooq08:34
*** panda has quit IRC08:34
*** panda has joined #oooq08:35
*** bogdando has joined #oooq08:41
*** apetrich has joined #oooq08:48
zbrmarios: i am back on WIP on collect-cmds change, as sagi pointed critical bug (redirection not workig), i already had alternative uploaded but i need to see it producing desired results.08:49
zbrlocal testing passed but is not enough ;)08:49
*** ykarel|lunch is now known as ykarel08:52
ykarelchkumar|rover, yes that issue is random and hit only some time08:52
*** jpena|off is now known as jpena08:53
ykarelzbr, sure i will check, but i would say get feedback from others as well as it's changing the management of that docs08:54
ykarelfor me main thing is it should be easily managable/editable08:54
marioszbr: ack08:54
*** rascasoft has joined #oooq08:54
ykareland should render fine in logs footer08:54
zbrykarel: first look at the outcome, you can see both, new one is lowercase.08:54
ykarelzbr, yup i saw that08:55
zbrykarel: i need initial feedback before investing more time in it.08:55
zbrmaybe even better if you want to takeover the challenge, we could make it a reality faster.08:56
zbrthe idea is that a static file is a dead end, because it never knows what you really collected, or if the urls are correct.08:56
zbrdynamic is the key, run after you already collected all files, the POC is using a simple ansible module to produce that, one using jinja2 inside, so it should easy to understand.08:57
zbrnot much magic involved.08:57
zbrmy initial attempt was to do it in ansible but soon i realised it would not work, ansible is too slow to do this. 10 lines of python did the trick,08:58
ykarelzbr, as first look, LGTM, would need additionally:- log list categorized, template outside of python file09:00
zbrykarel: i like that we think alike.09:00
ykarelzbr, so posted my +1 on the review with remarks, will remove the static changes from my review09:17
ykareljust keep vexxhost related changes09:18
*** rascasoft has quit IRC09:20
*** rascasoft has joined #oooq09:20
*** derekh has joined #oooq09:40
chkumar|roversshnaidm|afk, zbr https://review.rdoproject.org/r/#/c/24803/09:45
chkumar|roverplease have a look when free09:45
zbrchkumar|rover: e may not be the only users of this nodeset, you should not combine nodeswitching with its removal.09:47
chkumar|roverzbr, fixed09:47
zbrput removal separated and you have my full support.09:47
chkumar|roverzbr, yes sure09:48
chkumar|roverarxcruz, as we discussed yesterday, do we want to bump concurrency of standalone full tempest?09:49
arxcruzchkumar|rover: let's test09:49
zbrchkumar|rover: lets switch to rdo about that subject09:51
chkumar|roverzbr, aye09:51
pandamarios: I think you are right, but even last time I could not find the fatal log. And centos7_master logs for that time are missing.09:55
mariospanda: k lets wait for weshay|ruck maybe there is something else we're missing09:58
mariospanda: thanks for sanity check09:58
*** rfolco|bbl has joined #oooq10:08
*** jmasud has quit IRC10:11
*** jmasud has joined #oooq10:12
*** panda is now known as panda|bbl10:33
*** jmasud has quit IRC10:36
*** ysandeep|lunch is now known as ysandeep10:38
sshnaidm|afkzbr, I didn't understand your comment here: https://review.opendev.org/#/c/705623/10:39
mariosarxcruz: chkumar|rover: fs20 scenario only 3h35m @ https://review.rdoproject.org/r/#/c/24339/10:41
mariosarxcruz: are you still exploring there? can you please comment on https://review.opendev.org/#/c/701016/ ... this has been going on for a while. i'd rather not stall it... if we are going with an alternative approach then lets do that you can take over https://tree.taiga.io/project/tripleo-ci-board/task/1383 and i will abandon the patches there10:42
mariosarxcruz: starting to get sick of it ;)10:42
chkumar|roverzbr, how you solved re2-devel issue for centos8?10:45
chkumar|roverzbr, https://logserver.rdoproject.org/03/24803/3/check/tox-linters/13414c7/job-output.txt10:45
chkumar|roverzbr, _re2.cc:37:10: fatal error: re2/re2.h: No such file or directory10:45
*** jmasud has joined #oooq10:46
arxcruzmarios: chkumar|rover  this is great, we can have both concurrency 4 and split jobs10:49
arxcruzthis will decrease the time a lot10:50
arxcruzmarios: yesterday we merge concurrency = 4 on fs02010:50
arxcruzwe should not see timeouts now10:50
chkumar|roverFYI!, if you are sending patches on rdo-jobs, tox jobs is broken, working on fixing it10:52
mariosarxcruz: ah maybe this is why the time went to 3.35 then10:54
arxcruzmarios: not really, because i see your patch have concurrency 310:54
*** jmasud has quit IRC10:55
arxcruzmarios: but periodic fs020 today pass ;)10:56
arxcruzand with less time than before, but we are still in the edge, any change we might get timeouts again10:56
*** udesale_ has joined #oooq11:13
*** udesale has quit IRC11:16
zbrsshnaidm|afk: i replied. thanks for pointing to the redirection bug, i already fixed it in patchset no2, rdo already reported results.11:17
*** derekh has quit IRC12:07
*** derekh has joined #oooq12:08
*** tosky has joined #oooq12:12
* chkumar|rover headed home, will meet during program call12:29
*** jpena is now known as jpena|lunch12:33
*** soniya29 has quit IRC12:38
*** rfolco|bbl is now known as rfolco12:45
rfolcoweshay|ruck, you know where this is called ? 2020-02-05 11:57:28.632750 | primary | + tripleo-repos --mirror http://mirror.centos.org --rdo-mirror https://trunk.rdoproject.org highavailability opstools12:46
rfolcoweshay|ruck, its calling w/ highavailability and opstools args12:46
mariosrfolco: yeah here12:46
mariosrfolco: i was literally just commenting on it https://review.opendev.org/#/c/701937/12/roles/tripleo-repos/tasks/main.yml12:46
weshay|ruckrfolco, marios I have it working :)12:47
mariosrfolco: i am updating my patch on weshay|ruck latest stuff and commented there https://review.opendev.org/#/c/702993/11/roles/tripleo-repos/tasks/main.yml12:47
weshay|rucknew match up on the tripleo-repo role12:47
rfolcomarios, weshay|ruck ha and powertools should come for FREE12:47
weshay|ruckrfolco, they do12:47
weshay|ruck:)12:47
weshay|ruckif ur on the right dlrn12:47
rfolcowe should not specify as args then12:47
mariosrfolco: ack that's what i was asking in my comment at https://review.opendev.org/#/c/702993/11/roles/tripleo-repos/tasks/main.yml12:47
weshay|ruckwe dont12:48
mariosrfolco: i'll remove highavailability12:48
rfolcoweshay|ruck, look this https://review.opendev.org/#/c/701937/12/roles/tripleo-repos/tasks/main.yml12:48
rfolcoit raises an error here12:48
rfolcohttps://logserver.rdoproject.org/75/24775/11/check/tripleo-build-containers-centos-8-master/34a8afe/job-output.txt12:48
rfolcomarios, cool :)12:48
mariosrfolco: see my replies ^^12:48
weshay|ruckrfolco, please follow my latest changes12:48
mariosweshay|ruck: i don't understand why we have 2 patches12:49
rfolcoweshay|ruck, ok we need a semaphore to update patches at the same time :)12:49
mariosweshay|ruck: should i abandon my one?12:49
mariosweshay|ruck: i don't see the point having to sync patches12:50
weshay|ruckmarios, not sure.. up to you guys.. I split off a branch to poke at it as well12:50
weshay|ruckmarios, you can keep doing what you were doing or jump into my patch12:50
weshay|ruckdon't care12:50
mariosweshay|ruck: well ^^ 'up to you guys' not really, i mean if i vote to keep my patch then you will just keep updating your one too :D12:50
mariosso?12:50
rfolcoyeah its one or the other12:50
weshay|ruckwant to chat in 5min?12:51
mariosweshay|ruck: sure12:51
weshay|ruckk k12:51
mariosweshay|ruck: did you see the email re promoter and ppc containers12:51
mariosweshay|ruck: i am not sure if there was a problem or i missed something more in logs.. i asked panda|bbl for sanity check too12:51
mariosweshay|ruck: as per the email, i think it was working OK (it has error for ppc pull, but we expected those, and then it continues with ignore_errors)12:52
weshay|ruckok ready marios rfolco12:55
rfolcowhere12:55
weshay|ruckhttps://meet.google.com/vui-awyj-qbh?authuser=112:56
*** rlandy has joined #oooq13:01
chkumar|roverweshay|ruck, program call?13:04
*** nhicher has quit IRC13:05
*** ysandeep is now known as ysandeep|away13:07
chkumar|roversshnaidm|afk, zbr rlandy https://review.rdoproject.org/r/#/c/24803/ is good to go13:11
chkumar|roverrdo jobs tox jobs broken currently13:11
marios2020-02-05 11:11:29.884912 | ubuntu-bionic | ./tripleo_repos/tests/test_main.py:110:43: E502 the backslash is redundant between brackets13:11
*** ratailor has quit IRC13:12
zbrchkumar|rover: you removed an useful comment,               # needed to avoid accidental call of /usr/bin/tox13:12
chkumar|roverzbr, you want I can readd back13:13
chkumar|roverzbr, done13:14
zbradded a -2 until we get confirmation about EPEL part.13:16
zbrtouching base/pre.yaml is highly sensitive13:16
zbrif we get the ok, wf it. but i just want confirmation from wes and rdo.13:17
chkumar|roverzbr, epel is disabled upstream13:17
chkumar|roverzbr, for re2-devel it comes from epel13:18
chkumar|roverzbr, i can parameterize it based on tox job13:19
*** ykarel is now known as ykarel|away13:27
weshay|ruckchkumar|rover, you need to sync?13:31
*** amoralej is now known as amoralej|lunch13:31
chkumar|roverweshay|ruck, yes13:31
weshay|ruckchkumar|rover, https://meet.google.com/vui-awyj-qbh?authuser=113:32
*** jpena|lunch is now known as jpena13:35
weshay|ruckrlandy, anyone else using bm env d?13:37
weshay|ruckrlandy, chandan and I are wondering about running rhel8 fs001 on bm13:37
rlandyweshay|ruck: I have been using env E but currently nobody is on env D13:38
rlandyweshay|ruck: we were going to use it for the bootcamp13:38
weshay|ruckrlandy,  can you chat w/ us for a hot sec https://meet.google.com/vui-awyj-qbh?authuser=113:38
rlandybut I think we can get away with one env13:38
arxcruzzbr: https://review.opendev.org/#/c/704762/ please :)13:41
mariosweshay|ruck: rfolco: looks like mark abandoned https://review.opendev.org/#/c/692368/13:46
mariosweshay|ruck: rfolco: so perhaps master is correct as folco is trying already ? we don't need a patch?13:46
rfolcomarios, yes, but merged code includes a lot of skips13:46
rfolcomarios, docker/base/Dockerfile.j2:    # FIXME(mgoddard): Not available for CentOS 8 yet.13:47
mariosrfolco: i see13:47
rfolcoi am trying to comment out ceph to see what happens13:47
rfolcoceph excludes*13:47
*** saneax has quit IRC13:49
chkumar|roverzbr, weshay|ruck rlandy sshnaidm|afk marios arxcruz panda|bbl rfolco found something interesting https://github.com/cs01/termpair13:49
arxcruzcool13:50
arxcruzit's a tmate for browser13:50
weshay|ruckchkumar|rover, https://review.rdoproject.org/r/#/c/24803/ comments on that14:01
*** amoralej|lunch is now known as amoralej14:01
chkumar|roverweshay|ruck, yes, I will hack based on job type14:02
weshay|ruckchkumar|rover, k.. be really careful here14:02
zbrchkumar|rover: afaik, you need it only for docs for the moment, enable for these only and we shoud be safe, i deal was to have a enable-epel role, as is easier to reuse.14:03
chkumar|roverzbr, it is needed for linters14:03
zbri think i even have an upstream CR for adding epel option to setup-repos, argument.14:03
chkumar|roverprecommit is pulling re2-devel14:03
chkumar|rovermay be ansible lint14:04
zbri remember fixing this issue one or two weeks ago,... bit strange.14:04
*** dtantsur is now known as dtantsur|brb14:05
zbrchkumar|rover: you see https://github.com/rdo-infra/rdo-jobs/blob/master/bindep.txt#L1314:09
zbrthe reson for not conditon is because it comes from epel14:10
zbreven if you put epel-release here it may not help as it is not run as a script14:10
zbrevrika! i remembered where14:11
weshay|ruckchkumar|rover, https://beaker.engineering.redhat.com/view/rdoci-hp-01.v100.rdoci.lab.eng.rdu2.redhat.com#power-settings14:12
zbrchkumar|rover: it was when i moved molecule to ansible zuul, try to sync deps from https://github.com/ansible-community/molecule/blob/master/bindep.txt14:14
zbri do no think that re2-devel is mandatory, ZAC is already using centos-8 for tox jobs.14:15
*** ykarel|away is now known as ykarel14:20
*** TrevorV has joined #oooq14:47
weshay|ruckrlandy, chkumar|rover https://code.engineering.redhat.com/gerrit/#/c/189436/7/zuul.d/projects.yaml14:47
*** dpawlik has joined #oooq14:48
*** panda|bbl is now known as panda14:49
*** sshnaidm|afk is now known as sshnaidm14:51
*** ykarel is now known as ykarel|away15:08
mariospanda: o/ are you available for a quick call at some point please should be brief. i am adding a ppc_manifests switch into the manifest push so we can disable ppc but keep the rest with a switch. having some issues decoding that product https://github.com/rdo-infra/ci-config/blob/a0e0049280218c37b9f9a3ad5ee2d8f884437e6b/ci-scripts/infra-setup/roles/promoter/molecule/container-push/playbook.yml#L126 in15:16
mariosparticular where containers is set. so it fails now when ppc is disabled cos it tries to check the ppc ones too.15:16
weshay|ruckrfolco, marios ok.. have a build running w/o patches15:18
weshay|ruckError: No matching repo to modify: influxdb.15:18
weshay|ruckon the base container15:19
weshay|ruckI think rfolco mentioned that15:19
weshay|ruckrfolco, we should probably update the tripleo-ci-testing pin on centos-815:20
weshay|ruckmarios, rfolco k.. we have wf on the tripleo-repos patch15:24
mariosweshay|ruck: ack cool will be one less thing15:24
pandamarios: today ?15:24
mariospanda: sure if you can15:25
chkumar|roverweshay|ruck, I am logging off now, will follow the pad tomorrow15:25
chkumar|roversee ya15:25
chkumar|roverweshay|ruck, please take care of the cix15:25
weshay|ruckchkumar|rover, ya.. I got you :)15:25
mariospanda: if not then i'll have to reserve a slot for tomorrow morning np15:25
weshay|ruckchkumar|rover, thanks.. get some rest15:25
chkumar|roverweshay|ruck, on tempest side cix, all patches are about to merge, final is still pending15:25
chkumar|roverweshay|ruck, this two jobs needs attension http://zuul.opendev.org/t/openstack/builds?job_name=tripleo-ci-centos-7-scenario002-standalone&job_name=tripleo-ci-centos-7-scenario004-standalone&branch=stable%2Fstein&pipeline=check15:26
pandamarios: in 30 minutes ok or too late for you ?15:26
chkumar|roverred from long time15:26
chkumar|roveror file a bug tomorrow15:26
chkumar|roverweshay|ruck, on scenar02 adlee is looking15:26
weshay|ruckchkumar|rover, nice15:26
chkumar|roverthat's it15:26
mariospanda: yes fine thanks15:26
*** chkumar|rover is now known as raukadah15:27
raukadahsee ya15:27
rlandyraukadah: weshay|ruck: I'm merging the review to add the fs021 job definition15:28
*** fmount has quit IRC15:52
*** dtantsur|brb is now known as dtantsur15:53
*** fmount has joined #oooq15:53
pandamarios: where ?16:00
mariospanda: o/ so perhaps we can skip the call... worked out the thing i was stuck on. if you like though you can have a look at that please https://review.rdoproject.org/r/#/c/24807/16:00
mariospanda: if you would like to we can still talk but I got past the issue i had16:00
mariospanda: i added an extra test in the playbook for the container push scenario so we can check if its ok16:01
pandamarios: looking16:01
mariospanda: ack no rush can be later but thanks and sorry to mess you around (and thanks for being available to talk anyway)16:02
mariospanda:++16:02
*** Goneri has joined #oooq16:02
*** Trevor_V has joined #oooq16:03
*** openstackstatus has joined #oooq16:04
*** ChanServ sets mode: +v openstackstatus16:04
weshay|ruckrfolco, do you know anything about the influxdb error?16:05
weshay|ruckError: No matching repo to modify: influxdb.16:05
*** TrevorV has quit IRC16:07
rfolcoweshay|ruck, yes, I am mocking it like I did with the others16:08
weshay|ruckrfolco,  ah geez..16:08
rfolcoweshay|ruck, we need a patch somewhere16:08
weshay|ruckk.. /me looks at your hackmd16:08
mariosweshay|ruck: rfolco: panda: created a new task for that ppc thing * https://tree.taiga.io/project/tripleo-ci-board/task/1511 (under consolidate promoter tests * https://tree.taiga.io/project/tripleo-ci-board/us/1493 )16:09
weshay|ruckmarios, thanks16:09
rfolcoweshay|ruck, for now I am just creating a fake repo and doing this in dockerfile16:10
rfolcoCOPY influxdb.repo /etc/yum.repos.d/.16:10
rfolcoI can't find why cinder is being skipped16:10
pandamarios: who sets the ppc_manifest var ?16:13
mariospanda: well for now we can default it in the vars/main. afterwards we want to add some conditional set_fact in the mainfest_push.yml... weshay|ruck was thinking along the lines of 'if the first 5 ppc containers fail, disable it'16:14
rfolcoweshay|ruck, we still have bizarre conditions in kolla code like this16:14
rfolco        {% if distro_package_manager == 'dnf' or base_distro not in ['centos'] %}16:14
rfolco            {% set cinder_base_packages = cinder_base_packages + [16:14
rfolco                'qemu-img'16:14
rfolco            ] %}16:14
mariospanda: commented there https://review.rdoproject.org/r/#/c/24807/1/ci-scripts/container-push/roles/containers-promote/tasks/manifest-push.yml16:15
sshnaidmzbr, wrt: https://review.opendev.org/#/c/705623/ I think it's worth to leave redirection to files in commands themselves, the construction "cmd" > "file" makes it overcomplicated, error prone, unpredictable results.16:18
sshnaidmzbr, and you don't need this to run it in parallel16:18
weshay|ruckrfolco, marios related patch https://review.opendev.org/#/c/704970/3/kolla/template/repos.yaml16:19
sshnaidmzbr, you don't have locks for same file, so why not to include them in the "cmd" body16:19
*** rascasoft_ has joined #oooq16:19
owalshweshay|ruck: is it ok to recheck https://review.opendev.org/704880?16:19
*** rascasoft has quit IRC16:19
weshay|ruckowalsh, no need.. it should kick after https://review.opendev.org/#/c/704919/ merges16:20
weshay|ruckowalsh, I'll watch it.. I'm only 90% sure of that fact :)16:21
owalshweshay|ruck: ack, didn't realise zuul was that smart :-)16:21
zbri disagree here, doing the redirection ourselves does simplify the command definitions and assure consistency of how they are handled.16:21
weshay|ruckrfolco, marios so influx works on rhel816:22
zbrfor the moment parallel execution is not part of the change, it is considered but not part.16:22
zbralso this approach is much safer than current one where is very easy for a user to miss adding a redirection and spam ansible with too much stdout output.16:23
zbrby keeping the redirection in one place, we assure it is kept that way.16:23
*** udesale_ has quit IRC16:24
weshay|ruckrfolco, when you have a sec can you please update https://hackmd.io/dSagCbocQ4KSVEZR1uf8Tw16:24
rlandyweshay|ruck: raukadah: https://code.engineering.redhat.com/gerrit/#/c/191279/ - let's see how far that gets16:28
rlandyI have some suspicion that we will have to do more steps to get the right release file on the virt undercloud. but we'll see16:28
rfolcoweshay|ruck, will do asap... I just wanted to get some progress with master, but some are skipped somehow... I can't find where16:29
*** jmasud has joined #oooq16:29
weshay|ruckrlandy++16:29
weshay|ruckre: influx CentOS one is not usable due to wrong arch name used (arm64 instead of16:30
weshay|ruckaarch64).16:30
weshay|ruckrfolco, added influx to the skip list got passed it16:35
weshay|ruckalso noticed.. kolla creates the influx.repo for the containers16:35
weshay|ruckname = InfluxDB Repository - RHEL $releasever16:35
weshay|ruckbaseurl = https://repos.influxdata.com/rhel/$releasever/$basearch/stable16:35
weshay|ruckenabled = 116:35
weshay|ruckgpgcheck = 116:35
weshay|ruckgpgkey = https://repos.influxdata.com/influxdb.key16:35
weshay|ruck[zuul@wes base]$16:35
rfolcoweshay|ruck, influx is just noise.16:35
weshay|ruckrfolco, marios job is getting to the build https://review.rdoproject.org/zuul/stream/cdb92414fad74d388fae1a1177d3f945?logfile=console.log16:36
weshay|ruckheh.. then failing16:36
weshay|ruckhttps://logserver.rdoproject.org/95/24795/3/check/tripleo-build-containers-centos-8-master/cdb9241/logs/build.log16:38
weshay|ruck2020-02-05 16:36:06 | sudo: buildah: command not found16:39
*** panda is now known as panda|bbl16:40
weshay|ruckrfolco, sorry pita here..16:42
weshay|ruckrfolco, not seeing where we ensure buildah is installed16:42
weshay|ruckthinking to add to the bindep16:42
weshay|ruckmaybe it's preinstalled in rhel?16:42
rfolcoweshay|ruck, https://github.com/openstack/tripleo-ci/blob/master/roles/build-containers/tasks/pre.yaml#L4616:43
rfolcoshould have installed16:43
weshay|ruckthanks16:44
rfolcohttps://review.rdoproject.org/r/#/c/24775/11/zuul.d/build-containers.yaml@5716:44
rfolcoweshay|ruck, I just don't know why it didn't install it16:44
weshay|ruck020-02-05 16:34:36.803151 | TASK [build-containers : Install Buildah]16:44
weshay|ruck2020-02-05 16:34:36.840759 | primary | skipping: Conditional result was False16:44
weshay|ruckrfolco, I'll get it16:44
weshay|ruckrlandy, rhel bm needs this patch https://review.opendev.org/#/c/702100/16:51
rlandyok - adding16:52
*** bogdando has quit IRC16:54
*** marios is now known as marios|out16:56
weshay|ruckrfolco, does the containers build need to run as root or sudo?17:08
rfolcoweshay|ruck, manually I run as normal user17:09
rfolcoopenstack overcloud container image build     --kolla-config-file /home/centos/build/kolla-build.conf  --exclude opendaylight --use-buildah > build.log 2>&117:09
weshay|ruckhttp://pastebin.test.redhat.com/83338617:10
rfolcoweshay|ruck, try to run that command inside your base container, you'll see what is failing17:13
rfolcoweshay|ruck, one of those repos probably doesn't exist17:13
weshay|ruckrfolco, just got perm errors17:14
weshay|ruckbut will double check17:14
weshay|ruckthanks17:14
weshay|ruckrfolco, so do I need to copy the repos from /etc/yum.repos.d/ to the base container w/ these kolla names?17:17
weshay|rucklike .. PowerTools17:18
*** marios|out has quit IRC17:18
rfolcofirst time I built I used rhel's method, build a base one and kolla builds on top of it17:19
rfolcoand I injected all repos there in the dockerfile17:19
rfolcothen built the base one17:19
rfolcosudo buildah bud -t centos:8 .17:19
rfolconow I am trying like you17:19
rfolcowithout a base one17:20
rfolcoso let me see what we need to do with repos17:20
rfolcoweshay|ruck, yeah if you don't build a base one injecting repos, you'll miss opstools, then ha, influx, etc17:24
weshay|ruckrfolco, ah cool17:24
*** soniya29 has joined #oooq17:24
rfolco[root@b0ee3447d0f4 /]# dnf -y install dnf-plugins-core centos-release-opstools     && dnf config-manager --enable PowerTools     && dnf config-manager --disable influxdb && dnf -y distro-sync --security --sec-severity=Important --sec-severity=Critical     && dnf clean all && rm -rf /var/cache/dnf17:24
weshay|ruckrfolco, so this file https://review.opendev.org/#/c/702993/17/roles/build-containers/templates/Dockerfile.centos8.j217:24
rfolcoNo match for argument: centos-release-opstools17:24
*** aakarsh has joined #oooq17:25
sshnaidmzbr, I think we have to do redirection in task, sorry, in your patch it gives unpredictable results, and this is really confusing for future users, if you want I can explain in mtg17:25
rfolcoyes should be this one. As long as you build a base one called centos, it will be used by kolla17:26
weshay|ruck2020-02-05 17:24:15.927847 | TASK [build-containers : Install Buildah]17:26
weshay|ruck2020-02-05 17:24:27.472439 | primary | changed17:26
weshay|ruckfixed the install of buildah17:26
sshnaidmzbr, maybe if you run the whole cmd in subshell, then maybe.. not sure17:26
weshay|ruckrfolco, if you can pastebin what you think https://review.opendev.org/#/c/702993/17/roles/build-containers/templates/Dockerfile.centos8.j2 should look like17:26
weshay|ruckthat would help us sync17:27
zbrsshnaidm: please commend on review with example.17:27
sshnaidmzbr, example of what?17:27
zbrof when it breaks17:28
rfolcoweshay|ruck, line 127 is my dockerfile http://pastebin.test.redhat.com/83338917:28
weshay|ruckoh I see17:28
sshnaidmzbr, can you please re-read my message again?17:29
rfolcoweshay|ruck, re: buildah install, cool thanks17:29
rfolcoweshay|ruck, don't ask me why, the base container I built w/ sudo buildah bud... then openstack build command without sudo17:30
rfolcoI had to build*17:30
weshay|ruckrfolco, don't think we can use epel btw17:30
rfolcoweshay|ruck, epel is fake17:30
weshay|ruckk17:31
rfolcothose repos are fake except ceph_master17:31
rfolcoits just to get something built17:31
rfolcoweshay|ruck, also need to patch kolla... kolla/image/build.py, remove skips for centos8 like ceph, collectd17:32
rfolcoweshay|ruck, on top of master ^17:32
weshay|ruckwhich patch?17:32
rfolcoI hacked it manually, if this works now, I'll send out the patch17:32
*** rascasoft_ has quit IRC17:32
weshay|ruckk17:33
weshay|ruckthanks17:33
rfolcosorry, didn't get a chance to structure the steps yet17:33
rfolcogoing on trial and error methodology17:33
rfolcoweshay|ruck, for example, what we have for influx? you know? who knows ?17:34
weshay|ruckrfolco, /me experimenting w/ the skip list for now17:36
weshay|ruckwe should get that w/ ceph17:36
weshay|ruckwhen the repos are avail17:36
rfolcoweshay|ruck, ceph is in skip list, so barbican-api, cinder-* will be skipped17:37
rlandyweshay|ruck: 1-on-1? or do you want to skip it?17:37
weshay|ruckrlandy, I can chat17:37
*** aakarsh has quit IRC17:38
*** dtantsur is now known as dtantsur|afk17:48
*** jaosorior has quit IRC17:49
*** jpena is now known as jpena|off17:53
weshay|ruckhttp://git.app.eng.bos.redhat.com/git/tripleo-environments.git/tree/roles/prep-internal-host17:58
weshay|ruckrlandy, ^17:59
*** dsneddon has joined #oooq18:04
*** soniya29 has quit IRC18:07
*** tesseract has quit IRC18:11
*** chem has quit IRC18:16
*** amoralej is now known as amoralej|off18:16
*** dpawlik has quit IRC19:04
*** dpawlik has joined #oooq19:05
rfolcoweshay|ruck, RUN echo 'binary not yet available for centos'19:09
rfolcoin nova-libvirt19:09
*** holser has quit IRC19:15
rfolconeed to find out what changed in centos8 for libvirt19:20
rfolcohttps://github.com/openstack/kolla/blob/master/docker/nova/nova-libvirt/Dockerfile.j2#L46-L5419:20
rfolcoweshay|ruck, ^19:20
weshay|ruckk.. sec in 1-119:27
weshay|ruckrfolco, 5min chat?19:34
rfolcoyeah19:34
rfolcoweshay|ruck, gmeet?19:35
weshay|ruckhttps://meet.google.com/twm-pdro-dbp?authuser=119:36
weshay|ruckrfolco, https://etherpad.openstack.org/p/kolla-centos819:42
weshay|ruckrlandy, sorry to bug you..19:50
weshay|ruckwhat was the internal repo w/ Dockerfiles.. that's not in kolla?19:50
* weshay|ruck looking at https://hackmd.io/6gE-LZYMSzmoP3IFt7puKg19:50
weshay|ruckhttps://pkgs.devel.redhat.com/cgit/containers/19:51
weshay|ruckmaybe19:51
weshay|ruck2020-02-05 19:25:18 | Stderr: 'error building at STEP "RUN dnf -y install ceph-common cyrus-sasl-scram libguestfs libvirt-client libvirt-daemon libvirt-daemon-config-nwfilter libvirt-daemon-driver-nwfilter openvswitch trousers qemu-kvm edk2-ovmf libseccomp openssl-perl openstack-nova-migration podman     && dnf clean all && rm -rf /var/cache/dnf": error while running runtime: exit status 1\n'19:53
rlandyweshay|ruck: checking20:06
*** jtomasek has quit IRC20:06
rlandyhttp://pkgs.devel.redhat.com/cgit/containers/openstack-nova-api20:06
rlandyweshay|ruck: they are all in their own repos20:06
rlandybase is in http://pkgs.devel.redhat.com/cgit/containers/openstack-base20:07
rlandyweshay|ruck: take 7 patches20:07
*** jmasud has quit IRC20:11
weshay|ruckrfolco, ok.. adapting this https://opendev.org/openstack/tripleo-ci/src/branch/master/roles/build-containers/tasks/main.yaml#L108-L14420:12
rfolcoweshay|ruck, yes. This is a way when you want a custom base20:12
rfolcoweshay|ruck, or change that centos8.dockerfile + pre tasks20:13
*** jmasud has joined #oooq20:14
*** holser has joined #oooq20:41
sshnaidmrlandy, so do you want to change "release" to "baremetal_provision" in py script?20:43
rlandysshnaidm: yes20:43
rlandysshnaidm: I have another patch ready20:44
sshnaidmrlandy, cool20:44
rlandybut your patch is testing now20:44
rlandysshnaidm: I'd like to see it complete testing before I switch20:44
sshnaidmrlandy, ack20:44
weshay|ruckrfolco, fyi.. ceph is not in the expected to build20:45
weshay|rucklist20:45
rfolcoweshay|ruck, don't see any skips in the code or errors in the log20:51
mjturekrfolco: was able to upload a known working image today. Upon pulling it from rdoproject.org, I still get the EOF error20:58
rfolcomjturek, wtf20:59
mjturekrfolco: it's still that first chunk21:01
*** jfrancoa has quit IRC21:01
rfolcomjturek, what are you using to push/pull21:01
rfolcomjturek, nm, sorry I am mixing problems21:08
mjturekrfolco: docker for push and pull (just to be clear)21:14
mjturekrfolco: do you know what registry software they use?21:15
baha(Also, just to reiterate, we were able to push these images up to and down from dockerhub just fine)21:15
rfolcoyeah must be on server side since I get the same EOF21:15
rfolcosir, unplug your modem and re-plug it back21:16
rfolcomjturek, I am almost giving you dummy suggestions like this ^21:16
mjtureklololol21:17
bahaare you sure your build server is connected to the internet21:18
*** tosky has quit IRC21:18
bahaI mentioned this to mjturek, but I'm wondering if it's possible that the registry is tossing new uploads, since the base container isn't actually changing hash/version when we upload it. Would be something to check with the RDO folks21:23
bahaIf we had an upload that didn't work quite right and future uploads are being ignored, that could cause it21:24
*** rfolco has quit IRC21:27
*** holser has quit IRC21:30
*** rfolco has joined #oooq21:31
*** rfolco has quit IRC21:35
mjturekbaha: unfortunately don't know enough about registries to know if that's plausible or not21:38
bahasame21:38
bahaI would hope checksumming would occur at some step in the chain to prevent that, but have no other immediate ideas21:39
*** yolanda has joined #oooq22:35
*** Trevor_V has quit IRC22:40
*** rlandy is now known as rlandy|bbl23:25

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