opendevreview | Matthias Runge proposed openstack/project-config master: Panko has been deprecated and moved to unmaintained https://review.opendev.org/c/openstack/project-config/+/921090 | 06:03 |
---|---|---|
*** elodilles_ooo is now known as elodilles | 06:32 | |
opendevreview | Merged openstack/project-config master: Set inmotion max servers to 0 https://review.opendev.org/c/openstack/project-config/+/921072 | 07:10 |
opendevreview | Jan Marchel proposed openstack/project-config master: Add Ai Anomaly Detection component to NebulOuS https://review.opendev.org/c/openstack/project-config/+/920996 | 07:51 |
*** liuxie is now known as liushy | 08:41 | |
opendevreview | James Page proposed openstack/project-config master: Prep sunbeam single charm repos for retirement https://review.opendev.org/c/openstack/project-config/+/917827 | 09:11 |
opendevreview | Merged opendev/storyboard-webclient master: Stop using deprecated javascript jobs https://review.opendev.org/c/opendev/storyboard-webclient/+/921080 | 09:49 |
frickler | fyi promotion of ^^ failed with (I think) some dockerhub credentials issue https://zuul.opendev.org/t/openstack/build/7dd4739d05b544c399413b0ddf344a23 | 10:31 |
*** thuvh1 is now known as thuvh | 10:45 | |
Guest8493 | frickler: 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 Guest8598 | 15:14 | |
opendevreview | Monty Taylor proposed zuul/zuul-jobs master: Add support for installing node via NVM https://review.opendev.org/c/zuul/zuul-jobs/+/921304 | 15:56 |
clarkb | frickler: 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 |
clarkb | there 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 away | 16:04 |
clarkb | corvus: ^ any concerns with doing that in the zk db by hand? | 17:04 |
corvus | clarkb: if the provider is being deleted, you can use `nodepool erase` | 17:49 |
clarkb | corvus: ack maybe we do that after https://review.opendev.org/c/openstack/project-config/+/921074 lands | 17:50 |
clarkb | corvus: and ya the cloud is going away so that it can be redeployed on a more up to date platform | 17:51 |
clarkb | so I think we can go through those cleanup steps and whatever is left behind we run the erase command against | 17:51 |
corvus | yeah that sounds like a good plan | 18:27 |
opendevreview | Tony Breeds proposed opendev/system-config master: Add golang based docker compose tool. https://review.opendev.org/c/opendev/system-config/+/920760 | 18:31 |
opendevreview | Tony Breeds proposed opendev/system-config master: Fixups: for jitsi-meet https://review.opendev.org/c/opendev/system-config/+/921320 | 18:31 |
opendevreview | Tony Breeds proposed opendev/system-config master: Add an opendev specific build of mediawiki https://review.opendev.org/c/opendev/system-config/+/921321 | 18:31 |
opendevreview | Tony Breeds proposed opendev/system-config master: DNM: Initial dump or mediawiki role and config https://review.opendev.org/c/opendev/system-config/+/921322 | 18:31 |
clarkb | tonyb: 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 stack | 18:38 |
tonyb | Well the 2 mediawiki chnages rely on the golang docker compose tool, but Yeah I can move jitsi-meet to the top of the stack | 18:40 |
tonyb | But yes jitsi meet stuff is unrelated just things I noticed while doing the MediaWiki stuff | 18:41 |
opendevreview | Tony Breeds proposed opendev/system-config master: Fixups: for jitsi-meet https://review.opendev.org/c/opendev/system-config/+/921320 | 18:43 |
opendevreview | Tony Breeds proposed opendev/system-config master: Add golang based docker compose tool. https://review.opendev.org/c/opendev/system-config/+/920760 | 18:43 |
opendevreview | Tony Breeds proposed opendev/system-config master: Add an opendev specific build of mediawiki https://review.opendev.org/c/opendev/system-config/+/921321 | 18:43 |
opendevreview | Tony Breeds proposed opendev/system-config master: DNM: Initial dump or mediawiki role and config https://review.opendev.org/c/opendev/system-config/+/921322 | 18:43 |
clarkb | ah ok | 18:54 |
clarkb | tonyb: 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 change | 19:46 |
clarkb | until tomorrow morning if we want to try to confirm that first | 19:46 |
clarkb | corvus: 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 image | 19:49 |
clarkb | builds says it provides | 19:49 |
clarkb | corvus: 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 |
clarkb | corvus: I'm going to find lunch now, but will try to read the logs more carefully afterwards | 19:50 |
clarkb | I wonder if it is a side effect of using docker.io as the image location prefix | 19:51 |
clarkb | (just something I noticed in the error, but lunch now) | 19:52 |
corvus | https://zuul.opendev.org/t/openstack/build/e19f758a0f3d4d47bc8fa0d437bc43fc/log/zuul-info/inventory.yaml#131 <-- that shows it knew about the image build | 19:52 |
corvus | so it's not a provides/requires zuul config thing, it's a problem at the ansible job content level | 19:52 |
corvus | 3.1 vs 3.10 | 19:54 |
corvus | did the image build job have the tag as a float not a string? | 19:54 |
tonyb | Looks like it may have | 19:55 |
corvus | yep left a comment on https://review.opendev.org/920939 | 19:56 |
tonyb | - 3.10 vs - '3.10' | 19:56 |
* tonyb goes AFK for a bit | 19:57 | |
Clark[m] | Oh wow nice catch | 20:01 |
opendevreview | Clark Boylan proposed opendev/system-config master: Add Gerrit 3.10 image builds and testing https://review.opendev.org/c/opendev/system-config/+/920939 | 20:31 |
opendevreview | Clark Boylan proposed opendev/system-config master: Add Gerrit 3.10 upgrade testing https://review.opendev.org/c/opendev/system-config/+/921060 | 20:31 |
clarkb | lets see if that is happier | 20:31 |
clarkb | tonyb: I left some comments on the wiki announce etherpad surrounded by []s and then also edited some spelling things | 21:46 |
clarkb | corvus: fwiw fixing that yaml type error seems to ahve gotten jobs running as expected. Thank you for the extra eyeballs on that | 21:49 |
corvus | \o/ yw! | 21:50 |
clarkb | I 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 later | 21:50 |
clarkb | tonyb: I've also done a first pass at the mediwiki changes. Let me know if you have any questions about what I wrote | 22:47 |
clarkb | frickler: 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 changes | 22:50 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!