Thursday, 2022-06-30

fungii wouldn't mind greater retention anyway00:34
opendevreviewMerged opendev/grafyaml master: Strip id/uid from .json input  https://review.opendev.org/c/opendev/grafyaml/+/83321200:35
opendevreviewMerged opendev/grafyaml master: Use fixed datasource UID  https://review.opendev.org/c/opendev/grafyaml/+/84807000:35
*** rlandy|bbl is now known as rlandy01:02
*** rlandy is now known as rlandy|out01:07
opendevreviewIan Wienand proposed openstack/project-config master: grafana: Add infra deployment overview  https://review.opendev.org/c/openstack/project-config/+/83321303:48
opendevreviewIan Wienand proposed openstack/project-config master: grafana: add helper script  https://review.opendev.org/c/openstack/project-config/+/83321403:49
*** ysandeep|PTO is now known as ysandeep03:50
*** chandankumar is now known as raukadah04:03
opendevreviewIan Wienand proposed openstack/project-config master: grafana: Add infra deployment overview  https://review.opendev.org/c/openstack/project-config/+/83321304:37
opendevreviewIan Wienand proposed openstack/project-config master: grafana: add helper script  https://review.opendev.org/c/openstack/project-config/+/83321404:37
opendevreviewIan Wienand proposed openstack/project-config master: [wip] update nodepool dib graph  https://review.opendev.org/c/openstack/project-config/+/84821204:37
opendevreviewIan Wienand proposed openstack/project-config master: [wip] update nodepool dib graph  https://review.opendev.org/c/openstack/project-config/+/84821204:51
ianwfrickler: re small discussion the other day : http://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_a36/848212/2/check/project-config-grafana/a36c3ac/screenshots/nodepool-dib-status.png05:11
ianwi remember now -- it's mapping the return code of dib.  so *usually* that's 1 on error, but can range 0->128.  if it flip-flops it can get averaged into a "weird" number05:12
ianwbut it makes for a fun heat-graph map.  basically anywhere there's colour is a failed build, the shade giving you a vague feeling about the return code05:13
ianw"fun" if you like looking at where it's broken, anyway :)05:14
fricklerianw: I like that, certainly gives a quick view of how much brokenness there is/was05:26
fricklerclarkb: yes, I will not be at the PTG. it's sad that the foundation couldn't be convinced to keep it virtual. couple of teams will continue to do that anyway, maybe we could, too?05:36
opendevreviewIan Wienand proposed openstack/project-config master: grafana: update nodepool-dib status page  https://review.opendev.org/c/openstack/project-config/+/84821205:53
ianwfrickler: if you feel like fiddling with it, you can pull 848212 then run ./grafana/run-grafana.sh and it will appear on :3000.  login admin/password and you can edit it, then the little "share" button up the top to save as json05:58
ianwwould appreciate any feedback on that workflow, if you find it easy, etc.05:59
*** akahat is now known as akahat|ruck06:47
*** ysandeep is now known as ysandeep|afk06:49
*** jpena|off is now known as jpena07:10
*** akahat is now known as akahat|ruck10:08
*** bhagyashris_ is now known as bhagyashris10:25
*** rlandy|out is now known as rlandy10:39
*** ysandeep|afk is now known as ysandeep10:50
*** dviroel_ is now known as dviroel11:20
*** marios is now known as marios|afk11:24
*** marios|afk is now known as marios12:00
*** dasm|off is now known as dasm13:03
opendevreviewJulia Kreger proposed openstack/diskimage-builder master: DNM: Network Manager logging to Trace for Debugging  https://review.opendev.org/c/openstack/diskimage-builder/+/84760013:14
clarkbfrickler: yes doing something virtually is an option. Considering we've skipped the last couple and how the ones prior to that went it does feel a bit like we're going to be working from scratch so no real pre planned stuff14:46
clarkbconsidering fungi and I are likely to be at the physical event we should probably do any virtual stuff offset from that (its just easier to do than when traveling)14:46
opendevreviewMatthew Thode proposed openstack/diskimage-builder master: update default python for gentoo to 3.10  https://review.opendev.org/c/openstack/diskimage-builder/+/84830714:50
fungiyes, i wouldn't want to try to do a virtual collaboration during the week of the in-person ptg, since it's not necessary to schedule them into the same week anyway14:57
*** ysandeep is now known as ysandeep|out14:59
clarkbthinking on this a bit more we've skipped previous virtual ptg in part because if we need to schedule time to discuss things I don't feel we need an event for that. I'm happy to schedule something like that and call it a ptg or do it whenever. Basically calling out that we shouldn't feel like a PTG is necessary to have that sort of interaction. If you'd like to haev a more in depth15:28
clarkb"face to face" virtual meeting on a topic let us know and I'm sure we can schedule something15:28
*** marios is now known as marios|out15:50
clarkbfungi: do you have time for https://review.opendev.org/c/openstack/project-config/+/848033 and child today? If so I'm in a good place to pay attention to that and make sure it doesnt' make zuul sad16:27
fungiyep! approved16:28
clarkbthanks16:28
*** jpena is now known as jpena|off16:28
opendevreviewMerged openstack/project-config master: Remove windmill from zuul tenant config  https://review.opendev.org/c/openstack/project-config/+/84803316:41
opendevreviewJulia Kreger proposed openstack/diskimage-builder master: DNM: Network Manager logging to Trace for Debugging  https://review.opendev.org/c/openstack/diskimage-builder/+/84760016:50
clarkbok windmill removal created one new openstack tenant error. Nothing major, but looking into the best way to address that now16:55
clarkbLooks like x/ansible-role-shade should also be removed from the tenant as an idle project related to windmill. Also shade doesn't exist anymore16:56
opendevreviewClark Boylan proposed openstack/project-config master: Remove x/ansible-role-shade from Zuul tenant config  https://review.opendev.org/c/openstack/project-config/+/84831817:01
clarkbfungi: ^ that should fix it (also its parent is good to go too)17:01
opendevreviewJeremy Stanley proposed opendev/system-config master: Redirect all Mailman sites from HTTP to HTTPS  https://review.opendev.org/c/opendev/system-config/+/84831917:07
clarkbfungi: ^ that change lgtm. One thing I wondered is if you wanted to do it only for the non multisite setup first as that will be easier to test as you don't have the weird runtime rules for multisite to contend with17:18
clarkbbut I suspect its fine either way since it seems existing lists don't change until you manually modify them17:19
fungiyeah, the testing will be the same either way, and reverting if it goes wrong for unanticipated reasons17:20
fungii've already manually tested the redirect part on the lists.openinfra.dev site just for my own edification, so it's more to do with the mailman config17:21
clarkbfungi: did you want to review https://review.opendev.org/c/openstack/project-config/+/848034 too or should i just approve that one?17:21
fungii approved it a couple minutes ago17:21
clarkboh my refresh must've happened just before17:23
clarkbthanks17:23
funginp17:23
clarkband you've reviewed the gitea testing refactor. Have you had a chance to look at the 1.17 WIP? Its kind of amazing how much stuff they've gotten into this release. For that reason I'm starting to think we should upgrade after .1 comes out unless there is an urgent reason before17:24
fungii did not get that far yet, no sorry17:25
clarkbno rush. I think its more interesting for others to see what is coming rather than in depth review at this point17:26
opendevreviewMerged openstack/project-config master: Remove x/neutron-classifier from Zuul tenant config  https://review.opendev.org/c/openstack/project-config/+/84803417:29
opendevreviewMerged openstack/project-config master: Remove x/ansible-role-shade from Zuul tenant config  https://review.opendev.org/c/openstack/project-config/+/84831817:29
clarkbok that seems to have cleared up the new error I created and no additional errors have fallen out17:55
clarkb#status log removed windmill/*, x/ansible-role-shade, and x/neutron-classifier from Zuul openstack tenant config.17:55
opendevstatusclarkb: finished logging17:56
fungithanks!17:59
opendevreviewMerged openstack/diskimage-builder master: Use internal dhcp client for centos 9-stream and beyond  https://review.opendev.org/c/openstack/diskimage-builder/+/84801718:20
*** dasm is now known as dasm|off20:42
*** dviroel is now known as dviroel|afk20:42
opendevreviewThiago Paiva Brito proposed zuul/zuul-jobs master: Remove zuul_work_dir from role  https://review.opendev.org/c/zuul/zuul-jobs/+/84833420:44
clarkbwe are down from 98 to 83 repos with deprecated zuul queue syntax. progress20:55
opendevreviewThiago Paiva Brito proposed zuul/zuul-jobs master: Remove zuul_work_dir from role  https://review.opendev.org/c/zuul/zuul-jobs/+/84833421:00
opendevreviewThiago Paiva Brito proposed zuul/zuul-jobs master: Adding git commit message guidelines check  https://review.opendev.org/c/zuul/zuul-jobs/+/84833922:08
*** rlandy is now known as rlandy|bbl22:29

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