| fungi | comparing against ip address (which is tracked in inventory) also works, granted you still have to cross-reference | 00:03 |
|---|---|---|
| corvus | yeah i compared the ips with the inventory diff | 00:03 |
| corvus | i think that's good enough. | 00:03 |
| fungi | i'd be in favor of also having instance ids in the inventory | 00:03 |
| corvus | #status log replaced all zuul-merger hosts with new jammy nodes | 00:03 |
| opendevstatus | corvus: finished logging | 00:03 |
| corvus | zuul seems to think it still has a full complement of mergers after i issued the deletes, so that's good :) | 00:04 |
| clarkb | corvus: with mergers I don't expect problems but for other services any issue with a ne whost showing up registering as if it were an existing host? | 00:05 |
| clarkb | I think nodepool builders us ethat info to determine if there are local things on disk to curate | 00:05 |
| clarkb | maybe component registration is smarter now. I think it may use a uuid ? I'll have to look closer later | 00:06 |
| *** mmalchuk_ is now known as mmalchuk | 00:37 | |
| mmalchuk_ | test | 00:49 |
| *** jroll3 is now known as jroll | 04:20 | |
| *** amoralej is now known as amoralej|lunch | 11:06 | |
| *** amoralej|lunch is now known as amoralej | 12:49 | |
| *** amoralej|off is now known as amoralej | 12:52 | |
| clarkb | I have approved https://review.opendev.org/c/opendev/system-config/+/884284 to continue the quay revert. No resync was necessary since I last updated accessbot | 15:12 |
| *** amoralej is now known as amoralej|off | 15:38 | |
| fungi | awesome | 15:48 |
| *** tosky_ is now known as tosky | 16:20 | |
| clarkb | fungi: thank you for looking into that failure and rechecking | 16:20 |
| fungi | np | 16:22 |
| fungi | we've had a few of those recently, i've noticed | 16:22 |
| opendevreview | Merged opendev/system-config master: Revert "Move system-config irc bots into quay.io" https://review.opendev.org/c/opendev/system-config/+/884284 | 17:27 |
| fungi | deploy finished | 17:39 |
| clarkb | sorry I got a bit distracted. I'll dobule check things then proceed with the next item in the list | 18:16 |
| clarkb | ya looks like those updated on eavesdrop as expected. | 18:20 |
| clarkb | next up is gerritbot. corvus do you want to review these changes (eg https://review.opendev.org/c/opendev/gerritbot/+/884286) or should I proceed? | 18:20 |
| corvus | lgtm | 18:24 |
| clarkb | ok approved as that was next in the list | 18:25 |
| opendevreview | Merged opendev/gerritbot master: Revert "Move gerritbot to quay.io" https://review.opendev.org/c/opendev/gerritbot/+/884286 | 18:35 |
| clarkb | promote looks good. I'll approve the system-config change to swap the source of the image now | 18:45 |
| clarkb | oh wait no I did a combined change for that one. SO I need to approve https://review.opendev.org/c/opendev/statusbot/+/884287 and https://review.opendev.org/c/openstack/ptgbot/+/884288 first | 18:46 |
| clarkb | corvus: ^ any chance you might be interested in those too? | 18:46 |
| corvus | done :) | 19:00 |
| clarkb | those are both on their way in now. Hopefully I can get the system-config update landed shortly after lunch | 19:08 |
| opendevreview | Merged opendev/statusbot master: Revert "Move statusbot to quay.io" https://review.opendev.org/c/opendev/statusbot/+/884287 | 19:20 |
| opendevreview | Merged openstack/project-config master: Add Kubernetes Power Manager app to StarlingX https://review.opendev.org/c/openstack/project-config/+/884250 | 19:29 |
| corvus | clarkb: fungi there's a conversation in #zuul:opendev.org about podman -- it's looking like the way we use bind mounts means that we either need to run podman as root, or need to run the command inside the container as root (but that's a fake userns root, it would really be the user that started the container). running podman as root is the path of least resistance (it's pretty similar to what we do now with docker. but i think we need to be | 19:34 |
| corvus | okay with this before proceeding (or else, further delay until we figure out a better solution). | 19:34 |
| clarkb | as mentioned in the zuul room I think this is roughly equivalent to running the way it was under docker. Maybe we should double check that and if so I think its fine as it is no worse | 19:37 |
| clarkb | ok approing the system-config update now | 20:02 |
| clarkb | it only had one +2 but is a straightforward change unlike some of the others | 20:03 |
| opendevreview | Merged opendev/system-config master: Revert "Move pull external IRC bot images from quay.io" https://review.opendev.org/c/opendev/system-config/+/884289 | 20:27 |
| fungi | that has deployed, as also evidenced by gerritbot leaving the channel due to a restart | 20:31 |
| clarkb | also statusbot | 20:32 |
| clarkb | quick someone push something to see it push messages :) | 20:33 |
| clarkb | https://review.opendev.org/c/opendev/grafyaml/+/884291 is next which will allow https://review.opendev.org/c/openstack/project-config/+/884293 https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/884294 and https://review.opendev.org/c/opendev/system-config/+/884292 to merge | 20:34 |
| clarkb | cc corvus if you have a second particularly for the first one in that list | 20:34 |
| clarkb | the others are straightforward but the job updtes are always worth a look if I can get that | 20:34 |
| corvus | all +2 | 20:42 |
| clarkb | thank you thank you | 20:45 |
| opendevreview | Merged opendev/grafyaml master: Revert "Migrate grafyaml container images to quay.io" https://review.opendev.org/c/opendev/grafyaml/+/884291 | 20:51 |
| clarkb | the promote job for that just succeeded. I'm approving the other changes now | 20:54 |
| clarkb | lodgeit is next up with https://review.opendev.org/c/opendev/lodgeit/+/884296 and https://review.opendev.org/c/opendev/system-config/+/884298 cc corvus | 20:56 |
| clarkb | then I'll abandon my changes for moving things to quay.io and we'll be "done" | 20:56 |
| opendevreview | Merged openstack/project-config master: Revert "Pull grafyaml from quay.io" https://review.opendev.org/c/openstack/project-config/+/884293 | 21:24 |
| opendevreview | Merged opendev/system-config master: Revert "Pull grafyaml from quay.io" https://review.opendev.org/c/opendev/system-config/+/884292 | 21:28 |
| *** janders1 is now known as janders | 21:34 | |
| clarkb | and now lodgeit | 21:47 |
| *** sfinucan is now known as stephenfin | 21:52 | |
| opendevreview | Merged opendev/lodgeit master: Revert "Move lodgeit image publication to quay.io" https://review.opendev.org/c/opendev/lodgeit/+/884296 | 22:00 |
| clarkb | the only changes not yet approved are the zuul changes. The last system-config change has been approved as promotion succeeded and we have a new image | 22:18 |
| fungi | cool! | 22:19 |
| fungi | thanks for working through those | 22:19 |
| opendevreview | Merged opendev/system-config master: Revert "Pull lodgeit from quay.io" https://review.opendev.org/c/opendev/system-config/+/884298 | 22:53 |
| fungi | that has deployed now too | 22:58 |
| clarkb | agreed and paste still works https://paste.opendev.org/show/bPFw66NxFCdsudS8ibjr/ | 23:03 |
| clarkb | That leaves us just with the zuul changes for pulling base images | 23:03 |
| clarkb | and I'll abandon the other changes I had pushed to move to quay to remove any confusion | 23:04 |
| clarkb | four changes abandoned and I think I can call this basically done | 23:05 |
| clarkb | thank you for the reviews and debugging and discussiond and planning that went into this. Was a "fun" one | 23:06 |
| ianw | thank you for driving all the not fun bits! | 23:14 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!