Thursday, 2023-01-26

*** ysandeep|out is now known as ysandeep|holiday01:28
*** rlandy|bbl is now known as rlandy02:02
*** rlandy is now known as rlandy|out02:15
marios\o06:12
chandankumarmarios: o/07:02
marios\o07:14
*** amoralej|off is now known as amoralej07:32
Tengu|roverhello there07:35
Tengu|roverlast day as rover :).07:37
Tengu|rovertime to squash issues, so that handover will be smooth07:37
Tengu|roverbhagyashris|ruck: guess we'll set some call with Pooja and Doug for the passover?07:51
Tengu|rover*handover. whatever.07:51
amoralejhi, we are hitting issues related to pyroute2 version with master jobs in neutron07:53
amoralejdunno if you are seeing similar ones, i'll update pyroute2 but it may take some time as we need to get it in fedora too07:53
Tengu|roveramoralej: ah? maybe linked to the neutron component line being a bit red (with some green)?07:59
Tengu|roverI was wanting to investigate it, it fails on tempest.07:59
Tengu|roverfolks, care to add https://review.opendev.org/c/openstack/tripleo-quickstart/+/871730 to your review list?07:59
reviewbotI have added your review to the Review list07:59
amoralejTengu|rover, you have logs that i can check?07:59
Tengu|roveramoralej: gimme a few, I'm switching from tabs to tabs08:00
amoralejhttps://logserver.rdoproject.org/openstack-periodic-integration-main/review.rdoproject.org/rdoinfo/master/weirdo-antelope-promote-puppet-scenario002-centos-stream-9/4588687/logs/weirdo-project/logs/neutron/dhcp-agent.txt.gz08:00
amoralejthat's the kind of logs i've seen08:00
amoralejnote that's ml2/ovs, not ovn08:00
Tengu|roveramoralej: https://bugs.launchpad.net/tripleo/+bug/2003884 is the LP I opened in order to track things, didn't really investigate yet08:01
amoralejbut i think ovn also fails08:01
amoralejhttps://logserver.rdoproject.org/openstack-component-network/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-9-scenario007-standalone-network-master/64f1e09/logs/undercloud/var/log/containers/neutron/dhcp-agent.log.txt.gz08:02
amoralejyeah, same error08:02
Tengu|roverwow, if that wasn't a good timing :)08:02
Tengu|roveramoralej: care to comment in the LP?08:02
amoralejsure08:02
Tengu|rover++ thanks!!08:02
Tengu|roverI don't think I'd find it alone08:03
Tengu|roverand you even have an LP for the pinning :)08:03
Tengu|roveror update. or whatever patch is needed :)08:03
mariosTengu|rover: probably only doug around today bhagyashris|ruck and pooja are out for public holiday (regarding handoff)08:05
mariosTengu|rover: do you want to send a meeting invite for half hour before scrum?08:05
amoralejTengu|rover, i'm working to update pyroute2, it it takes too long we can pin neutron but i prefer to avoid it if possible08:05
Tengu|rovermarios: ah, right, I just saw Rabi talking about public holidays.08:05
Tengu|rovermarios: sure08:05
Tengu|roveramoralej: sure - at least we have a cause, a way out, so things are under control :)08:06
Tengu|roverthis is lovely08:06
Tengu|rovermarios: stupid question.... which doug again? ^^'08:07
Tengu|rovergot it08:09
Tengu|roverinvite sent08:09
mariosTengu|rover: also add rlandy_ for that meeting08:18
Tengu|roveroh, right08:20
Tengu|roverok you added her already08:21
*** jpena|off is now known as jpena08:32
Tengu|roverrlandy_: heya! I'm working on the tripleo-ci change allowing to get the latest image - I introduced a stupid thing (tabs instead of spaces, due to some wild copy-paste)11:02
Tengu|roverrlandy_: would get that change green first, then we can concentrate on the downstream testproject.11:02
Tengu|roversince the change triggers some image-build, we'll at least know a bit more.11:03
Tengu|rovermarios: should I kick the openstack-component-tripleo in order to trigger a promotion of that component?11:09
Tengu|roverthat would make things clearer. iirc it's running once a day now?11:09
marioso/ Tengu|rover did the haproxy pin help us? (or is the line red for something else?11:15
marioslooking11:15
Tengu|rovermarios: well, it didn't re-kick, soooo11:15
mariosah 11:15
Tengu|roverI just launched one periodic in testproject to check the state.11:15
Tengu|roverhttps://review.rdoproject.org/r/c/testproject/+/4670111:15
Tengu|rovershould help seeing how things are now.11:15
Tengu|roverI'd rather avoid passing a red line to the next rr shift :)11:16
Tengu|roverit's running periodic-tripleo-ci-centos-9-scenario004-standalone-tripleo-master right now11:16
*** rlandy|out is now known as rlandy11:16
Tengu|roverSHOULD help hopefully.11:16
mariosTengu|rover: just concerned about rekicking the whole line because resources 11:17
Tengu|rovermarios: that's why I kick just one job11:17
Tengu|rover:)11:17
mariosTengu|rover: but we should verify with a test at least 11:17
mariosright11:17
Tengu|roverif it's green, the, we can hope either next run will be fine, or maybe consider a re-kick.11:18
Tengu|roverif it's red.... #digging11:18
mariosso in fact... it didn't run yet today right? Tengu|rover https://github.com/rdo-infra/review.rdoproject.org-config/blob/master/zuul.d/upstream.yaml#L455 11:18
Tengu|rovermarios: well, if it ran, it failed, because pin wasn't in place.11:19
Tengu|roveras said, it merged earlier today11:19
Tengu|roverand... lemme check11:19
mariosTengu|rover: so no need to kick it will run this evening 11:19
Tengu|rover2023-01-25 20:30  it did run yesterday11:19
mariosTengu|rover:         - time: '30 19 * * *'11:19
mariosfrom 13:18 < marios> so in fact... it didn't run yet today right? Tengu|rover https://github.com/rdo-infra/review.rdoproject.org-config/blob/master/zuul.d/upstream.yaml#L455 11:19
rlandyTengu|rover: hi - hang in there11:20
Tengu|rovermarios: it did run, took about 1h, so it matches.11:20
Tengu|roverrlandy: I'll go for lunch in a few11:20
rlandybhagyashris|ruck I think is on holiday11:20
Tengu|roverrlandy: yep, she is11:20
rlandyTengu|rover: np - will check in with you later11:20
Tengu|rovermarios: I'll wait for the testproject result then, and that's about it.11:20
rlandyTengu|rover: ok - I'm still working on 17.111:20
Tengu|roverI'll update the hackmd accordingly.11:20
Tengu|roverrlandy: same, with the tripleo-ci patch11:21
Tengu|roverrlandy: had to update it for some stupid tabs vs space - and job is running upstream. It should be greener - and if it's ok, we'll be able to re-kick downstream testproject.11:21
Tengu|roverok, have to go - lunch is ready. Back in ~1h11:21
*** dviroel|out is now known as dviroel11:32
*** dviroel_ is now known as dviroel11:45
rlandyTengu|rover: I think I know what the issue is11:49
rlandytrying another approach11:49
Tengu|roverrlandy: back.12:06
Tengu|roverrlandy: I think the tripleo-ci patch is working now.12:06
Tengu|roverat least it still didn't crash the checks.12:06
Tengu|rovermarios: dang.. either pin isn't in, or there's something still crashing tripleo component.12:08
Tengu|roverlemme inestigate.12:08
Tengu|rover(or the pin didn't help)12:08
Tengu|rovermeh. same error.12:09
Tengu|roveramoralej: heya! just a stupid question: is the rdoinfo pin for puppet-haproxy really correct? Or is there anything to do in order to get it actually applied? I launched a job ~1h ago (so rdoinfo patch was properly merged by then), and it seems to still install the wrong puppet-haproxy version...12:13
Tengu|roverrlandy: did I do anything wrong with the way I pass the vars down?12:13
mariosTengu|rover: o/ link to logs? 12:13
Tengu|rovermarios: https://logserver.rdoproject.org/01/46701/3/check/periodic-tripleo-ci-centos-9-scenario004-standalone-tripleo-master/d58a725/logs/undercloud/var/log/extra/package-list-installed.txt.gz12:14
Tengu|roverhere we see the puppet-haproxy 6.4.1-0.20230124153846.95ca912.el9 - it's the wrong version12:14
mariosnot sure lets wait for amoralej to double check ( amoralej do we need to wait for something else i.e. https://review.rdoproject.org/r/c/rdoinfo/+/46709 should be available as soon as it merges no?)12:20
Tengu|rovermarios: ah :). basically the same question. good, I'm not alone12:21
mariosTengu|rover: yeah in fact that 95ca912 is the breaking version (one we want to avoid) https://github.com/puppetlabs/puppetlabs-haproxy/commit/95ca912f77966ed0da7a5bec9265cdacbf09da4c12:21
mariosTengu|rover: cant see something else in logs 14:21 < Tengu|rover> marios: ah :). basically the same question. good, I'm not alone12:21
Tengu|rovermarios: exactly. we want the path just before12:21
Tengu|rovermarios: 13:13 CET - at least I have it here.12:22
mariosTengu|rover: then... re-add vote there please :) https://review.opendev.org/c/openstack/puppet-tripleo/+/871807 and lets start also pushing that direction 12:24
mariosTengu|rover: can you please add info into the cix about the pin not helping us with link to logs etc12:25
Tengu|roversure.12:25
Tengu|roverdone.12:27
Tengu|roverok, Takashi's patch seems all good, jobs are running tempest, meaning they could deploy haproxy.12:32
Tengu|roverso we may just get the fix before the pin is effective/applied/whatever12:32
Tengu|roverrlandy: afaik tripleo-ci patch is good now. Lemme know if you want to sync on the 17.1 side.12:33
rlandyTengu|rover: yep - sorry was in meeting - can chat now12:33
Tengu|roverrlandy: :) so, what's your idea? why would the param not be properly passed down? :/12:34
rlandyTengu|rover: https://meet.google.com/qdu-jqkb-phe?pli=1&authuser=012:34
Tengu|roverjoining12:35
Tengu|roverrlandy: tripleo-ci-internal-jobs is up-to-date now.12:52
Tengu|roverno more merge conflict in addition.12:52
Tengu|roverwe can re-kick the internal testproject.12:52
Tengu|roverwait... what... arrg12:53
Tengu|roverNOW it's rebased. forgot the git review -d doesn't rebase....12:53
Tengu|roverrlandy: kicked the testproject.12:54
amoralejTengu|rover, marios sorry i need to force rebuild as we are reverting to previous version12:55
amoralejgimme a while12:55
Tengu|roveramoralej: ah, that's what you said yesterday - I didn't get there was something manual to do.12:55
amoralejit's done via dummit commit to the distgit12:55
Tengu|rover'k12:56
Tengu|roverrdoinfo distgit?12:56
Tengu|roveror puppet-haproxy... ?12:56
amoralejTengu|rover, https://review.rdoproject.org/r/c/puppet/puppet-haproxy-distgit/+/4672412:57
amoralejthat one12:57
Tengu|roverok!12:58
Tengu|roverah, yeah. it's really, really a dummy commit XD12:58
mariosthanks amoralej /me vaguely remembers the dummy commit now... 13:03
amoralejit's tricky, i know :)13:03
marios(and it was not even *that* long ago from 2022 https://review.rdoproject.org/r/c/openstack/openstacksdk-distgit/+/43214 )13:05
Tengu|roveramoralej: anddd... is it also needed when we remove a pin? "asking for a friend" (truth: proper fix will be nudged to gate in the following 30 minutes or so, so we'll get it before EOD)13:08
amoralejTengu|rover, nop13:09
Tengu|rovergood. so just plain revert on rdoinfo, and once it's merge, next «whatever» will rebuild latest13:09
amoralejas removing the pin is moging ahead13:09
amoralejTengu|rover, exactly13:09
Tengu|roverhmm, I could propose the revert now, with a depends-on the proper fix?13:09
amoralejTengu|rover, whichç's the fix?13:12
Tengu|roveramoralej: https://review.opendev.org/c/openstack/puppet-tripleo/+/87180713:12
Tengu|roveramoralej: https://review.rdoproject.org/r/c/rdoinfo/+/46725  revert's ready.13:15
amoraleji think that should be fine13:17
Tengu|roversince I wasn't really sure about the delay for the proper fix.... 13:17
amoralejthe only point is to make sure the new commit of tripleo is rebuilt in rdo before revert is merged, but that should be just minutes13:17
Tengu|roverin the end the pin may have been useless, but...13:17
Tengu|roveramoralej: so that's why I added the depends-on13:17
Tengu|roverI can -2 it for the time being if needed.13:18
Tengu|rovermarking as WIP for now.13:18
rlandyTengu|rover: ugh - templating13:18
amoralejthat's good, just in case13:18
Tengu|roverrlandy: uh?13:19
rlandyhttps://sf.hosted.upshift.rdu2.redhat.com/zuul/t/tripleo-ci-internal/stream/e821b6e9b4724b2d87f91d9fa721d796?logfile=console.log13:19
*** amoralej is now known as amoralej|lunch13:19
Tengu|roverrlandy: "end of stream".13:19
rlandyTengu|rover: TASK [discover-latest-image : Fetch publication page]13:19
rlandy2023-01-26 13:06:52.259054 | primary | ERROR13:19
rlandy2023-01-26 13:06:52.259485 | primary | {13:19
rlandy2023-01-26 13:06:52.259541 | primary |   "msg": "An unhandled exception occurred while templating '{{ base_url }}'13:19
Tengu|roverguess it didn't like something13:19
Tengu|roverfuuuun13:19
Tengu|roverwould be good to see what block it hit...13:20
* Tengu|rover has far, far too many tabs opened13:21
Tengu|roverrlandy: lemme do another check..13:24
rlandymarios: dviroel: Tengu|rover: I'm on a meeting  - will miss RR sync13:31
mariosTengu|rover:++13:41
mariosnp rlandy 13:41
Tengu|roverrlandy: gimme a moment to update the downstream things, I've renamed parameters to match things in a better "ansible" way.13:42
Tengu|roverrlandy: hmmm.... not sure if you kicked the recheck just too soon13:48
Tengu|rovergood13:49
Tengu|roverdviroel: fyi, the testproject for tripleo component: https://review.rdoproject.org/r/c/testproject/+/4670113:49
Tengu|rover(added to the rr hackmd)13:50
dviroelTengu|rover: ack, tks14:00
dviroelmarios: rlandy: tomorrow I will start some hours late, I have a doc appt in my morning. I should be around for or podified meeting14:02
*** dasm|off is now known as dasm14:03
dasmo/14:03
rlandyTengu|rover: passed???14:14
rlandydviroel: no worries  - will cover you14:14
rlandyhttps://sf.hosted.upshift.rdu2.redhat.com/logs/90/438890/14/check/tripleo-rhel-9-buildimage-ironic-python-agent-rhos-17.1/f147b74/logs/build.log.txt.gz14:15
Tengu|roverrlandy: yeeeahhh14:17
Tengu|rover\o/14:17
rlandyTengu|rover: looks like it took 9.1 image14:17
rlandyas base14:17
Tengu|roverdon't tell me that.14:17
*** amoralej|lunch is now known as amoralej14:37
chandankumarmarios: rlandy we are adding crc rhel/centos-8 based label in downstream zuul15:08
chandankumarin case you need for Adoption ci work15:08
mariosthanks chandankumar 15:12
chandankumarDo let me know when you start wiring up the job15:13
marioschandankumar: you mean c9 though right? "adding crc rhel/centos-8 " 15:13
marioschandankumar: ok we can talk next week about that15:13
chandankumarcrc is based on rhel/centos-815:13
chandankumarcs/rhel-9 have some issue15:13
chandankumardaniel is already on it15:13
chandankumarsure15:14
marioschandankumar: ah ok (you mean in zuul/label has issues ) ... cos my dev box is 9-stream with crc 15:14
chandankumarmarios: nah, the issue is related to openstack cloud with cs/rhel-9 crc node15:15
chandankumarit will not block your testing15:15
mariosack 15:15
* dviroel lunch15:40
chandankumarsee ya people tomorrow o/15:51
*** marios is now known as marios|out17:02
Tengu|roverOK Team - signing off. See you tomorrow17:05
*** Tengu|rover is now known as Tengu17:05
*** amoralej is now known as amoralej|off17:27
*** jpena is now known as jpena|off17:32
*** dviroel is now known as dviroel|out20:58
*** dasm is now known as dasm|off23:13
dasm|offo/23:13

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