Friday, 2024-10-18

opendevreviewYoshi Kadokawa proposed openstack/project-config master: Add Cinder Huawei charm  https://review.opendev.org/c/openstack/project-config/+/86758905:06
opendevreviewYoshi Kadokawa proposed openstack/project-config master: Add Cinder Huawei charm  https://review.opendev.org/c/openstack/project-config/+/86758905:45
opendevreviewMerged opendev/system-config master: Add rabbitmq caching mirrors  https://review.opendev.org/c/opendev/system-config/+/93189711:13
Clark[m]fungi: I'm getting sorted after the school run. If you get a chance can you look at the gitea upgrade change? I think we can proceed with that if we are happy with it then depending on timing decide if we want to do Gerrit after as well14:49
fungisure, looking...14:54
opendevreviewMerged opendev/system-config master: Cleanup python 3.10 bullseye images  https://review.opendev.org/c/opendev/system-config/+/93210215:00
clarkbthanks!15:03
fungigitea change lgtm, i'm disappearing to grab lunch but should be around later to help test15:07
clarkbfungi: thanks, should I approve it in like half an hour or so to try and get it to land when you're back?15:09
clarkbI'll operate under that plan unless I haer otherwise. It takes a bit to gate too so we can always -2 it during that time15:10
fungii'm back and available to help once it's in16:50
fungilooks like you approved it around an hour ago16:51
clarkbyup I've been waiting patiently16:51
clarkbOnce that is done we can decide if we want to proceed with the gerrit update too: https://review.opendev.org/c/opendev/system-config/+/932314 I suspect we can do this Monday afternoon ish after PTG things as an alternative16:54
fungii'm around for the rest of today, so happy to help keep an eye on the gerrit upgrade too. today seems like a better choice than monday if we have time collectively17:01
clarkbya it should work for me as well17:02
clarkboh I'm noticing there may be a node hold for the gerrit update too17:04
clarkbfungi: https://200.225.47.13/ is the held node if that is useful in your reviewing17:06
funginice. sorry i've been kinda out of touch for the past few days but seems like tonyb was on the ball with this17:10
fungiseems working17:11
fungithe other thing that didn't help is that for some reason my workstation, which i rely on as a remote bastion when i'm travelling, apparently senses if the monitors get turned off and powers itself down completely, i guess it's something deep inside systemd power management but haven't remembered to investigate17:12
fungiso the last few times i've gone anywhere, when i turn my monitors off manually, i forget that's going to result in my workstation shutting down17:13
clarkbmine does not do this. However I do have the occasionaly (like once a quarter) kernel panic due to some bug in the amd driver when the monitor goes to sleep17:13
clarkbI don't think you use a desktop environment but some of them have fancy power management settings that could do stuff like that too17:14
fungithat too sounds bad, but i don't think that's what i'm seeing because it's cleanly shut down and fully powered off17:14
fungiand yeah, the closest thing i have to a desktop environment is ratpoison, but that's more of a window manager17:14
fungistartx is basically my desktop environment17:14
fungiand whatever commands i stick in my ~/.xinitrd17:14
clarkbthe gitea change should merge around when the hourly jobs end. Good timing17:16
opendevreviewMerged opendev/system-config master: Update Gitea to 1.22.3  https://review.opendev.org/c/opendev/system-config/+/93196617:27
clarkbhourly jobs are done too so this should move fairly quickly17:27
clarkbpromote job was successful the infra-prod job is starting now.17:28
clarkbgitea09 has updated. I can browse system-config there17:29
clarkbI am able to clone system-config from gitea09 as well17:32
clarkbthis seems to be generally working. Just need to make sure all 6 are happy when done and also that replication is working17:32
fungiyeah, 09 is working for me17:37
clarkball 6 are upgraded now17:37
clarkbhttps://zuul.opendev.org/t/openstack/build/869e83a4103949adb9de4d514f73654e and the job was successful. So I think we just need to look for replication success17:38
clarkbnothing has merged sicne the upgrade but we could try to fetch specific change refs /me looks for new patchests17:38
clarkbhttps://review.opendev.org/c/openstack/keystone/+/925020 patchset 15 would've been uploaded right around when gitea09 updated but I'm not finding anything newer17:39
clarkbI guess we wait for someone to push something then check. In the meantime fungi I guess you can review https://review.opendev.org/c/opendev/system-config/+/932314 and if that looks good to you we can approve it then plan to do a manual gerrit update later today after the images publish?17:40
fungiyep, also lgtm, approved17:42
clarkbthe process for that is wait for image promotion then docker-compose pull, docker-compose down, mv gerrit replication plugin waiting queue dir aside, docker-compose up -d, then wait for gerrit to start and then wait for diffs to render again17:44
clarkbhttps://review.opendev.org/c/starlingx/utilities/+/932555 I'm checking if this new patchset replicated now17:47
clarkb`git fetch origin refs/changes/55/932555/6` then `git show FETCH_HEAD` works and the sha matches the new patchset sha17:48
clarkbreplication seems ot be working17:48
fungilooks right17:56
clarkbcurrent timing says the change should merge in about 17 minutes. If that is accurate we can probably get this done before I find lunch18:17
opendevreviewMerged opendev/system-config master: Update Gerrit images to 3.9.7 and 3.10.2  https://review.opendev.org/c/opendev/system-config/+/93231418:30
clarkbfungi: ^ those need to promote but then I think we can proceed18:30
clarkbhttps://hub.docker.com/layers/opendevorg/gerrit/3.9/images/sha256-712af444c67136ec5b21dc3c46059698da795871a4d447a5526cc8e39df69921?context=explore image updated. Now we wait for the infra-prod-service-review job to end I guess just so that we don't interfere with one another18:33
clarkbI started a screen18:33
fungiyeah, was waiting to get the notifications from the promote pipeline18:33
fungiconnected18:33
clarkbopendevorg/gerrit   3.9       d78266d324d0 is the existing image18:33
fungilgtm18:33
clarkbnow to figure out the waiting queue path18:34
fungiyeah, "2 months ago" so should refresh when the promote finishes18:34
fungi(and after a pull)18:35
clarkblooks like the project-config chnge that merged shouldn't involve gerrit either18:35
clarkber that is about to merge18:35
clarkbfungi: I think we are ready to proceed? I've prepped the waiting queue move command in another window that I can copy over when that step is reached18:39
clarkbI also notified the openstack release team18:39
fungiagreed, promote/deploy finished18:39
clarkbok I'll pull the new image, down gerrit, move the waiting queue, then start gerrit again18:40
clarkbopendevorg/gerrit   3.9       9ba502024800 new image18:41
clarkbproceeding now18:41
clarkb[2024-10-18T18:42:35.312Z] [main] INFO  com.google.gerrit.pgm.Daemon : Gerrit Code Review 3.9.7-5-g71c50e4e33-dirty ready18:42
clarkbI can load changes but not all diffs yet18:43
fungithat looks right18:43
clarkbya the version looks good and early startup functionality is as expected (half working but not fully)18:43
clarkbin another couple of minutes everything else should be happy too I hope18:44
clarkb[DiskCache-Prune-0] INFO  com.google.gerrit.server.cache.h2.H2CacheImpl : Cache jdbc:h2:file:///var/gerrit/cache/gerrit_file_diff size (2.44g) is greater than maxSize (128.00m), pruning18:45
clarkbI noticed this in the logs. I suspect that we should consider increasing that cache size? Maybe to 512MB?18:45
clarkbI get diffs now on a random change I hadn't oepned in a long time18:46
clarkband in random change I have never loaded before18:46
fungiyeah, that does seem fairly small18:46
clarkb[DiskCache-Prune-0] INFO  com.google.gerrit.server.cache.h2.H2CacheImpl : Cache jdbc:h2:file:///var/gerrit/cache/git_modified_files size (290.89m) is greater than maxSize (128.00m), pruning18:46
clarkbthis is a different cache that maybe should be increased in size too18:46
clarkbI'll grep for DiskCache-Prune and see if there are any others then push up a change after lunch18:47
clarkbhttps://review.opendev.org/c/airship/promenade/+/932740 this change goet a new patchset after the restart18:47
clarkbso I think that covers being able to push changes. I'll check that new patchset replicated to gitea now too18:48
clarkbyup I'm able to `git fetch origin refs/changes/40/932740/2` and origin is: origin https://opendev.org/airship/promenade (fetch)18:49
clarkbfungi: anything else you think we should check?18:50
clarkbgit show FETCH_HEAD matches the ps2 sha18:50
funginot that i can think of, everything seems to be working for me18:51
clarkbhttps://paste.opendev.org/show/bk4pTIuQLCsWaF3dVVF7/ this is my grep for the cache pruning. A number of those are floating at the limit which I think we can leave alone18:52
clarkbI'll just bump up those that are obviously not keeping up18:52
clarkbfungi: should I close the screen session?18:53
fungiyeah, i think we're good now18:55
clarkbdone18:55
clarkbtonyb: thank you for preparing the change and holding a node to make this quick and easy18:55
opendevreviewMerged openstack/project-config master: Add #openstack-eventlet-removal to accessbot  https://review.opendev.org/c/openstack/project-config/+/93198318:56
clarkbok lunch now then I'll work on that cache change later18:57
JayFWould it be possible to get https://review.opendev.org/c/openstack/project-config/+/931991 merged today and the initial groups populated (I am a reasonable choice for that if you want)? I'd like to have this new core structure in place before the end of the PTG.19:08
fungimake it so19:15
fungier, engage?19:15
fungii can add rpittau as the initial user in those groups once the deploy job completes19:15
opendevreviewMerged openstack/project-config master: Proposed new Ironic core structure  https://review.opendev.org/c/openstack/project-config/+/93199119:20
JayFthanks fungi 20:03
JayFrpittau: ^ those groups should be created now, if you want to get them populated I can do the follow-up changes for project-config.20:03
fungii should be able to add him as the initial member after i get out of the meeting i'm in20:06
JayFhe's hopefully gone until monday morning at this point, given his tz20:07
JayFso no rush20:07
JayFthat message was left expecting he reads it Monday :D 20:07
fungiperfect!20:07
opendevreviewClark Boylan proposed opendev/system-config master: Increase size limits for some Gerrit caches  https://review.opendev.org/c/opendev/system-config/+/93276320:53
clarkbI tried to base that on what the docs say we should aim for while still being conservative about memory. In theory we should set those even larger but since memory is a scarce resource and the old defaults are 10m I suspect this will still be a big improvement20:54

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