Thursday, 2023-09-28

*** ykarel|away is now known as ykarel04:53
*** dasm is now known as Guest155507:16
opendevreviewMerged openstack/project-config master: Allow Magnum cores to toggle WIP state  https://review.opendev.org/c/openstack/project-config/+/89648311:56
opendevreviewElod Illes proposed openstack/project-config master: [check-release-approval] Fix distributed leadership handling  https://review.opendev.org/c/openstack/project-config/+/89678512:00
fungielodilles: see also https://review.opendev.org/81549712:22
elodillesfungi: ah, i forgot about that patch12:23
opendevreviewRadosÅ‚aw Piliszek proposed openstack/project-config master: Add nebulous/slo-violation-detector  https://review.opendev.org/c/openstack/project-config/+/89679413:50
opendevreviewMerged openstack/project-config master: Add nebulous/slo-violation-detector  https://review.opendev.org/c/openstack/project-config/+/89679414:36
*** Guest1555 is now known as dasm14:48
*** d34dh0r5- is now known as d34dh0r5315:10
tinwood_fungi, qq: how hard is to delete old stable branches these days?  It used to be a stop the infra and do it all at once. (At least that's how I interpreted it - which may be very wrong!) Is it something that's relatively easy to do; we have a ton of branches we want to delete and just tag instead.16:06
fungitinwood_: oh, i don't think we ever needed downtime to delete branches, only to rename projects16:32
fungitinwood_: if the branches are stable branches in openstack projects that the release management team handles, talk to them as they usually take care of it in batches16:33
fungiif they're not under release management, you can give a group (probably your release group) permission in acls to delete branches directly16:34
fungitinwood_: in either case, you'll need to make sure any outstanding changes targeting those branches either get merged or abandoned, gerrit won't allow deletion of branches with open changes16:35
tinwood_ah, we control the release process, so it should be us; we may have to change the acls, and thanks for the heads up on patchs!  Okay, that sounds great!  thanks for the info.16:44
fungitinwood_: my pleasure, let me know if you need pointers, but if you grep the gerrit/acls directory in openstack/project-config for "delete" you should find examples16:45
tinwood_fungi, awesome; will do.16:45
fungitinwood_: looks like ironic's acl has it, for the same reasons16:46
tinwood_fungi, +116:47
JayFtinwood_: I can unearth the runbook I used to do this for the manually-managed bugfix branches, if that's useful16:57
JayFtinwood_: just git tag/branch delete commands but I found it comforting to have them prewritten and reviewed 16:57
tinwood_JayF, that would be great! thanks!16:58
JayFhttps://etherpad.opendev.org/p/IronicBugfixBranchCleanupRFR16:58
JayFobviously you'll have to replace the tag strings/shas/messages as needed16:59
tinwood_yeah, that's good; it's always handy to have the options on the commands!17:06
clarkbtinwood_: one thing to keep in mind is that we run regular git garbage collection on the repos we host. This is necessary for repacking in particular, but it will delete content if no more refs point to it.18:47
clarkbThis is why we tag branches before we delete them. The tag acts a ref to keep the content around when we garbage collect18:48
fungithough also, changes have named refs pointing to them which should keep their commits from being gc'd, so at worst you lose the merge commits that stitched them into the branch, not the changes themselves19:16
fungicatcher in the rye conspiracy19:22
fungiall garbage collection is, really19:23
fungi(no not really)19:23
opendevreviewGhanshyam proposed openstack/openstack-zuul-jobs master: Add stable/2023.2 to periodic-stable templates  https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/89683220:12

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