ysandeep | reviewbot, please add in review list: https://review.opendev.org/c/openstack/tripleo-heat-templates/+/871555 | 01:29 |
---|---|---|
reviewbot | I have added your review to the Review list | 01:29 |
*** ysandeep is now known as ysandeep|afk | 02:42 | |
*** ysandeep|afk is now known as ysandeep | 04:17 | |
marios | o/ | 05:57 |
ysandeep | \o | 06:02 |
ysandeep | reviewbot, please add in reviewlist: https://code.engineering.redhat.com/gerrit/c/openstack/tripleo-ci-internal-config/+/438957 | 06:12 |
reviewbot | I have added your review to the Review list | 06:13 |
ysandeep | Tengu|rover, thanks for review yesterday, I have updated/replied to your comments ^^ | 06:14 |
*** ysandeep is now known as ysandeep|afk | 06:17 | |
*** ysandeep|afk is now known as ysandeep | 07:21 | |
Tengu|rover | hey there | 07:30 |
Tengu|rover | ysandeep: will check in a few :) | 07:30 |
*** bhagyashris_ is now known as bhagyashris|ruck | 07:34 | |
ysandeep | Tengu|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|rover | ysandeep: doing some more comments - gimme 5 minutes :) | 07:39 |
ysandeep | sure | 07:39 |
marios | \o | 07:39 |
Tengu|rover | marios: hello there | 07:41 |
Tengu|rover | ysandeep: commented out :). | 07:41 |
Tengu|rover | up to you for the rest - you get my +2 in the current state (with explanation as to "why") | 07:42 |
Tengu|rover | wow... we got a master promotion yesterday \o/ | 07:44 |
Tengu|rover | though I think I have to double-check the components, network was a bit old iirc. (master or Zed... have to double-check) | 07:44 |
ysandeep | Tengu|rover, thanks for the review.. regarding your fs39 comment - depends-on with testproject is not possible as this is a zuul trusted project | 07:45 |
Tengu|rover | ah. zed is fine with network.. | 07:45 |
Tengu|rover | ysandeep: ah, fine then. | 07:45 |
Tengu|rover | so I'm all good with your patch :). Thanks for hunting down that one | 07:47 |
Tengu|rover | DNS are usually an interesting thing :) | 07:47 |
ysandeep | Tengu|rover, Updated https://code.engineering.redhat.com/gerrit/c/openstack/tripleo-ci-internal-config/+/438957 to address marios 's comment | 08:01 |
Tengu|rover | ysandeep: sorry, a small nit I overlooked :/ | 08:04 |
Tengu|rover | my bad | 08:04 |
ysandeep | Tengu|rover, nw, thanks for the detailed review, checking how "nm_service_conf.changed " differes from "nm_service_conf is changed" differs. | 08:06 |
ysandeep | I got service_conf.changed from stackoverflow: https://stackoverflow.com/questions/57571765/restart-service-when-service-file-changes-when-using-ansible | 08:06 |
Tengu|rover | ysandeep: 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|rover | probably from within dark corner of the ansible doc. | 08:07 |
Tengu|rover | ysandeep: https://groups.google.com/g/ansible-project/c/OYNsMYSim0c/m/w68KFqSyCQAJ | 08:08 |
Tengu|rover | check Matt answer :) | 08:08 |
ysandeep | thanks reading | 08:08 |
* Tengu|rover loves his brain, but sometimes it's unsettling when things are flowing with no reasons | 08:08 | |
ysandeep | Tengu++ thanks | 08:09 |
ysandeep | I 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#31 | 08:09 |
Tengu|rover | ah, good - sorry, selective reading ^^'. That one was alone at the bottom, so I saw it | 08:15 |
ysandeep | done | 08:16 |
marios | Tengu|rover: bhagyashris|ruck: o/ joining | 08:29 |
*** ysandeep is now known as ysandeep|lunch | 08:30 | |
Tengu|rover | hmmm networking on master is lagging. | 09:01 |
Tengu|rover | pushing a testproject in order to see if the component promotes. | 09:01 |
Tengu|rover | seems to be tempest. | 09:01 |
Tengu|rover | that was the one. not Zed. master. | 09:01 |
*** ysandeep|lunch is now known as ysandeep | 10:06 | |
* marios quick run biab | 10:18 | |
bhagyashris|ruck | ykarel, need you insights on https://review.opendev.org/c/openstack/neutron/+/871640/3#message-580b83c87db19c59ea0467e9ea739266242b0b20 | 10:57 |
ykarel | bhagyashris|ruck, you mean the tox job failures? | 11:02 |
ykarel | if yes then latest PS will fix that | 11:02 |
bhagyashris|ruck | ykarel, ok thanks! and your review will also help to merge that soon :-) | 11:03 |
ykarel | bhagyashris|ruck, sure don't have +2 in stable branches but will review, will get others on it once job passes | 11:04 |
bhagyashris|ruck | ykarel, ok thanks :) | 11:04 |
*** rlandy|out is now known as rlandy | 11:14 | |
ysandeep | review time o/ | 11:16 |
ysandeep | rlandy, ^^ | 11:17 |
marios | rlandy_: ^^ | 11:17 |
*** dviroel_ is now known as dviroel | 11:21 | |
rlandy | pojadhav: you around? | 11:31 |
rlandy | can you join review time? | 11:31 |
ysandeep | rlandy, marios wohoo we passed earlier failing step for ovb train jobs :D | 11:51 |
ysandeep | https://sf.hosted.upshift.rdu2.redhat.com/zuul/t/tripleo-ci-internal/stream/d8e528716591446fa47e28158265a971?logfile=console.log | 11:51 |
marios | great ysandeep | 11:53 |
*** amoralej is now known as amoralej|lunch | 11:58 | |
pojadhav | rlandy, sorry i was stepped out | 12:09 |
*** ysandeep is now known as ysandeep|afk | 12:09 | |
rlandy | pojadhav: responded to your comments - pls see there are two sets of patches - one that changes all needed for integration and one for component/check | 12:13 |
pojadhav | rlandy, yeah.. checking the same | 12:13 |
rlandy | akahat: will be there in a sec | 12:33 |
Tengu|rover | folks, 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 |
chandankumar | Tengu|rover: yes, by using depends-on | 13:06 |
Tengu|rover | chandankumar: 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 ysandeep | 13:22 | |
*** amoralej|lunch is now known as amoralej | 13:26 | |
pojadhav | rlandy, 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-74 | 13:30 |
marios | thx pojadhav | 13:31 |
* pojadhav brb | 13:32 | |
pojadhav | chandankumar, 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-74 | 13:51 |
*** dasm|off is now known as dasm | 13:59 | |
dasm | o/ | 13:59 |
pojadhav | chandankumar, tc/ua/pm sync | 14:01 |
*** ysandeep is now known as ysandeep|dinner | 14:28 | |
*** ysandeep|dinner is now known as ysandeep | 15:27 | |
chandankumar | rlandy: 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 repo | 15:27 |
chandankumar | tomorrow | 15:28 |
rlandy | chandankumar: ack - will do after ci meeting | 15:28 |
dasm | arxcruz: what kind of monitor are you using? is it 34" or 49"? | 15:38 |
Tengu|rover | ok, so we apparently have Network + TripleO components in red on master | 15:46 |
Tengu|rover | though the TripleO seems to be linked to infra issue... need to investigate a bit more. | 15:46 |
Tengu|rover | nope. ok. | 15:47 |
arxcruz | dasm: 34" ultrawide | 16:00 |
dasm | arxcruz: can you share a link to the one you have? | 16:02 |
arxcruz | dasm: 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 |
arxcruz | i'm not finding my model anymore in the website | 16:04 |
dasm | arxcruz: ack, thx. I've been planning to update my setup. I was looking at samsung 49" or dell 34" but those are pretty expensive | 16:05 |
arxcruz | dasm: yup, this is not so expensive as the samsung and dell, but it's not cheap though | 16:06 |
arxcruz | my dream is the new lg lc something | 16:06 |
Tengu|rover | ok, 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 |
dasm | arxcruz: thanks | 16:08 |
*** dviroel is now known as dviroel|lunch | 16:08 | |
Tengu|rover | back in 5. | 16:08 |
arxcruz | dasm: 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 |
chandankumar | see ya people tomorrow o/ | 16:08 |
dasm | arxcruz: i'm not playing on monitor. for that i have steam deck or pc+tv. | 16:09 |
arxcruz | so you may find non-gaming versions that are cheaper | 16:09 |
dasm | the one you mentioned is $300 so it's not such bad deal. | 16:10 |
dasm | i've been thinking about doing either 2x 34" or for now 1x 34" + 1x24" 1080p | 16:10 |
arxcruz | yeah, it was the price i paid | 16:10 |
dasm | it's not $1.2 or more for 49" xD | 16:11 |
arxcruz | dasm: it's a ultrawide, you might not need a second monitor | 16:11 |
arxcruz | and on linux you can virtually split the monitor with xrandr | 16:11 |
Tengu|rover | rlandy, 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|rover | I don't think I ever pinned a version so far. | 16:12 |
dasm | arxcruz: currently i have 3x 24" 1080p. 1x 34" is kinda 2x 24 so less space. | 16:13 |
arxcruz | dasm: 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 |
arxcruz | so i like it | 16:15 |
dasm | thanks. i wanted to talk to someone with a real life experience. | 16:15 |
dasm | when i checked 34" in store, it looked tiny xD | 16:16 |
rlandy | Tengu|rover: tripleo or network? | 16:18 |
marios | Tengu|rover: which branch | 16:18 |
marios | Tengu|rover: eg something like that https://github.com/redhat-openstack/rdoinfo/blob/11b0f28935bfcd8c7e65963757b56665c371b5d4/tags/wallaby.yml#L773 | 16:19 |
marios | Tengu|rover: for pin | 16:19 |
Tengu|rover | so: tripleo component on master will need a pin - related to https://bugs.launchpad.net/tripleo/+bug/2003882 | 16:19 |
Tengu|rover | for 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/2003884 | 16:20 |
rlandy | Tengu|rover: rdoinfo - we can propose a patch ... amoralej will need to approve I think | 16:20 |
Tengu|rover | (at least the tripleo one was easy to spot) | 16:20 |
Tengu|rover | hmmmmm rdoinfo... that's gerrithub iirc | 16:21 |
Tengu|rover | ah, nope, rdoproject. ok | 16:21 |
Tengu|rover | marios: errr so that would be..... antelope-uc.yml? | 16:22 |
amoralej | yes, please propose the patch you need | 16:22 |
amoralej | Tengu|rover, it's review.rdoproject.org/r | 16:23 |
Tengu|rover | amoralej: since I'm targeting master, it will be in antelope.yml? | 16:23 |
Tengu|rover | since you're around :) | 16:23 |
amoralej | antelope-uc.yml | 16:23 |
Tengu|rover | ah | 16:23 |
amoralej | but which package? | 16:23 |
Tengu|rover | amoralej: puppetlabs-haproxy | 16:23 |
amoralej | that shouldn't be pinned in master | 16:23 |
amoralej | lemme check | 16:23 |
amoralej | we usually just follow master in puppet modules | 16:24 |
Tengu|rover | amoralej: yeah, well, following master is creating this: https://bugs.launchpad.net/tripleo/+bug/2003882 | 16:24 |
amoralej | ah, you want to pin ? | 16:24 |
amoralej | then yes, yuy need to send patch to rdoinfo | 16:24 |
amoralej | and we'll have to rebuild it | 16:24 |
Tengu|rover | amoralej: at least temporarly in order to make the component pass. | 16:24 |
amoralej | yeah, sure, np | 16:25 |
Tengu|rover | need to find a working hash | 16:25 |
amoralej | you can get the last promoted one, igues? | 16:25 |
Tengu|rover | ah, name is "puppet-haproxy" | 16:25 |
amoralej | yes | 16:25 |
Tengu|rover | amoralej: and.. guess it would be something like 9fbf4e6 ? | 16:26 |
Tengu|rover | yep. seems to be the commit just before the one that broke us. | 16:27 |
amoralej | Tengu|rover, yep, that's the one in last promoted-components | 16:29 |
marios | Tengu|rover: yeah antelope i think | 16:29 |
Tengu|rover | maybe we won't need to pin. In touch with Takashi, I'll update as soon as I know more. | 16:29 |
marios | Tengu|rover: not -uc (upper-constraints? but probably amoralej should confirm for us) | 16:29 |
* marios off in few | 16:30 | |
* pojadhav out | 16:34 | |
Tengu|rover | amoralej: https://review.rdoproject.org/r/c/rdoinfo/+/46709 | 16:36 |
Tengu|rover | lemme know if I did it right | 16:36 |
Tengu|rover | amoralej: why don't we see -uc for the other branches? | 16:37 |
*** ysandeep is now known as ysandeep|out | 16:37 | |
amoralej | Tengu|rover, nop | 16:37 |
amoralej | all the stable branches follow u-c | 16:37 |
amoralej | only 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 issues | 16:38 |
amoralej | for stable, we always follow u-c | 16:38 |
Tengu|rover | ah, ok. | 16:38 |
Tengu|rover | I get it, more or less. | 16:39 |
*** gthiemon1e is now known as gthiemonge | 16:40 | |
Tengu|rover | ok... time for me to sign off for the day. amoralej thanks for the pointer and vote :). | 16:41 |
Tengu|rover | I'll submit a revert as soon as there's a proper fix. | 16:41 |
*** marios is now known as marios|out | 16:43 | |
dasm | eliadcohen: o/ have a good night! | 16:43 |
*** dviroel|lunch is now known as dviroel | 17:14 | |
*** jpena is now known as jpena|off | 17:25 | |
*** amoralej is now known as amoralej|off | 17:38 | |
eliadcohen | dasm, god night! | 19:03 |
* dasm feels embarased :P | 19:05 | |
eliadcohen | :) | 19:05 |
dasm | eliadcohen: no idea how i ended up with pinging you :) it's 11am for me :) | 19:05 |
eliadcohen | that's cool, 2pm for me | 19:05 |
eliadcohen | Never too late in the day to wish someone random a good night :) That's what I always say | 19:06 |
dasm | haha | 19:06 |
dasm | eliadcohen: http://www.total-knowledge.com/~ilya/mips/ugt.html | 19:07 |
dasm | ^^ | 19:07 |
eliadcohen | dasm, thanks for this! Will assimilate! | 19:08 |
eliadcohen | Good (UGT) Night! (Need to drive home before the snow gets me) | 19:08 |
dasm | good night! :) | 19:09 |
*** dviroel is now known as dviroel|out | 20:52 | |
* dasm => offline | 21:51 | |
dasm | o/ | 21:51 |
*** dasm is now known as dasm|off | 21:51 | |
rlandy | bye | 22:23 |
*** rlandy is now known as rlandy|bbl | 23:21 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!