| Clark[m] | Maybe use irrelevant-files? | 00:00 |
|---|---|---|
| fungi | my rough draft on the penultimate patchset of the ptgbot handler addition broke out all the letsencrypt-* roles and all the files under letsencrypt-create-certs so as to exclude handlers/main.yaml | 00:02 |
| * fungi finds | 00:02 | |
| ianw | that says though "all of the files changed match this list"; not sure it can exclude just one file | 00:03 |
| fungi | yeah, https://review.opendev.org/804791 patchset 10 | 00:05 |
| fungi | i think that's what you're talking about? | 00:06 |
| ianw | fungi: yep, that works too | 00:11 |
| opendevreview | Merged opendev/system-config master: Build our gerrit images on Bullseye https://review.opendev.org/c/opendev/system-config/+/809286 | 00:12 |
| opendevreview | Ian Wienand proposed opendev/system-config master: letsencrypt: avoid running on handler changes https://review.opendev.org/c/opendev/system-config/+/812765 | 00:39 |
| opendevreview | Merged opendev/system-config master: ptgbot: rename site to ptg.opendev.org https://review.opendev.org/c/opendev/system-config/+/812755 | 00:48 |
| opendevreview | Merged opendev/system-config master: ptgbot: Add ServerAlias for ptg.openstack.org https://review.opendev.org/c/opendev/system-config/+/812763 | 00:48 |
| ianw | i haven't seen cla updates, but i might restart gerrit on the new images in a few hours anyway; that gives me a few hours to monitor it and a bit longer to shake out anything before we try a 3.3 image | 00:53 |
| fungi | sounds good, i'm still somewhat around too | 00:55 |
| Clark[m] | Ya apparently there are some edits they have to sort out. I think it's fine if we end up needing to do a second restart later | 00:56 |
| Clark[m] | I'm sort of around but shifting to less and less around | 00:57 |
| fungi | and now i'm nodding off ;) | 01:36 |
| ianw | looks like ptg.opendev.org and ptg.openstack.org are all responding, the bot is in the channel ... i think we can say it is done! | 02:09 |
| ianw | probably close for to the golden hour for putting gerrit onto bullseye. i'll pull the images and confirm everything | 03:08 |
| ianw | "opendevorg/gerrit@sha256:4b708dd0fea6854d0d83624cb01c09cf41fca3d289780496c3a2f98b79a3e529" | 03:50 |
| ianw | that matches dockerhub | 03:50 |
| ianw | #status log restarted gerrit with sha256:4b708dd0... to pick up bullseye based images | 03:57 |
| opendevstatus | ianw: finished logging | 03:57 |
| *** ysandeep|out is now known as ysandeep | 06:39 | |
| *** jpena|off is now known as jpena | 06:47 | |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/diskimage-builder master: Correct path for CentOS 9 stream base image https://review.opendev.org/c/openstack/diskimage-builder/+/806819 | 08:30 |
| *** ykarel is now known as ykarel|lunch | 08:34 | |
| *** jpena is now known as jpena|brb | 09:05 | |
| *** ykarel|lunch is now known as ykarel | 09:52 | |
| *** jpena|brb is now known as jpena | 10:00 | |
| opendevreview | Ananya proposed opendev/elastic-recheck rdo: Fix ER bot to report back to gerrit with bug/error report https://review.opendev.org/c/opendev/elastic-recheck/+/805638 | 10:16 |
| opendevreview | Shnaidman Sagi (Sergey) proposed zuul/zuul-jobs master: Include podman installation with molecule https://review.opendev.org/c/zuul/zuul-jobs/+/803471 | 10:19 |
| *** jpena is now known as jpena|lunch | 11:23 | |
| *** jpena|lunch is now known as jpena | 12:23 | |
| *** raukadah is now known as chandankumar | 12:28 | |
| *** Gonri[m] is now known as Goneri | 12:29 | |
| Goneri | Him I get an "Address unreachable" when I try to reach review.opendev.org with ipv6. | 12:31 |
| mordred | Gonéri: I'm not really here - but I can connect to the server over ipv6 | 12:33 |
| Goneri | mordred: Thanks for the feedback. I can ping ipv6.google.com just fine. So it may be a route problem somewhere else. | 12:34 |
| fungi | 64 bytes from review02.opendev.org (2604:e100:1:0:f816:3eff:fe52:22de): icmp_seq=1 ttl=46 time=71.8 ms | 12:35 |
| fungi | i'm hitting it just fine from my house over my isp's v6 routes (no tunneling) | 12:35 |
| frickler | Goneri: can you do a traceroute and show it at paste.openstack.org? | 12:36 |
| fungi | and feel free to redact the first few hops if you're concerned about giving away your local routes | 12:36 |
| fungi | though also a traceroute from only one side can be misleading, since routing is often asymmetric, so the place you see responses stop may just be the point of route divergence | 12:37 |
| fungi | but it can't hurt | 12:37 |
| Goneri | I will contact my provider directly, it looks they've got the route wrong. | 12:38 |
| fungi | oh neat | 12:38 |
| fungi | good luck! | 12:38 |
| fungi | let us know if we can help | 12:38 |
| fungi | worst case we can try to get vexxhost involved if it's anything to do with their bgp6 announcements or something | 12:38 |
| *** mrkiko_ is now known as mrkiko | 13:37 | |
| *** ykarel_ is now known as ykarel|away | 13:52 | |
| fungi | python 3.11.0a1 was tagged today, i've already discovered that pyyaml is broken on it | 14:13 |
| mrkiko | fungi: corvus: thanks a lot again for the improvements!! | 14:15 |
| corvus | mrkiko: you're welcome! and i haven't forgotten there's more to do... :) | 14:23 |
| fungi | corvus: did you see the mention earlier this week that on switching to the change view the cursor seems to jump to the sync counter in the corner? i wonder if forcing the focus to the first selectable widget on view change would solve that | 14:24 |
| corvus | fungi: that sounds like a good solution | 14:25 |
| fungi | the way to do that is an event trigger? i'm still struggling to grasp the model for urwid | 14:26 |
| corvus | fungi: i think maybe at the bottom of the changescreen constructor | 14:26 |
| fungi | would it be sufficient to explicitly set focus to the change-id link, or should some sort of recursion be used to find the first selectable? | 14:28 |
| opendevreview | Dong Zhang proposed zuul/zuul-jobs master: WIP: Implement role for limiting zuul log file size https://review.opendev.org/c/zuul/zuul-jobs/+/813034 | 14:32 |
| opendevreview | Javier Peña proposed openstack/project-config master: Give RDO 3rd Party CI user permissions on Packstack https://review.opendev.org/c/openstack/project-config/+/813035 | 14:36 |
| icey | hey, is tehre anything holding back these project-config and governance changes: https://review.opendev.org/c/openstack/project-config/+/809012 https://review.opendev.org/c/openstack/governance/+/809011 ? | 14:48 |
| icey | s/tehre/there | 14:49 |
| opendevreview | Dong Zhang proposed zuul/zuul-jobs master: WIP: Implement role for limiting zuul log file size https://review.opendev.org/c/zuul/zuul-jobs/+/813034 | 14:49 |
| clarkb | icey: I've approved the gerrit project addition but can't speak to the openstackgovernance stuff | 14:50 |
| icey | thanks clarkb! | 14:51 |
| icey | is there a better place to ask? | 14:51 |
| fungi | icey: #openstack-tc for the governance changes | 14:52 |
| clarkb | icey: #openstack-infra tends to have a better cross section of opendev and openstack people. Though I think the idea was always that that channel's use would slowly drive to zero and you can ask #opendev for opendev things and various openstack channels for for their stuff. #openstack-tc for governance in this case | 14:52 |
| opendevreview | Dong Zhang proposed zuul/zuul-jobs master: WIP: Implement role for limiting zuul log file size https://review.opendev.org/c/zuul/zuul-jobs/+/813034 | 14:52 |
| fungi | yeah, the openstack tc are the folks with approval rights on the openstack/governance repo | 14:53 |
| icey | awesome, thanks again! | 14:53 |
| fungi | #openstack-infra is still more or less the home of the tact sig | 14:53 |
| opendevreview | Merged openstack/project-config master: Add the cinder-netapp charm to Openstack charms https://review.opendev.org/c/openstack/project-config/+/809012 | 15:00 |
| opendevreview | Tristan Cacqueray proposed opendev/system-config master: Update gerritbot-matrix version to include change number in notifications https://review.opendev.org/c/opendev/system-config/+/813040 | 15:02 |
| mrkiko | corvus: :) :) | 15:28 |
| mrkiko | fungi: thanks for re-reporting; | 15:28 |
| fungi | mrkiko: i'll look into creating a fix for that as soon as my current meetings are over | 15:29 |
| mrkiko | fungi: ahaha, don't worry. thanks a lot instead. | 15:29 |
| fungi | it's my pleasure, i'm always happy to fix accessability bugs | 15:30 |
| *** ysandeep is now known as ysandeep|dinner | 15:48 | |
| corvus | fungi: yeah, i think just hard-code to that | 15:49 |
| fungi | will give it a shot, thanks | 15:50 |
| *** marios is now known as marios|out | 16:04 | |
| *** hjensas is now known as hjensas|afk | 16:12 | |
| clarkb | fungi: ianw: I think we should write a change to do the gerrit swap to the 3.3 images and start a little document on the plan we intend to take sunday/monday. Rouhgly I think the process is either land change for 3.3, wait for it to apply, pull, down only the gerrit container, run init on 3.3 with mariadb up, start gerrit container. Or manually edit docker-compose.yaml. Down gerrit | 16:18 |
| clarkb | container only, run init, start gerrit, land change to reflect the update. | 16:18 |
| clarkb | Then we can followup with job cleanups to stop building 3.2 images, start building 3.4 images, update upgrade job to test 3.3 to 3.4 upgrade etc | 16:18 |
| clarkb | Its nto a ton of work but would be good to think it through before sunday :) | 16:18 |
| clarkb | I can start work on that probably after lunch. I have a few things to get done before that | 16:19 |
| clarkb | corvus: also is the plan still to do a zuul restart today? I wasn't suer what other changes have landed in addition to the bug fixes since the last restart | 16:19 |
| fungi | clarkb: planning today sounds good, as i will try to not be at the computer tomorrow | 16:26 |
| clarkb | and ya you should enjoy a day off | 16:32 |
| fungi | it will be an exciting day of influenza vaccines and updated passport photos | 16:32 |
| fungi | the usual sorts of things one does to unwind | 16:33 |
| clarkb | drive through flu shots for the whole family have revolutionized my vaccine getting. Seriously what was once 4 separate appoibtments is now half and hour on a saturday morning | 16:33 |
| *** ralonsoh_ is now known as ralonsoh | 16:35 | |
| corvus | clarkb: yeah, i think a zuul restart today would be good; might just look at doing that after lunch? | 16:37 |
| fungi | i expect to be around then as well | 16:37 |
| clarkb | corvus: sounds good | 16:37 |
| opendevreview | Merged openstack/project-config master: Set launchpad bug Fix Released after adding comment https://review.opendev.org/c/openstack/project-config/+/801376 | 16:38 |
| *** jpena is now known as jpena|off | 16:46 | |
| opendevreview | Wes Wilson proposed opendev/system-config master: Update ICLA to reference OpenInfra https://review.opendev.org/c/opendev/system-config/+/813055 | 16:58 |
| opendevreview | Ananya proposed opendev/elastic-recheck rdo: Fix ER bot to report back to gerrit with bug/error report https://review.opendev.org/c/opendev/elastic-recheck/+/805638 | 17:10 |
| opendevreview | Wes Wilson proposed opendev/system-config master: Consistent quotes https://review.opendev.org/c/opendev/system-config/+/813059 | 17:13 |
| *** ysandeep|dinner is now known as ysandeep | 17:18 | |
| opendevreview | Wes Wilson proposed opendev/system-config master: Consistent quotes https://review.opendev.org/c/opendev/system-config/+/813055 | 17:37 |
| opendevreview | Wes Wilson proposed opendev/system-config master: Consistent quotes https://review.opendev.org/c/opendev/system-config/+/813060 | 17:37 |
| opendevreview | Wes Wilson proposed opendev/system-config master: Consistent quotes https://review.opendev.org/c/opendev/system-config/+/813055 | 17:57 |
| opendevreview | Ananya proposed opendev/elastic-recheck rdo: Fix ER bot to report back to gerrit with bug/error report https://review.opendev.org/c/opendev/elastic-recheck/+/805638 | 18:03 |
| opendevreview | Wes Wilson proposed opendev/system-config master: Update ICLA to reference OpenInfra https://review.opendev.org/c/opendev/system-config/+/813055 | 18:04 |
| clarkb | fungi: do you think we should get someone other than ourselves to review 813055 just to be sure we aren't getting foundation thing straight through? I don't expect anyone would have issue with that but maybe we should have ianw and/or corvus ack it anyway? | 18:05 |
| fungi | can't hurt, it's a fairly sensitive document | 18:06 |
| fungi | and there's no rush on it | 18:06 |
| clarkb | Luca notes that a new gerrit 3.3.x release will be happening today or tomorrow | 18:09 |
| clarkb | we should rebuild our images to pick that up once it happens | 18:09 |
| fungi | good call | 18:10 |
| *** ysandeep is now known as ysandeep|out | 18:39 | |
| opendevreview | Clark Boylan proposed opendev/system-config master: Fix gerrit image dep in zuul config https://review.opendev.org/c/opendev/system-config/+/813068 | 18:50 |
| clarkb | infra-root ^ that should be an easy one. Probably good to land that before 813055 | 18:51 |
| opendevreview | Jeremy Stanley proposed opendev/system-config master: Turn on mod_rewrite for PTG vhost https://review.opendev.org/c/opendev/system-config/+/813069 | 18:51 |
| opendevreview | Jeremy Stanley proposed opendev/system-config master: Add ptgbot serveralias for redirecting PTG site https://review.opendev.org/c/opendev/system-config/+/813070 | 18:51 |
| opendevreview | Jeremy Stanley proposed opendev/system-config master: Update management documentation for ptgbot service https://review.opendev.org/c/opendev/system-config/+/813071 | 18:51 |
| fungi | diablo_rojo_phone: ianw: clarkb: ^ those are also ptgbot fixes | 18:51 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Start building gerrit 3.4 https://review.opendev.org/c/opendev/system-config/+/792047 | 18:59 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Upgrade Gerrit to 3.3 https://review.opendev.org/c/opendev/system-config/+/813072 | 18:59 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Test upgrade from Gerrit 3.3 to 3.4 https://review.opendev.org/c/opendev/system-config/+/813073 | 18:59 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Remove Gerrit 3.2 images https://review.opendev.org/c/opendev/system-config/+/813074 | 18:59 |
| clarkb | ok found time just before lunch to write up that stack. I'm not sure any of those changes are complete, but I think that is roughyl the set of things we want to have for the gerrit upgrade and post upgrade cleanup | 18:59 |
| fungi | on the topic of reducing our dependence on wiki.o.o, tristanC has a series of changes up to add etherpad support to statusbot, which i haven't had time to review yet but might be nice for us to have | 19:00 |
| fungi | i can imagine configuring with an unguessable rw pad id and then publishing the link (perhaps via a redirect or proxy) to the read-only version of it, or even embedding that into some other page | 19:01 |
| fungi | or even using the r-o api to grab the markup version of the pad contents and then rendering them however we want | 19:02 |
| clarkb | fungi: question on https://review.opendev.org/c/opendev/system-config/+/813071 | 19:02 |
| clarkb | And now I should go make lunch | 19:03 |
| fungi | answered | 19:04 |
| opendevreview | Kendall Nelson proposed openstack/project-config master: Add diablo_rojo as OP to #openinfra-events https://review.opendev.org/c/openstack/project-config/+/813076 | 19:14 |
| opendevreview | Merged opendev/system-config master: Fix gerrit image dep in zuul config https://review.opendev.org/c/opendev/system-config/+/813068 | 19:25 |
| opendevreview | Jeremy Stanley proposed opendev/system-config master: Turn on mod_rewrite for PTG vhost https://review.opendev.org/c/opendev/system-config/+/813069 | 19:34 |
| opendevreview | Jeremy Stanley proposed opendev/system-config master: Add ptgbot serveralias for redirecting PTG site https://review.opendev.org/c/opendev/system-config/+/813070 | 19:34 |
| opendevreview | Jeremy Stanley proposed opendev/system-config master: Update management documentation for ptgbot service https://review.opendev.org/c/opendev/system-config/+/813071 | 19:34 |
| opendevreview | Jeremy Stanley proposed opendev/system-config master: Update ptgbot jobs to use #opendev-sandbox channel https://review.opendev.org/c/opendev/system-config/+/813077 | 19:34 |
| opendevreview | Merged openstack/project-config master: Add diablo_rojo as OP to #openinfra-events https://review.opendev.org/c/openstack/project-config/+/813076 | 19:40 |
| opendevreview | Ananya proposed opendev/elastic-recheck rdo: Fix ER bot to report back to gerrit with bug/error report https://review.opendev.org/c/opendev/elastic-recheck/+/805638 | 19:53 |
| clarkb | fungi: forsome reason I thought they were hosting it under openinfra fro the same reasons we moved the irc channel | 19:56 |
| fungi | the repo wasn't moved out of the openstack namespace, it's currently tracked as belonging to openstack's tact sig | 19:57 |
| clarkb | fungi: re https://review.opendev.org/c/opendev/system-config/+/813077 I wouldn't expect we need to set the var in the group vars and the defaults. It should be enough to set the value in defaults, but it doesn't hurt to also set it in group vars | 19:57 |
| fungi | clarkb: i think the problem is the defaults is never hit because we set it in our global group vars | 19:57 |
| clarkb | ah | 19:57 |
| fungi | it's set in inventory/service/group_vars/eavesdrop.yaml | 19:58 |
| fungi | so we need to override that with the test-specific group vars | 19:58 |
| clarkb | got it, then ya we need to override in the zuul group vars file. We might consider dropping the default in that case but doesn't hurt to have it there iether | 19:58 |
| clarkb | a more typical pattner would be to have the role express what we want out of production then only override in zuul vars | 19:59 |
| fungi | i considered that, but then we have a required variable with no default provided in the role | 19:59 |
| fungi | and yeah, we could do it that way instead | 19:59 |
| opendevreview | Clark Boylan proposed openstack/project-config master: Update gerrit image dep for jeepyb triggered builds https://review.opendev.org/c/openstack/project-config/+/813080 | 20:06 |
| opendevreview | Clark Boylan proposed openstack/project-config master: Update jeepyb to trigger Gerrit 3.3 image builds https://review.opendev.org/c/openstack/project-config/+/813081 | 20:06 |
| clarkb | fungi: https://review.opendev.org/c/openstack/project-config/+/813080 is a similar fixup to https://review.opendev.org/c/opendev/system-config/+/813068 | 20:07 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Remove Gerrit 3.2 images https://review.opendev.org/c/opendev/system-config/+/813074 | 20:07 |
| clarkb | https://review.opendev.org/q/hashtag:gerrit-3.3+status:open I'm trying to accumulate these changes under that hashtag. Though i notice that only the change owner can set hashtag on those repos :/ | 20:09 |
| clarkb | fungi: oh I hadn't realized that https://review.opendev.org/c/opendev/system-config/+/813055/ failed CI. Note that my earlier change wasn't related to that. Looking at the failures we seem to do enough validation of the cla content to ensure we are serving static content properly that we need to update the change for that too | 20:14 |
| clarkb | I'll ask wes if he wants to do that or I can update it | 20:14 |
| fungi | oh, good point, i saw the failures and assumed they were related to the bullseye image change, i should have dug deeper there | 20:14 |
| clarkb | assert 'OpenStack Project Individual Contributor License Agreement' in cmd.stdout is what we check but the cla is being updated to say 'OpenInfra Foundation Individual Contributor License Agreement' | 20:15 |
| fungi | indeed, good test! | 20:16 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Update ICLA to reference OpenInfra https://review.opendev.org/c/opendev/system-config/+/813055 | 20:17 |
| clarkb | fungi: ^ I think that will fix the testing | 20:17 |
| clarkb | https://review.opendev.org/c/opendev/system-config/+/813073/ getting a +1 from zuul the first time around scares me :) | 20:18 |
| clarkb | (that updates the upgrade testing we do for gerrit from 3.2 -> 3.3 to 3.3 -> 3.4) | 20:18 |
| clarkb | I forgot about that test but we added it when we got the paths for static file serving wrong | 20:18 |
| clarkb | If we can land https://review.opendev.org/c/opendev/system-config/+/813055 today and restart on the updated image that would be a good point to stop for merging things pre upgrade then hashtag:gerrit-3.3 stuff should be ready to land Sunday as we work through the process | 20:22 |
| clarkb | Looks like we lost the matrix bridge almost an hour ago | 20:24 |
| clarkb | I'm chatting with corvus on matrix since he isn't here right now | 20:35 |
| clarkb | I think we'd like to go ahead and do that zuul restart nowish. | 20:35 |
| clarkb | I don't see any openstack release activity | 20:35 |
| clarkb | I'll let them know anyway | 20:35 |
| fungi | okay, i'm around for a few more minutes before i need to switch to dinner prep for a bit | 20:36 |
| clarkb | corvus reports the restart is beginning | 20:37 |
| clarkb | well the process to restart is beginning | 20:37 |
| fungi | thanks! | 20:37 |
| clarkb | corvus notes that we'll be relying on the periodic cleanup routine in zuul to clean up the leaked entries. So we should see that graph trend the other direction after about an hour when the routine starts | 20:40 |
| clarkb | basically the restart itself isn't going to clean things up | 20:40 |
| fungi | ahh, very cool | 20:40 |
| fungi | and a great test of the cleanup | 20:41 |
| clarkb | ya | 20:41 |
| fungi | has reenqueuing started yet? | 20:51 |
| clarkb | yes | 20:52 |
| clarkb | as expected https://grafana.opendev.org/d/5Imot6EMk/zuul-status?viewPanel=37&orgId=1 doesn't show dramatic update to the znodes, but we should expect that cleanup in an hour or so | 20:52 |
| clarkb | it is up and reenqueuing has begun. When reeqeuing is done I'll #status log the info | 20:53 |
| fungi | thanks! | 20:54 |
| ianw | clarkb: thanks for the gerrit work. will look over in a little, and start a upgrade checklist etherpad | 20:57 |
| clarkb | ianw: sounds good | 20:57 |
| clarkb | also I think for the 3.3.7 release we'll want a change that bumps up the tag versions of plugins on the 3.3 build to do that. Assuming that doesn't arrive until tomorroe I can push that up and self approve it I guess | 20:58 |
| ianw | fungi: i thought i did test the ptg redirects, were they not working? | 20:58 |
| clarkb | ianw: thsi is a new redirect because ptgbot.opendev.org was apparently communicated in non zero locations | 20:59 |
| clarkb | oh you mean the mod rewrite thing? | 20:59 |
| ianw | https://review.opendev.org/c/opendev/system-config/+/813069/ where it enables mod_rewrite | 20:59 |
| ianw | the ptgbot thing is fine, but i guess we need to add back dns and LE generation? | 21:00 |
| *** dviroel is now known as dviroel|out | 21:02 | |
| clarkb | ya | 21:04 |
| clarkb | fungi: ^ were there LE changes? | 21:04 |
| clarkb | #status log restarted all of zuul on commit 96c61208a064a2c5cabe1446bafdd58d1d1387c4 | 21:04 |
| opendevstatus | clarkb: finished logging | 21:05 |
| ianw | i can add those bits in a bit too | 21:05 |
| ianw | just have to eat some breakfast and reset a few things as we had a power outage overnight | 21:06 |
| fungi | i'll have to catch up after dinner | 21:07 |
| opendevreview | Merged openstack/project-config master: Update gerrit image dep for jeepyb triggered builds https://review.opendev.org/c/openstack/project-config/+/813080 | 21:09 |
| fungi | ianw: the http to https redirect worked but https to https did not | 21:10 |
| fungi | adding the rewriteengine directive on the server temporarily solved it for me | 21:11 |
| fungi | http to https was using a redirect directive which works fine on its own, but the rewrites on the https vhost were being ignored | 21:12 |
| simondodsley | whatever the outage just was I now see patch 810331.11 running in both gate and check. Can someone kill the check job, but leave the gate running? | 21:13 |
| clarkb | as mentioned in #zuul before you moved over, I think I'm happy to let both buildsets run as I don't expect it will cause problems. Though I need to pop out now for a school run so can't dequeue anyway | 21:14 |
| clarkb | maybe fungi or ianw can do that if we need to | 21:14 |
| simondodsley | clarkb, thanks | 21:16 |
| ianw | fungi: ahh. i don't try to hit the http sites but i guess firefox's default behaviour is to go there. i did think they switched it to https first | 21:22 |
| fungi | ianw: it's configurable in ff's settings | 21:22 |
| ianw | well it found one problem, but made me miss another ... i guess it's even :) | 21:23 |
| fungi | regardless, with the rewriteengine addition to the config it should be fine. part of the confusion there is mixing redirect and rewrite directives | 21:24 |
| fungi | took me a little while to figure out what wasn't working exactly | 21:24 |
| fungi | ianw: since the fix hasn't deployed yet, if you browse to https://ptg.openstack.org/ptg.html you'll see it doesn't (yet) correct the domain in the url | 21:25 |
| opendevreview | Ian Wienand proposed opendev/zone-opendev.org master: Re-add ptgbot site https://review.opendev.org/c/opendev/zone-opendev.org/+/813088 | 21:26 |
| fungi | thanks! i was just about to write that one after your prescient comment reminded me | 21:27 |
| ianw | clarkb: if you have a quick sec for https://review.opendev.org/c/opendev/system-config/+/812765 that will stop us running the LE job for changes like this | 21:28 |
| fungi | ianw: minor issue noted inlinne on 813088 | 21:30 |
| opendevreview | Ian Wienand proposed opendev/zone-opendev.org master: Re-add ptgbot site https://review.opendev.org/c/opendev/zone-opendev.org/+/813088 | 21:31 |
| *** diablo_rojo_phone is now known as Guest2133 | 21:35 | |
| opendevreview | Merged opendev/system-config master: Update ptgbot jobs to use #opendev-sandbox channel https://review.opendev.org/c/opendev/system-config/+/813077 | 21:35 |
| opendevreview | Merged opendev/system-config master: Turn on mod_rewrite for PTG vhost https://review.opendev.org/c/opendev/system-config/+/813069 | 21:35 |
| fungi | c | 21:39 |
| fungi | hah, you're not my mutt window | 21:39 |
| opendevreview | Merged opendev/zone-opendev.org master: Re-add ptgbot site https://review.opendev.org/c/opendev/zone-opendev.org/+/813088 | 21:44 |
| ianw | clarkb: starting on https://etherpad.opendev.org/p/gerrit-upgrade-3.3 | 21:51 |
| ianw | clarkb: good point, i'll start mariadb separately because sometimes the pull refreshes that image too, so good to give it a restart | 22:05 |
| clarkb | wfm | 22:07 |
| opendevreview | Merged opendev/system-config master: Add ptgbot serveralias for redirecting PTG site https://review.opendev.org/c/opendev/system-config/+/813070 | 22:21 |
| opendevreview | Merged opendev/system-config master: Update management documentation for ptgbot service https://review.opendev.org/c/opendev/system-config/+/813071 | 22:21 |
| ianw | clarkb: does gerrit write out backup files, or should we do something like a hash comparision? | 22:38 |
| clarkb | ianw: we would need to check ourselves. Maybe just backup those fiels so we can diff them and correct the ansible templates if necessary | 22:38 |
| clarkb | though as noted there I seem to have checked this previously and at least for our test install of gerrit there was no change | 22:38 |
| fungi | i suppose automating an exact comparison in an upgrade test wouldn't be hard. automating an inexact comparison tolerant of trivial changes however, far harder | 22:55 |
| fungi | clarkb: if you get time, https://review.opendev.org/812527 was the bridge upgrade check we mentioned, looks like it implies we could upgrade bridge.o.o to focal if we want | 22:57 |
| fungi | not urgent, but system-config-run-base-ansible-devel does succeed on it as well | 22:58 |
| fungi | so that theory panned out at least | 22:58 |
| clarkb | cool and ya I just pushed a zuul bug fix, let me double check the upgrade etherpad again and then I can review it | 22:58 |
| clarkb | ianw: left a comment on step 9 that I think is important if you can dobule check that | 23:01 |
| clarkb | fungi: do you have the etherpad url for the mega upgrade from 2.13 to 3.2 handy? that would give us more info on the init command to run | 23:02 |
| clarkb | though I'm reasonably confident we should be using what we use in ansible for testing | 23:02 |
| clarkb | also we halved our znodes in zk when zuul cleanup ran | 23:03 |
| clarkb | we are about 9k above our previous steady state | 23:03 |
| clarkb | ianw: would you be willing to review https://review.opendev.org/c/opendev/system-config/+/813055 as well? | 23:07 |
| clarkb | that is the cla update we thoguht might be coming tomorrow. I don't want to approve it sicne the submitter and current reviewers are foundation staff, would be good to have another party just sanity check us | 23:08 |
| ianw | clarkb: sure, i don't know about the leagalese but it looks like it does what it says :) | 23:15 |
| ianw | what does d/b/a mean? | 23:16 |
| ianw | ... https://en.wikipedia.org/wiki/Trade_name | 23:17 |
| ianw | TIL | 23:17 |
| clarkb | ya its the lightweight way of renaming yourself aiui | 23:21 |
| clarkb | otherwise the process is far more invovled | 23:22 |
| fungi | clarkb: i can find it, just a sec | 23:28 |
| fungi | clarkb: https://etherpad.opendev.org/p/gerrit-upgrade-2021 | 23:30 |
| ianw | clarkb: i'll keep an eye on the gerrit twitter, but it does seem to run on a EU-ish TZ | 23:31 |
| ianw | for a 3.3.7 release | 23:31 |
| clarkb | fungi: I think that is the server upgrade not service upgrade? | 23:31 |
| clarkb | ianw: thanks | 23:31 |
| fungi | clarkb: ahh, yep, i'll dig back farther | 23:31 |
| clarkb | https://etherpad.opendev.org/p/gerrit-2.16-upgrade was in that etherpad I think that is what I was looking for | 23:32 |
| clarkb | ianw: ya ^ shows us using the init commands like I put in the new etherpad from ansible, I think we should update to mimic on the 3.3 task list | 23:32 |
| ianw | yep, agree and updated that step | 23:32 |
| fungi | clarkb: https://etherpad.opendev.org/p/opendev-gerrit-3.2-upgrade-plan | 23:33 |
| clarkb | ah that was the other one, cool we have lots of info :) | 23:33 |
| fungi | maybe? | 23:33 |
| clarkb | ianw: re updating the plugins what I typically do is open the google source page for each plugin and check if there is a stable-3.3 branch for it. If not use the latest tag which should be 3.3.7 post release | 23:38 |
| clarkb | ianw: then I think I remember that we also have to change the Dockerfile otherwise the jobs don't run the way we want | 23:38 |
| clarkb | the change I made for updating mordreds 3.4 build change does that if you compare patchsets (but there was a rebase so that might be a little ugly) | 23:39 |
| clarkb | interesting the matrix bridge to oftc sees our communication here but if I send a message from the matrix side it does not go through | 23:40 |
| ianw | yep, some like the its-base seem to have not make a 3.3 branch | 23:41 |
| ianw | i'm not sure that happens by default, or only at the point it needs a separate branch because master merged something incompat | 23:41 |
| ianw | (e.g. the zuul-summary-plugin current has no branch) | 23:41 |
| clarkb | I suspect that for many it is a compat thing | 23:42 |
| clarkb | but the way they do branches has always confused me | 23:42 |
| ianw | yes they merge upwards instead of what i feel is more logical which is cherry-picking changes you need downwards | 23:43 |
| clarkb | for a half second I was worried that it seemed we weren't allocating nodes maybe we quickly as I woudl expect, but it seems things are happy looking at grafana | 23:59 |
| clarkb | just normal slowness (the cla update change in particular is waiting on a node, but I expect that is just normal wait though I hanve't look at logs for it) | 23:59 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!