Monday, 2019-05-06

*** d0ugal has quit IRC00:06
*** altlogbot_3 has quit IRC02:41
*** altlogbot_2 has joined #oooq02:42
*** ykarel has joined #oooq03:15
*** altlogbot_2 has quit IRC03:31
*** altlogbot_3 has joined #oooq03:31
*** gkadam has joined #oooq03:42
*** gkadam has quit IRC03:42
*** udesale has joined #oooq03:53
*** raukadah has quit IRC04:00
*** chandankumar has joined #oooq04:01
*** saneax has joined #oooq04:09
*** ykarel is now known as ykarel|afk04:21
*** ykarel|afk has quit IRC04:26
*** ykarel|afk has joined #oooq04:48
*** ykarel_ has joined #oooq04:52
*** ykarel|afk has quit IRC04:55
*** sanjayu_ has joined #oooq05:05
*** saneax has quit IRC05:08
*** sanjayu__ has joined #oooq05:24
*** ykarel_ is now known as ykarel05:25
*** sanjayu_ has quit IRC05:26
*** marios has joined #oooq05:38
*** ratailor has joined #oooq05:46
*** skramaja has joined #oooq06:22
*** udesale has quit IRC06:23
*** udesale has joined #oooq06:24
*** quiquell has joined #oooq06:28
*** kopecmartin|off is now known as kopecmartin06:52
*** ccamacho has joined #oooq06:57
*** holser_ has joined #oooq06:58
*** holser_ has quit IRC06:58
*** holser_ has joined #oooq06:59
*** tesseract-RH has joined #oooq07:05
*** amoralej|off is now known as amoralej07:09
*** altlogbot_3 has quit IRC07:13
*** altlogbot_3 has joined #oooq07:17
*** apetrich has joined #oooq07:22
*** tosky has joined #oooq07:28
*** arxcruz|rover is now known as arxcruz07:37
*** bogdando has joined #oooq07:42
*** dmellado has quit IRC07:44
*** dmellado has joined #oooq07:44
*** jpena|off is now known as jpena07:49
*** udesale has quit IRC07:55
*** udesale has joined #oooq07:55
pandanote it's holiday in Ireland today07:58
* marios raises guiness 07:58
quiquellreverse jet lag :-(08:00
*** ykarel is now known as ykarel|lunch08:08
*** ykarel|lunch is now known as ykarel08:50
*** altlogbot_3 has quit IRC09:11
*** altlogbot_0 has joined #oooq09:14
*** altlogbot_0 has quit IRC09:15
*** altlogbot_2 has joined #oooq09:15
*** zbr is now known as zbr|rover09:18
zbr|roverhi! i am back in business, for two days :p09:19
quiquellzbr|rover: o/09:27
quiquellzbr|rover: I have an ansible-lint issue09:27
quiquellzbr|rover: maybe you can help me09:27
zbr|roverquiquell: sure, show me.09:27
quiquellzbr|rover: send you a priv with tmate09:28
ykareltosky, kopecmartin can you followup on https://review.opendev.org/#/c/655698/, seems to be stuck on PTL vote09:29
ykarelcan you get this and move it09:29
toskyykarel: chandankumar already pinged him - the summit/PTG week probably didn't help (I think no one of us was there)09:30
ykareltosky, ack, anyway good to send reminder whenever he is available09:31
zbr|roverquiquell: interesting the lint error does not reproduce on m machine but is clearly related to the attempt to load the /var/ssh/id_rsa file during linting, and i kinda know why: lookup() is running on the host/controller and it may be run during linting09:46
quiquellzbr|rover: running ansible-lint directly works ok at my machine09:48
quiquellzbr|rover: but running with pre-commit fails, would  be good to know how pre-commit calls ansible-lint09:48
quiquellmayeb you have /var/ssh/id_rsa at your machine :-/ ?09:48
zbr|roverquiquell: now that i look at the code, i think is unreleated to that warning. i was was able to reproduce the error on one machine, but not on another.09:49
quiquellso, where is the error ?09:51
zbr|roverquiquell: i am almost sure is related to ansible version, because ansible lint has a very loose req to it https://github.com/ansible/ansible-lint/blob/master/setup.cfg#L7809:55
zbr|roveri will find which one is buggy and add a more restrictive range for it to use09:55
zbr|roverbut i am almost sure the bug is in ansible code09:55
quiquellack09:56
quiquellthanks09:56
*** udesale has quit IRC10:34
*** udesale has joined #oooq10:35
*** skramaja has quit IRC10:40
*** ccamacho has quit IRC11:04
zbr|roverquiquell: i updated the review and included the workaround related to yaml, it did pass linting now.11:07
zbr|roversee https://review.rdoproject.org/r/#/c/20548/23/.pre-commit-config.yaml11:08
quiquellzbr|rover: I see, thanks mate !11:08
zbr|roveri hope everyone reads it because we are likely to encounter it again, and finding the culprit was not so obvious11:08
zbr|rovermainly we can tell pre-commit to use additional reqs conditions in order to bypass bugs like this. it works the same way as with tox.11:09
*** udesale has quit IRC11:17
*** altlogbot_2 has quit IRC11:18
*** altlogbot_1 has joined #oooq11:20
*** jaosorior has joined #oooq11:21
*** jpena is now known as jpena|lunch11:32
*** rfolco|ruck has joined #oooq11:50
*** amoralej is now known as amoralej|lunch12:00
rfolco|ruckzbr|rover, o/12:05
*** ratailor has quit IRC12:07
zbr|roverrfolco|ruck: need to sync on something?12:09
rfolco|ruckzbr|rover, not really, unless you need to chat about anything12:09
zbr|rovernot really.12:10
ykarelzbr|rover, rfolco|ruck there are couple of issues in master, queens, just in case u are not already aware12:11
*** rlandy has joined #oooq12:11
ykareli pinged Arx in morning seeing |rover in his nick12:11
rfolco|ruckykarel, ok, looking at the promotion jobs, thank you12:11
ykarelrfolco|ruck, master fix:- https://review.opendev.org/#/c/625261/, queens Phase1 fix:- https://review.opendev.org/#/c/657334/12:13
ykarelmay be other issues are there, /me aware of these two only ^^12:13
arxcruzchandankumar: yeah, i 'm working on the heat_stack_owner12:14
chandankumararxcruz: I think we can enable in tht itself https://opendev.org/openstack/puppet-heat/src/branch/master/manifests/keystone/auth.pp#L11312:17
chandankumarwe need to chnage this value to true i think12:17
arxcruzchandankumar: problem is where? :/12:18
chandankumararxcruz: in our standalone deployment heat_stack_owner does not exists12:18
arxcruzchandankumar: or maybe in os_tempest sneusre this is created instead of assum12:18
chandankumarin osa side in os_heat they care of that12:18
chandankumarhttps://opendev.org/openstack/openstack-ansible-os_heat/src/branch/master/tasks/heat_service_setup.yml#L6212:19
arxcruzshit12:19
*** udesale has joined #oooq12:19
chandankumararxcruz: in TQE, when we deploy overcloud then we use to create that role12:19
*** altlogbot_1 has quit IRC12:19
chandankumarhttps://opendev.org/openstack/tripleo-quickstart-extras/src/branch/master/roles/overcloud-deploy/templates/overcloud-deploy-post.sh.j2#L3812:19
arxcruzi don't know how tho make this link between the options :/12:19
*** altlogbot_2 has joined #oooq12:19
chandankumarfinding the example12:19
*** udesale has quit IRC12:25
*** udesale has joined #oooq12:26
*** jpena|lunch is now known as jpena12:29
*** aakarsh has quit IRC12:35
weshayrfolco|ruck zbr|rover can you guys put up a patch on ci-config/jenkins to run check on https://trello.com/c/b92G3x3A/960-cixlp1826591tripleociproa-phase1-jobs-failing-with-undefined-variables12:39
weshaychandankumar fyi ^12:39
weshayfyi. that job is also not working yet, needs a playbook actually12:40
weshayand a version bump in ansible12:40
chandankumararxcruz: we need to add another param here https://opendev.org/openstack/tripleo-heat-templates/src/branch/master/deployment/heat/heat-api-container-puppet.yaml#L17412:41
arxcruzchandankumar: i don't think so, i think we can do it in the yaml file on tripleo-quickstart side12:42
chandankumararxcruz: heat::keystone::auth::configure_delegated_roles: {get_param: true}12:42
chandankumarsorry12:43
chandankumararxcruz: https://opendev.org/openstack/tripleo-heat-templates/src/branch/master/deployment/heat/heat-engine-container-puppet.yaml#L14612:43
zbr|roverweshay: i am now trying to find out what failed at http://logs.openstack.org/36/656636/1/check/dib-functests-bionic-python3/a0b6139/logs/fedora-minimal_build-succeeds.FAIL.log12:44
weshayk12:45
zbr|rover... not clear to me why that fails, if anyone has better eyes...12:46
arxcruzchandankumar: we need to override it on tripleo side12:46
arxcruzi mean, on tripleo-quickstart side12:46
*** zul has joined #oooq12:46
zbr|roverweshay: damn, this was non voting, no need to fix it.12:47
chandankumararxcruz: no no, we need to first replace the above with get_param12:47
chandankumararxcruz: and replace the same get_param value here https://opendev.org/openstack/tripleo-heat-templates/src/branch/master/ci/environments/scenario001-standalone.yaml#L4212:49
arxcruzchandankumar: ok, i'll work on that :)12:50
rfolco|ruckweshay, so the request is to add a playbook to ci-config/jenkins that imports the new collect-logs role ?12:51
chandankumararxcruz: https://opendev.org/openstack/tripleo-heat-templates/src/branch/master/deployment/heat/heat-engine-container-puppet.yaml#L5712:51
chandankumargrep for HeatWorkers12:52
chandankumarit will be a good example :-)12:52
weshayrfolco|ruck I'll explain now if you want /me joins blue early for the scrum12:54
rfolco|ruckok12:54
rfolco|ruckweshay, joined https://bluejeans.com/587845809712:55
chandankumarweshay: ack!12:56
*** aakarsh has joined #oooq12:59
weshayrfolco|ruck https://review.opendev.org/#/c/655938/13:00
chandankumararxcruz: kopecmartin rlandy marios quiquell weshay scrum time13:01
chandankumarpanda: ^^13:02
*** amoralej|lunch is now known as amoralej13:02
quiquellconnecting13:03
chandankumarweshay: zbr|rover we can migrate our taiga stuff from here to https://teams.fedoraproject.org/discover13:05
*** altlogbot_2 has quit IRC13:15
*** altlogbot_3 has joined #oooq13:16
weshaypanda you around?13:19
weshayfyi https://etherpad.openstack.org/p/tripleo-ptg-train13:19
*** altlogbot_3 has quit IRC13:21
*** sanjayu_ has joined #oooq13:22
*** altlogbot_1 has joined #oooq13:24
*** sanjayu__ has quit IRC13:24
*** altlogbot_1 has quit IRC13:25
*** altlogbot_0 has joined #oooq13:25
*** ykarel is now known as ykarel|afk13:35
weshayhttps://review.rdoproject.org/r/#/c/20548/13:36
*** udesale has quit IRC13:46
*** ykarel|afk is now known as ykarel13:46
*** udesale has joined #oooq13:47
*** vinaykns has joined #oooq13:47
weshayrfolco|ruck and all... I have to drop14:04
weshayrfolco|ruck I'll work w/ you on the doc today14:04
rfolco|ruckack14:04
chandankumarweshay: rlandy rfolco|ruck zbr|rover arxcruz marios quiquell https://review.opendev.org/#/c/655850/ and https://review.opendev.org/#/c/654934/ these are the final patches for moving to new collect-logs, feel free to thave a look and merge , thanks :-)14:12
rlandyrfolco|ruck: you ready for this ??? :)14:12
*** Vorrtex has joined #oooq14:14
marioschandankumar: ack14:15
rfolco|ruckrlandy, you are scaring me, do you see any potential breakages ?14:15
quiquellchandankumar: will look at them tomorrow14:15
rlandyrfolco|ruck: we will find out14:15
*** quiquell is now known as quiquell|off14:15
*** udesale has quit IRC14:16
*** sanjayu_ has quit IRC14:17
rfolco|ruckrlandy, chandankumar +1 thx14:19
*** udesale has joined #oooq14:21
rlandyrfolco|ruck: zbr|rover: weshay: https://review.opendev.org/#/c/654934 - planning to merge this ... just fyi on possible fallout14:21
rlandyasked chandankumar to hold this until today where we have a bigger team to deal with issues14:21
rfolco|ruckrlandy, ack14:21
rlandychandankumar: k - w+'ed - let the chaos begin14:25
*** ykarel is now known as ykarel|away14:27
weshayrlandy will need to chat for 5 min at the top of the hour14:46
rlandyweshay: ok14:47
rfolco|ruckweshay, questions about ci-config/jenkins change to the new collect-logs.... we should replace this https://github.com/rdo-infra/ci-config/blob/4458e0f911904cc6d663aa5766297c9fd44baa11/jenkins/jobs/scripts/tripleo-upstream-collect-logs.sh ... and bump ansible version ?14:51
rfolco|ruckbrb14:55
chandankumarrlandy: rfolco|ruck thanks :-)14:57
rlandyweshay: joined your bj15:00
weshayrlandy k.. /me going15:03
rlandyweshay: https://review.opendev.org/#/c/644770/15:18
weshayrfolco|ruck I don't think we need to replace https://github.com/rdo-infra/ci-config/blob/4458e0f911904cc6d663aa5766297c9fd44baa11/jenkins/jobs/scripts/tripleo-upstream-collect-logs.sh15:24
rfolco|ruckjob is failing to clone collect-logs https://ci.centos.org/view/rdo/view/tripleo-gate/job/tripleo-quickstart-extras-gate-master-delorean-full-featureset052/3314/console15:24
weshaythe only thing that changed was that the role moved to another repo, it should be at run time in the same place15:24
weshayrfolco|ruck  git fetch --tags --progress ssh://weshayutin@review.opendev.org:29418/openstack/ansible-role-collect-logs.git refs/changes/72/657372/115:27
weshayfatal: Couldn't find remote ref refs/changes/72/657372/115:27
weshayhappens on my box too.. how is that related to the script?15:27
rfolco|ruck657372 isn't an ansible-role-collect-logs patch15:30
weshayrfolco|ruck k.. I'm still missing what you are communicating.... maybe you can show me?15:32
rfolco|ruckweshay, sure15:33
ykarel|awayyup depends-on messes it, same GERRIT_REFSPEC15:33
rfolco|ruckykarel|away, you are omnipresent15:34
rfolco|rucklolz15:34
ykarel|awayrfolco|ruck, this issue was discussed few days back with rlandy15:34
ykarel|awayso i remember15:34
rfolco|ruck:)15:34
ykarel|awaybut didn't foolowup ahead15:34
weshayrfolco|ruck on my blue15:35
rfolco|ruckfor some reason the jenkins jobs is trying to fetch the ref which is the patch that trigered the jobs15:35
rfolco|ruckweshay, k15:35
rlandyykarel|away: rfolco|ruck: weshay: ci,centos is an issue in multiple levels15:35
rlandyat the job will never run15:35
rlandyand we can't get to change things on our own in ci.centos15:36
rlandyis a check job worth it at this point?15:36
rlandyif so, we need to merge weshay's abandoned change15:36
rlandywe tested collect_logs with promotion pipeline jobs15:37
chandankumarrfolco|ruck: I will miss the bug traige call, logging out15:41
*** chandankumar is now known as raukadah15:41
raukadahsee ya guys, tomorrow :-)15:42
*** Goneri has joined #oooq15:50
*** bogdando has quit IRC15:51
*** ykarel|away has quit IRC15:53
weshayrfolco|ruck https://review.opendev.org/#/c/655938/15:54
*** ykarel has joined #oooq16:17
rfolco|ruckweshay, thanks for your patience16:24
weshaynp16:24
rfolco|ruckweshay, https://bugs.launchpad.net/tripleo/+bug/182790616:29
openstackLaunchpad bug 1827906 in tripleo "invalid option for include_tasks: tags --> breaks ansible 2.7" [High,Triaged]16:29
zbr|roverrfolco|ruck: weshay : the change that prevented f28 to pass in periodical (dib/virtualenv) was merged few hours ago, we should no longer see that error.16:35
zbr|roverwhat is not clear to me is where to see if the new images were build16:35
zbr|rover@oooq: if you see ansible-lint failing for no clear reason, check https://github.com/yaml/pyyaml/issues/297 -- it seems that newer pyyaml can cause this in very few cases (not sure why, but using old version works).16:37
weshayarxcruz rlandy there was some doc on running zuul-runner w/ nova16:38
weshayI think one of you pointed me at it16:38
weshayanyone have a link to that?16:38
weshaygot it rlandy in that review you sent https://review.opendev.org/#/c/644770/9/doc/source/user/runner.rst16:41
rlandyyep16:46
*** dsneddon has joined #oooq16:46
weshayrlandy arxcruz making notes here: https://docs.google.com/document/d/17WOTM5kbtfoTP4HQQB135-AC9Rwuvs59ogWQsPdgHRs/edit#16:47
*** holser_ has quit IRC16:49
weshayrlandy arxcruz I've invited tristan to that doc16:55
*** amoralej is now known as amoralej|off16:56
rlandyweshay: ack - I plan to ping him later this evening when he comes on line16:56
weshayrfolco|ruck can you move the bug triage to a much earlier time where chandan can participate16:57
rfolco|ruckweshay, sure16:57
weshaythanks..  and we'll skip today16:57
rfolco|ruckweshay, done, 8am your time. Please let me know if it's too early16:59
*** jpena is now known as jpena|off17:10
*** jaosorior has quit IRC17:14
*** kopecmartin is now known as kopecmartin|off17:17
mariosweshay: still being uploaded but fyi at http://www.mariosandreou.com/tripleo/2019/05/06/open-infrastructure-summit-denver-3.html17:18
mariosthat takes far too long to put together17:18
mariosttyl17:19
weshayk.. thanks17:19
mariosi think it should show up in planet tripleo assuming that still a thing17:19
weshayit will hit my feed then :)17:21
weshaymarios++17:23
mariosweshay: ack np it doesn't go into huge detail anywhere but i think i got all the things I went to there17:23
mariosweshay: (with pointers to more info)17:23
*** marios has quit IRC17:24
*** tesseract-RH has quit IRC17:24
*** ykarel has quit IRC17:25
*** udesale has quit IRC17:27
weshayrfolco|ruck rlandy can you guys spare a minute to review https://docs.google.com/document/d/17WOTM5kbtfoTP4HQQB135-AC9Rwuvs59ogWQsPdgHRs/edit#17:33
rlandyyeah17:36
rfolco|ruckweshay, in a bit, yes17:37
rlandyweshay: that's a lot of work for 1 1/2 days17:41
weshayya.. not expecting to get through all of it17:42
rlandyyeah - like a whole sprint for the whole team17:42
weshayI'm good at identifying work17:42
rlandyso for today/tomorrow, we pick anything from that list we like?17:44
weshayfor tues/wed17:44
rlandyor the list is prioritized?17:44
weshayadded my thoughts to the instructions17:46
*** ykarel has joined #oooq17:48
rlandyweshay: have time to go over baseos work?18:03
weshayrlandy sure.. give me 5min please18:11
weshayrlandy ready when ever18:20
rlandyweshay - k- joinng your bj18:20
*** Goneri has quit IRC18:41
*** Goneri has joined #oooq18:42
*** ykarel is now known as ykarel|away18:49
*** jtomasek has quit IRC19:13
*** ykarel|away has quit IRC19:22
*** quiquell has joined #oooq19:39
*** Vorrtex has quit IRC19:39
*** apetrich has quit IRC19:39
*** quiquell|off has quit IRC19:39
*** apetrich has joined #oooq19:40
*** dsneddon has quit IRC19:44
*** Vorrtex has joined #oooq20:32
*** dsneddon has joined #oooq20:37
*** dsneddon has quit IRC20:42
*** dsneddon has joined #oooq20:42
*** zul has quit IRC20:47
*** Vorrtex has quit IRC20:51
*** Goneri has quit IRC22:07
*** vinaykns has quit IRC22:24
*** zbr|rover has quit IRC23:01
*** tosky has quit IRC23:32
rlandyweshay: hi23:34
weshayrlandy hey23:35
rlandyweshay: pls see #zuul23:36
rlandyweshay: will copy that conversation to your test day notes23:36
*** aakarsh has quit IRC23:41
weshaythe running of the job we have nailed down23:43
rlandyweshay: yep - looking at the comments - I think we will be right back where we started23:44
rlandywe could also claim to run with one or two lines23:45
rlandythose lines hide a whole lot of the same complexity23:45
weshayrlandy it's interestng how much faith we're putting into unmerged code here w/ zuul-runner23:49
weshayrlandy we would probably bolt that into the existing new repro code I think23:50
weshayright?23:50
weshaybecause it doesn't get you any hardware23:51
weshayany nodes23:51
weshayrlandy will be a good discussion to have over the next few days23:53
rlandyweshay: I think we are not going to gain much other than maybe software factory support?23:54
rlandyweshay: will see what arxcruz says tomorrow - I plan on working on libvirt multinode23:55
weshayrlandy what we are working *is* a software factory use case23:56
*** rlandy has quit IRC23:59

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