Thursday, 2020-01-23

*** rfolco has quit IRC00:17
*** rfolco has joined #oooq00:23
*** rf0lc0 has joined #oooq00:25
*** rfolco has quit IRC00:27
*** jaosorior has quit IRC00:28
*** rf0lc0 has quit IRC00:29
*** rf0lc0 has joined #oooq00:32
*** rf0lc0 has quit IRC00:38
* ssbarnea uploaded an image: ima_759aa5c.png (209KB) < https://matrix.org/_matrix/media/r0/download/matrix.org/NAQUMLyUpOROnfbaZNmJVPFG >00:39
*** ysandeep has joined #oooq01:03
*** ysandeep has quit IRC02:02
*** yolanda has quit IRC02:12
*** yolanda has joined #oooq02:17
*** ysandeep has joined #oooq02:24
*** yolanda has quit IRC02:27
*** yolanda has joined #oooq02:34
*** wznoinsk has quit IRC02:58
*** apetrich has quit IRC03:11
*** rlandy|bbl is now known as rlandy03:17
*** udesale has joined #oooq04:11
*** ratailor has joined #oooq05:21
*** surpatil has joined #oooq05:39
*** raukadah is now known as chkumar|rover06:09
*** ykarel|away is now known as ykarel06:17
*** ccamacho has quit IRC06:26
*** Guest6 has joined #oooq06:27
*** Guest6 has quit IRC06:28
*** jmasud has joined #oooq06:29
*** skramaja has joined #oooq06:30
*** jaosorior has joined #oooq06:47
*** jfrancoa has joined #oooq06:48
*** marios has joined #oooq07:05
*** ykarel is now known as ykarel|lunch07:23
*** holser has joined #oooq07:35
*** surpatil has quit IRC07:45
*** surpatil has joined #oooq07:49
*** ccamacho has joined #oooq07:50
*** chem_ has joined #oooq07:51
*** jbadiapa has joined #oooq07:55
*** tesseract has joined #oooq07:56
*** Tengu has quit IRC07:59
*** gouthamr has quit IRC07:59
*** owalsh has quit IRC07:59
*** chem has quit IRC07:59
*** dtantsur|afk has quit IRC07:59
*** ssbarnea has quit IRC08:01
*** dtantsur|afk has joined #oooq08:02
*** irclogbot_1 has quit IRC08:04
*** irclogbot_3 has joined #oooq08:04
*** jpena|off is now known as jpena08:05
*** tosky has joined #oooq08:13
*** ysandeep has quit IRC08:13
*** amoralej|off is now known as amoralej08:30
*** udesale_ has joined #oooq08:31
*** udesale_ has quit IRC08:33
*** udesale_ has joined #oooq08:33
*** udesale has quit IRC08:34
*** ykarel|lunch is now known as ykarel08:34
*** owalsh has joined #oooq08:45
*** Tengu has joined #oooq08:45
*** gouthamr has joined #oooq08:45
*** d0ugal has quit IRC08:46
*** ysandeep has joined #oooq08:46
*** d0ugal has joined #oooq08:46
*** openstackstatus has quit IRC08:48
*** d0ugal has joined #oooq08:48
*** d0ugal has quit IRC08:49
*** d0ugal has joined #oooq08:53
*** udesale_ has quit IRC08:54
*** d0ugal has quit IRC08:57
*** d0ugal has joined #oooq08:59
*** udesale has joined #oooq09:02
*** saneax has joined #oooq09:04
*** jtomasek has joined #oooq09:04
mariosback in a bit09:16
*** marios has quit IRC09:16
*** bogdando has joined #oooq09:17
*** dtantsur|afk is now known as dtantsur09:36
zbro/09:43
*** ratailor has quit IRC09:44
chkumar|rover\o/09:49
*** ratailor has joined #oooq09:51
*** apetrich has joined #oooq09:57
*** derekh has joined #oooq09:58
*** marios has joined #oooq09:59
*** ysandeep has quit IRC10:04
zbrchore: https://review.rdoproject.org/r/#/c/24639/10:04
zbrmarios: i was wondering if it would ok to add +W from start on fixed that need to go in ASAP, without putting the reviews. AFAIK, zuul will start the gate only when all requirements are met.10:06
zbrit would like saying: merge it as soon as it gets reviews and passed jobs.10:06
zbrmore related to to rdo10:06
*** jaosorior has quit IRC10:08
*** marios has quit IRC10:08
*** marios has joined #oooq10:13
*** sshnaidm|afk is now known as sshnaidm10:16
*** holser has quit IRC10:17
zbri think i am going to write a bot to remove myself from CC on DNM reviews10:21
zbrmarios: can you rebase https://review.rdoproject.org/r/#/c/23212/ ?10:25
*** surpatil has quit IRC10:31
zbrarxcruz: can i drop https://review.rdoproject.org/r/#/c/24340/ ?10:32
arxcruzzbr: yes, why not10:33
*** ykarel is now known as ykarel|mtg10:35
*** holser has joined #oooq10:37
*** marios has quit IRC10:54
*** udesale has quit IRC10:56
*** holser has quit IRC10:57
*** holser has joined #oooq10:58
*** marios has joined #oooq11:00
marioszbr: o/ sorry connection issues gimme sec11:00
zbrmarios: sure, i feel your pain, got some issues yesterday.11:00
marioszbr: done thanks for checking11:08
mariosthats an old review and forgotten about11:09
zbri did bit of cleanup this morning, inspired by sshnaidm CR on tci11:09
zbrprobably we need a bot11:09
sshnaidmyou can vote for this patch, btw11:10
zbrsshnaidm: i wanted to +2 right away but i was on mobile, gerrit UI is impossible to use on mobile.11:10
sshnaidmgerrti UI is impossible to use, you can stop here11:11
zbrnewer versions are much better, but we are lacking behind a lot.11:11
*** jbadiapa has quit IRC11:12
zbrpaying the cost of lacking behing current stable.11:12
zbrthere is only one case where lacking worth doing: if you know that you are going to drop that tech-dep soon.11:12
zbrwhich is not the case, gerrit, ansible, python,... not going away.11:13
sshnaidmzbr, marios, the first promoter: https://review.opendev.org/#/c/703789/2/scripts/mirror-server/promote.sh11:21
sshnaidmso nostalgic11:21
marios:D11:21
zbri am not11:21
mariossshnaidm: looks much nicer than the current one11:21
marios;)11:22
mariossshnaidm: arxcruz: chkumar|rover: can you please check comments @ https://review.rdoproject.org/r/#/c/24612/  when you next have some time for that sort of stuff thanks11:33
mariosalso chkumar|rover any idea about that fs20 and fs1 there they have never passed. have they both really been broken for so long? https://review.rdoproject.org/r/#/c/24339/11:34
marioswhat on earth11:34
arxcruzmarios: done11:35
zbrsshnaidm: ok to wf your removal?11:35
marios;)https://review.rdoproject.org/zuul/builds?job_name=periodic-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-master hm some green there chkumar|rover :/ not sure what's going on. it only changes tempest test black/white list nothing else11:35
sshnaidmzbr, not yet, let DF people to look at it11:35
mariosarxcruz: i see thanks for clarifying11:36
zbri need https://review.rdoproject.org/r/#/c/24639/ now11:39
sshnaidmzbr, 7200 to wait for molecule job?? please no11:40
zbrin same cases is needed.11:40
sshnaidmzbr, you can split the job and not to run all scenarios on every patch11:41
sshnaidmzbr, which cases11:41
zbrwe already did this on ansible zuul instance, spliting can be considered but should not be enforced. in fact 30min may not be enough for a single scenario.11:42
zbrat this moment 2000 would be ok, if you want to be more conservative.11:42
sshnaidmzbr, then it's bad scenario11:42
chkumar|rovermarios, checking11:45
*** ykarel|mtg is now known as ykarel11:48
*** holser has quit IRC11:49
zbrok to stick to 3600?11:49
zbrshould be a good compromise, i think11:49
marioschkumar|rover: wdyt about that please when you get a chance https://review.opendev.org/#/c/701016/7/config/general_config/featureset020.yml@21111:51
zbrsshnaidm: i need your blessing on https://github.com/ansible/molecule/pull/248611:55
sshnaidmzbr, cool11:58
sshnaidmzbr, where will it run?11:58
zbrzao11:58
sshnaidmansible-zuul, I see11:59
zbrsshnaidm: is already active to gate!11:59
sshnaidmgreat11:59
zbri will ping gundalow to transfer the repository to ansible-community11:59
zbrnot very happy about that but... i cannot fight it11:59
*** jpena is now known as jpena|lunch12:00
sshnaidmzbr, podman is passed as I see12:00
sshnaidmskipped I mean12:00
zbrto my surprise yes, but there are few skips. we need to be careful with these.12:00
sshnaidmzbr, you mean moving ansible/molecule there?12:00
weshay|ruck0/12:01
zbrhttps://github.com/ansible/molecule/pull/248612:01
zbrsshnaidm: to https://github.com/ansible-community/ -- drivers are already there12:01
zbrcollections will be there12:01
*** rf0lc0 has joined #oooq12:01
sshnaidmzbr, yeah.. should it be a problem?12:01
weshay|ruckchkumar|rover, hola12:02
chkumar|roverweshay|ruck, hello12:02
sshnaidmzbr, as I see all podman tests skipped, maybe need to install podman for them12:02
zbrless official, longer url, but more flexibility12:02
sshnaidmzbr, less official sounds as an advantage in this case12:02
zbrprobably12:02
*** jaosorior has joined #oooq12:14
*** ysandeep has joined #oooq12:18
weshay|ruckchkumar|rover, https://trello.com/c/l3oBsoiI/1302-cixbz1785692ospphase1osp16image-prepare-fails-on-registryredhatio-openshift4-ose-grafana12:23
*** jaosorior has quit IRC12:24
*** jaosorior has joined #oooq12:25
rf0lc0weshay|ruck, chkumar|rover pre-planning?12:31
weshay|ruckrf0lc0, https://meet.google.com/jfx-pwrz-meb?authuser=112:32
rf0lc0I was at the one in the invite12:32
rf0lc0https://meet.google.com/ena-buhw-spy12:32
rf0lc0ok12:32
*** ssbarnea has joined #oooq12:37
*** pojadhav is now known as pojadhav|away12:40
kopecmartinzbr: any chance you can point me to an example on how i can define variables for all playbooks in molecule? i tried to just copy the vars from playbook to molecule.yml, but that didn't work for some reason :/12:56
zbrkopecmartin: yeah, I will bit later. I was waiting for that question, is not really obvious.12:57
kopecmartinzbr: yeah, it's not, i couldn't find anything in the doc12:58
*** ykarel is now known as ykarel|afk12:58
rf0lc0planning meeting agenda https://hackmd.io/IhMCTNMBSF6xtqiEd9Z0Kw?both13:00
rf0lc0https://meet.google.com/tbr-rgjz-cjc?authuser=113:01
chkumar|roverbrb in 15 mins13:01
*** jpena|lunch is now known as jpena13:16
zbrkopecmartin: search for group_vars on https://molecule.readthedocs.io/en/stable/configuration.html13:16
zbryou will understand, it will be like passing them from the cli with -e13:17
zbrput them in "all" group13:17
*** skramaja has quit IRC13:19
*** ysandeep has quit IRC13:22
*** udesale has joined #oooq13:23
chkumar|rovermigi, rlandy I am not seeing any difference in these two files http://pkgs.devel.redhat.com/cgit/containers/openstack-nova-api/tree/extend_start.sh?h=rhos-17.0-rhel-8 and https://github.com/openstack/kolla/blob/master/docker/nova/nova-api/extend_start.sh13:27
chkumar|roverwe have so many repos why?13:27
zbrsshnaidm: what do you want to do about sova on centos-7? the python-yaml package comes from epel-release on that platform.13:27
zbrthat is why i missed to see it, as i got epel-release enabled.13:28
sshnaidmzbr, what do you mean? Our jobs run on centos7 now13:28
zbris related to removal of pip install from yaml13:28
migichkumar|rover: the diff is here: https://github.com/openstack/kolla/tree/master/docker/nova/nova-api vs https://github.com/openstack/kolla/tree/master/docker/nova/nova-api13:29
migidamn, wrong13:29
migichkumar|rover: http://pkgs.devel.redhat.com/cgit/containers/openstack-nova-api/tree/?h=rhos-17.0-rhel-813:29
zbrsshnaidm: https://review.opendev.org/#/c/703781/10/molecule/sova/prepare.yml13:29
zbrthis needs to go away. we have two options13:30
sshnaidmzbr, what's the problem? It worked fine so far13:30
zbrit worked because it was not tested correctly13:30
zbrin fact that line was put on molecule when it was supposed to be part of the role13:31
zbris like cheating the tests: building an imaginary setup to make them pass13:31
sshnaidmzbr, no, it shouldn't be part of the role in any case13:31
zbrin reality, on centos-7, you will not find pyyaml.13:31
sshnaidmzbr, and it was testing everything13:31
sshnaidmzbr, please check logs from previous runs13:32
zbra role must be self-contained. if it needs something, it needs to either install it itself or call roles that do this.13:32
zbrthat is not something i am willing to debate much about it13:32
sshnaidmzbr, strongly disagree on that13:33
zbrthere are workarounds, we only need to decide which approach to take13:33
sshnaidmdon't mix requirements and role itself13:34
sshnaidmzbr, I'd like to understand what problem you have with this scenario before any change you do for it13:35
zbrsshnaidm: please, show me these requiremetns, PyYAML is not mentioned anywhere in the README of that role.13:35
chkumar|roverweshay|ruck, https://opendev.org/openstack/tripleo-ci/src/branch/master/roles/tripleo-repos/tasks/main.yml#L3213:36
sshnaidmzbr, why should reqs be in readme? In which else role do you see it?13:36
zbrin fact the only place where is used is inside molecule/sova/playbook.yml which confirms my theory: hiding a requirement to make it run.13:36
sshnaidmzbr, pyyaml is installed with ansible in other places13:37
sshnaidmmolecule doesn't have it, that's why this workaround13:37
sshnaidmzbr, for better test install ansible instead of pyyaml, if you like it more13:37
kopecmartinzbr: thanks, it works now13:39
zbrsshnaidm: this sova task make me waste almost the entire workiing day yesterday, it needs to be fixed somehow.13:39
*** marios is now known as marios|call13:39
sshnaidmzbr, your slow internet shouldn't be a reason for code changes :)13:40
sshnaidmzbr, host a local pypi mirror13:40
zbrtbh, AFAIK we even had a policy regarding not installing stuff from pip. and is not much about this,  I already explained this several times yesterday: pypyaml is not a pure python wheel, it requires gcc compilation, in fact is a big project. take a lot of time to build, each time you run.13:42
zbrnot somethign you want to run as part of functional testing on n platforms13:42
zbrmainly that single task did double the total molecule scenario runtime13:42
zbrit it was about installing six module, it did not care so much. is like one second.13:43
*** ratailor has quit IRC13:43
zbrsadly pyyaml is a very reasource intensive installation13:43
sshnaidmzbr, I don't care much about molecule test time in CI because it's anyway less than any tripleo job13:43
zbrunrelated to traffic13:43
sshnaidmzbr, then install it by package, or install ansible and it will take it as deps (I hope)13:44
zbrsshnaidm: you do have to care,  I already had to propose timeout bumping today, because is tox based, its default is 1800, which 30min.13:44
sshnaidmzbr, you proposed timeout on rdo-jobs, it doesn't run tripleo jobs13:45
sshnaidmzbr, we're talking about upstream CI13:45
zbrin fact the timeout is the same, 1800 comes from upstream if I remember well.13:45
sshnaidmzbr, let's start from the beginning, do you think it shouldn't be tested on centos7 while we run 95% of our job on centos7?13:46
zbrsshnaidm: i do want to test it not only on centos-7, but also on 8 and ubuntu13:47
*** jaosorior has quit IRC13:47
sshnaidmzbr, I've asked explicitly about centos713:47
chkumar|roverweshay|ruck, wes sorry I rechecked this https://code.engineering.redhat.com/gerrit/#/c/189436/ I am not sure your env will vanished13:48
zbrlet me rephrase: i want the testing job to run without hacks on clean os.13:48
zbrthat pyyaml installation should not be in playbook.yaml13:49
chkumar|rovermarios|call, regarding fs01, few got failed due to infra issue13:50
zbrmainly that is an undocumented requirement13:50
marios|callchkumar|rover: it hasn't passed once on that review :(13:50
marios|callchkumar|rover: same for fs2013:50
marios|callchkumar|rover: fs10 is pretty good but always very close to just above 3 hours13:50
marios|callchkumar|rover: which is why we need that timeout bump13:50
marios|callchkumar|rover: i added your suggestions at https://review.opendev.org/#/c/701016/9/config/general_config/featureset010.yml@103  check it when you have time thanks13:51
marios|callchkumar|rover: but still TBD if we want fs1 or 10 mainly blocked on them passing at least once !?!13:51
chkumar|roverah I commented back13:51
chkumar|roverchecking again13:52
sshnaidmzbr, we don't document requirements in oooq, and it's not a reason not to install it at all13:52
sshnaidmzbr, I don't care about way you install pyyaml, but it must be there before sova task13:53
sshnaidmzbr, I think it's pretty obvious13:53
*** ykarel|afk is now known as ykarel13:55
zbrsshnaidm: we are talking here about ansible-role-collect-logs upstream role, which makes the stuff above even more weird13:56
marios|callpanda: sent call for tomorrow sync please check if the time is ok for you cc ua chkumar|rover fyi13:59
marios|callrf0lc0: rlandy: weshay|ruck: sent call for tomorrow container sync please nack if we need to reschedule13:59
rf0lc0marios|call, ack14:00
rlandymarios|call: wfm14:00
rlandythanks14:00
*** jbadiapa has joined #oooq14:01
*** jbadiapa has quit IRC14:01
*** holser has joined #oooq14:06
*** jpena is now known as jpena|off14:12
*** holser has quit IRC14:12
*** dsneddon has quit IRC14:16
sshnaidmmarios|call, can you add me to this review?14:18
marios|callsshnaidm: which one? fs20? there https://tree.taiga.io/project/tripleo-ci-board/task/1383 please thanks14:20
*** jpena|off is now known as jpena14:20
sshnaidmmarios|call, I mean more adding reviewers to the patch itself, it make them more visible in dashboard14:21
*** holser has joined #oooq14:21
*** holser has quit IRC14:22
marios|callsshnaidm: added you https://review.opendev.org/#/c/701016/ ack right i usually do but have been asking here for now to tempest folks14:22
marios|callgood idea will add more thanks sshnaidm14:22
*** holser has joined #oooq14:22
arxcruzsshnaidm: sorry, didn't notice my mic was on :)14:24
sshnaidmarxcruz, np, meet.g allows it :)14:24
sshnaidmunlike bj14:25
arxcruzsshnaidm: ;)14:25
marios|callrf0lc0: we can't hear you14:33
chkumar|roverweshay|ruck, arxcruz https://bugs.launchpad.net/tripleo/+bug/1860673 please have a look when free,14:34
openstackLaunchpad bug 1860673 in tripleo "test_cross_tenant_traffic tempest tests failed with Timed out waiting for 10.0.0.112 to become reachable from 10.0.0.118 in fs020 master" [Critical,Triaged]14:34
chkumar|roverbrb for 1514:34
*** saneax has quit IRC14:36
arxcruzchkumar|rover: seems to be network issues14:38
arxcruzprobably some firewall rule14:39
*** marios|call is now known as marios14:45
weshay|ruckchkumar|rover, thanks14:47
weshay|ruckchkumar|rover, can you put up a testproject change in internal sf on the two patches from Alex14:47
weshay|ruckfor osp-1614:47
*** dsneddon has joined #oooq14:50
chkumar|roverrlandy, want to start the podman call right now?14:51
chkumar|roverif free?14:51
rlandychkumar|rover: ack and I will help you with weshay|ruck request ^^14:52
rlandyall related14:52
chkumar|roverweshay|ruck, fs039 job is broken from long time do we have bug open?14:52
rlandychkumar|rover: joined meeting14:53
chkumar|roverrlandy, https://code.engineering.redhat.com/gerrit/#/c/189436/14:55
sshnaidmmarios, does 010 have something 001 doesn't have in api tests?14:56
rlandychkumar|rover: http://git.app.eng.bos.redhat.com/git/ansible-role-tripleo-modify-image.git/15:00
mariossshnaidm: not sure but other way around fs1 is tls i believe so we might prefer it. having said that, it has been very unstable for me in the test fs1 and fs2015:02
mariossshnaidm: but fs10 is pretty green few runs now15:02
sshnaidmmarios, worth to compare list of running tests in both15:03
sshnaidmmarios, I think we have 1ctrl_1comp with fs 001 job, which might be closer to fs02015:04
weshay|ruckchkumar|rover, no.. leave fs03915:05
weshay|ruckchkumar|rover, I need a testproject job on osp-16 that includes https://review.opendev.org/#/q/topic:bug/1860585+(status:open+OR+status:merged)15:05
mariossshnaidm: thanks ack please comment on gerrit and i will revise it with comments thanks15:06
sshnaidmmarios, fyi https://github.com/rdo-infra/rdo-jobs/blob/master/zuul.d/ovb-jobs.yaml#L1515:06
sshnaidmmarios, ack, np15:07
chkumar|roverrlandy, https://hackmd.io/cThSvUCiRzeHvi8ekq6zWw#Proposal15:08
*** holser has quit IRC15:08
weshay|ruckchkumar|rover, please pleaes :)15:08
weshay|ruckchkumar|rover, https://review.opendev.org/#/q/topic:bug/1860585+(status:open+OR+status:merged)15:08
*** holser has joined #oooq15:09
chkumar|roverrlandy, https://github.com/rdo-infra/review.rdoproject.org-config/blob/master/zuul/rdo.yaml#L108415:09
chkumar|roverrlandy, https://tree.taiga.io/project/tripleo-ci-board/us/132215:10
rlandychkumar|rover: https://code.engineering.redhat.com/gerrit/#/c/189440/7/.zuul.yaml15:10
chkumar|roverrlandy, https://tree.taiga.io/project/tripleo-ci-board/task/134015:11
*** apetrich has quit IRC15:11
chkumar|roverrlandy, https://review.rdoproject.org/r/#/c/23249/4/zuul.d/projects.yaml15:13
*** dsneddon has quit IRC15:15
rlandychkumar|rover: http://git.app.eng.bos.redhat.com/git/openstack/sf-config.git/tree/resources/tripleo-ci-internal.yaml15:15
*** TrevorV has joined #oooq15:17
weshay|ruckrlandy, can you wf? https://code.engineering.redhat.com/gerrit/#/c/190197/115:17
*** Goneri has joined #oooq15:18
rlandyweshay|ruck: no15:18
rlandywe need to fix permissions on that15:18
rlandygo in gerrot15:18
rlandysec15:18
chkumar|roverrlandy, https://review.rdoproject.org/r/2324915:19
chkumar|roverrlandy, https://review.rdoproject.org/r/#/c/23246/15:20
rlandyweshay|ruck: will fix that for you in a bit - juts on call with chkumar|rover15:21
chkumar|roverrlandy, https://code.engineering.redhat.com/gerrit/19019815:21
weshay|ruckrlandy, k15:22
*** holser has quit IRC15:24
*** holser has joined #oooq15:25
rlandyhttp://git.app.eng.bos.redhat.com/git/ansible-role-tripleo-modify-image.git/commit/?h=rhos-16.0-trunk-patches&id=7c0f7fff0a17a6ab65a9d48fffbc9447362c762715:25
rlandychkumar|rover: ^^15:25
weshay|ruckchkumar|rover, was given ack to disable encyrption in fs027.. /me works on master promotion15:27
weshay|ruckchkumar|rover, fyi https://review.opendev.org/70400815:32
chkumar|roverrlandy, git clone ssh://chkumar@code.engineering.redhat.com/ansible-role-tripleo-modify-image15:37
*** ksambor has quit IRC15:40
chkumar|roverweshay|ruck, do you have test review up?15:44
chkumar|roverdoign right now15:44
chkumar|roverweshay|ruck, https://review.rdoproject.org/r/#/c/24638/15:45
*** dsneddon has joined #oooq15:45
rlandyweshay|ruck: k - fixing the permissions ...15:46
weshay|ruckchkumar|rover, thank you :)15:48
chkumar|roverweshay|ruck, removed promotion blocker from the bug15:48
chkumar|roverweshay|ruck, https://bugs.launchpad.net/tripleo/+bug/186067315:49
openstackLaunchpad bug 1860673 in tripleo "test_cross_tenant_traffic tempest tests failed with Timed out waiting for 10.0.0.112 to become reachable from 10.0.0.118 in fs020 master" [Critical,Triaged]15:49
chkumar|roversee ya tomorrow weshay|ruck15:49
*** chkumar|rover is now known as raukadah15:49
*** Trevor_V has joined #oooq15:53
*** TrevorV has quit IRC15:57
*** jbadiapa has joined #oooq15:57
*** jpena is now known as jpena|off16:06
weshay|ruckmarios, rlandy muther may I? https://review.rdoproject.org/r/#/c/24624/16:15
rlandyweshay|ruck: checking jenkins config16:16
weshay|ruckrlandy, /me watching https://ci.centos.org/view/rdo/view/promotion-pipeline/job/rdo_trunk-promote-queens-current-tripleo/333/console16:17
weshay|ruckhttps://ci.centos.org/job/tripleo-quickstart-promote-queens-rdo_trunk-minimal/290/console16:17
weshay|ruckrather16:17
rlandyweshay|ruck: https://github.com/rdo-infra/ci-config/blob/master/jenkins/jobs/tripleo-upstream-builders.yml#L14116:18
weshay|ruckrlandy, ah. https://review.rdoproject.org/r/#/c/24635/16:19
weshay|ruckI thought it merged already16:19
weshay|ruckfew more min16:19
rlandyweshay|ruck: k - now you can16:20
weshay|ruckthanks16:22
*** ccamacho has quit IRC16:26
*** tosky has quit IRC16:28
weshay|ruckrf0lc0, question for you16:31
rf0lc0weshay|ruck, all ears16:31
weshay|ruckrf0lc0, https://github.com/rdo-infra/review.rdoproject.org-config/blob/master/playbooks/tripleo-ci-periodic-base/containers-build.yaml#L2016:32
weshay|ruckis the token needed there? I guess it probably is.. because kolla builds and pushes to rdo?16:32
rf0lc0yes16:33
rf0lc0when push=true16:33
weshay|ruckrf0lc0, k.. now for an opinion16:33
weshay|ruckI've been updated other areas of ci.. so that we deliniate between ci_config_repo and rdo-infra and config16:34
weshay|ruckby using two vars16:34
weshay|ruckci_config_repo and ci_config_repo_protected16:34
weshay|ruckwould that help or be annoying to carry on and make it consistent16:34
weshay|rucke.g. https://review.rdoproject.org/r/#/c/24635/16:34
*** marios is now known as marios|out16:35
rf0lc0ci-config naming is confusing enough16:37
weshay|ruckaye.. because it can point to two diff repos16:37
rlandyweshay|ruck: raukadah: https://code.engineering.redhat.com/gerrit/#/c/190206/116:37
weshay|ruckwe inherit that variable.. in some playbooks so you wouldn't know which one ur using16:37
weshay|ruckis my concern16:37
weshay|ruckrlandy, ah thanks16:38
rlandyweshay|ruck: while we fangle the testproject permissions16:38
weshay|ruckI was trying to submit one. .but could not16:38
weshay|ruckk16:38
rlandyweshay|ruck: see #rhos-delivery16:38
weshay|ruckrlandy, so I wonder what happens if I do a depends-on w/ an upstream change16:39
weshay|ruckwill it apply?16:39
rlandyupdated .gitreview16:39
raukadahrlandy, how you fixed the permission?16:39
rlandyweshay|ruck: it should16:39
weshay|ruckrlandy, ok.. will try16:39
rlandyraukadah: jjoyce did it16:39
rlandyonly alex could commit before16:39
raukadahrlandy, does it got fixed for all repo?16:39
rlandyraukadah: ack16:40
rlandyraukadah: pull again16:40
*** tesseract has quit IRC16:41
weshay|ruckrf0lc0, wdyt?16:41
rlandyweshay|ruck: no, not like that16:42
weshay|ruckmy update?16:42
rlandyweshay|ruck: upstream review will not take downstream if they are not branchless16:42
weshay|ruckoh16:42
rlandythis review is on the rhos-16 patches branch16:42
weshay|ruckaye16:42
rlandyso we pick up alex's changes in downstream16:42
rlandygetting16:43
weshay|ruckonly see ur patch https://code.engineering.redhat.com/gerrit/#/q/project:ansible-role-tripleo-modify-image+branch:rhos-16.0-trunk-patches16:43
rlandyweshay|ruck: http://git.app.eng.bos.redhat.com/git/ansible-role-tripleo-modify-image.git/log/?h=rhos-16.0-trunk-patches16:43
rf0lc0weshay|ruck, I am not what you asking... ci_config_repo should be the non-protected IMHO. just config_repo_trusted would be good naming convention.16:43
rlandymy patch?16:43
rlandyForce docker formatAlex Schultz21 hours3-1/+616:44
* weshay|ruck see's https://code.engineering.redhat.com/gerrit/#/c/189911/16:44
rlandyalex's one is merged on the branch16:44
weshay|ruckrlandy, https://code.engineering.redhat.com/gerrit/#/q/owner:%22Alex+Schultz%2216:44
weshay|ruckok16:44
rlandyI'm confused16:44
rlandyweshay|ruck: ok now?16:45
weshay|ruckso.. if his patch is on the branch https://code.engineering.redhat.com/gerrit/#/c/190206/16:45
weshay|ruckthat should be enough16:45
raukadahrlandy, depends on is not going to work16:45
rlandyraukadah: I know16:46
raukadahas build-test-packages does not support internal stuff16:46
raukadahwhich we need to work on very soon16:46
rlandyI don;t think  the depends on will take16:47
weshay|ruckrlandy,  meh..  ok16:47
rlandyweshay|ruck: only because it's in the branch already16:48
raukadahmay be need to hack into opsinfo file16:48
rlandyno biggy16:48
rlandyshould be fine16:48
rlandyok ... getting back to adding podman change jobs16:48
*** dsneddon has quit IRC16:48
weshay|ruckzbr, left some comments on compression change16:54
*** ykarel is now known as ykarel|away16:56
*** derekh has quit IRC16:57
*** marios|out has quit IRC16:59
*** jbadiapa has quit IRC17:00
*** udesale has quit IRC17:05
*** bogdando has quit IRC17:12
raukadahsshnaidm, rlandy https://review.opendev.org/#/c/704008/ it works tested, time to merge it and get master promotion17:20
rlandylooking17:20
weshay|ruckraukadah, k k.. /me looking17:21
weshay|ruckraukadah, get out of here.. let's sync tomorrow17:21
raukadahok ok17:21
rf0lc0kick raukadah17:22
weshay|ruckraukadah, this has to merge.. then I'll remove 27 from criteria17:23
rlandyraukadah: https://code.engineering.redhat.com/gerrit/#/c/190212/17:23
*** dtantsur is now known as dtantsur|afk17:24
*** dsneddon has joined #oooq17:24
zbrweshay|ruck: ok to rename var to artcl_gzip instead of artcl_gzip_only which i find confusing17:25
weshay|ruckzbr, agree..17:25
weshay|ruckartcl_gzip makes more sense17:26
weshay|ruckthanks17:26
*** jpena|off is now known as jpena17:27
*** dsneddon has quit IRC17:29
weshay|ruckhrm.. I think I can drop this https://opendev.org/openstack/tripleo-quickstart/src/branch/master/roles/libvirt/setup/undercloud/tasks/main.yml#L362-L36917:32
rlandyhmmm...no show on https://code.engineering.redhat.com/gerrit/#/c/190206/ - although it has the right clone17:42
*** pojadhav|away is now known as pojadhav17:52
zbrweshay|ruck: https://review.rdoproject.org/r/#/c/24639/17:56
weshay|ruckzbr++17:57
*** derekh has joined #oooq18:02
*** dsneddon has joined #oooq18:03
*** jaosorior has joined #oooq18:03
*** dsneddon has quit IRC18:11
*** amoralej is now known as amoralej|off18:14
weshay|rucksshnaidm, fyi https://review.rdoproject.org/r/2465218:15
*** dsneddon has joined #oooq18:16
sshnaidmweshay|ruck, is it needed?18:17
weshay|rucksshnaidm, will help keep things where they should be18:18
*** dsneddon has quit IRC18:21
*** derekh has quit IRC18:32
*** holser has quit IRC18:34
*** dsneddon has joined #oooq18:36
*** jmasud has quit IRC18:41
*** fmount has quit IRC18:45
*** jpena is now known as jpena|off18:48
*** jaosorior has quit IRC18:54
*** fmount has joined #oooq18:56
*** jtomasek has quit IRC19:07
rlandyweshay|ruck: tbh ... testing of these rhos-branch patches isn't working :)19:39
rlandyhttps://code.engineering.redhat.com/gerrit/#/c/190206/ should have picked up the change afaict19:40
rlandy2020-01-23 13:12:03 | (9/335): ansible-role-tripleo-modify-image-1.1. 282 kB/s |  26 kB     00:0019:41
*** openstackstatus has joined #oooq20:07
*** ChanServ sets mode: +v openstackstatus20:07
*** sshnaidm is now known as sshnaidm|afk20:09
*** jaosorior has joined #oooq20:10
*** dsneddon has quit IRC20:19
*** jmasud has joined #oooq20:22
weshay|ruckrlandy,  my first infrared review lolz https://github.com/redhat-openstack/infrared/issues/38220:26
weshay|ruckoops20:26
weshay|ruckhttps://github.com/redhat-openstack/infrared/issues/38220:26
weshay|ruckrlandy, you proud of me?20:26
weshay|rucklolz20:26
rlandyweshay|ruck: wow ... that quick20:27
*** jfrancoa has quit IRC20:43
*** jmasud has quit IRC20:51
*** dsneddon has joined #oooq20:54
rlandyweshay|ruck: have a few minutes to chat about podman jobs triggering downstream?20:58
*** dsneddon has quit IRC21:00
weshay|ruckrlandy, ya.. need another go at infrared.. they are using python2 on rhel821:00
weshay|ruck:(((21:00
rlandyno rush21:01
rlandyweshay|ruck: another matter ...21:04
rlandy2020-01-23 13:16:14.428527 | primary | TASK [setup/undercloud : Start undercloud vm] **********************************21:04
rlandy2020-01-23 13:16:14.459260 | primary | Thursday 23 January 2020  13:16:14 +0000 (0:00:00.094)       1:00:02.726 ******21:04
rlandy2020-01-23 13:16:14.482640 | primary | fatal: [rdo-ci-fx2-06-s4.v103.rdoci.lab.eng.rdu2.redhat.com]: FAILED! => {}21:04
rlandy2020-01-23 13:16:14.482693 | primary |21:04
rlandy2020-01-23 13:16:14.482711 | primary | MSG:21:04
rlandy2020-01-23 13:16:14.482722 | primary |21:04
rlandy2020-01-23 13:16:14.483015 | primary | The conditional check 'ansible_version.full|version_compare('2.3','<')' failed. The error was: template error while templating string: no filter named 'version_compare'. String: {% if ansible_version.full|version_compare('2.3','<') %} True {% else %} False {% endif %}21:04
rlandynot hitting ci.centos?21:05
rlandyhttps://github.com/geerlingguy/ansible-role-varnish/issues/9021:05
zbrweshay|ruck: leaving now, maybe you can have a look at and tell me why the role did not run and leave a not, https://review.opendev.org/#/c/704020/21:08
zbron tox/molecule it seems to be working.21:08
weshay|ruckzbr, I'm working on fucking infrared21:09
weshay|ruckwhy do they install the vbmc on the virthost21:09
zbrhaha, what can I say, enjoy that homebrew stuff21:09
weshay|ruckBAHH21:09
rlandyhttps://ci.centos.org/job/tripleo-quickstart-promote-stein-current-tripleo-delorean-minimal/276/consoleFull21:16
rlandyraukadah: weshay|ruck: https://review.opendev.org/#/c/704066/1 - should also fix broken ci.centos21:18
*** Trevor_V has quit IRC21:18
*** irclogbot_3 has quit IRC21:18
*** yolanda has quit IRC21:18
*** panda has quit IRC21:18
*** jpena|off has quit IRC21:19
rlandyzbr: suffering PTSD from your former job?21:20
*** openstackstatus has quit IRC21:20
*** irclogbot_1 has joined #oooq21:25
*** jaosorior has quit IRC21:25
*** dsneddon has joined #oooq21:36
weshay|ruckadd the virtbmc role as the next role we're shareing21:36
*** dsneddon has quit IRC21:41
*** jmasud has joined #oooq21:45
*** jmasud has quit IRC21:57
*** yolanda has joined #oooq22:01
*** panda has joined #oooq22:01
*** jpena|off has joined #oooq22:01
*** jmasud has joined #oooq22:03
*** dsneddon has joined #oooq22:08
*** jmasud has quit IRC22:10
*** jmasud has joined #oooq22:10
*** jmasud has quit IRC22:10
*** dsneddon has quit IRC22:15
*** tosky has joined #oooq22:16
*** jmasud has joined #oooq22:29
*** Goneri has quit IRC22:37
*** dsneddon has joined #oooq22:43
*** jmasud has quit IRC22:45
*** dsneddon has quit IRC22:47
*** aakarsh has joined #oooq23:03
*** jmasud has joined #oooq23:06
*** aakarsh has quit IRC23:10
*** dsneddon has joined #oooq23:11
*** dsneddon has quit IRC23:16
*** jmasud has quit IRC23:20
*** jmasud has joined #oooq23:24
*** jmasud has quit IRC23:26
*** ysandeep has joined #oooq23:31
*** apetrich has joined #oooq23:40
*** dsneddon has joined #oooq23:51
*** jmasud has joined #oooq23:56
weshay|ruckrlandy, I have a fix for that23:56
weshay|ruckand a bug23:56
*** dsneddon has quit IRC23:56
weshay|ruckrlandy, raukadah https://review.opendev.org/#/c/704046/23:56

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