Friday, 2021-12-03

*** rlandy|ruck is now known as rlandy|out00:55
*** rcastillo is now known as rcastillo|out01:02
*** sshnaidm is now known as sshnaidm|off02:57
*** ysandeep|out is now known as ysandeep|ruck04:54
*** pojadhav|afk is now known as pojadhav05:00
*** raukadah is now known as chandankumar06:12
*** ysandeep|ruck is now known as ysandeep|afk06:16
mariosysandeep|afk: o/ hey you forgot to add meet link for the handover call today 06:23
ysandeep|afkmarios, thanks for pointing that, added 06:25
mariosthanks06:26
marioschandankumar: o/ please check i didn't get what you want me to do there https://review.opendev.org/c/openstack/tripleo-ci/+/818222/48#message-3eab7f43102d25fdf86e7f2833c7dd2db97289a5 06:57
pojadhavfolks, please review when free https://review.rdoproject.org/r/c/rdo-jobs/+/36718 https://review.rdoproject.org/r/c/rdo-jobs/+/3695906:57
mariosok this is ridiculous now rlandy ... i mean getting up early is one thing but not even going to sleep is another !07:03
chandankumarmarios: sorry for the confusion, commented back07:05
marioschandankumar: ack i see, sure, once we remove it from the release file then we can remove it from here too.07:06
chandankumarmarios: thanks :-)07:06
pojadhavchandankumar, thank you for merging my changes :)07:25
*** jpena|off is now known as jpena08:02
*** amoralej|off is now known as amoralej08:03
*** ysandeep|afk is now known as ysandeep08:07
*** ysandeep is now known as ysandeep|ruck08:08
soniya29arxcruz|rover, can you review this please https://review.opendev.org/c/openinfra/python-tempestconf/+/82016908:46
*** ysandeep|ruck is now known as ysandeep|lunch08:58
mariosysandeep|lunch: chandankumar: dviroel_: o/ please review whenyou get a chance https://review.opendev.org/c/openstack/tripleo-ci/+/820318/08:59
*** ysandeep|lunch is now known as ysandeep10:13
*** ysandeep is now known as ysandeep|ruck10:16
ysandeep|ruckchandankumar, hey i was reviewing https://review.opendev.org/c/openstack/tripleo-ci/+/820172 , Do we have a testproject somewhere?10:31
ysandeep|rucktrying to check what's the value of ensure_pip_virtualenv_command var (which comes from ensure_pip role) in this case10:31
*** dviroel_ is now known as dviroel11:09
dviroelmorning11:09
*** rlandy_ is now known as rlandy|ruck11:14
pojadhavrlandy|ruck, morning11:14
rlandy|ruckpojadhav: hello11:15
pojadhavrlandy|ruck, chandankumar will miss review meeting with you as he is taken his wife at hospital for routine checkup. 11:15
rlandy|ruckpojadhav: thanks for heads up11:15
rlandy|ruckmarios: :)11:16
rlandy|ruckmarios: lol - I'm starting to get used to 5am11:16
mariosrlandy|ruck: :)11:16
mariosrlandy|ruck: i saw you earlier it must have been middle of th enight for you11:16
mariosrlandy|ruck: or it was your bot ;)11:16
mariosrlandy|ruck: when you get a sec ... https://review.opendev.org/c/openstack/tripleo-ci/+/820318 as discussed on the email11:18
rlandy|ruckmarios: ok - W+ing that11:20
marios rlandy|ruck: ack thanks11:20
rlandy|ruckI think we will need to add those roles to the -rhel base job11:20
rlandy|ruckin the end11:20
rlandy|ruckbut let's see11:20
mariosrlandy|ruck: k, i mean worst case if we still cant test it then we do plan b merge it and pray ;)11:21
mariosand fix the issue after 11:21
ysandeep|ruckshadowing in zuul can easily get confusing if not complicated :) 11:21
rlandy|ruckysandeep|ruck: marios: yeah - went back and forth with nicholas on this one11:22
rlandy|ruckzuul can't resolve the depends 11:22
rlandy|ruckysandeep|ruck: arxcruz|rover: anything burning in ruck/rover land?11:24
rlandy|ruckmarios: I think we will be able to test it after we merge the base situation11:24
ysandeep|ruckrlandy|ruck, things are looking better now, metalsmith fix is now in master line (running now)11:25
rlandy|ruckI am thinking we will need to add those roles like this: https://review.opendev.org/c/openstack/tripleo-ci/+/820257/1/zuul.d/base.yaml11:25
ysandeep|ruckrlandy|ruck, we had very long queues in rdo - i didn't recheck component failed jobs today so that things can get settled11:25
rlandy|ruckrhel8 is what the downstream container builds are using11:26
rlandy|ruckysandeep|ruck: ack - queues were hanging for 8 or 9 hours11:26
rlandy|ruckasked infra to reset al11:26
rlandy|ruckall11:26
rlandy|ruckso they only started again in my night hours11:26
ysandeep|ruckrlandy|ruck, sshnaidm|off fix merged - things are okay in check/gate now11:26
rlandy|ruckysandeep|ruck: yeah - luck sshnaidm|off is still around11:27
rlandy|ruckthat guy rocks!!!11:27
* rlandy|ruck will close CIX11:27
ysandeep|rucksshnaidm++ thank you! You are amazing :) 11:27
ysandeep|ruckrlandy|ruck, sc010 removal from everywhere: https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/37010 and https://review.rdoproject.org/r/c/rdo-jobs/+/37011 , we can do definition removal in a follow up11:30
ysandeep|ruckmarios, chandankumar please add https://review.rdoproject.org/r/q/topic:%22move_to_sc010-kvm%22+(status:open%20OR%20status:merged) and https://review.rdoproject.org/r/c/rdo-jobs/+/36890 in your review list11:32
rlandy|ruckysandeep|ruck: thanks - +2'ed both11:32
ysandeep|ruckrlandy|ruck, this one also: https://review.rdoproject.org/r/c/rdo-jobs/+/36890 11:32
rlandy|ruckbhagyashris: pojadhav: rcastillo|out's patch is in merge conflict 11:33
rlandy|ruckas pojadhav's patch merged before11:33
rlandy|ruckbhagyashris: pojadhav: pls rebase https://review.rdoproject.org/r/c/rdo-jobs/+/3697411:34
rlandy|ruckand I'll approve it again11:34
bhagyashrisrlandy|ruck, will do that11:34
rlandy|ruckysandeep|ruck: jfrancoa is working on that job11:35
rlandy|ruckif it starts passing soon, then we should keep it - holding on this for a week11:35
ysandeep|ruckrlandy|ruck, ack please -1 with your comment 11:35
rlandy|ruckjbadiapa: ysandeep|ruck: https://review.opendev.org/c/openstack/tripleo-quickstart/+/818766 - can we merge this?11:36
* rlandy|ruck checks with harold as well11:36
ysandeep|ruckrlandy|ruck, c7 issue cleared after sagi patch.. I think we can post check-ci-minimal now to confirm everythings works11:36
ysandeep|ruckrlandy|ruck, anyway ^^ lgtm +211:38
rlandy|ruckysandeep|ruck: posted again11:38
rlandy|ruckysandeep|ruck: there is an issue getting tempest router11:39
rlandy|ruckbut at least this gets us past the boot problem11:39
rlandy|ruckand the c7 problem11:39
rlandy|ruckwe can bug the router problem next11:39
ysandeep|ruckjust a note for us.. we will need to update https://review.opendev.org/c/openstack/tripleo-quickstart/+/818766/11/roles/libvirt/defaults/main.yml#69 condition to remove wallaby once sbaker patches are in11:39
rlandy|ruckysandeep|ruck: I don;t think so11:40
rlandy|ruckuefi is only default on master11:40
rlandy|ruckwhole_disk_images11:40
rlandy|ruckis on wallaby11:40
ysandeep|ruckThey don't plan to uefi on wallaby? i think plan is also to cherrypick that on wallaby11:40
jbadiaparlandy|ruck: from my side it can be merged11:41
jbadiapaBut not sure about the ysandeep|ruck comment.11:41
jbadiapaI think I saw womething regarding the whole_disk_images on wallaby11:41
rlandy|ruckwhole_disk_images yes11:42
jbadiapahttps://review.opendev.org/c/openstack/tripleo-quickstart/+/813553 11:43
jbadiapa^merged yesterday11:43
arxcruz|roversoniya29: did tyou test that patch ?11:45
ysandeep|ruckrlandy|ruck, left a comment for sbaker on hardprov chat to confirm if he plan to backport https://review.opendev.org/c/openstack/tripleo-heat-templates/+/809961 in wallaby11:45
rlandy|ruckysandeep|ruck: yeah  - thanks - we have no confirmation on that yet11:46
jbadiaparlandy|ruck: I'm changing the patch to add uefi to wallaby11:47
rlandy|ruckjbadiapa: not yet pls11:47
jbadiapaah, ok11:47
rlandy|ruckjbadiapa: so far only master11:47
rlandy|ruckwe have no back port on https://review.opendev.org/c/openstack/tripleo-heat-templates/+/809961 yet11:48
ysandeep|ruckjbadiapa, iiuc.. that would be required only after https://review.opendev.org/c/openstack/tripleo-heat-templates/+/809961 cheery-pick on wallaby11:48
rlandy|ruckmarios: chandankumar: ysandeep|ruck: pls review https://issues.redhat.com/browse/RHOSZUUL-70111:48
rlandy|rucklet me know your thoughts11:48
ysandeep|ruckrlandy|ruck, reading 11:49
jbadiapawe might need to get this  https://review.opendev.org/c/openstack/puppet-ironic/+/820086  merged11:49
ysandeep|ruckjbadiapa, yeah i think waiting on Tobias , we have less core members on that repo11:50
rlandy|ruckdviroel: hi - pls see marios comments on https://review.rdoproject.org/r/c/rdo-infra/ci-config/+/3691111:53
rlandy|ruckso we can decide on merge11:53
soniya29arxcruz|rover, nope..will add testproject to it11:53
dviroelrlandy|ruck: looking11:54
ysandeep|ruckrlandy|ruck, +1 to build c09 image via dib and also rhel9 image 11:54
* ysandeep|ruck reading details11:55
rlandy|ruckjfrancoa: hey - you all set with the node debug you needed?11:55
rlandy|ruckarxcruz|rover: hey - pls ping before you are EoD and let's get a status of your patches to hand off11:56
arxcruz|roverrlandy|ruck: yes 11:56
bhagyashrisrlandy|ruck, done https://review.rdoproject.org/r/c/rdo-jobs/+/3697411:57
jfrancoarlandy|ruck: hey! yes, thanks a lot!11:57
bhagyashrisrcastillo|out, pojadhav ^11:57
pojadhavbhagyashris, ack11:57
jfrancoarlandy|ruck: I received the credentials and I hope to find some time in the afternoon to debug it11:57
rlandy|ruckbhagyashris: thanks - pls vote and I'll merge11:57
rlandy|ruckjfrancoa: just fyi - hold time is olny 24 hours11:58
rlandy|ruckonly11:58
ysandeep|ruckarxcruz|rover, rr sync12:00
jfrancoarlandy|ruck: oh, thanks for the info...good to know (I don't have much time left then)12:22
*** arxcruz|rover is now known as arxcruz12:43
chandankumarysandeep|ruck: testing venv patch here https://review.rdoproject.org/r/c/testproject/+/3699712:48
chandankumarrlandy|ruck: looking12:48
pojadhavrlandy|ruck, https://code.engineering.redhat.com/gerrit/q/topic:%22ansible_code_remove%22+(status:open%20OR%20status:merged)12:48
chandankumarmarios: ysandeep|ruck I will be not around on monday and tuesday, will back on Wed.12:57
chandankumarpojadhav: bhagyashris ^^12:57
chandankumarsoniya29: ^^12:57
bhagyashrischandankumar, ack12:57
ysandeep|ruckchandankumar, o/ enjoy your PTOs12:57
marioschandankumar: k have a good one13:00
chandankumarmarios: might also hit this https://paste.openstack.org/raw/811433/ after using repo setup13:00
chandankumarin container build13:01
pojadhavchandankumar, ack13:01
marioschandankumar: k noting... havent seen that one before13:02
chandankumarlogged a bug related to curl https://bugs.launchpad.net/tripleo/+bug/195315613:20
chandankumarrlandy|ruck: marios ysandeep|ruck https://review.opendev.org/c/openstack/tripleo-ci/+/820172 13:31
chandankumarand this also https://review.opendev.org/c/openstack/tripleo-common/+/820382 when free, thanks!13:33
*** amoralej is now known as amoralej|lunch13:39
marioschandankumar: ack 13:39
rlandy|ruckchandankumar: ysandeep|ruck and I are on agile training13:40
rlandy|ruckpls add to review list13:40
chandankumarrlandy|ruck: aye13:41
*** pojadhav is now known as pojadhav|brb13:48
chandankumarmarios: now cs9 is offically announced https://blog.centos.org/2021/12/introducing-centos-stream-9/13:51
marioschandankumar: :)13:52
rlandy|ruckwow - we've arrived13:59
*** rcastillo|out is now known as rcastillo14:04
rlandy|ruckrcastillo: hi - just w+'ed your patch14:07
rlandy|ruckrcastillo: feel free to pick up another 14:07
rcastilloneat :D14:08
rcastillowill do14:08
ysandeep|ruckrlandy|ruck, have we already changed upstream sc010 to use kvm instead of qemu? In follow up of chandankumar comment here: https://review.rdoproject.org/r/c/rdo-jobs/+/37011/1//COMMIT_MSG#7 14:14
rlandy|ruckysandeep|ruck: ack - we have14:14
*** amoralej|lunch is now known as amoralej14:14
rlandy|ruckwill respond14:14
rlandy|ruckysandeep|ruck: chandankumar: responded on  https://review.rdoproject.org/r/c/rdo-jobs/+/3701114:17
chandankumarrlandy|ruck: ysandeep|ruck thanks!14:18
chandankumarmarios: I have tested repo-setup toci patch, looks good with results also, good to go from my side14:19
marioschandankumar: thanks14:19
ysandeep|ruckrlandy|ruck, thanks! imho.. once its all settled - i think we should just rename our sc010-kvm (in periodic lines) - just sc010 to be in sync with upstream.14:22
rlandy|ruckysandeep|ruck: sure14:30
mariosrlandy|ruck: put some numbers down re rhos-dev-stage and replied to your mail feel free to forward on if you are sending some reply 14:32
mariosrlandy|ruck: (i put some numbers)14:32
rlandy|ruckmarios: ack - will do the same for ovb14:33
chandankumarrlandy|ruck: Dec 3 reviews are mostly reviewed https://hackmd.io/FGMoCiRfSNa8puA1BpTQ-Q?view#Dec-3-2021 please +w or merge when free, thanks!14:35
rlandy|ruckchandankumar: thanks - will look after training14:36
marioschandankumar: what is the workaround for curl-minmal i am hitting in latest test run 14:42
marioschandankumar: https://logserver.rdoproject.org/17/36817/9/check/periodic-tripleo-build-containers-stream9-development/30e1b09/logs/container-builds/b1aff13d-6872-401e-97e8-a9d7e83a91d9/base/base-build.log14:42
chandankumarmarios: https://review.opendev.org/c/openstack/tripleo-common/+/82038214:42
marioschandankumar: thanks adding14:43
chandankumarmarios: tested here https://review.rdoproject.org/r/c/testproject/+/3699714:43
marioschandankumar: also trying https://review.opendev.org/c/openstack/tripleo-quickstart/+/820374 14:44
chandankumarmarios: ah, I missed this patch14:45
chandankumarthanks!14:45
marioschandankumar: hmm on second thoughts actually that one needs to be tested with the upstream/opendev jobs... 14:45
marioschandankumar: still good to sanity check it doesn't break something rdo side 14:45
chandankumaryup totally agree14:45
chandankumaronce I will be back, I will add the container build job upstream first14:46
marioschandankumar: k yeah c9 content provider14:46
soniya29chandankumar, ack14:49
arxcruzrlandy|ruck: have time very quickly ?14:56
rlandy|ruckarxcruz: ack 14:56
arxcruzrlandy|ruck: https://meet.google.com/_meet/uyk-zjxe-igb14:57
*** dviroel is now known as dviroel|lunch15:10
Tengurlandy|ruck: LADA has a new feature allowing to find where's a package (zstream? compose?) - it's only for downstream though. Lemme know if such a feature would be good for the upstream as well.15:12
Tenguysandeep|ruck: -^^  (should have voiced the "for downstream only")15:16
*** pojadhav|brb is now known as pojadhav15:20
ysandeep|ruckyeah it got that part when you mentioned "zstream and compose" that this is downstream only.15:20
rlandy|ruckmarios: your patch is almost merged - setting up downstream check of remaining patch15:21
ysandeep|ruckFor Upstream - If lada can tell current fix is at what stage - current , consistent , tripleo-ci-testing or current-tripleo that would be awesome for us.15:22
rlandy|ruckwe have those stages in LADA already15:22
rlandy|ruckTengu already implemented all that for us15:22
rlandy|ruckso I think we are good upstream15:22
ysandeep|ruckoh nice, I didn't noticed that's implemented already15:26
ysandeep|ruckjust tested that works15:26
Tenguysandeep|ruck, rlandy|ruck good then :). So we're covered.15:28
rlandy|ruckTengu: I think so - we'll ping if we're missing something going forward15:28
Tenguperfect!15:28
Tengugood example of Agile ;).15:28
chandankumarsee ya on Wed :-)15:28
*** chandankumar is now known as raukadah15:29
ysandeep|ruckrlandy|ruck, worth covering downstream component line stages as well ?15:29
rlandy|ruckysandeep|ruck: yep15:29
rlandy|ruckhmmm ...15:29
rlandy|ruckthought that was picked up by the stages added15:30
rlandy|ruckI mean 15:30
rlandy|ruck current , consistent , tripleo-ci-testing etc, is upstream and downstream15:30
ysandeep|ruckrlandy|ruck, that's implemented already too.. :)15:35
* ysandeep|ruck amazed to see the improvements on lada since the last time i visited that15:35
ysandeep|ruckTengu++ thank youuuuuu! 15:36
Tenguysandeep|ruck: you're welcome :)15:37
Tengurlandy|ruck: the new feature is more oriented on "package" than "patch", and allows to find out "when was that package available in compose and/or zstream".15:38
Tengunote: the feature already existed. It wasn't exposed in the UI.15:38
rlandy|ruckmarios: job accepted in downstream now15:44
mariosrlandy|ruck: nice :)16:00
mariosrlandy|ruck: did you have to do something else or just the base merge upstream? did it merge /me checks16:00
mariosok great16:01
rlandy|ruckmarios: nothing16:03
rlandy|ruckw/o base change, it's fine16:03
mariosrlandy|ruck: https://sf.hosted.upshift.rdu2.redhat.com/zuul/t/tripleo-ci-internal/status#295749 16:04
mariosrlandy|ruck: nice16:04
mariosgood call rlandy|ruck++16:06
jfrancoarlandy|ruck: big thanks for the node! I think I found out why was my job failing... I missed the subtle difference that the "openstack tripleo deploy" and "openstack tripleo upgrade" should be run as sudo16:07
jfrancoaand of course I was getting crazy logs of permission denials, which ended up filling up the disk16:07
jfrancoait took me two weeks to find the damn difference :-D16:08
rlandy|ruckjfrancoa: thanks for tracking that down16:11
mariosrlandy|ruck: going afk in a few time for some cardio ;)16:15
mariosrlandy|ruck: will close within the hour if you need something ping i can see before i go16:15
rlandy|ruckmarios: sur e- will +2 your patch if downstream passes16:15
*** marios is now known as marios|afk16:15
marios|afkrlandy|ruck: thanks16:15
rlandy|ruckyou can merge in your morning16:15
rlandy|ruckmarios|afk: see you monday 16:15
*** dviroel|lunch is now known as dviroel16:28
ysandeep|ruckrlandy|ruck, fyi.. fs001 master failed on tempest .. retriggered here : https://review.rdoproject.org/r/c/testproject/+/2844616:30
ysandeep|ruckthat's the only one left for promotion 16:31
rlandy|ruckysandeep|ruck: ok - you're EoD - I'll check it out16:31
rlandy|ruckand raise LP if needed16:31
ysandeep|ruckthanks!16:31
pojadhavrlandy|ruck, i am facing issue while adding sc10-kvm component master job https://code.engineering.redhat.com/gerrit/c/openstack/tripleo-ci-internal-jobs/+/296123/1#message-c1695162f7aa64e0a182f6d3827072663b29fd5c16:34
rlandy|ruckpojadhav: k, give me a few - will look16:36
pojadhavrlandy|ruck, ack16:36
* pojadhav will be back after sometime16:36
*** pojadhav is now known as pojadhav|dinner16:36
* ysandeep|ruck out, see you next week o/16:38
*** ysandeep|ruck is now known as ysandeep|out16:38
rlandy|ruckysandeep|out: have a good weekend16:43
rlandy|ruckpojadhav|dinner: looking16:43
*** jpena is now known as jpena|off16:51
*** pojadhav|dinner is now known as pojadhav16:53
*** marios|afk is now known as marios16:59
mariosthanks rlandy|ruck16:59
rlandy|ruckmarios: looks like it passes downstream17:00
rlandy|ruckcan't promise it uses the right role 17:00
rlandy|ruckbut it won;t hose anything17:00
mariosrlandy|ruck: checking 17:00
mariosrlandy|ruck: yeah some green there looks promising https://sf.hosted.upshift.rdu2.redhat.com/zuul/t/tripleo-ci-internal/status#295749 17:00
mariosrlandy|ruck: some still running17:00
mariosrlandy|ruck: but good one on the sanity check 17:00
rlandy|ruckmarios: going to +217:00
mariosrlandy|ruck: lets wait17:00
mariosrlandy|ruck: we got this far17:00
mariosrlandy|ruck: waiting for all the green https://sf.hosted.upshift.rdu2.redhat.com/zuul/t/tripleo-ci-internal/status#295749 17:01
mariosrlandy|ruck: but if you can worfklow it before you leave it would be great17:01
mariosrlandy|ruck: if all is good that is17:01
rlandy|ruckmarios: https://code.engineering.redhat.com/gerrit/c/testproject/+/19067217:01
mariossame jobs? me checks17:02
rlandy|ruckgi t a clean run there17:02
rlandy|ruckit's the check jobs17:02
mariosrlandy|ruck: ack fair enough 17:02
mariosrlandy|ruck: not all the same but enough 16/17 containers images 17:02
mariosrlandy|ruck: i just worfklowed it17:03
rlandy|ruckmarios: ok17:03
dviroelmarios: rlandy|ruck 17:04
dviroelhttps://review.rdoproject.org/r/c/rdo-infra/ci-config/+/3691117:04
dviroeli just remove that addition to dev script, need to improve that, but the fix continues the same17:04
mariosdviroel: ack but monday 17:04
* marios status and out17:04
rlandy|ruckdviroel: ok by me - you'll catch the other core reviewers on monday17:05
rlandy|ruckdviroel: rcastillo: we got confirmation that dariusz's laptop shipped17:06
rlandy|ruckso hopefully we will have an easier time with his onboarding17:06
dviroelsure, doesn't block anything, just a small fix no rush17:06
dviroel++17:06
*** marios is now known as marios|out17:08
pojadhavrlandy|ruck, I am out.. pls drop me notes, I will pickup from that notes. thanks!17:24
*** pojadhav is now known as pojadhav|out17:24
rlandy|ruckpojadhav|out: ack - have a good weekend17:24
pojadhav|outsame to you :)17:25
rlandy|ruckpojadhav|out: k - see the issue - going to edit patch17:44
rlandy|rucklunch - biab17:49
*** amoralej is now known as amoralej|off18:03
*** dviroel_ is now known as dviroel18:49
*** dviroel is now known as dviroel|afk18:59
*** rcastillo is now known as rcastillo|out23:09

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