| clarkb | https://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 them | 00:28 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/project-config master: Move os-(apply|collect|refresh)-config to heat https://review.opendev.org/c/openstack/project-config/+/901916 | 01:06 |
| * yoctozepto can't wait to get this approved https://review.opendev.org/c/openstack/project-config/+/901591 (please consider) | 08:00 | |
| fungi | clarkb: when you're around, you potentially had questions about that one? ^ otherwise i would have approved it last week | 13:27 |
| opendevreview | Merged openstack/project-config master: Move os-(apply|collect|refresh)-config to heat https://review.opendev.org/c/openstack/project-config/+/901916 | 13: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 |
| fungi | Clark[m]: any worse than https://opendev.org/nebulous/activemq ? | 15:01 |
| fungi | presumably this will be the same sort of thing (collection of helm charts and dockerfiles), i.e. "packaging" for use of influxdb in the nebulous platform | 15:02 |
| fungi | but hopefully yoctozepto can confirm | 15:03 |
| Clark[m] | I think I have the same concerns with that repo. At best it is confusing. | 15:07 |
| fungi | i 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 kernel | 15:17 |
| fungi | it's common for distributions to name the git repositories of their packaging files after the software being packaged | 15:18 |
| fungi | just like the https://src.fedoraproject.org/rpms/weechat repo isn't a fork of weechat | 15: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 proceed | 15:32 |
| Clark[m] | Mostly I didn't want it to land over a big holiday weekend when no one was looking | 15:32 |
| fungi | okay, thanks | 15:33 |
| opendevreview | Merged openstack/project-config master: Add NebulOuS InfluxDB repo https://review.opendev.org/c/openstack/project-config/+/901591 | 15:54 |
| yoctozepto | yeah, I thought that namespacing makes it clear | 16:21 |
| yoctozepto | thanks for approving | 16:21 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Upgrade gitea to 1.20.6 https://review.opendev.org/c/opendev/system-config/+/902094 | 16:24 |
| clarkb | infra-root ^ we should probably try to get that in relatively soon | 16:24 |
| fungi | looks safe enough based on the changelog | 16:27 |
| fungi | maybe we can merge it after the opendev meeting? | 16:27 |
| clarkb | that works for me though I'll probably be afk for a bit after the meeting | 16:29 |
| frickler | hmm, 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-dns | 16:31 |
| fungi | looks like it probably started around the time gerrit began making rebases "on behalf of" | 16:32 |
| clarkb | all of that info is from the ssh stream events events | 16:32 |
| clarkb | it is possible the info you want it still available but we need to read it out of a different json field now | 16:33 |
| fungi | johnsom: did you use the "rebase" button in the webui for that, or push a local rebase? | 16:33 |
| johnsom | fungi The button | 16:33 |
| fungi | i wonder if the event contents differ based on which way it's done | 16:33 |
| fungi | thanks, that was my suspicion | 16:33 |
| fungi | and yeah, we can recreate different scenarios in opendev/sandbox while recording the ssh event stream to see what fields get populated with which values | 16:34 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Update Gerrit 3.8 images to 3.8.3 https://review.opendev.org/c/opendev/system-config/+/901992 | 16:41 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Add gerrit 3.9 image builds https://review.opendev.org/c/opendev/system-config/+/901468 | 16:41 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Add gerrit 3.8 to 3.9 upgrade testing https://review.opendev.org/c/opendev/system-config/+/901469 | 16:41 |
| clarkb | I'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 upstream | 16:41 |
| tonyb | +2 on merging 902094 around the meeting today | 16:47 |
| opendevreview | Tony 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/+/901628 | 17:09 |
| opendevreview | Tony Breeds proposed opendev/system-config master: Add inventory/LE records for mirror02.dfw.rax https://review.opendev.org/c/opendev/system-config/+/902008 | 17:09 |
| opendevreview | Tony 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/+/901504 | 17:09 |
| opendevreview | Tony Breeds proposed opendev/system-config master: [testinfra] Update Ansible canary. https://review.opendev.org/c/opendev/system-config/+/902098 | 17:09 |
| opendevreview | Tony Breeds proposed opendev/zone-opendev.org master: Switch CNAME records to new mirrors https://review.opendev.org/c/opendev/zone-opendev.org/+/902100 | 17:42 |
| clarkb | tonyb: 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 anyway | 17:43 |
| tonyb | clarkb: Sure that works too. If you push that change I'll rebase on top of it. | 17:45 |
| clarkb | tonyb: I think we can do it in the other order. I +2'd your change | 17:45 |
| tonyb | clarkb: I can do it but the commit message would be more meaningful if you do it | 17:45 |
| tonyb | Mine would be somethign like "Because Clark said so ;p" | 17:46 |
| clarkb | basically lets not worry about that until after the mirror stuff is done | 17:46 |
| tonyb | clarkb: Okay that works for me | 17:46 |
| clarkb | but as a followup we can remove the test entirely | 17:46 |
| opendevreview | Merged opendev/zone-opendev.org master: Add DNS records for mirror02.dfw.rax https://review.opendev.org/c/opendev/zone-opendev.org/+/902007 | 18:06 |
| opendevreview | Elod Illes proposed openstack/project-config master: [check-release-approval] Fix distributed leadership handling https://review.opendev.org/c/openstack/project-config/+/896785 | 18:16 |
| clarkb | the gitea 1.20.6 job failed in post due to wht I think is an error in the local registry. I rechecekd it | 18:39 |
| clarkb | if it persists we may have some debugging to do :? | 18:39 |
| fungi | clarkb: yeah, i had already rechecked it about 1.5 hours earlier | 18:44 |
| clarkb | oh I missed that first one. was it the same issue? | 18:44 |
| fungi | it reported success about 30 seconds after your recheck comment | 18:44 |
| clarkb | oh bah | 18:44 |
| clarkb | I'm just bad at reading comments | 18:45 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Add gerrit 3.8 to 3.9 upgrade testing https://review.opendev.org/c/opendev/system-config/+/901469 | 19:40 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Add gerrit 3.9 image builds https://review.opendev.org/c/opendev/system-config/+/902109 | 19:40 |
| JayF | Hmm. 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 backend | 20:05 |
| JayF | jamespage: did you change your gerrit email, perhaps? | 20:05 |
| JayF | I guess you all didn't push new gerrit, I wonder what's up | 20:05 |
| JayF | hmm, looks like email behind it changed and maybe some TC metadata needs updating | 20:06 |
| clarkb | JayF: the apis have gotten more restrictive around user lookups | 20:06 |
| JayF | well, I'm seeing in https://review.opendev.org/admin/groups/c88faeb0c52f35ee530be8defcc9627ea87b28df,members that another email appears for James | 20:06 |
| JayF | and that the query works for my primary email | 20:06 |
| clarkb | but I think if you are logged in / doing authenticated requests you should still see that info | 20:06 |
| clarkb | JayF: the gerrit 3.8 release notes had more info | 20:07 |
| JayF | so I think this might just be a change in email account | 20:07 |
| clarkb | https://www.gerritcodereview.com/3.8.html ^F secondary | 20:07 |
| opendevreview | Brian Rosmaita proposed openstack/project-config master: Implement openstack-unmaintained-core group https://review.opendev.org/c/openstack/project-config/+/902110 | 20:07 |
| clarkb | its 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 address | 20:07 |
| JayF | I don't know which is the likely change, but I can confirm updating the email fixes it | 20:08 |
| JayF | thanks for quacking with me at this | 20:08 |
| JayF | I'll push the update to governance | 20:08 |
| fungi | just noticed in debian/unstable upgrade notes, ssh-keygen will now start generating elliptic curve keys as the default instead of rsa | 20:47 |
| fungi | berstein flavor, not nsa | 20:48 |
| fungi | er, bernstein | 20:48 |
| clarkb | gerrit 3.9.0-rc6 works according to our testing. I've reported that info back upstream | 20:57 |
| clarkb | fungi: interesting openssh 9.3p2-3.1 on tumbleweed is still -t rsa by default | 20:58 |
| fungi | this is 9.5p1 | 21:05 |
| tonyb | JayF: 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 |
| tonyb | JayF: see: https://paste.opendev.org/show/bXSw2xaqd5A4Wy57Xvfb/ | 21:13 |
| JayF | Changing the email seems to have fixed it, and someone else on the TC noted that the email was the updated one in his PTL record | 21:13 |
| JayF | so I think it was just outdated metadata in the TC members yaml | 21:13 |
| tonyb | JayF: Oh okay. I guess you can update the memberid to the new one. | 21:14 |
| JayF | https://review.opendev.org/c/openstack/governance/+/902111 :) | 21:14 |
| tonyb | And just to correct my misunderstanding, memberid in members.yaml is the OIF memberid NOT the gerrit account_id sorry for the noise | 21:25 |
| opendevreview | Merged opendev/system-config master: Upgrade gitea to 1.20.6 https://review.opendev.org/c/opendev/system-config/+/902094 | 21:35 |
| opendevreview | Merged opendev/system-config master: Add openstack.org to certcheck domains https://review.opendev.org/c/opendev/system-config/+/896688 | 21:35 |
| fungi | infra-prod-service-gitea is running in deploy for the upgrade | 21:39 |
| fungi | seeing "Powered by Gitea | 21:43 |
| fungi | Version: v1.20.6" in the footers now | 21:43 |
| fungi | (not sure why there's an embedded newline in that string when i copy it from the browser, but whatevs) | 21:44 |
| fungi | deployment completed | 21:45 |
| fungi | stuff seems to be browseable normally for me | 21:45 |
| tonyb | because there is in the HTML? https://paste.opendev.org/show/b8FV7e8ysQYRciowzv96/ | 21:46 |
| fungi | yeah, i guess my point was i don't know why they included newlines there | 21:48 |
| fungi | and why my browser didn't condense them into a space | 21:49 |
| tonyb | I suppose the template is building up the footer and not set to trim whitespace. | 21:56 |
| tonyb | fungi, clarkb: I checked each of the backends just in case and yup v1.20.6 on them all \o/ | 21:58 |
| tonyb | fungi: I'm happy with https://review.opendev.org/c/openstack/project-config/+/898118 , any reason I shouldn't +2+W it? | 22:15 |
| tonyb | I don't think it'd be destructive in anyway but I'm feeling cautous for some reason | 22:16 |
| fungi | tonyb: it's long overdue. only thing it can conceivably break is our ability to merge project creation changes anyway | 22:35 |
| clarkb | tonyb: thank you for checking the giteas. I think you acn approve 898118 | 23:03 |
| tonyb | clarkb: yw and done. | 23:04 |
| tonyb | I cleaned (abandoned) up some old zone-opendev.org updates, and have been trying to review things since 1st Sept. | 23:05 |
| tonyb | once I've gone through that I'll try older ones to see what is still worthwhile | 23:06 |
| opendevreview | Merged openstack/project-config master: Check proposed project names for problems https://review.opendev.org/c/openstack/project-config/+/898118 | 23:07 |
| clarkb | arg in my haste to get the gerrit stuff tested for upstream I accidentallied a squash | 23:14 |
| * clarkb fixes it | 23:14 | |
| opendevreview | Clark Boylan proposed opendev/system-config master: Add gerrit 3.9 image builds https://review.opendev.org/c/opendev/system-config/+/901468 | 23:15 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Add gerrit 3.8 to 3.9 upgrade testing https://review.opendev.org/c/opendev/system-config/+/901469 | 23:15 |
| tonyb | I think I'm done for today. I have an early meeting tomorrow. | 23:18 |
| clarkb | good night! | 23:22 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!