Thursday, 2021-07-08

dviroelweshay|ruck: ack, lets sync tomorrow 00:46
rlandy|bblweshay|ruck: ok - thanks - will relook at it02:29
*** ykarel|away is now known as ykarel04:34
*** zbr is now known as Guest16405:03
*** ysandeep|away is now known as ysandeep05:18
ysandeepbhagyashris|ruck, chkumar|rover fyi.. network component in master promotion was stuck  http://pastebin.test.redhat.com/977817  because https://review.rdoproject.org/zuul/builds?job_name=periodic-tripleo-ci-centos-8-multinode-1ctlr-featureset010-network-master is failing due to infra issues from last 2 days.07:04
ysandeepretrigged via testproject here: https://review.rdoproject.org/r/c/testproject/+/32054 07:04
*** amoralej|off is now known as amoralej07:07
*** ykarel_ is now known as ykarel07:14
chkumar|roverysandeep: checking now07:17
*** jpena|off is now known as jpena07:19
bhagyashris|ruckysandeep, thanks07:25
chkumar|roverysandeep: bhagyashris|ruck https://bugs.launchpad.net/tripleo/+bug/193499408:07
ysandeepchkumar|rover: ack , thanks, i noticed sc010 was not in network promotion criteria https://github.com/rdo-infra/ci-config/blob/master/ci-scripts/dlrnapi_promoter/config/CentOS-8/component/master.yaml#L54-L55 08:11
chkumar|roverysandeep: yes, thanks, adding it in the comment, moving it to CIX will to get it fixed.08:13
bhagyashris|ruckchkumar|rover, ack08:16
*** sshnaidm_ is now known as sshnaidm08:25
ysandeepchkumar|rover, weshay|ruck removed ovb jobs from criteria for component and enabled to_build: true a while back.. i just noticed he missed to revert those patches.. reverted those here: https://review.rdoproject.org/r/q/topic:%22c8-stream-hostname%22+(status:open%20OR%20status:merged) 08:26
chkumar|roverysandeep: will take a look, thanks :-)08:33
*** ykarel is now known as ykarel|lunch08:41
ysandeepchkumar|rover, thanks!08:48
*** ysandeep is now known as ysandeep|lunch08:49
mariosneeds workflow please https://review.opendev.org/c/openstack/tripleo-ci/+/793120 08:57
mariosneed votes please at https://review.opendev.org/c/openstack/tripleo-heat-templates/+/79983209:17
*** ysandeep|lunch is now known as ysandeep09:51
ysandeepmarios, checking  https://review.opendev.org/c/openstack/tripleo-ci/+/793120 09:51
mariosthanks ysandeep i was about to self workflow it :)09:51
ysandeepi already voted there, Let me check the last comments b/w you and wes then i will +w it09:52
mariosysandeep: ack 09:52
ysandeepdone09:55
mariosthank you ysandeep 09:58
ysandeepmy pleasure, looks like you already got reviews on your other tht patch.. i only have +1 there10:00
mariosysandeep: yeah thanks10:04
*** ykarel|lunch is now known as ykarel10:46
*** jpena is now known as jpena|lunch11:31
*** slaweq_ is now known as slaweq11:42
*** amoralej is now known as amoralej|lunch12:02
weshay|ruckrlandy, https://review.rdoproject.org/r/q/owner:pojadhav%2540redhat.com12:09
weshay|ruckrlandy, https://review.opendev.org/c/openstack/tripleo-ci/+/793120/8/zuul.d/upgrades-jobs-templates.yaml#13012:21
chkumar|roverweshay|ruck: Tengu jpena|lunch https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/34441 for mirror sync status12:25
*** jpena|lunch is now known as jpena12:27
Tenguchkumar|rover: yeah, saw it, voted and commented out :)12:27
Tenguwould be really, really good getting those info in some dashboard with some history.12:27
chkumar|roverTengu: will add improvements in next patchset12:27
Tengu+112:28
ysandeepweshay|ruck: https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/34453 review request when you have time.12:45
sshnaidmdviroel, commented on https://review.opendev.org/c/openstack/tripleo-repos/+/796529 , you can fix issues in this patch or in followup, I don't mind. If in followup I can +w.12:53
dviroelsshnaidm: tks, reading your comments am12:57
dviroels/at/atm12:57
dviroelsshnaidm: both work for me. we'll propably come with a follow up change + units test for this module12:58
bhagyashris|ruckarxcruz, weshay|ruck chkumar|rover frenzy_friday zbr rlandy scrum time13:00
marioszbr: i want to spontaneously sing something to you, i suspect you have never heard this before13:01
marioszbr: its coming home its coming home its coming13:01
marios(stop me if you know it)13:01
zbrmarios: lol, not into football but I have pub 10m away from my house, i got my fair share of the song ;)13:03
*** amoralej|lunch is now known as amoralej13:03
marioszbr: :) i am sure that's why i'm singing it at you13:04
ysandeeprlandy, count me in for c8 deps work till i go on PTOs.. we should meet soon to figure out next steps.13:12
bhagyashris|ruckhttps://review.rdoproject.org/r/c/rdo-infra/ci-config/+/3267913:12
*** poojajadhav is now known as pojadhav13:16
bhagyashris|ruckreviews please https://review.opendev.org/c/openstack/tripleo-repos/+/795333 & https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/793331 13:17
bhagyashris|ruckhttps://review.opendev.org/q/topic:tripleo-get-hash13:17
bhagyashris|ruckhttps://review.opendev.org/q/topic:wallaby-upgrade-jobs13:23
mariosreviews please https://review.opendev.org/c/openstack/tripleo-repos/+/795333 & https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/793331 13:24
mariosalso reviews here please https://review.opendev.org/q/topic:wallaby-upgrade-jobs  13:24
bhagyashris|ruckmarios, https://hackmd.io/1lXAxUH7SkG0mVcGHrdkQw13:32
mariosthanks bhagyashris|ruck 13:33
weshay|ruckmarios, you'll like this one https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/34441/113:41
mariosweshay|ruck: ack adding to reviews :)13:42
mariosnice one chkumar|rover this should be very useful 13:42
Tengumarios, weshay|ruck and there's also that one, though WIP and waiting for some httpd related metrics to come: https://review.rdoproject.org/r/c/rdo-infra/ansible-role-dlrn/+/3443814:03
mariosTengu: thanks noting for review list14:04
rlandypojadhav: hi - reviewing your patches ...14:17
rlandywhich approach did you decide on for https://review.rdoproject.org/r/c/config/+/34432?14:17
*** ykarel_ is now known as ykarel|away14:18
rlandylooks like a combination of the two right now14:18
weshay|ruckrlandy, ping me later to catch on scen000 and anything else14:32
rlandyweshay|ruck: ack14:33
rlandyhave a few things to cover14:33
rlandyweshay|ruck: found my mistake re: scenario000 and ceph14:33
rlandyfixing that now14:33
rlandycopy/paste/idiot error14:34
weshay|ruckheh.. I'm famous for those14:35
weshay|ruckchkumar|rover, bhagyashris|ruck rlandy sshnaidm I think vexx is back to normal.. ack to close infra cix cards?14:38
rlandyweshay|ruck: yay - party!14:38
rlandyalso we need to send jpena flowers for that follow up14:39
sshnaidm++14:41
bhagyashris|ruckweshay|ruck, ack14:42
bhagyashris|ruckweshay|ruck, chkumar|rover arxcruz https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/799578 please add in your review list14:42
weshay|ruckyo arxcruz thanks for https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/34442  can we put the go build in the dockerfile?14:44
weshay|ruckwe don't often rerun the setup on the cockpits14:44
weshay|ruckthat's bad on us.. but it's the truth14:45
weshay|ruckbhagyashris|ruck, are you turning the promoter back to full ON?14:48
bhagyashris|ruckweshay|ruck, yes14:49
bhagyashris|ruckdoing14:49
weshay|ruckthank you :)14:49
weshay|ruckchkumar|rover, fyi ^14:49
bhagyashris|ruckDone14:50
weshay|ruckbhagyashris|ruck, chkumar|rover wallaby is promoting :)  /me will poke at mirrors after the promotion is done14:51
bhagyashris|ruckweshay|ruck, ack14:52
ysandeepchkumar|rover++ nice work on https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/34441/1 , Using threading will reduce the script run time: http://paste.openstack.org/show/807276/ , okay in current version.. just that I was learning threading and tried applying in this case.14:59
ysandeepUsing concurrent threading here reduced the script run time by 5 secs from  8.34 seconds to 2.7 seconds14:59
ysandeepweshay|ruck, I am working on ruck_rover.py to reduce script run time with threading/multiprocessing15:02
weshay|ruckheh k15:02
weshay|ruckysandeep, I was thinking we probably need to break this into classes and perhaps files15:03
weshay|ruckthere's quite a bit of duplication15:03
ysandeepweshay|ruck, yeah now we have to :) its ~800 lines already.. I never thought it would be soo big initially15:04
ysandeepweshay|ruck i think multithreading can do magic for ruck_rover.py.. i tried that and it reduced chandhan script run time from 8s to 2s http://paste.openstack.org/show/807276/15:06
weshay|ruckok.. go for it :)15:06
weshay|ruckysandeep, that paste is the mirror monitoring15:07
ysandeepweshay|ruck, yes i tried to applied it on chandan mirror monitoring script - and it reduced the run time from 8s to 2s15:08
weshay|ruckoh roger15:08
rlandyweshay|ruck: pojadhav: ok - reviewed patches and left comments ... we can look at those again this afternoon15:10
weshay|ruckrock15:10
chkumar|roverysandeep: sure, will make the improvement there15:11
ysandeepthanks! 15:13
arxcruzweshay|ruck: i'll check how to do it 15:20
weshay|ruckysandeep, so easy to check a running promotion now :)15:31
weshay|ruckhttp://pastebin.test.redhat.com/97803615:31
ysandeepweshay|ruck, yes this is getting better and better :), the only issue I see now is its taking time to get the results - basically because we download logs to find reason of failure / Find build history from zuul api - which I am trying to speed with multithreading.15:36
ysandeepweshay|ruck, rlandy review request when you have time: https://code.engineering.redhat.com/gerrit/c/tripleo-environments/+/252228 15:43
rlandyk15:43
ysandeepweshay|ruck, ^^ ruck_rover.py found there is an issue :)15:43
*** ysandeep is now known as ysandeep|away15:44
rlandysshnaidm: pls re-review https://review.opendev.org/c/openstack/ansible-role-tripleo-modify-image/+/79323415:49
rlandylet me know if any concerns about merging this logging into the modify-image role15:50
rlandyysandeep|away: ^^ requesting your review also15:50
*** marios is now known as marios|out16:07
sshnaidmrlandy, commented16:10
rlandysshnaidm: thanks - will update16:10
sshnaidmrlandy, nothing critical, we can merge and then to improve later if need16:11
rlandysshnaidm: there is no rush - I'll adjust the patch16:15
rlandyhmmm - delorean used in centos 9 doesn't include component-tripleo16:16
rlandyoh it does16:16
rlandyI take that back16:16
*** jpena is now known as jpena|off16:22
akahatReview Request: https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/34313, https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/32679, https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/3434116:25
pojadhavrlandy, sorry was away for dinner.. 16:30
pojadhavi have updated patch https://review.rdoproject.org/r/c/config/+/34432 instead of removing.. I have renamed it and added depends on this patch : https://review.rdoproject.org/r/c/rdo-jobs/+/3443716:31
pojadhavas per the discussion on the patch..16:32
weshay|ruckchkumar|rover, bhagyashris|ruck wallaby promoted.. and the mirrors are updated http://paste.openstack.org/show/807283/16:37
*** amoralej is now known as amoralej|off16:41
rlandypojadhav: thanks16:47
*** pojadhav is now known as pojadhav|away16:52
weshay|ruckarxcruz, https://bugs.launchpad.net/tripleo/+bug/193507217:59
rlandyweshay|ruck: pls ping when you want to meet18:01
rlandystill building containers so whenever 18:01
rlandyhttps://logserver.rdoproject.org/65/31165/46/check/periodic-tripleo-ci-centos-8-scenario000-standalone-master/8240833/logs/undercloud/home/zuul/standalone_deploy.log.txt.gz18:01
rlandylatest scenario000 fail 18:02
rlandygot keystone building18:02
weshay|ruckk.. talking to phil rlandy sec18:40
rlandyweshay|ruck: k - whenever you're ready18:40
dviroelo/20:58
dviroeltomorrow is a local holiday here, but i'll be around, ping me if need anything20:58
*** rlandy is now known as rlandy|biab21:18
*** dviroel is now known as dviroel|out21:44
*** rlandy|biab is now known as rlandy22:38

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!