Tuesday, 2023-11-28

clarkbhttps://etherpad.opendev.org/p/2023-opendev-annual-report Starting to draft the opendev annual report here. Or trying to. No rael content yet. Feel free to add ideas if you have them00:28
opendevreviewTakashi Kajinami proposed openstack/project-config master: Move os-(apply|collect|refresh)-config to heat  https://review.opendev.org/c/openstack/project-config/+/90191601:06
* yoctozepto can't wait to get this approved https://review.opendev.org/c/openstack/project-config/+/901591 (please consider)08:00
fungiclarkb: when you're around, you potentially had questions about that one? ^ otherwise i would have approved it last week13:27
opendevreviewMerged openstack/project-config master: Move os-(apply|collect|refresh)-config to heat  https://review.opendev.org/c/openstack/project-config/+/90191613:49
Clark[m]yoctozepto: fungi: my concern is over naming confusion. The repo isn't actually an influxdb repo right? Should it be called something else?14:45
fungiClark[m]: any worse than https://opendev.org/nebulous/activemq ?15:01
fungipresumably this will be the same sort of thing (collection of helm charts and dockerfiles), i.e. "packaging" for use of influxdb in the nebulous platform15:02
fungibut hopefully yoctozepto can confirm15:03
Clark[m]I think I have the same concerns with that repo. At best it is confusing.15:07
fungii guess it doesn't seem confusing to me in the context of the namespace, any more than https://salsa.debian.org/kernel-team/linux doesn't make me expect it will contain a fork of the linux kernel15:17
fungiit's common for distributions to name the git repositories of their packaging files after the software being packaged15:18
fungijust like the https://src.fedoraproject.org/rpms/weechat repo isn't a fork of weechat15:24
Clark[m]Fwiw I had similar concerns with the openstack Debian packaging being confusing because it was a fork. I don't intend on -1'ing the change I just wanted to call it out and it others have considered it and don't find it problematic I think you can proceed15:32
Clark[m]Mostly I didn't want it to land over a big holiday weekend when no one was looking15:32
fungiokay, thanks15:33
opendevreviewMerged openstack/project-config master: Add NebulOuS InfluxDB repo  https://review.opendev.org/c/openstack/project-config/+/90159115:54
yoctozeptoyeah, I thought that namespacing makes it clear16:21
yoctozeptothanks for approving16:21
opendevreviewClark Boylan proposed opendev/system-config master: Upgrade gitea to 1.20.6  https://review.opendev.org/c/opendev/system-config/+/90209416:24
clarkbinfra-root ^ we should probably try to get that in relatively soon16:24
fungilooks safe enough based on the changelog16:27
fungimaybe we can merge it after the opendev meeting?16:27
clarkbthat works for me though I'll probably be afk for a bit after the meeting16:29
fricklerhmm, if someone rebases someone else's patch, gerritbot used to mention the name of the rebaser, now it mentions the name of the original author, which didn't actually perform any action, that's confusing. see https://review.opendev.org/c/openstack/designate/+/901871 and the recent msg in #openstack-dns16:31
fungilooks like it probably started around the time gerrit began making rebases "on behalf of"16:32
clarkball of that info is from the ssh stream events events16:32
clarkbit is possible the info you want it still available but we need to read it out of a different json field now16:33
fungijohnsom: did you use the "rebase" button in the webui for that, or push a local rebase?16:33
johnsomfungi The button16:33
fungii wonder if the event contents differ based on which way it's done16:33
fungithanks, that was my suspicion16:33
fungiand yeah, we can recreate different scenarios in opendev/sandbox while recording the ssh event stream to see what fields get populated with which values16:34
opendevreviewClark Boylan proposed opendev/system-config master: Update Gerrit 3.8 images to 3.8.3  https://review.opendev.org/c/opendev/system-config/+/90199216:41
opendevreviewClark Boylan proposed opendev/system-config master: Add gerrit 3.9 image builds  https://review.opendev.org/c/opendev/system-config/+/90146816:41
opendevreviewClark Boylan proposed opendev/system-config master: Add gerrit 3.8 to 3.9 upgrade testing  https://review.opendev.org/c/opendev/system-config/+/90146916:41
clarkbI've put the 3.8.3 change first in the list so that we aren't held back by 3.9 weirdness. Then I updated our 3.9 chnage to depends on upstream fixes partiailly in order to test that those fixes work in order to provide feedback upstream16:41
tonyb+2 on merging 902094 around the meeting today16:47
opendevreviewTony Breeds proposed opendev/system-config master: Add inventory/LE records for mirror02.bhs1.ovh and mirror03.gra1.ovh  https://review.opendev.org/c/opendev/system-config/+/90162817:09
opendevreviewTony Breeds proposed opendev/system-config master: Add inventory/LE records for mirror02.dfw.rax  https://review.opendev.org/c/opendev/system-config/+/90200817:09
opendevreviewTony Breeds proposed opendev/system-config master: Add a helper script for doing the LVM setup on mirror nodes.  https://review.opendev.org/c/opendev/system-config/+/90150417:09
opendevreviewTony Breeds proposed opendev/system-config master: [testinfra] Update Ansible canary.  https://review.opendev.org/c/opendev/system-config/+/90209817:09
opendevreviewTony Breeds proposed opendev/zone-opendev.org master: Switch CNAME records to new mirrors  https://review.opendev.org/c/opendev/zone-opendev.org/+/90210017:42
clarkbtonyb: I'm tempted to remove the testing for ansible galaxy entirely... I don't think it works at all on our mirror proxies due to them making massive changes anyway17:43
tonybclarkb: Sure that works too.  If you push that change I'll rebase on top of it.17:45
clarkbtonyb: I think we can do it in the other order. I +2'd your change17:45
tonybclarkb: I can do it but the commit message would be more meaningful if you do it17:45
tonybMine would be somethign like "Because Clark said so ;p"17:46
clarkbbasically lets not worry about that until after the mirror stuff is done17:46
tonybclarkb: Okay that works for me17:46
clarkbbut as a followup we can remove the test entirely17:46
opendevreviewMerged opendev/zone-opendev.org master: Add DNS records for mirror02.dfw.rax  https://review.opendev.org/c/opendev/zone-opendev.org/+/90200718:06
opendevreviewElod Illes proposed openstack/project-config master: [check-release-approval] Fix distributed leadership handling  https://review.opendev.org/c/openstack/project-config/+/89678518:16
clarkbthe gitea 1.20.6 job failed in post due to wht I think is an error in the local registry. I rechecekd it18:39
clarkbif it persists we may have some debugging to do :?18:39
fungiclarkb: yeah, i had already rechecked it about 1.5 hours earlier18:44
clarkboh I missed that first one. was it the same issue?18:44
fungiit reported success about 30 seconds after your recheck comment18:44
clarkboh bah18:44
clarkbI'm just bad at reading comments18:45
opendevreviewClark Boylan proposed opendev/system-config master: Add gerrit 3.8 to 3.9 upgrade testing  https://review.opendev.org/c/opendev/system-config/+/90146919:40
opendevreviewClark Boylan proposed opendev/system-config master: Add gerrit 3.9 image builds  https://review.opendev.org/c/opendev/system-config/+/90210919:40
JayFHmm. New Gerrit appears to potentially have broken TC tooling, it seems, I think? 404 to https://review.opendev.org/accounts/james.page@ubuntu.com with "Account 'james.page@ubuntu.com' not found" -- either that or some metadata changed on the backend20:05
JayFjamespage: did you change your gerrit email, perhaps?20:05
JayFI guess you all didn't push new gerrit, I wonder what's up20:05
JayFhmm, looks like email behind it changed and maybe some TC metadata needs updating20:06
clarkbJayF: the apis have gotten more restrictive around user lookups20:06
JayFwell, I'm seeing in https://review.opendev.org/admin/groups/c88faeb0c52f35ee530be8defcc9627ea87b28df,members that another email appears for James20:06
JayFand that the query works for my primary email20:06
clarkbbut I think if you are logged in / doing authenticated requests you should still see that info20:06
clarkbJayF: the gerrit 3.8 release notes had more info20:07
JayFso I think this might just be a change in email account20:07
clarkbhttps://www.gerritcodereview.com/3.8.html ^F secondary20:07
opendevreviewBrian Rosmaita proposed openstack/project-config master: Implement openstack-unmaintained-core group  https://review.opendev.org/c/openstack/project-config/+/90211020:07
clarkbits possible that James' primary email address changed long ago but it wasn't until the 3.8 upgrade that you stopped being able to query by the secondary address20:07
JayFI don't know which is the likely change, but I can confirm updating the email fixes it20:08
JayFthanks for quacking with me at this20:08
JayFI'll push the update to governance20:08
fungijust noticed in debian/unstable upgrade notes, ssh-keygen will now start generating elliptic curve keys as the default instead of rsa20:47
fungiberstein flavor, not nsa20:48
fungier, bernstein20:48
clarkbgerrit 3.9.0-rc6 works according to our testing. I've reported that info back upstream20:57
clarkbfungi: interesting openssh 9.3p2-3.1 on tumbleweed is still -t rsa by default20:58
fungithis is 9.5p121:05
tonybJayF: So I think jamespage *may* have altered his account.  https://opendev.org/openstack/governance/src/branch/master/reference/members.yaml#L50-L55 gives us his gerrit account memberid but that doesn't seem to be there.21:13
tonybJayF: see: https://paste.opendev.org/show/bXSw2xaqd5A4Wy57Xvfb/21:13
JayFChanging the email seems to have fixed it, and someone else on the TC noted that the email was the updated one in his PTL record21:13
JayFso I think it was just outdated metadata in the TC members yaml21:13
tonybJayF: Oh okay.  I guess you can update the memberid to the new one.21:14
JayFhttps://review.opendev.org/c/openstack/governance/+/902111 :)21:14
tonybAnd just to correct my misunderstanding, memberid in members.yaml is the OIF memberid NOT the gerrit account_id sorry for the noise21:25
opendevreviewMerged opendev/system-config master: Upgrade gitea to 1.20.6  https://review.opendev.org/c/opendev/system-config/+/90209421:35
opendevreviewMerged opendev/system-config master: Add openstack.org to certcheck domains  https://review.opendev.org/c/opendev/system-config/+/89668821:35
fungiinfra-prod-service-gitea is running in deploy for the upgrade21:39
fungiseeing "Powered by Gitea21:43
fungiVersion: v1.20.6" in the footers now21:43
fungi(not sure why there's an embedded newline in that string when i copy it from the browser, but whatevs)21:44
fungideployment completed21:45
fungistuff seems to be browseable normally for me21:45
tonybbecause there is in the HTML? https://paste.opendev.org/show/b8FV7e8ysQYRciowzv96/21:46
fungiyeah, i guess my point was i don't know why they included newlines there21:48
fungiand why my browser didn't condense them into a space21:49
tonybI suppose the template is building up the footer and not set to trim whitespace.21:56
tonybfungi, clarkb: I checked each of the backends just in case and yup v1.20.6 on them all \o/21:58
tonybfungi: I'm happy with https://review.opendev.org/c/openstack/project-config/+/898118 , any reason I shouldn't +2+W it?22:15
tonybI don't think it'd be destructive in anyway but I'm feeling cautous for some reason22:16
fungitonyb: it's long overdue. only thing it can conceivably break is our ability to merge project creation changes anyway22:35
clarkbtonyb: thank you for checking the giteas. I think you acn approve 89811823:03
tonybclarkb: yw and done.23:04
tonybI cleaned (abandoned) up some old zone-opendev.org updates, and have been trying to review things since 1st Sept.23:05
tonybonce I've gone through that I'll try older ones to see what is still worthwhile23:06
opendevreviewMerged openstack/project-config master: Check proposed project names for problems  https://review.opendev.org/c/openstack/project-config/+/89811823:07
clarkbarg in my haste to get the gerrit stuff tested for upstream I accidentallied a squash23:14
* clarkb fixes it23:14
opendevreviewClark Boylan proposed opendev/system-config master: Add gerrit 3.9 image builds  https://review.opendev.org/c/opendev/system-config/+/90146823:15
opendevreviewClark Boylan proposed opendev/system-config master: Add gerrit 3.8 to 3.9 upgrade testing  https://review.opendev.org/c/opendev/system-config/+/90146923:15
tonybI think I'm done for today.  I have an early meeting tomorrow.23:18
clarkbgood night!23:22

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