Thursday, 2023-05-25

fungicomparing against ip address (which is tracked in inventory) also works, granted you still have to cross-reference00:03
corvusyeah i compared the ips with the inventory diff00:03
corvusi think that's good enough.00:03
fungii'd be in favor of also having instance ids in the inventory00:03
corvus#status log replaced all zuul-merger hosts with new jammy nodes00:03
opendevstatuscorvus: finished logging00:03
corvuszuul seems to think it still has a full complement of mergers after i issued the deletes, so that's good :)00:04
clarkbcorvus: 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
clarkbI think nodepool builders us ethat info to determine if there are local things on disk to curate00:05
clarkbmaybe component registration is smarter now. I think it may use a uuid ? I'll have to look closer later00:06
*** mmalchuk_ is now known as mmalchuk00:37
*** jroll3 is now known as jroll04:20
*** amoralej is now known as amoralej|lunch11:06
*** amoralej|lunch is now known as amoralej12:49
*** amoralej|off is now known as amoralej12:52
clarkbI have approved to continue the quay revert. No resync was necessary since I last updated accessbot15:12
*** amoralej is now known as amoralej|off15:38
*** tosky_ is now known as tosky16:20
clarkbfungi: thank you for looking into that failure and rechecking16:20
fungiwe've had a few of those recently, i've noticed16:22
opendevreviewMerged opendev/system-config master: Revert "Move system-config irc bots into"
fungideploy finished17:39
clarkbsorry I got a bit distracted. I'll dobule check things then proceed with the next item in the list18:16
clarkbya looks like those updated on eavesdrop as expected.18:20
clarkbnext up is gerritbot. corvus do you want to review these changes (eg or should I proceed?18:20
clarkbok approved as that was next in the list18:25
opendevreviewMerged opendev/gerritbot master: Revert "Move gerritbot to"
clarkbpromote looks good. I'll approve the system-config change to swap the source of the image now18:45
clarkboh wait no I did a combined change for that one. SO I need to approve and first18:46
clarkbcorvus: ^ any chance you might be interested in those too?18:46
corvusdone :)19:00
clarkbthose are both on their way in now. Hopefully I can get the system-config update landed shortly after lunch19:08
opendevreviewMerged opendev/statusbot master: Revert "Move statusbot to"
opendevreviewMerged openstack/project-config master: Add Kubernetes Power Manager app to StarlingX
corvusclarkb: fungi there's a conversation in 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 be19:34
corvusokay with this before proceeding (or else, further delay until we figure out a better solution).19:34
clarkbas 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 worse19:37
clarkbok approing the system-config update now20:02
clarkbit only had one +2 but is a straightforward change unlike some of the others20:03
opendevreviewMerged opendev/system-config master: Revert "Move pull external IRC bot images from"
fungithat has deployed, as also evidenced by gerritbot leaving the channel due to a restart20:31
clarkbalso statusbot20:32
clarkbquick someone push something to see it push messages :)20:33
clarkb is next which will allow and to merge20:34
clarkbcc corvus if you have a second particularly for the first one in that list20:34
clarkbthe others are straightforward but the job updtes are always worth a look if I can get that20:34
corvusall +220:42
clarkbthank you thank you20:45
opendevreviewMerged opendev/grafyaml master: Revert "Migrate grafyaml container images to"
clarkbthe promote job for that just succeeded. I'm approving the other changes now20:54
clarkblodgeit is next up with and cc corvus20:56
clarkbthen I'll abandon my changes for moving things to and we'll be "done"20:56
opendevreviewMerged openstack/project-config master: Revert "Pull grafyaml from"
opendevreviewMerged opendev/system-config master: Revert "Pull grafyaml from"
*** janders1 is now known as janders21:34
clarkband now lodgeit21:47
*** sfinucan is now known as stephenfin21:52
opendevreviewMerged opendev/lodgeit master: Revert "Move lodgeit image publication to"
clarkbthe 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 image22:18
fungithanks for working through those22:19
opendevreviewMerged opendev/system-config master: Revert "Pull lodgeit from"
fungithat has deployed now too22:58
clarkbagreed and paste still works
clarkbThat leaves us just with the zuul changes for pulling base images23:03
clarkband I'll abandon the other changes I had pushed to move to quay to remove any confusion23:04
clarkbfour changes abandoned and I think I can call this basically done23:05
clarkbthank you for the reviews and debugging and discussiond and planning that went into this. Was a "fun" one23:06
ianwthank you for driving all the not fun bits!23:14

Generated by 2.17.3 by Marius Gedminas - find it at!