| opendevreview | Frikin Evgenii proposed openstack/diskimage-builder master: Add variable for check installing python3 in yum element https://review.opendev.org/c/openstack/diskimage-builder/+/856577 | 03:20 |
|---|---|---|
| *** ysandeep|out is now known as ysandeep | 05:17 | |
| *** luigi is now known as luigi-mtg | 07:05 | |
| *** jpena|off is now known as jpena | 07:20 | |
| *** ysandeep is now known as ysandeep|afk | 07:53 | |
| opendevreview | Rafal Lewandowski proposed openstack/diskimage-builder master: Added cloud-init growpart element https://review.opendev.org/c/openstack/diskimage-builder/+/855856 | 07:55 |
| *** ysandeep|afk is now known as ysandeep | 08:25 | |
| opendevreview | Rafal Lewandowski proposed openstack/diskimage-builder master: Added cloud-init growpart element https://review.opendev.org/c/openstack/diskimage-builder/+/855856 | 10:34 |
| *** rlandy|out is now known as rlandy | 10:37 | |
| *** ysandeep is now known as ysandeep|afk | 10:42 | |
| *** dviroel|afk is now known as dviroel | 11:31 | |
| opendevreview | Rafal Lewandowski proposed openstack/diskimage-builder master: Added cloud-init growpart element https://review.opendev.org/c/openstack/diskimage-builder/+/855856 | 11:41 |
| *** diablo_rojo_phone is now known as Guest2493 | 11:45 | |
| *** ysandeep|afk is now known as ysandeep | 12:04 | |
| *** dasm|off is now known as dasm | 13:31 | |
| *** iurygregory_ is now known as iurygregory | 14:04 | |
| *** jpodivin_ is now known as jpodivin | 14:44 | |
| *** dviroel is now known as dviroel|afk | 15:35 | |
| *** ysandeep is now known as ysandeep|dinner | 15:36 | |
| *** marios is now known as marios|out | 15:56 | |
| clarkb | corvus: fungi: do we know why gertty doesn't respond to the comment thread properly with fungi's latest comment on https://review.opendev.org/c/opendev/system-config/+/851248 ? | 16:01 |
| clarkb | there have been a coupel of times where I've missed fungi's responses to things because gerrit doesn't display them in the expected context (based on web ui behavior) | 16:01 |
| corvus | clarkb: the new comment system is completely different. i have a wip change that is half-done to add support for it. | 16:03 |
| corvus | i can push it up if others want to try to use it / help finish it. but it's not really usable yet. | 16:03 |
| fungi | yeah, gertty doesn't have a "reply to inline comment" feature, you just make inline comments | 16:04 |
| clarkb | ok I may need to retrain myself to fully open all the diffs to view inlien commenst then | 16:04 |
| opendevreview | James E. Blair proposed ttygroup/gertty master: WIP: support inline comment threads https://review.opendev.org/c/ttygroup/gertty/+/860588 | 16:04 |
| clarkb | I don't think it is urgent just wanted to make sure it was understood / known (sounds like it is but wip) | 16:05 |
| fungi | gertty shows my inline comment after ianw's, so to me it looks like a continuation of that discussion, but i don't know what it looks like on the webui. i'll take a look | 16:05 |
| clarkb | fungi: inside the file diff it is fine. But on the top level of the change in the comments view it is completely disconnected | 16:05 |
| corvus | that's where i'm at so far ^ -- it's really squirrely though; widgets disappear sometimes and you have to exit a screen and go back, to get them to show up, etc. definitely don't use it for anything important. | 16:05 |
| fungi | clarkb: okay, that helps. yeah i only ever view inline comments actually inline (as in looking at the diff) so i hadn't noticed there was even another place they got displayed | 16:06 |
| *** ysandeep|dinner is now known as ysandeep|out | 16:06 | |
| corvus | (i can use it to make one or two replies correctly; beyond that, the bugs are too much to deal with) | 16:06 |
| clarkb | fungi: it can really speed up processing review followups without new code to see the responses at the top leve like that | 16:07 |
| fungi | corvus: tangential, but do you know where that address originally came from/what it was? https://review.opendev.org/c/opendev/system-config/+/851248/77/inventory/service/host_vars/lists01.opendev.org.yaml#57 | 16:08 |
| fungi | it's been 10 years, so i'm not holding out hope | 16:08 |
| corvus | fungi: nope; maybe git history would suggest a clue? | 16:08 |
| fungi | the ip address now belongs to iweb/orock from what i can glean from whois | 16:08 |
| fungi | corvus: git history has it imported as part of the original server config | 16:09 |
| fungi | mordred's first change adding the config for the listserv | 16:09 |
| fungi | so i'm guessing it was entered into the config directly on the server prior to that | 16:09 |
| fungi | my best guess is that was the original server's ip address or something, and maybe rackspace has sold off the address space in the years since | 16:10 |
| clarkb | I'm going to find breakfast then I'll approve the ansible version bump | 16:10 |
| clarkb | fungi: were you going to update the mm3 change? I'm happy to do it too, but don't forget to rebase the image fork change on top of it too | 16:15 |
| fungi | i was going to ask if you or ianw were working on addressing those comments before doing so myself, but happy to | 16:16 |
| fungi | just don't want to duplicate work | 16:16 |
| clarkb | heh ya it becomes confusing when we all end up hacking on the thing. I'll let you do it since you arleady started responding to ianw | 16:17 |
| clarkb | But let me know if I can help | 16:17 |
| fungi | you bet | 16:17 |
| clarkb | I've double checked that the opendev service announce thread, fungi's heads up to openstack and the chagne itself for the ansible 6 bump have not elicited any concerns. I'm approve the change now as a result | 16:20 |
| fungi | sounds good, thanks! | 16:21 |
| clarkb | my typing and english have been poor today too. Apologies | 16:21 |
| fungi | no worse than mine | 16:24 |
| *** soniya29|ruck is now known as soniya29 | 16:30 | |
| opendevreview | Merged openstack/project-config master: Update OpenDev tenants' default ansible to 6 https://review.opendev.org/c/openstack/project-config/+/860474 | 16:31 |
| clarkb | and now we wait for that to deploy | 16:32 |
| clarkb | the ansible default version update should deploy in the next few minutes | 16:46 |
| clarkb | I'm watching 3 jobs run with ansible 6 after the default update | 17:00 |
| clarkb | and 2/3 of have succeeded. Last one is still running. I'll try to keep my eyes open for anything unexpected | 17:03 |
| fungi | sounds great so far! | 17:03 |
| *** dviroel|afk is now known as dviroel | 17:09 | |
| clarkb | my go-enry fix for finding "vendored" files in gitea landed https://github.com/go-enry/go-enry/pull/136. Hopefully that means after a gitea upgrade or two we'll be fixed | 17:23 |
| fungi | awesome, thanks! | 17:23 |
| clarkb | actually looks like they already made a go-enry 2.8.3 release. Let me see if I can work out an update to gitea to pull that in | 17:24 |
| clarkb | https://github.com/go-gitea/gitea/pull/21360 | 17:40 |
| *** jpena is now known as jpena|off | 18:20 | |
| *** dasm is now known as dasm|off | 18:22 | |
| clarkb | infra-root https://review.opendev.org/c/openstack/project-config/+/856326 should be ready to go now | 19:49 |
| clarkb | thats key rotation now that openstack releas is done | 19:49 |
| clarkb | also I hvaen't seen any failures or errors that appear related to ansible 6 | 19:51 |
| *** dviroel is now known as dviroel|afk | 20:32 | |
| *** dasm|off is now known as dasm | 20:49 | |
| clarkb | looks like gitea landed my go enry update. I haev no idea if that will get backported to 1.17 but should be in 1.18 at least | 21:29 |
| *** rlandy is now known as rlandy|bbl | 21:58 | |
| *** Guest2493 is now known as diablo_rojo_phone | 22:04 | |
| *** dasm is now known as dasm|off | 22:15 | |
| *** rlandy|bbl is now known as rlandy|out | 22:15 | |
| opendevreview | Merged opendev/system-config master: Update our Gerrit images https://review.opendev.org/c/opendev/system-config/+/859885 | 23:27 |
| clarkb | ianw ^ fyi. | 23:33 |
| ianw | ++ | 23:33 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!