Wednesday, 2023-05-10

opendevreviewJeremy Stanley proposed openstack/project-config master: Add a convenient normalize-gerrit-acl tox testenv
*** amoralej|off is now known as amoralej07:21
*** Guest924 is now known as diablo_rojo_phone08:11
*** amoralej is now known as amoralej|lunch10:56
*** amoralej|lunch is now known as amoralej12:39
*** iurygregory_ is now known as iurygregory14:25
clarkbI've gone ahead and approved the three changes related to and including ^15:11
clarkbcorvus: ianw fwiw after sleeping on it I think I like the consistency of for users. It makes things slightly more complicated on the zuul setup side, but I think having consistent jobs for users ends up being simpler for them15:12
clarkbcorvus: it does need rereview if we want to proceed in that direction (sounds like we do)15:23
opendevreviewMerged openstack/project-config master: Add an "apply" transformation which applies all
opendevreviewMerged openstack/project-config master: Add a convenient normalize-gerrit-acl tox testenv
opendevreviewMerged openstack/project-config master: Make option indenting a selectable transformation
*** gthiemon1e is now known as gthiemonge15:55
clarkbfungi: did you want to weigh in on vs before we land one? I think corvus and I are heavily leaning to 882724 and ianw is +2 on that change now as well16:18
fungii can, but not until later today16:22
corvus+2 from me16:26
fungiwithout paying attention to which one was preferred by whom, i also liked the effect of 882724 making the behavior toggleable through a var, defaulting to the old workflow19:51
fungiand the fact that it finished implementation which was missing but documented in the playbook19:51
fungi+2'd but did not approve since i've got other things going on and won't be able to devote much attention if it causes unforeseen problems (though it shouldn't since the default behavior looks to be preserved)19:53
clarkbthanks I'm going to approve it now and once it lands I'll also approve the grafyaml updates which should exercise it19:58
clarkbcorvus: once that lands you may want to approve one or more of the zuul/* base python image updates as well as those should be a good sanity check too20:05
opendevreviewMerged zuul/zuul-jobs master: Gracefully handle use of intermediate registry in container upload role
clarkbgrafyaml change is approved20:12
opendevreviewMerged opendev/grafyaml master: Migrate grafyaml container images to
clarkb looks good to me so I Think this is working20:20
opendevreviewClark Boylan proposed opendev/grafyaml master: Fix up tag release container build jobs
clarkbeverything seems to have worked but I noticed ^ was missed20:25
clarkbI don't think anything else has had a similar miss so far20:26
opendevreviewClark Boylan proposed opendev/system-config master: Pull grafyaml from
clarkbfungi: ^ that is a minor typo fix if you have time for a quick rereview20:27
clarkbI'm going to look at cleaning up the change tags for things that have migrated now20:30
clarkbfor example on all the change tags should be removed20:32
clarkbthat seemed easy enough. I just need to make a list and go through them all20:33
clarkband that should now be done for all of the images that have been pushed so far20:40
clarkbI'm going to proceed with lodgeit now. Then after that is etherpad, refstack, hound20:43
opendevreviewMerged openstack/project-config master: Pull grafyaml from
corvusclarkb: you ready for ?20:51
clarkbcorvus: yes I think all of those changes on the zuul side are good to go20:52
clarkbthe base images are on quay now and the fix we just landed for change tags appears to be working so I think we're rolling forward now20:52
corvuscool, i'll approve the whole set then20:53
clarkbcorvus: on the opendev side I'm finishing up grafyaml and moving on to lodgeit. Then will approve the chnage that updates etherpad, hound, and refstack20:54
clarkbonce those are all done I need to go and write more changes and gitea, gerrit, mailman3 will be next up20:54
opendevreviewMerged opendev/lodgeit master: Move lodgeit image publication to
clarkblodgeit promotion failed. I'll take a look at that momentarily21:10
opendevreviewClark Boylan proposed opendev/lodgeit master: Fix quay publishing
clarkbargh I knew I needed to fix all of these and thought I had. Will try to be extra careful and check for that now21:12
clarkbanyway 882896 now needed21:13
opendevreviewClark Boylan proposed opendev/system-config master: Migrate jinja-init to
opendevreviewClark Boylan proposed opendev/system-config master: Migrate gitea images to
clarkbthese last two can be ignored for now. I won't get to them until tomorrow but I wanted a headstart on the conversion because we do things like buildkit21:18
clarkbchanges that need reviews today are and
corvusclarkb: i should be able to manually delete quay tags now right?21:20
clarkbcorvus: yes I believe so. I already did so for images under opendevorg/21:21
corvusi will do so for zuul now21:21
clarkbcorvus: you can select them in the web ui using check boxes then there is an action drop down to select delete tags. It gives you a little summar of what it will delete and you hit go21:21
corvusyeah, and i think you can filter for "change" and then hit the select-all checkbox21:22
corvusokiedokie all done21:26
opendevreviewClark Boylan proposed opendev/system-config master: Migrate gerrit images to
clarkbis anyone able to review 882894 and 882896? Would be good to have both grafyaml and lodgeit completely marked off so I can focus on other things :)21:41
fungii can in a sec21:44
fungii approved both of those21:48
opendevreviewMerged opendev/system-config master: Pull grafyaml from
opendevreviewMerged opendev/grafyaml master: Fix up tag release container build jobs
opendevreviewMerged opendev/lodgeit master: Fix quay publishing
clarkbThat worked ^ I've approved the change to pull lodgeit from the new location22:04
clarkbour complicated gerrit stuff isn't working with a naive translation of container build jobs22:08
opendevreviewClark Boylan proposed opendev/system-config master: Migrate gerrit images to
clarkbok I think maybe that will fix the issue I just saw22:16
opendevreviewMerged opendev/system-config master: Pull lodgeit from
clarkb new lodgeit image seems to work22:33
clarkbeverything in the quay eitherpad up to line 72 should be done now22:35
clarkbgerrit and gitea images appear to be building in the proposed changes too22:35
fungithe zuul change hit unit test failures in the gate23:28
opendevreviewClark Boylan proposed opendev/system-config master: Migrate gerrit images to

Generated by 2.17.3 by Marius Gedminas - find it at!