| -@gerrit:opendev.org- OpenStack Proposal Bot proposed: [openstack/project-config] 990586: Normalize projects.yaml https://review.opendev.org/c/openstack/project-config/+/990586 | 02:25 | |
| @fungicide:matrix.org | catching up from last night's discussion, happy to help with a gerrit restart but i'm planning to disappear for lunch after my morning meetings wrap up, expect i'll be gone between 15:00 and 16:50 utc | 12:50 |
|---|---|---|
| @fungicide:matrix.org | and yeah, i expect it'll all just magic itself | 12:50 |
| -@gerrit:opendev.org- ayyappa proposed: [openstack/project-config] 990651: Add repo app-ejbca for starlingx https://review.opendev.org/c/openstack/project-config/+/990651 | 13:43 | |
| @mordred:waterwanders.com | fungi: it would be really neat to magically understand what happened :) | 13:46 |
| @jim:acmegating.com | i may not be around all day, but i can help get things started before Clark shows up, so i'll approve the gerrit build change | 13:51 |
| @fungicide:matrix.org | thanks. we might be able to squeeze a restart in before i leave for lunch, even | 13:53 |
| @fungicide:matrix.org | turns out my 14-15z meeting is cancelled this week so i won't be as distracted | 13:53 |
| -@gerrit:opendev.org- ayyappa proposed: [openstack/project-config] 990651: Add repo app-ejbca for starlingx https://review.opendev.org/c/openstack/project-config/+/990651 | 13:54 | |
| -@gerrit:opendev.org- ayyappa proposed: [openstack/project-config] 990651: Add repo app-ejbca for starlingx https://review.opendev.org/c/openstack/project-config/+/990651 | 14:06 | |
| @clarkb:matrix.org | Ya I've got a school run to do in a few minutes but then I should be back | 14:20 |
| @fungicide:matrix.org | zuul's estimating another half hour to merge anyway | 14:21 |
| @fungicide:matrix.org | and deploy could very well end up losing the race against hourly jobs | 14:21 |
| @fungicide:matrix.org | oh, i was looking at elapsed time, it hasn't even started running system-config-run-review-3.13 yet | 14:30 |
| @fungicide:matrix.org | okay, zuul does think it still could merge before the top of the hour | 14:40 |
| -@gerrit:opendev.org- Zuul merged on behalf of Clark Boylan: [opendev/system-config] 990570: Build gerrit replication plugin using stable branches https://review.opendev.org/c/opendev/system-config/+/990570 | 15:00 | |
| @clarkb:matrix.org | Now to see if deploy or hourly wins the enqueue race | 15:01 |
| @fungicide:matrix.org | merged about 40 seconds too late i think, but they're all waiting behind a reconfigure event | 15:01 |
| @jim:acmegating.com | aww, missed the top of the hour by 42 seconds :) | 15:01 |
| @clarkb:matrix.org | We have some variance in the hourly pipeline so deploy can still win iirc | 15:02 |
| @fungicide:matrix.org | yeah, unblocked now but they did indeed get scheduled in chronological order | 15:02 |
| @clarkb:matrix.org | Hourly for the semaphore I think | 15:02 |
| @clarkb:matrix.org | * Hourly got the semaphore I think | 15:02 |
| @fungicide:matrix.org | right, but at least the promotes are able to run in parallel with that | 15:02 |
| @fungicide:matrix.org | and i'm overdue to step away for my lunch appointment, will check on things in ~1.5 hours when i get back, or help with the restart at that point if it hasn't happened yet | 15:04 |
| @clarkb:matrix.org | https://quay.io/repository/opendevorg/gerrit/manifest/sha256:736d3f4e5d6260fc334526fe1c3ca629d6db8d9bbd09345f005f47ae708235c2 is the new image | 15:06 |
| @clarkb:matrix.org | how does this look for when we're ready to actually restart things: #status notice Gerrit will be restarted to pick up a bugfix in the replication plugin. You may notice a short outage of a few minutes. | 15:06 |
| @clarkb:matrix.org | f4812a7aaf95 appears to be the image we are running with right now | 15:08 |
| @clarkb:matrix.org | `docker-compose -f /etc/gerrit-compose/docker-compose.yaml down && mv ~gerrit2/review_site/data/replication/ref-updates/waiting ~gerrit2/tmp/waiting_queue_2026-05-29 && rm ~gerrit2/review_site/cache/{gerrit_file_diff,git_file_diff,git_modified_files,modified_files,comment_context}-v2.* && docker-compose -f /etc/gerrit-compose/docker-compose.yaml up -d` This is the command I'll plan to run to do the actual restart | 15:10 |
| @clarkb:matrix.org | I'll start a root screen on review03 and pull the image now | 15:10 |
| @clarkb:matrix.org | I've pulled the image and checked it against the quay.io url above. That lgtm | 15:13 |
| @clarkb:matrix.org | infra-root should I proceed with sending that status notice and then run the restart command or would we prefer to wait for fungi to return? | 15:13 |
| @jim:acmegating.com | Clark: i'm around for the restart if you want to go for it | 15:30 |
| @clarkb:matrix.org | Ok I will get started momentarily | 15:31 |
| @clarkb:matrix.org | #status notice Gerrit will be restarted to pick up a bugfix in the replication plugin. You may notice a short outage of a few minutes. | 15:31 |
| @status:opendev.org | @clarkb:matrix.org: sending notice | 15:31 |
| @clarkb:matrix.org | once that is complete I will run the docker compose down etc in the command above (it is also commented out in the screen for the moment) | 15:32 |
| @jim:acmegating.com | is that command documented somewhere? | 15:33 |
| @clarkb:matrix.org | corvus: probably not :( the normal docker compose down and up will work. This just optimizes a few things. But it is a good comment that this should be documented | 15:33 |
| @jim:acmegating.com | i'll work on that, mostly to see if i'm missing anything :) | 15:33 |
| @clarkb:matrix.org | the waiting queue cleanup avoids a few thousand tracebacks on startup that make things look bad (and hard to see real issues with all that spam) and the cache file cleanups helps contain their growth | 15:34 |
| @clarkb:matrix.org | since those files don't really shrink taking advantage of the downtime to remove them and restart from 0 is a nice thing to do | 15:34 |
| -@status:opendev.org- NOTICE: Gerrit will be restarted to pick up a bugfix in the replication plugin. You may notice a short outage of a few minutes. | 15:34 | |
| @status:opendev.org | @clarkb:matrix.org: finished sending notice | 15:34 |
| @clarkb:matrix.org | ok I will proceed with the restart now | 15:34 |
| @clarkb:matrix.org | stopping gerrit takes a few minutes typically. like 3-5 I think | 15:35 |
| @clarkb:matrix.org | `[2026-05-29T15:37:23.245Z] [main] INFO com.google.gerrit.pgm.Daemon : Gerrit Code Review 3.12.7-dirty ready` | 15:37 |
| @clarkb:matrix.org | I can view diffs like https://review.opendev.org/c/opendev/system-config/+/990570/1/zuul.d/docker-images/gerrit.yaml | 15:37 |
| @jim:acmegating.com | yep lgtm | 15:38 |
| @clarkb:matrix.org | I'm going to check show-queue and if that looks settled i'll issue a replicate start command for something like bindep | 15:39 |
| @jim:acmegating.com | yeah i was just looking at the queues | 15:39 |
| @clarkb:matrix.org | they look pretty empty (which is good) | 15:40 |
| @jim:acmegating.com | yeah, i think it's just doing current events maybe | 15:40 |
| @jim:acmegating.com | yep, mostly idle with the occasional replication for a new change | 15:40 |
| @clarkb:matrix.org | yup and then it lists the diskcachepruner things which I believe are scheduled for the future | 15:41 |
| @clarkb:matrix.org | they just end up on the list when you start the service | 15:41 |
| @clarkb:matrix.org | I issued `replication start opendev/bindep` via ssh and my ssh command exited 0 | 15:42 |
| @clarkb:matrix.org | I believe that jeepyb was seeing a non zero exit with the error message which generated the traceback | 15:42 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [opendev/system-config] 990679: Document current gerrit restart process https://review.opendev.org/c/opendev/system-config/+/990679 | 15:42 | |
| @jim:acmegating.com | should we see general replication tasks in the queue soon? | 15:43 |
| @jim:acmegating.com | oh you just did bindep sorry | 15:43 |
| @jim:acmegating.com | i had in my mind you were going to replicate all; nvm. i don't think that's necessary, i think testing one repo is fine. | 15:43 |
| @jim:acmegating.com | is it the case that the current running tasks are not in the queue? | 15:44 |
| @jim:acmegating.com | (like if we see a task at the top like replicate to gitea09 does that mean that we're currently running replicate to giteo08, or that we're currently running replicate to gitea09) | 15:46 |
| @clarkb:matrix.org | corvus: I saw some the first time I ran show queue | 15:43 |
| @clarkb:matrix.org | there are some now `51840bab 15:41:34.354 [f16bbfc4] push ssh://git@gitea09.opendev.org:222/openstack/nova.git [refs/changes/41/981141/meta]` | 15:44 |
| @clarkb:matrix.org | so I think automatic replication si working but I'll try to find positive confirmation of this via a gerrit change update that got replicated | 15:44 |
| @clarkb:matrix.org | https://opendev.org/opendev/system-config/commit/2d71e37b6684b3305740a6b4d4781e619117c762 your change appears to have replicated | 15:45 |
| @clarkb:matrix.org | corvus: I think if the state column of the task is empty then it is currently executing | 15:47 |
| @clarkb:matrix.org | corvus: otherwise things that are queued up/waiting for threads or just schedueld for another time have data within the state column | 15:47 |
| @jim:acmegating.com | yeah -- show-queue and ps show the same tasks | 15:48 |
| @clarkb:matrix.org | one of the things we also like to do after a restart is trigger reindexing of changes | 15:48 |
| @jim:acmegating.com | so currently running tasks *are* in show-queue | 15:48 |
| @clarkb:matrix.org | to avoid the duplicate change problem | 15:48 |
| @clarkb:matrix.org | and that will enqueue a lot of stuff and create queues as well as tasks in progress which may make this easier to see | 15:48 |
| @clarkb:matrix.org | should I go ahead and issue a reindex of changes now? | 15:48 |
| @jim:acmegating.com | Clark: before doing that, is there anything we want to do with drizzle? | 15:49 |
| @jim:acmegating.com | my understanding is that we want to re-push into gerrit, so i think the answer to my question is no | 15:49 |
| @jim:acmegating.com | but i just wanted to check before we fill queues | 15:49 |
| @clarkb:matrix.org | drizzle shouldn't have any changes yet right? So I don't think we need to figure out drizzle before reindexing existing changes | 15:49 |
| @mordred:waterwanders.com | I *think* based on yesterday's investigation that a manual clone/push (probably doing it the jeepyb way) would be a thing to try. but yeah - agree, doesn't need to be sorted before reindexing | 15:50 |
| @clarkb:matrix.org | and ya I think the next step with drizzle is to repush it. But Let me double check that it didn't magically get refs created after the resetart | 15:50 |
| @clarkb:matrix.org | ya drizzle.git/refs/heads is still an empty dir and drizzle.git/packed-refs only has tags and refs/meta/config in it | 15:51 |
| @jim:acmegating.com | reindex sgtm | 15:51 |
| @clarkb:matrix.org | so I think we're still ni the same state as before there with four missing branches and issuing a reindex of changes should be fine | 15:51 |
| @clarkb:matrix.org | ack I'll trigger that now | 15:51 |
| @jim:acmegating.com | Clark: what's the command you're using for that? i'll add it to the docs | 15:53 |
| @clarkb:matrix.org | corvus: `gerrit index start changes --force` via ssh on port 29418 | 15:53 |
| @mordred:waterwanders.com | I swear I did a local reproduction of the jeepyb steps to check the fsck and the heads - and I cannot find the dir I did it in :) | 15:53 |
| @clarkb:matrix.org | the force is required beacuse the index version doesn't change. If gerrit sees the index version is the same it won't reindex by default | 15:53 |
| @clarkb:matrix.org | and show-queue -w has lots of entries now | 15:53 |
| @clarkb:matrix.org | corvus: and you can see state is `waiting ...` for things that are queued | 15:54 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [opendev/system-config] 990679: Document current gerrit restart process https://review.opendev.org/c/opendev/system-config/+/990679 | 15:55 | |
| @clarkb:matrix.org | this typically takes about half an hour or so to reindex | 15:56 |
| @clarkb:matrix.org | not terribly quick but much quicker than it used to be | 15:56 |
| @clarkb:matrix.org | that should finish around when fungi returns. Did we want to proceed with pushing drizzle content then? | 15:57 |
| @clarkb:matrix.org | I'll leave the screen up for now too in case we want to use that for the push (though maybe someone will do it locally?) | 15:57 |
| @jim:acmegating.com | yeah, no need to wait on the drizzle stuff | 16:00 |
| @clarkb:matrix.org | oh fungi when you get back can you rerun the command that failed for you when doing replication start? Just to be sure we're using a known reproducer? I did `replication start opendev/bindep` which I think is sufficient, but figured we should double check with a known reproducer which you had iirc | 16:01 |
| @jim:acmegating.com | https://paste.opendev.org/show/bEdKVOKKuw6L6BRNCJw9/ | 16:02 |
| @jim:acmegating.com | we're just going to manually run those commands right? | 16:03 |
| @mordred:waterwanders.com | your way was easier than mine: https://paste.opendev.org/show/bRGwYC6v96VRWKkLnQbY/ | 16:03 |
| @mordred:waterwanders.com | (but yes, I think those are the right reproducing sequence) | 16:03 |
| @jim:acmegating.com | those look very similar except one swap in order | 16:03 |
| @mordred:waterwanders.com | might not be a bad idea to do a git for-each-ref refs/copy/ after the fsck | 16:04 |
| @clarkb:matrix.org | I think we can try to have manage-projects do it again by running manage-projects (I think there may even be a flag to do one project instead of all of them). But the downside to that is we won't be able to control each step and debug as we go so I do think manually running the commands manage-projects would run may be useful | 16:04 |
| @clarkb:matrix.org | mordred: ++ exactly that we would miss that if we ran manage-projects rather than the individual commands | 16:04 |
| @jim:acmegating.com | i'm happy to run the commands; i can get started on that | 16:04 |
| @jim:acmegating.com | i have added myself to bootstrappers and am starting the process now | 16:06 |
| @jim:acmegating.com | ``` | 16:08 |
| corvus@fuligin /t/d/drizzle (main)> git for-each-ref refs/copy/ | ||
| 269bf1ede95c17736f45d7fa7cdc2172a677892a commitrefs/copy/heads/drizzle-7.0 | ||
| 3873683eeb4cf650f9cdceece80eca10ca888f23 commitrefs/copy/heads/drizzle-7.1 | ||
| b4de3d25e752315ef7fd020efb3f14ef62bc151d commitrefs/copy/heads/drizzle-7.2 | ||
| 51e535c15ba38fb5c09bd101a374bba14d7c3487 commitrefs/copy/heads/main | ||
| ``` | ||
| @jim:acmegating.com | that's everything before the push; ready to push now? | 16:08 |
| @clarkb:matrix.org | ok so fsck didn't delete the refs | 16:08 |
| @clarkb:matrix.org | (but it didn't for me or mordred either) | 16:08 |
| @clarkb:matrix.org | mordred: ^ you probably want to double check those refs? But it looks good from my perspective without checking the specific refs | 16:08 |
| @mordred:waterwanders.com | darn. I was hoping that would magically break and be the smoking cun. | 16:09 |
| @mordred:waterwanders.com | yeah - that looks right. main definitely is | 16:09 |
| @clarkb:matrix.org | reindexing is nearing the halfway point | 16:09 |
| @jim:acmegating.com | remote: You need 'Create' rights to create new references. | 16:10 |
| @mordred:waterwanders.com | *wow* - and you're in project bootstrappers! | 16:10 |
| @clarkb:matrix.org | corvus: add yourself to admins too I think | 16:10 |
| @jim:acmegating.com | oh i uh | 16:10 |
| @jim:acmegating.com | may not have added corvus to bootstrappers | 16:10 |
| @clarkb:matrix.org | but maybe this was the issue if bootstrappers is not in admins? | 16:10 |
| @jim:acmegating.com | i think i added corvus.admin :) | 16:10 |
| @clarkb:matrix.org | ah that would do it :) | 16:10 |
| @mordred:waterwanders.com | :) | 16:11 |
| @jim:acmegating.com | yeah because that's what the instructions say :) | 16:11 |
| @jim:acmegating.com | https://docs.opendev.org/opendev/system-config/latest/sysadmin.html#force-merging-a-change | 16:11 |
| @jim:acmegating.com | anyway, i'll just push as corvus.admin :) | 16:11 |
| @jim:acmegating.com | okay... so... there were more warning messages than my very large scrollback can handle, so i lost the top of this. but here's a sample of them, and then the bottom: https://paste.opendev.org/show/bkKN2EGM46kcLL7Ulce0/ | 16:12 |
| @mordred:waterwanders.com | huh. skip-validation | 16:13 |
| @mordred:waterwanders.com | so - like - it's trying to run commit validation | 16:13 |
| @jim:acmegating.com | i can try again and tee it to a logfile if we want to see stuff at the top | 16:13 |
| @jim:acmegating.com | but yeah, that validation message is completely new to me i don't know whats up | 16:13 |
| @mordred:waterwanders.com | maybe that's also a new gerrit behavior and it's just been a while since a new project imported preexisting state? | 16:14 |
| @clarkb:matrix.org | if this is the issue jeepyb hit then A) drizzle's history was a stress test after all :) and B) its weird that we didn't log the error in the manage-project log. But also we're learning things | 16:14 |
| @jim:acmegating.com | or the pre-existing state was smaller? | 16:14 |
| @clarkb:matrix.org | mordred: a project of this size in particular. It is complaining about the total number of commits I think | 16:14 |
| @mordred:waterwanders.com | apparently you can add "-o skip-validation" to the git push commabnd | 16:14 |
| @mordred:waterwanders.com | which we probably want to do in manage-projects for completeness | 16:14 |
| @clarkb:matrix.org | https://gerrit-review.googlesource.com/Documentation/user-upload.html#skip_validation ya | 16:14 |
| @clarkb:matrix.org | https://gerrit-review.googlesource.com/Documentation/error-too-many-commits.html | 16:15 |
| @jim:acmegating.com | (i ran it again and captured the whole log; nothing interesting at the top. the log file was 27987 lines, most of those commit message warnings) | 16:16 |
| @mordred:waterwanders.com | I don't fully understand why that didn't cause m-p to error out | 16:16 |
| @clarkb:matrix.org | so that 10000 limit is a default config value for the server. We could change it, but also having a sanity check seems fine and then have jeepyb's import process override it via the option | 16:16 |
| @clarkb:matrix.org | corvus: is it exiting 0? | 16:16 |
| @jim:acmegating.com | exit 1 | 16:16 |
| @clarkb:matrix.org | huh then ya really weird this didn't generate an error in jeepyb. Maybe we don't properly check errors when pushing the code but do when we replicate? | 16:17 |
| -@gerrit:opendev.org- Monty Taylor https://matrix.to/#/@mordred:inaugust.com proposed: [opendev/jeepyb] 990685: Add skip-validation when importing from upstream https://review.opendev.org/c/opendev/jeepyb/+/990685 | 16:18 | |
| @jim:acmegating.com | ready for me to `git push -o skip-validation ssh://corvus.admin@review.opendev.org:29418/drizzle/drizzle.git '+refs/copy/heads/*:refs/heads/*'` | 16:18 |
| @jim:acmegating.com | oh mordred you put it on the end in jeepyb, i put it at the start | 16:19 |
| @jim:acmegating.com | which one do we want to do? :) | 16:19 |
| @mordred:waterwanders.com | we should be running: " log.debug("Return code: %s" % p.returncode) | 16:19 |
| log.debug("Command said: %s" % out.strip())" in git_command | ||
| @mordred:waterwanders.com | so I'd have expected a logged non-zero return code at least | 16:19 |
| @clarkb:matrix.org | corvus: the documented version is at the start | 16:19 |
| @mordred:waterwanders.com | corvus: oh, I'll do it at the start | 16:19 |
| @clarkb:matrix.org | mordred: I don't know that we log at a debug level | 16:19 |
| -@gerrit:opendev.org- Monty Taylor https://matrix.to/#/@mordred:inaugust.com proposed: [opendev/jeepyb] 990685: Add skip-validation when importing from upstream https://review.opendev.org/c/opendev/jeepyb/+/990685 | 16:20 | |
| @jim:acmegating.com | all of that output, including the reject lines, went to stderr | 16:20 |
| @jim:acmegating.com | nothing to stdout | 16:20 |
| @jim:acmegating.com | not sure if jeepyb merges them or not | 16:20 |
| @clarkb:matrix.org | and then as far as pushing it goes. I think we all knew this was a project with a big history and that was why I asked some questions before we landed the chagne to import it. I don't think the risks around that have changed. Gerrit is just checking more aggressively than we expected. So ya I think we can proceed with pushing | 16:20 |
| @mordred:waterwanders.com | Clark: good point. and yeah, looks like we're ignoring exit codes | 16:20 |
| @clarkb:matrix.org | mordred: aha ok mystery probably solved then | 16:21 |
| @mordred:waterwanders.com | we invoke run_command in a form that returns the error code, but then at the callsite don't do anything about it, only trapping for exceptions | 16:22 |
| @mordred:waterwanders.com | so it looks like the callsite might have expected the git_command to throw on error, but that's not the model :) | 16:22 |
| @clarkb:matrix.org | at the very least we should probably log at an error level non zero return codes | 16:22 |
| @mordred:waterwanders.com | yeah | 16:23 |
| @jim:acmegating.com | ! [remote rejected] refs/copy/heads/main -> main (prohibited by Gerrit: not permitted: skip validation) | 16:23 |
| @clarkb:matrix.org | I don't know that I want to do massive manage-projects refactoring just due to how difficult it is to test and be confident in. But some extra logging would've helped tremendously here I think | 16:23 |
| @jim:acmegating.com | i feel like i'm at the dmv | 16:24 |
| @mordred:waterwanders.com | "Forge Author | 16:24 |
| Forge Committer | ||
| Forge Server | ||
| Push Merge Commits | ||
| " | ||
| @mordred:waterwanders.com | gah | 16:24 |
| @mordred:waterwanders.com | * ``` | 16:25 |
| Using the skip-validation option requires the user to have a specific set of permissions, in addition to those permissions already required to bypass review: | ||
| Forge Author | ||
| Forge Committer | ||
| Forge Server | ||
| Push Merge Commits | ||
| ``` | ||
| @clarkb:matrix.org | I'm guessing forge server is the missing permission for bootstrappers | 16:25 |
| @clarkb:matrix.org | that list is documented at https://gerrit-review.googlesource.com/Documentation/user-upload.html#skip_validation | 16:25 |
| @mordred:waterwanders.com | corvus: I also feel like you're at the dmv | 16:26 |
| @jim:acmegating.com | i'm going to add myself to administrators so that i can click the gui for access controls | 16:26 |
| @clarkb:matrix.org | ya looking at our documented all projects acls I see registered users has forge author, bootstrappers has forge commiter, bootstrappers has push merge. | 16:27 |
| @clarkb:matrix.org | I don't see a forge server acl | 16:27 |
| @jim:acmegating.com | it's there in the gui | 16:28 |
| @clarkb:matrix.org | https://gerrit-review.googlesource.com/Documentation/access-control.html#category_forge_server documents what that means | 16:28 |
| @clarkb:matrix.org | I'm not sure I fully understand it, but I guess the idea is that the check in there is one of the validations. So if you're skipping vlidations then you need the permission to forge the data and bypass validation? | 16:28 |
| @clarkb:matrix.org | mordred: this repo has no gerrit notedb history in it right? | 16:28 |
| @clarkb:matrix.org | (we aren't replicating refs/changes/ so that shouldn't actually matter) | 16:29 |
| @mordred:waterwanders.com | it does not | 16:29 |
| -@gerrit:opendev.org- Monty Taylor https://matrix.to/#/@mordred:inaugust.com proposed: [opendev/jeepyb] 990689: Log git errors if they happen https://review.opendev.org/c/opendev/jeepyb/+/990689 | 16:29 | |
| @clarkb:matrix.org | so ya I think this is a non issue for this repo. It would potentially come up if importing somethign that was filter branched and hosted ing errit itself? But even then I suppose we would want imports to just work? | 16:30 |
| @clarkb:matrix.org | corvus: when you say it is there in the gui you mean boostrappers has the permission or that we have the ability to add it in the gui for eg bootstrappers? | 16:30 |
| @jim:acmegating.com | ability to add | 16:30 |
| @mordred:waterwanders.com | yeah. I think when we say "please import this" we mean it | 16:31 |
| @clarkb:matrix.org | ack I think I'm good with adding it. But fungi should be back in a few minutes and it may be worth having his opinion on that too? | 16:31 |
| @jim:acmegating.com | is there a way we can tell gerrit to perform the validation on more commits? | 16:31 |
| @clarkb:matrix.org | yes there is a server wide config setting: https://gerrit-review.googlesource.com/Documentation/config-gerrit.html#receive.maxBatchCommits | 16:32 |
| @jim:acmegating.com | maybe we could increase that; we carefully control who can bypass review (typically just mp) | 16:33 |
| @clarkb:matrix.org | ya that would be another option. We would have to restart gerrit again to pick that up | 16:33 |
| @clarkb:matrix.org | any idea how many commits are in each of those branches? It sounded like maybe on the order of 27k? | 16:33 |
| @jim:acmegating.com | 22100 is the number of commits in main | 16:34 |
| @clarkb:matrix.org | maybe we bump that up to a round 32768? | 16:34 |
| @clarkb:matrix.org | reindexing completed with the expected 3 errors | 16:34 |
| @mordred:waterwanders.com | I think it's kind of fascinating that there's a number at all. | 16:34 |
| @jim:acmegating.com | mordred: yeah; i'm assuming it's something about how gerrit works with jgit and hooks into the git protocol | 16:35 |
| @mordred:waterwanders.com | yah | 16:35 |
| @clarkb:matrix.org | it is probably expensive to do the checks | 16:35 |
| @clarkb:matrix.org | and so they want to limit how much damage you can do to system performance. But if we do this once in a while and not regularly I don't expect it to be a major issue | 16:36 |
| @fungicide:matrix.org | okay, back now | 16:36 |
| @fungicide:matrix.org | wow that's a lot of discussion | 16:36 |
| @jim:acmegating.com | yeah. i just pushed again (with the 10k limit), and gerrit uses a couple extra cpus during the push. doesn't affect the memory usage. whole thing takes 12.5 seconds. | 16:37 |
| @clarkb:matrix.org | fungi: tl;dr is we think replication start commands are working again (you should probably double check since you got it to fail yseterday). THen corvus manually ran the manage-project push commands and found the failure source. It is related to https://gerrit-review.googlesource.com/Documentation/user-upload.html#skip_validation there is a limit of 10000 commits to push at once by default and we hit that limit | 16:37 |
| @jim:acmegating.com | so maybe if we bump this to 32k, the result is that gerrit consumes 2 cpus for 30 seconds. definitely notable, but not going to kill us for something that almost never happens. | 16:38 |
| @clarkb:matrix.org | corvus: agreed | 16:38 |
| @mordred:waterwanders.com | and ... project bootstrappers doesn't have the ACL to bypass the validation manually, is missing a setting | 16:38 |
| -@gerrit:opendev.org- Monty Taylor https://matrix.to/#/@mordred:inaugust.com proposed: [opendev/jeepyb] 990689: Log git errors if they happen https://review.opendev.org/c/opendev/jeepyb/+/990689 | 16:39 | |
| @clarkb:matrix.org | specifically this one https://gerrit-review.googlesource.com/Documentation/access-control.html#category_forge_server We can change the limit via https://gerrit-review.googlesource.com/Documentation/config-gerrit.html#receive.maxBatchCommits | 16:39 |
| @mordred:waterwanders.com | speaking of - I split that from the validation patch. they're not related ^^ | 16:39 |
| @jim:acmegating.com | i'm okay with either approach, slight preference for increasing the limit instead of skipping validation, but like, very very slight. | 16:39 |
| @jim:acmegating.com | afk for 5 min while fungi catches up | 16:40 |
| @clarkb:matrix.org | I'm good with that too. But also I think forge server is probably also safe | 16:40 |
| @clarkb:matrix.org | so ya let's see what if any input fungi has | 16:40 |
| @mordred:waterwanders.com | I kind of think both. add forge-server so that project-bootstrappers has the oomph when an admin uses it to evelvate permissions, but then instead of updating manage-projects to skip validation, leave that as a human override and instead increase the limit in config | 16:41 |
| @mordred:waterwanders.com | because when an admin decides to override something, it's weird to be told no | 16:41 |
| @clarkb:matrix.org | ya that seems reasonable. Give ourselves the escape hatch but rely on sanity checking for typical operations | 16:42 |
| @mordred:waterwanders.com | let's be honest - this specific scenario is not frequent. I'm guessing the initial import of nova was probably the last time we imported 10k commits in one go :) | 16:42 |
| @clarkb:matrix.org | when everyone reconvenes are we good with my closing the root screen on review03? | 16:42 |
| -@gerrit:opendev.org- Monty Taylor https://matrix.to/#/@mordred:inaugust.com proposed: [opendev/system-config] 990726: Increase receive.maxBatchCommits https://review.opendev.org/c/opendev/system-config/+/990726 | 16:47 | |
| @fungicide:matrix.org | oh, that is indeed a very interesting discovery | 16:48 |
| @fungicide:matrix.org | i never realized there was such a limit, but it makes some sense | 16:49 |
| -@gerrit:opendev.org- Monty Taylor https://matrix.to/#/@mordred:inaugust.com proposed: [opendev/system-config] 990727: Start mirroring ubuntu resolute container images https://review.opendev.org/c/opendev/system-config/+/990727 | 16:49 | |
| @clarkb:matrix.org | I am +2 on https://review.opendev.org/c/opendev/system-config/+/990726 but didn't approve it. Mainly because we need to restart Gerrit again to pick it up and I don't want the 3.13 upgrade in a week to be the time we do that. Do we want to approve it now and do another gerrit restart in an hour or two? | 16:51 |
| @jim:acmegating.com | i'm okay with that. | 16:55 |
| @clarkb:matrix.org | Ya I'm ok with it too as long as I'm not the only one around on a Friday to do it :) | 16:56 |
| @jim:acmegating.com | that is a concern. i may not be around by then. but i would be happy to restart tomorrow if it comes to it. or we could just restart on monday. | 16:56 |
| @clarkb:matrix.org | Ack Monday would work as well. | 16:57 |
| @jim:acmegating.com | i think we're still waiting for fungi's thoughts on the 2 approaches (one, the other, both, neither), yeah? | 16:57 |
| @clarkb:matrix.org | Worst case we can remove the setting before the 3.13 upgrade too I guess | 16:57 |
| @clarkb:matrix.org | Yes I think fungi should weigh in and then we can decide what to do next | 16:57 |
| @jim:acmegating.com | because that also sets us up for the question "do we also add the acl" and if the answer is yes, then i can push now without that change. (but doing so means we don't test that change.) | 16:58 |
| @fungicide:matrix.org | i'm still catching up on an eventful 1.5 hours of absence from several different spaces, but will prioritize feedback for this. i don't plan to disappear again until eod | 16:59 |
| -@gerrit:opendev.org- Douglas Lopes Pereira proposed: [openstack/project-config] 990730: Add PTP Metrics app to StarlingX https://review.opendev.org/c/openstack/project-config/+/990730 | 17:02 | |
| @jim:acmegating.com | i have removed myself from project bootstrappers and administrators | 17:14 |
| @harbott.osism.tech:regio.chat | do we have someone from airship around here? their config errors account for about half of the total errors in the openstack tenant by now | 17:43 |
| @fungicide:matrix.org | ildikov might be able to make some contacts | 17:44 |
| @clarkb:matrix.org | Doug Goldstein may also know who to talk to | 17:45 |
| @ildikov:matrix.org | I haven’t talked to folks in a while, but can try to reach them. We communicated in email, so happy to try make connections that way as well. | 17:49 |
| @harbott.osism.tech:regio.chat | just pointing them to https://zuul.opendev.org/t/openstack/config-errors?skip=0 to check the airship projects listed there would be great. they can come here if they need help | 17:50 |
| @fungicide:matrix.org | Clark: corvus: so the "two approaches" are 990726 and... what alternative? | 17:55 |
| @jim:acmegating.com | fungi: adding the "forge server identity" acl to project bootstrappers so we have the option of telling it to skip validation | 17:56 |
| @jim:acmegating.com | we think either in isolation is sufficient for this particular repo; we may have other larger repos in the future where we may need to skip validation | 17:57 |
| @clarkb:matrix.org | and as mordred mentioned we could do both things together. Update the ACL so that we have a manual out if we need it, but also bump the limit in 990726 so that cases like drizzle fall into the automatable category | 17:57 |
| @jim:acmegating.com | yep | 17:57 |
| @fungicide:matrix.org | if both were applied then since the automated import still uses project bootstrappers so would never exercise the increased limit, unless skipping validation was a separate manual action | 18:00 |
| @fungicide:matrix.org | er, forgive my frankensentence there | 18:00 |
| @clarkb:matrix.org | Yes you need an explicit -o skip-validarion argument set on the git push to skip validation | 18:00 |
| @clarkb:matrix.org | So manage-projects would not add that option and rely on the limit alone. If we have problems in the future we can manually opt into skipping validation via git push -o skip-validation | 18:01 |
| @fungicide:matrix.org | okay, so the suggestion does not include adding `-o skip-validation` to manage-projects | 18:01 |
| @clarkb:matrix.org | Not if we do both. I think it may if we only update the acl | 18:01 |
| @fungicide:matrix.org | i can definitely see arguments in favor of each. this is an unusual circumstance (first time we've hit it), so would we want to unknowingly allow it in future imports for other projects? | 18:02 |
| @fungicide:matrix.org | this gave us the opportunity to go, "wow that's a lot of commits you're importing, are you sure?" | 18:03 |
| @clarkb:matrix.org | Possibly not. I think that is an argument in favor of the config update. And keeping the use of the push flag manually doesn't negate that value I don't think as any use of it would be intentional | 18:03 |
| @fungicide:matrix.org | but we could also potentially do that when linting project creation requests in order to provide earlier feedback with less pain on our part | 18:04 |
| @clarkb:matrix.org | Yes though the upstream repo could change in the time between gating and deployment. But that seems like an unlikely situation | 18:04 |
| -@gerrit:opendev.org- Douglas Lopes Pereira proposed: [openstack/project-config] 990730: Add PTP Metrics app to StarlingX https://review.opendev.org/c/openstack/project-config/+/990730 | 18:05 | |
| @fungicide:matrix.org | also nothing (other than lack of efficiency) to keep a project from pushing that many changes through gerrit to merge with noop tests once the project is created | 18:05 |
| -@gerrit:opendev.org- Douglas Lopes Pereira proposed: [openstack/project-config] 990730: Add PTP Metrics app to StarlingX https://review.opendev.org/c/openstack/project-config/+/990730 | 18:06 | |
| @mordred:waterwanders.com | darn - I I should have thought of that - I totally could have claimed change 1000000 easily that way :) | 18:07 |
| @fungicide:matrix.org | anyway, i think i'm convinced just upping the limit is the better option | 18:07 |
| @fungicide:matrix.org | and i'm around for another restart once that lands, to make up for taking an inconveniently-timed lunch break earlier ;) | 18:07 |
| @clarkb:matrix.org | Ok do you want to approve the change then? Assuming you haven't yet | 18:08 |
| @fungicide:matrix.org | you assume incorrectly | 18:08 |
| @mordred:waterwanders.com | with that landed it's probably reasonable to just re-kick m-p and expect it to succeed | 18:08 |
| @clarkb:matrix.org | Assuming manage-projects will attempt to push again. It may bot | 18:08 |
| @fungicide:matrix.org | modulo a gerrit restart in between | 18:09 |
| @clarkb:matrix.org | * Assuming manage-projects will attempt to push again. It may not | 18:09 |
| @fungicide:matrix.org | i don't think gerrit will pick up that config change without restarting | 18:09 |
| @clarkb:matrix.org | Correct we need a Gerrit restart | 18:09 |
| -@gerrit:opendev.org- Monty Taylor https://matrix.to/#/@mordred:inaugust.com proposed: [opendev/jeepyb] 990689: Log git errors if they happen https://review.opendev.org/c/opendev/jeepyb/+/990689 | 18:10 | |
| @jim:acmegating.com | i'm afk for a bit; i yield the responsibility of doing manual pushes back to the collective. i'll try to check back later today and/or tomorrow if stuff needs doing | 18:10 |
| @mordred:waterwanders.com | also that ^^ if we're going to run m-p again ... assuming it can pass its integration test | 18:11 |
| @fungicide:matrix.org | also https://review.opendev.org/c/openstack/project-config/+/989446 is another project creation request which is ready to approve as soon as we're done fiddling with gerrit | 18:12 |
| @clarkb:matrix.org | I should be around today. Lunch is approaching but can work around that. The only other big thing in my list is to test etherpad 3.2.0 on the held node when I get a moment | 18:14 |
| @clarkb:matrix.org | I am hoping my bike will be functioning again and I can pick it up. But haven't heard anything on that yet | 18:16 |
| @fungicide:matrix.org | walks can also be good | 18:16 |
| @fungicide:matrix.org | (i say as someone who only occasionally tries to ride a very ergonomically terrible beach cruiser bike and ends up with a very sore behind) | 18:17 |
| @clarkb:matrix.org | That goes away after a week or two. But yes walks are also good. | 18:19 |
| @fungicide:matrix.org | i think this weekend we're planning for a ~6mi walk to the wright bros. memorial, up and over, then home again | 18:23 |
| @fungicide:matrix.org | if we go early enough, we can be done at the national park before it opens to vehicular traffic for the morning | 18:24 |
| @mordred:waterwanders.com | also, I imagine it'll be cooler that time of the day too | 18:30 |
| @mordred:waterwanders.com | (I assume it's already hot) | 18:30 |
| @fungicide:matrix.org | it's variable this time of year, though supposed to be cooler for the next few days again at least | 18:46 |
| @fungicide:matrix.org | though how warm or cool it feels is more to do with wind speed, direction, cloud cover, and relative humidity | 18:47 |
| @fungicide:matrix.org | our route follows the water mostly, so that helps | 18:47 |
| -@gerrit:opendev.org- Zuul merged on behalf of Monty Taylor https://matrix.to/#/@mordred:inaugust.com: [opendev/system-config] 990726: Increase receive.maxBatchCommits https://review.opendev.org/c/opendev/system-config/+/990726 | 18:54 | |
| @clarkb:matrix.org | that is deploying now and `/home/gerrit2/review_site/etc/gerrit.config` appears to have updated as expected | 18:57 |
| @clarkb:matrix.org | fungi: I'm about to start on lunch, but if you're able to drive the second restart I can put food down as necessary. Note since we already did a restart today the waiting queue move may need an additional suffix in the date/name | 18:57 |
| @fungicide:matrix.org | yep, happy to | 18:59 |
| @fungicide:matrix.org | there was no remaining screen session on review03 so i've started a new one | 19:00 |
| @fungicide:matrix.org | status notice The Gerrit service on review.opendev.org will be offline again monentarily while we restart for a configuration adjustment, but should return to service within a few minutes | 19:01 |
| @fungicide:matrix.org | that look good? | 19:01 |
| @clarkb:matrix.org | yup I stopped the old screen session. Andthat notice lgtm | 19:01 |
| @clarkb:matrix.org | I have attached to the screen you started | 19:02 |
| @fungicide:matrix.org | cool, i'll send that now | 19:07 |
| @fungicide:matrix.org | #status notice The Gerrit service on review.opendev.org will be offline again monentarily while we restart for a configuration adjustment, but should return to service within a few minutes | 19:07 |
| @status:opendev.org | @fungicide:matrix.org: sending notice | 19:07 |
| @fungicide:matrix.org | `docker-compose -f /etc/gerrit-compose/docker-compose.yaml down && mv ~gerrit2/review_site/data/replication/ref-updates/waiting ~gerrit2/tmp/waiting_queue_2026-05-29.1 && rm ~gerrit2/review_site/cache/{gerrit_file_diff,git_file_diff,git_modified_files,modified_files,comment_context}-v2.* && docker-compose -f /etc/gerrit-compose/docker-compose.yaml up -d` | 19:08 |
| @fungicide:matrix.org | that's what i've queued up in screen, based on the last restart | 19:08 |
| @clarkb:matrix.org | that command lgtm | 19:08 |
| @fungicide:matrix.org | i'll wait until statusbot is done, then execute it | 19:09 |
| -@gerrit:opendev.org- Douglas Lopes Pereira proposed: [openstack/project-config] 990730: Add PTP Metrics Exporter app to StarlingX https://review.opendev.org/c/openstack/project-config/+/990730 | 19:10 | |
| @status:opendev.org | @fungicide:matrix.org: finished sending notice | 19:10 |
| -@status:opendev.org- NOTICE: The Gerrit service on review.opendev.org will be offline again monentarily while we restart for a configuration adjustment, but should return to service within a few minutes | 19:10 | |
| @fungicide:matrix.org | engaging | 19:11 |
| @fungicide:matrix.org | graceful stop took 23.6s | 19:11 |
| @fungicide:matrix.org | on its way back up now | 19:11 |
| @clarkb:matrix.org | so quick when the caches are only a few hours old | 19:12 |
| @fungicide:matrix.org | webui is loading for me, `Powered by Gerrit Code Review (3.12.7-dirty)` | 19:12 |
| @fungicide:matrix.org | watching the gerritbot log for a new patchset-created event | 19:14 |
| @fungicide:matrix.org | but if anybody has something to revise and push, that could speed replication checking along | 19:15 |
| @fungicide:matrix.org | (i don't personally) | 19:15 |
| @fungicide:matrix.org | i'm also prepare to start reindexing with `gerrit index start changes --force` via the ssh api once that's confirmed | 19:16 |
| @clarkb:matrix.org | I don't and I'm deep in a bowl of curry | 19:16 |
| @clarkb:matrix.org | Can just push a useless DNM change if you need to | 19:16 |
| @fungicide:matrix.org | go deeper, curry priorities! | 19:16 |
| @fungicide:matrix.org | i'm in no hurry | 19:16 |
| @fungicide:matrix.org | madras? massaman? panang? | 19:17 |
| @clarkb:matrix.org | Thai green curry | 19:18 |
| @fungicide:matrix.org | yessss! one of my faves. with lots of fresh basil i hope | 19:18 |
| @clarkb:matrix.org | Unfortunately not. These are leftovers someone else got yesterday so I'm just cleaning out the fridge. Still tasty | 19:19 |
| @fungicide:matrix.org | next-day curry is often even better anyway | 19:20 |
| @fungicide:matrix.org | https://review.opendev.org/c/openstack/freezer-api/+/990735 got a new revision just a minute ago | 19:21 |
| @fungicide:matrix.org | 74a02dc662bf446d8c99e03b78248c9c6f92cd5a | 19:21 |
| @fungicide:matrix.org | the gitea link takes me to https://opendev.org/openstack/freezer-api/commit/74a02dc662bf446d8c99e03b78248c9c6f92cd5a which loads | 19:21 |
| @clarkb:matrix.org | excellent | 19:21 |
| @fungicide:matrix.org | i'll go ahead and start reindexing | 19:22 |
| @clarkb:matrix.org | sounds good | 19:22 |
| @fungicide:matrix.org | i'm filtering `~gerrit2/review_site/logs/error_log` for lines containing "Reindex" in the same screen session | 19:22 |
| @fungicide:matrix.org | just to give us a convenient place to monitor progress | 19:23 |
| @clarkb:matrix.org | then the last step would be for someone to try repushing the drizzle repo. We can potentially land 990730 and see if that does it for us. Or do it manually like corvus was doing earlier | 19:23 |
| @fungicide:matrix.org | in theory 989446 also will rerun manage-projects | 19:24 |
| @mordred:waterwanders.com | I think you'd need to update the project-cache to set pushed-to-gerrit to false for drizzle/drizzle | 19:24 |
| @fungicide:matrix.org | (990730 is not ready to merge yet) | 19:24 |
| @clarkb:matrix.org | Ah yup that is a better option | 19:24 |
| @mordred:waterwanders.com | but - flipping that to false and then re-running m-p would likely be the best test of whether the config fix fixed it | 19:25 |
| @clarkb:matrix.org | mordred: thats the json file right /me looks for it | 19:37 |
| @clarkb:matrix.org | `/opt/lib/jeepyb/project.cache` this is the file and yes the flag is set to true and would need to be toggled to false cc fungi | 19:38 |
| @clarkb:matrix.org | fungi: I +2'd https://review.opendev.org/c/openstack/project-config/+/989446 but didn't approve it so that we can edit that project.cache file if we want to use 989446 to try and resync drizzle/drizzle | 19:39 |
| @clarkb:matrix.org | but I think I'm comfortable with that if you are fungi | 19:40 |
| @mordred:waterwanders.com | ++ feels like the least manual re-try option | 19:40 |
| @fungicide:matrix.org | yeah, i was going to wait for reindexing to complete first, it's nearly at 50% niow | 19:40 |
| @clarkb:matrix.org | sounds good | 19:40 |
| @fungicide:matrix.org | 80% | 19:51 |
| @clarkb:matrix.org | looks like it completed with the expected 3 failuers again | 19:57 |
| @fungicide:matrix.org | agreed. sorry my broadband went offline for about 20 minutes but seems to have returned finally, at least for the moment | 20:10 |
| @fungicide:matrix.org | i switched `pushed-to-gerrit` to `false` for `drizzle/drizzle` in `/opt/lib/jeepyb/project.cache` | 20:12 |
| @fungicide:matrix.org | approving 989446 now | 20:12 |
| @clarkb:matrix.org | thanks | 20:13 |
| -@gerrit:opendev.org- Zuul merged on behalf of Sabbir Ahmed: [openstack/project-config] 989446: Add starlingx/app-machine-operator project https://review.opendev.org/c/openstack/project-config/+/989446 | 20:21 | |
| @clarkb:matrix.org | manage-projects should be running nowish | 20:23 |
| @clarkb:matrix.org | though I guess it does the giteas first? so maybe not quite yet. The job has staretd though | 20:23 |
| @clarkb:matrix.org | the job reports success | 20:28 |
| @clarkb:matrix.org | https://opendev.org/drizzle/drizzle is still not right. Maybe we didn't try to push I'll look at logs | 20:29 |
| @clarkb:matrix.org | https://paste.opendev.org/show/bE1gg3bhjQjAGwSBri1K/ ya I think it decided it didn't need to push for whatever reason | 20:29 |
| @clarkb:matrix.org | so not the outcome we wanted but also not an emergency. Maybe we just concede and try the manual push again? | 20:30 |
| @clarkb:matrix.org | I detached from the screen. Not sure that we need it any longer | 20:31 |
| @fungicide:matrix.org | looks like the deploy reported a few minutes ago | 20:36 |
| @fungicide:matrix.org | m-p did set that flag back to true when it ran, fwiw | 20:37 |
| @fungicide:matrix.org | i've closed out the screen session now | 20:37 |
| @fungicide:matrix.org | mordred: i added you as the initial member of drizzle-core | 20:38 |
| @mordred:waterwanders.com | fungi: \o/ thank you!! | 20:40 |
| @mordred:waterwanders.com | but also - yeah, opendev.org/drizzle/drizzle is still sad WTF | 20:41 |
| @fungicide:matrix.org | now we just need to get the heads pushed for the one repo, i guess the suggestion is manual `git push --force`? | 20:41 |
| @mordred:waterwanders.com | yeah - there's actually a set of commands we had in a paste - one sec | 20:41 |
| @fungicide:matrix.org | happy to temporarily stick myself in bootstrappers and do that | 20:42 |
| @mordred:waterwanders.com | fungi: https://paste.opendev.org/show/bRGwYC6v96VRWKkLnQbY/ is a paste I put together based on https://paste.opendev.org/show/bEdKVOKKuw6L6BRNCJw9/ | 20:43 |
| @clarkb:matrix.org | I'm probably going to pop out for the school run in 15 minutes but happy for fungi to do git pushing | 20:43 |
| @mordred:waterwanders.com | that has the specific ones jeepyb specifically uses to do things. | 20:43 |
| @fungicide:matrix.org | sure, i can do it now, just a sec | 20:43 |
| @mordred:waterwanders.com | woot! I see branches! | 20:47 |
| @fungicide:matrix.org | it's done and i see content at https://opendev.org/drizzle/drizzle including the expected branches now | 20:47 |
| @mordred:waterwanders.com | 2.3% m4 :) | 20:47 |
| @fungicide:matrix.org | please double-check it looks like you expect before you merge any reviews | 20:47 |
| @mordred:waterwanders.com | yup! That's the expected tip. now to push up a gitreview file :) | 20:48 |
| @mordred:waterwanders.com | that's probably the only repo in opendev that lists more perl than python code | 20:48 |
| @fungicide:matrix.org | i've removed my fungi.admin account from the project bootstrappers group again now | 20:48 |
| @mordred:waterwanders.com | thanks a bunch - now I get to do the fun stuff - like setting up proper multi-repo testing stuffs | 20:49 |
| @fungicide:matrix.org | perl... for great justice | 20:49 |
| @clarkb:matrix.org | What an adventure too. But we learned things about Gerrit so yay? | 20:49 |
| @mordred:waterwanders.com | right? I wasn't expecting nearly that much fun | 20:49 |
| @fungicide:matrix.org | feels like an after-school special | 20:50 |
| @clarkb:matrix.org | 158.69.66.89 is the held ehterpad 3.2.0 node. I have created a clarkb-test pad there and tested it with one firefox client and one chrome client. It seems to work | 22:03 |
| @clarkb:matrix.org | unfortunately it doesn't really test the session cleanup routine and the oom problem due to the lack of a decade of db records. But we can always go back to disabling the cleanups if necessary. | 22:03 |
| @fungicide:matrix.org | yeah, i think that's a fine approach | 22:05 |
| @fungicide:matrix.org | the held etherpad is working fine for me too | 22:10 |
| @clarkb:matrix.org | great. Maybe monday we can approve that change then? I think I'm done with upgrading servers for the day :) | 22:11 |
| @fungicide:matrix.org | yep, sounds great | 22:12 |
| @mordred:waterwanders.com | fwiw: https://review.opendev.org/c/drizzle/drizzle/+/990762 | 22:17 |
| @fungicide:matrix.org | woohoo! | 22:20 |
| @fungicide:matrix.org | it lives | 22:20 |
| @fungicide:matrix.org | is archon spren your llm agent companion? | 22:28 |
| @fungicide:matrix.org | i thought the name might be an anagram, but my algorithmic finder didn't come up with much. person ranch? sharpen corn? | 22:28 |
| @mordred:waterwanders.com | it's one of them :) Archon is the review agent - his job is to watch the gerrit event stream and review the changes (on a whitelist of opt-in projects of course) | 22:29 |
| @mordred:waterwanders.com | no - it's not an anagram. the archon name is just sort of made up. But in Sanderson's Stormlight Archive, spren are beings that are physical manifestations of human emotions, natural phenomena, and abstract concepts. So, sort of like souls of objects and concepts. | 22:30 |
| @mordred:waterwanders.com | so I've been personifying my various agents as-if they are spren. With the exception of my coding agent, sword-nimi,(aka nightblood) who is a direct reference to a sapient sword that was the result of someone from a different world having learned about spren from Roshar and infusing an object with life energy on their own. | 22:32 |
| @mordred:waterwanders.com | I may have a sanderson problem :) | 22:32 |
| @mordred:waterwanders.com | (nimi is an honorific like -san and is how that particular sword is referred to respectfully by one of his wielders. ) | 22:35 |
| @mordred:waterwanders.com | I particularly like this quote from one of the sanderson wikis ... "Personality: Despite its apocalyptic power, the sword possesses the naive personality of a child, cheerfully asking if it has "destroyed lots of evil today." ... which I think is very apt for an AI agent | 22:35 |
| @fungicide:matrix.org | okay, as long as the answer is geeky, i approve | 22:39 |
| @fungicide:matrix.org | and now i have more novels to add to the reading list i'll already never get through, thanks! | 22:40 |
| @mordred:waterwanders.com | oh - yeah. Let me help you with that: https://emonty.github.io/reading-order/#/?view=linear&categories.other-pub=false&categories.potential-plan=false&categories.future-plan=false&categories.unpublished=false&categories.apocryphal=false&categories.forthcoming=false | 22:43 |
| @fungicide:matrix.org | muahahahaha | 22:43 |
| @mordred:waterwanders.com | it turns out cosmere is MCU-like in that there are a bunch of interconnections. but he intends the various books and series to be able to read _without_ needing to read all of them | 22:43 |
| @fungicide:matrix.org | seems prolific | 22:44 |
| @mordred:waterwanders.com | however - I can report that it is enjoyable to read all of them, and then do all of them a second time and notice the references to things throughout | 22:44 |
| @mordred:waterwanders.com | the man is a machine | 22:44 |
| @clarkb:matrix.org | I picked up Esoteric Ebb recently which is basically a DnD reskin of Disco Elysium. Both games are essentially graphic choose your own adventure novels in video game form with some light world navigation | 22:44 |
| @clarkb:matrix.org | sorry the discussion about reading made me want to bring it up | 22:44 |
| @mordred:waterwanders.com | ++ I'll add that to my list :) | 22:44 |
| @mordred:waterwanders.com | speaking of books though - if you haven't all been preached to about the gospel of Dungeon Crawler Carl ... let me express my pleasure at how good book 8 (which just came out) was. HIGHLY recommend the audiobook versions, because the audiobook actor is really *really* good. (so good that the author and the audiobook actor frequently do appearances together, which is not the _most_ normal thing for a book series) | 22:46 |
| @clarkb:matrix.org | I've avoided Dungeon Crawler Carl mostly out of an annoyance that they don't do smaller format prints | 22:47 |
| @clarkb:matrix.org | I don't mind spending money to buy books but I hate the giant tomes that make it difficult to read | 22:47 |
| @mordred:waterwanders.com | I hear that. I've actually been enjoying hardbacks for things recently, but they are much harder to schlep around | 22:48 |
| @mordred:waterwanders.com | sandy reports that actually reading carl is also enjoyable, she's been reading them after we've listened to them. I'm going to try that next after I finish the Robin Hobb books I'm doing now | 22:48 |
| @clarkb:matrix.org | I think Dungeon Crawler Carl is an audible exclusive too? | 22:49 |
| @clarkb:matrix.org | for the audio version I mean | 22:49 |
| @mordred:waterwanders.com | yeah, they are. | 22:53 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!