Tuesday, 2024-06-04

opendevreviewMatthias Runge proposed openstack/project-config master: Panko has been deprecated and moved to unmaintained  https://review.opendev.org/c/openstack/project-config/+/92109006:03
*** elodilles_ooo is now known as elodilles06:32
opendevreviewMerged openstack/project-config master: Set inmotion max servers to 0  https://review.opendev.org/c/openstack/project-config/+/92107207:10
opendevreviewJan Marchel proposed openstack/project-config master: Add Ai Anomaly Detection component to NebulOuS  https://review.opendev.org/c/openstack/project-config/+/92099607:51
*** liuxie is now known as liushy08:41
opendevreviewJames Page proposed openstack/project-config master: Prep sunbeam single charm repos for retirement  https://review.opendev.org/c/openstack/project-config/+/91782709:11
opendevreviewMerged opendev/storyboard-webclient master: Stop using deprecated javascript jobs  https://review.opendev.org/c/opendev/storyboard-webclient/+/92108009:49
fricklerfyi promotion of ^^ failed with (I think) some dockerhub credentials issue https://zuul.opendev.org/t/openstack/build/7dd4739d05b544c399413b0ddf344a2310:31
*** thuvh1 is now known as thuvh10:45
Guest8493frickler: hrm. probably ok in this case, since we don't actually need new images published from this change - but also probably something that should be sorted eventually since those images are now the only publication artifact :) 13:32
*** dhill is now known as Guest859815:14
opendevreviewMonty Taylor proposed zuul/zuul-jobs master: Add support for installing node via NVM  https://review.opendev.org/c/zuul/zuul-jobs/+/92130415:56
clarkbfrickler: your held nodes are in inmotion. Do you still need those or can we clean them up safely as part of the inmotion wind down?16:04
clarkbthere are also 5 nodes that may be failing to delete. I think we may end up just removing those from the nodepool db by hand as the cloud will go away16:04
clarkbcorvus: ^ any concerns with doing that in the zk db by hand?17:04
corvusclarkb: if the provider is being deleted, you can use `nodepool erase`17:49
clarkbcorvus: ack maybe we do that after https://review.opendev.org/c/openstack/project-config/+/921074 lands17:50
clarkbcorvus: and ya the cloud is going away so that it can be redeployed on a more up to date platform17:51
clarkbso I think we can go through those cleanup steps and whatever is left behind we run the erase command against17:51
corvusyeah that sounds like a good plan18:27
opendevreviewTony Breeds proposed opendev/system-config master: Add golang based docker compose tool.  https://review.opendev.org/c/opendev/system-config/+/92076018:31
opendevreviewTony Breeds proposed opendev/system-config master: Fixups: for jitsi-meet  https://review.opendev.org/c/opendev/system-config/+/92132018:31
opendevreviewTony Breeds proposed opendev/system-config master: Add an opendev specific build of mediawiki  https://review.opendev.org/c/opendev/system-config/+/92132118:31
opendevreviewTony Breeds proposed opendev/system-config master: DNM: Initial dump or mediawiki role and config  https://review.opendev.org/c/opendev/system-config/+/92132218:31
clarkbtonyb: you may want to move the jitsi meet fixup onto its own stack as it seems unrelated to the other stuff. THen similarly put the two mediawiki chnages on their own stack18:38
tonybWell the 2 mediawiki chnages rely on the golang docker compose tool, but Yeah I can move jitsi-meet to the top of the stack18:40
tonybBut yes jitsi meet stuff is unrelated just things I noticed while doing the MediaWiki stuff18:41
opendevreviewTony Breeds proposed opendev/system-config master: Fixups: for jitsi-meet  https://review.opendev.org/c/opendev/system-config/+/92132018:43
opendevreviewTony Breeds proposed opendev/system-config master: Add golang based docker compose tool.  https://review.opendev.org/c/opendev/system-config/+/92076018:43
opendevreviewTony Breeds proposed opendev/system-config master: Add an opendev specific build of mediawiki  https://review.opendev.org/c/opendev/system-config/+/92132118:43
opendevreviewTony Breeds proposed opendev/system-config master: DNM: Initial dump or mediawiki role and config  https://review.opendev.org/c/opendev/system-config/+/92132218:43
clarkbah ok18:54
clarkbtonyb: https://review.opendev.org/c/opendev/system-config/+/921075 and https://review.opendev.org/c/openstack/project-config/+/921073 are next up for inmotion cleanup. For 921073 we still have nodepool nodes. Several stuck in deleting and three related to a hold for frickler. I suspect we can clean up the nodes frickler has held but not sure. We can probably delay on that change19:46
clarkbuntil tomorrow morning if we want to try to confirm that first19:46
clarkbcorvus: https://zuul.opendev.org/t/openstack/build/e19f758a0f3d4d47bc8fa0d437bc43fc/log/job-output.txt#16930 is the error from https://review.opendev.org/c/opendev/system-config/+/921060/ but you can see we have the soft dependency on the image build (which doesn't take effect here because the image build was in the parent chagne) and then also a requires on the image that the image19:49
clarkbbuilds says it provides19:49
clarkbcorvus: in a previous patchset I tried adding that job in the same change as the image build addition and that also failed with the same error with the soft dependency being enforced (so jobs ran one after another). I split things up like this in the latest patchset to see if the cross buildset behavior was different and it doesn't appear to be 19:50
clarkbcorvus: I'm going to find lunch now, but will try to read the logs more carefully afterwards19:50
clarkbI wonder if it is a side effect of using docker.io as the image location prefix19:51
clarkb(just something I noticed in the error, but lunch now)19:52
corvushttps://zuul.opendev.org/t/openstack/build/e19f758a0f3d4d47bc8fa0d437bc43fc/log/zuul-info/inventory.yaml#131  <-- that shows it knew about the image build19:52
corvusso it's not a provides/requires zuul config thing, it's a problem at the ansible job content level19:52
corvus3.1 vs 3.1019:54
corvusdid the image build job have the tag as a float not a string?19:54
tonybLooks like it may have19:55
corvusyep left a comment on https://review.opendev.org/92093919:56
tonyb - 3.10  vs - '3.10'19:56
* tonyb goes AFK for a bit19:57
Clark[m]Oh wow nice catch20:01
opendevreviewClark Boylan proposed opendev/system-config master: Add Gerrit 3.10 image builds and testing  https://review.opendev.org/c/opendev/system-config/+/92093920:31
opendevreviewClark Boylan proposed opendev/system-config master: Add Gerrit 3.10 upgrade testing  https://review.opendev.org/c/opendev/system-config/+/92106020:31
clarkblets see if that is happier20:31
clarkbtonyb: I left some comments on the wiki announce etherpad surrounded by []s and then also edited some spelling things21:46
clarkbcorvus: fwiw fixing that yaml type error seems to ahve gotten jobs running as expected. Thank you for the extra eyeballs on that21:49
corvus\o/ yw!21:50
clarkbI think that whole stack can probably be merged now as long as we're reasonably comfortable with the stoppage of 3.8 image builds. Oh side note we'll end up rebuilding 3.9 images when adding 3.10 images (to ensure they promote) and we probably want to restart gerrit on that later21:50
clarkbtonyb: I've also done a first pass at the mediwiki changes. Let me know if you have any questions about what I wrote22:47
clarkbfrickler: can you indicate if it is safe to cleanup your held nodes in inmotion if you end up around tomorrow? I'm happy to clean them up for you and then proceed with landing cleanup changes22:50

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