Wednesday, 2023-01-25

ysandeepreviewbot, please add in review list: https://review.opendev.org/c/openstack/tripleo-heat-templates/+/87155501:29
reviewbotI have added your review to the Review list01:29
*** ysandeep is now known as ysandeep|afk02:42
*** ysandeep|afk is now known as ysandeep04:17
marioso/05:57
ysandeep\o06:02
ysandeepreviewbot, please add in reviewlist: https://code.engineering.redhat.com/gerrit/c/openstack/tripleo-ci-internal-config/+/438957 06:12
reviewbotI have added your review to the Review list06:13
ysandeepTengu|rover, thanks for review yesterday, I have updated/replied to your comments ^^06:14
*** ysandeep is now known as ysandeep|afk06:17
*** ysandeep|afk is now known as ysandeep07:21
Tengu|roverhey there07:30
Tengu|roverysandeep: will check in a few :)07:30
*** bhagyashris_ is now known as bhagyashris|ruck07:34
ysandeepTengu|rover, good morning o/ no hurry.. I need to fix comment from marios to only check unbound for C8 case which sound reasonable as we only have unbound installed in C8 images.07:37
Tengu|roverysandeep: doing some more comments - gimme 5 minutes :)07:39
ysandeepsure07:39
marios\o07:39
Tengu|rovermarios: hello there07:41
Tengu|roverysandeep: commented out :).07:41
Tengu|roverup to you for the rest - you get my +2 in the current state (with explanation as to "why")07:42
Tengu|roverwow... we got a master promotion yesterday \o/07:44
Tengu|roverthough I think I have to double-check the components, network was a bit old iirc. (master or Zed... have to double-check)07:44
ysandeepTengu|rover, thanks for the review.. regarding your fs39 comment - depends-on with testproject is not possible as this is a zuul trusted project07:45
Tengu|roverah. zed is fine with network..07:45
Tengu|roverysandeep: ah, fine then. 07:45
Tengu|roverso I'm all good with your patch :). Thanks for hunting down that one07:47
Tengu|roverDNS are usually an interesting thing :)07:47
ysandeepTengu|rover, Updated https://code.engineering.redhat.com/gerrit/c/openstack/tripleo-ci-internal-config/+/438957 to address marios 's comment08:01
Tengu|roverysandeep: sorry, a small nit I overlooked :/08:04
Tengu|rovermy bad08:04
ysandeepTengu|rover, nw, thanks for the detailed review, checking how "nm_service_conf.changed " differes from "nm_service_conf is changed" differs.08:06
ysandeepI got service_conf.changed from stackoverflow: https://stackoverflow.com/questions/57571765/restart-service-when-service-file-changes-when-using-ansible08:06
Tengu|roverysandeep: yeah, I have a trigger in my brain that tells "is changed is better" but I don't remember where I've read it, unfortunately.08:07
Tengu|roverprobably from within dark corner of the ansible doc.08:07
Tengu|roverysandeep: https://groups.google.com/g/ansible-project/c/OYNsMYSim0c/m/w68KFqSyCQAJ08:08
Tengu|rovercheck Matt answer :)08:08
ysandeepthanks reading08:08
* Tengu|rover loves his brain, but sometimes it's unsettling when things are flowing with no reasons08:08
ysandeepTengu++ thanks08:09
ysandeepI have one more .changed, will replace that as well: https://code.engineering.redhat.com/gerrit/c/openstack/tripleo-ci-internal-config/+/438957/3/playbooks/tripleo-rdo-base/configure-nameserver.yaml#3108:09
Tengu|roverah, good - sorry, selective reading ^^'. That one was alone at the bottom, so I saw it08:15
ysandeepdone08:16
mariosTengu|rover: bhagyashris|ruck: o/ joining08:29
*** ysandeep is now known as ysandeep|lunch08:30
Tengu|roverhmmm networking on master is lagging.09:01
Tengu|roverpushing a testproject in order to see if the component promotes.09:01
Tengu|roverseems to be tempest.09:01
Tengu|roverthat was the one. not Zed. master.09:01
*** ysandeep|lunch is now known as ysandeep10:06
* marios quick run biab10:18
bhagyashris|ruckykarel, need you insights on https://review.opendev.org/c/openstack/neutron/+/871640/3#message-580b83c87db19c59ea0467e9ea739266242b0b2010:57
ykarelbhagyashris|ruck, you mean the tox job failures?11:02
ykarelif yes then latest PS will fix that11:02
bhagyashris|ruckykarel, ok thanks! and your review will also help to merge that soon  :-)11:03
ykarelbhagyashris|ruck, sure don't have +2 in stable branches but will review, will get others on it once job passes11:04
bhagyashris|ruckykarel, ok thanks :)11:04
*** rlandy|out is now known as rlandy11:14
ysandeepreview time o/11:16
ysandeeprlandy, ^^11:17
mariosrlandy_: ^^ 11:17
*** dviroel_ is now known as dviroel11:21
rlandypojadhav: you around?11:31
rlandycan you join review time?11:31
ysandeeprlandy, marios wohoo we passed earlier failing step for ovb train jobs :D11:51
ysandeephttps://sf.hosted.upshift.rdu2.redhat.com/zuul/t/tripleo-ci-internal/stream/d8e528716591446fa47e28158265a971?logfile=console.log11:51
mariosgreat ysandeep 11:53
*** amoralej is now known as amoralej|lunch11:58
pojadhavrlandy, sorry i was stepped out12:09
*** ysandeep is now known as ysandeep|afk12:09
rlandypojadhav: responded to your comments - pls see there are two sets of patches - one that changes all needed for integration and one for component/check12:13
pojadhavrlandy, yeah.. checking the same12:13
rlandyakahat: will be there in a sec12:33
Tengu|roverfolks, is there a way to test that patch downstream? https://review.opendev.org/c/openstack/tripleo-ci/+/871712 or, maybe, upstream using, say, latest cs9 image or something?13:00
chandankumarTengu|rover: yes, by using depends-on13:06
Tengu|roverchandankumar: Ronelle just told me downstream can depends-on that repo. So I'll do some testing then :).13:07
*** ysandeep|afk is now known as ysandeep13:22
*** amoralej|lunch is now known as amoralej13:26
pojadhavrlandy, marios, dasm|off, frenzy_friday : please add agenda for today's TC/UA/PM sync up https://hackmd.io/0ruKGb6eTwGvdvwWE4nk8g#2023-01-25-TCUAPM-planning-for-sprint-7413:30
mariosthx pojadhav 13:31
* pojadhav brb13:32
pojadhavchandankumar, sorry missed to ping you... feel free to add if any new topics : please add agenda for today's TC/UA/PM sync up https://hackmd.io/0ruKGb6eTwGvdvwWE4nk8g#2023-01-25-TCUAPM-planning-for-sprint-7413:51
*** dasm|off is now known as dasm13:59
dasmo/13:59
pojadhavchandankumar, tc/ua/pm sync14:01
*** ysandeep is now known as ysandeep|dinner14:28
*** ysandeep|dinner is now known as ysandeep15:27
chandankumarrlandy: dviroel when you get some time please review this tp https://review.rdoproject.org/r/c/testproject/+/46569 I will move this into rdo-jobs repo15:27
chandankumartomorrow15:28
rlandychandankumar: ack - will do after ci meeting15:28
dasmarxcruz: what kind of monitor are you using? is it 34" or 49"?15:38
Tengu|roverok, so we apparently have Network + TripleO components in red on master15:46
Tengu|roverthough the TripleO seems to be linked to infra issue... need to investigate a bit more.15:46
Tengu|rovernope. ok.15:47
arxcruzdasm: 34" ultrawide 16:00
dasmarxcruz: can you share a link to the one you have?16:02
arxcruzdasm: https://us.aoc.com/en-US/gaming/products/monitors/cu34g2x is the closest one, mine is 100hz and have pip to use two computers and split the monitor in 2 16:04
arxcruzi'm not finding my model anymore in the website16:04
dasmarxcruz: ack, thx. I've been planning to update my setup. I was looking at samsung 49" or dell 34" but those are pretty expensive16:05
arxcruzdasm: yup, this is not so expensive as the samsung and dell, but it's not cheap though 16:06
arxcruzmy dream is the new lg lc something 16:06
Tengu|roverok, so both network and tripleo components are hosed on master. network is a bit weird - tripleo, we have a potential culprit (that would lead to a pinning)16:08
dasmarxcruz: thanks16:08
*** dviroel is now known as dviroel|lunch16:08
Tengu|roverback in 5.16:08
arxcruzdasm: it doesn't have sound though, and it's a gamer monitor, if you're not into the fast response, you can get cheaper ones 16:08
chandankumarsee ya people tomorrow o/16:08
dasmarxcruz: i'm not playing on monitor. for that i have steam deck or pc+tv.16:09
arxcruzso you may find non-gaming versions that are cheaper 16:09
dasmthe one you mentioned is $300 so it's not such bad deal.16:10
dasmi've been thinking about doing either 2x 34" or for now 1x 34" + 1x24" 1080p16:10
arxcruzyeah, it was the price i paid 16:10
dasmit's not $1.2 or more for 49" xD16:11
arxcruzdasm: it's a ultrawide, you might not need a second monitor 16:11
arxcruzand on linux you can virtually split the monitor with xrandr 16:11
Tengu|roverrlandy, marios not sure what has to be done, but apparently we'll need to pin puppetlabs-haproxy version in order to unblock TripleO component promotion.16:12
Tengu|roverI don't think I ever pinned a version so far.16:12
dasmarxcruz: currently i have 3x 24" 1080p. 1x 34" is kinda 2x 24 so less space.16:13
arxcruzdasm: i think 4k is too much for my eyes, as i'm getting old, the 1440p is not so small but not so big 16:15
arxcruzso i like it 16:15
dasmthanks. i wanted to talk to someone with a real life experience.16:15
dasmwhen i checked 34" in store, it looked tiny xD16:16
rlandyTengu|rover: tripleo or network?16:18
mariosTengu|rover: which branch 16:18
mariosTengu|rover: eg something like that https://github.com/redhat-openstack/rdoinfo/blob/11b0f28935bfcd8c7e65963757b56665c371b5d4/tags/wallaby.yml#L773 16:19
mariosTengu|rover: for pin16:19
Tengu|roverso: tripleo component on master will need a pin - related to https://bugs.launchpad.net/tripleo/+bug/200388216:19
Tengu|roverfor network component, I'm not sure about the root cause yet - I just created the LP, but I don't have any data yet: https://bugs.launchpad.net/tripleo/+bug/200388416:20
rlandyTengu|rover: rdoinfo - we can propose a patch ... amoralej will need to approve I think16:20
Tengu|rover(at least the tripleo one was easy to spot)16:20
Tengu|roverhmmmmm rdoinfo... that's gerrithub iirc16:21
Tengu|roverah, nope, rdoproject. ok16:21
Tengu|rovermarios: errr so that would be..... antelope-uc.yml?16:22
amoralejyes, please propose the patch you need16:22
amoralejTengu|rover, it's review.rdoproject.org/r16:23
Tengu|roveramoralej: since I'm targeting master, it will be in antelope.yml?16:23
Tengu|roversince you're around :)16:23
amoralejantelope-uc.yml16:23
Tengu|roverah16:23
amoralejbut which package?16:23
Tengu|roveramoralej: puppetlabs-haproxy16:23
amoralejthat shouldn't be pinned in master16:23
amoralejlemme check16:23
amoralejwe usually just follow master in puppet modules16:24
Tengu|roveramoralej: yeah, well, following master is creating this: https://bugs.launchpad.net/tripleo/+bug/200388216:24
amoralejah, you want to pin ?16:24
amoralejthen yes, yuy need to send patch to rdoinfo16:24
amoralejand we'll have to rebuild it16:24
Tengu|roveramoralej: at least temporarly in order to make the component pass.16:24
amoralejyeah, sure, np16:25
Tengu|roverneed to find a working hash16:25
amoralejyou can get the last promoted one, igues?16:25
Tengu|roverah, name is "puppet-haproxy"16:25
amoralejyes16:25
Tengu|roveramoralej: and.. guess it would be something like 9fbf4e6 ?16:26
Tengu|roveryep. seems to be the commit just before the one that broke us.16:27
amoralejTengu|rover, yep, that's the one in last promoted-components16:29
mariosTengu|rover: yeah antelope i think16:29
Tengu|rovermaybe we won't need to pin. In touch with Takashi, I'll update as soon as I know more.16:29
mariosTengu|rover: not -uc (upper-constraints? but probably amoralej should confirm for us)16:29
* marios off in few16:30
* pojadhav out16:34
Tengu|roveramoralej: https://review.rdoproject.org/r/c/rdoinfo/+/4670916:36
Tengu|roverlemme know if I did it right16:36
Tengu|roveramoralej: why don't we see -uc for the other branches?16:37
*** ysandeep is now known as ysandeep|out16:37
amoralejTengu|rover, nop16:37
amoralejall the stable branches follow u-c16:37
amoralejonly in master we have two builders, one following u-c (which is the one we use) and one that follows master branch for everything which we use only to anticipate build issues16:38
amoralejfor stable, we always follow u-c16:38
Tengu|roverah, ok.16:38
Tengu|roverI get it, more or less.16:39
*** gthiemon1e is now known as gthiemonge16:40
Tengu|roverok... time for me to sign off for the day. amoralej thanks for the pointer and vote :).16:41
Tengu|roverI'll submit a revert as soon as there's a proper fix.16:41
*** marios is now known as marios|out16:43
dasmeliadcohen: o/ have a good night!16:43
*** dviroel|lunch is now known as dviroel17:14
*** jpena is now known as jpena|off17:25
*** amoralej is now known as amoralej|off17:38
eliadcohendasm, god night!19:03
* dasm feels embarased :P19:05
eliadcohen:)19:05
dasmeliadcohen: no idea how i ended up with pinging you :) it's 11am for me :)19:05
eliadcohenthat's cool, 2pm for me19:05
eliadcohenNever too late in the day to wish someone random a good night :) That's what I always say19:06
dasmhaha19:06
dasmeliadcohen: http://www.total-knowledge.com/~ilya/mips/ugt.html19:07
dasm^^19:07
eliadcohendasm, thanks for this! Will assimilate!19:08
eliadcohenGood (UGT) Night! (Need to drive home before the snow gets me)19:08
dasmgood night! :)19:09
*** dviroel is now known as dviroel|out20:52
* dasm => offline21:51
dasmo/21:51
*** dasm is now known as dasm|off21:51
rlandybye22:23
*** rlandy is now known as rlandy|bbl23:21

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