*** ykarel|away is now known as ykarel | 04:53 | |
*** dasm is now known as Guest1555 | 07:16 | |
opendevreview | Merged openstack/project-config master: Allow Magnum cores to toggle WIP state https://review.opendev.org/c/openstack/project-config/+/896483 | 11:56 |
---|---|---|
opendevreview | Elod Illes proposed openstack/project-config master: [check-release-approval] Fix distributed leadership handling https://review.opendev.org/c/openstack/project-config/+/896785 | 12:00 |
fungi | elodilles: see also https://review.opendev.org/815497 | 12:22 |
elodilles | fungi: ah, i forgot about that patch | 12:23 |
opendevreview | Radosław Piliszek proposed openstack/project-config master: Add nebulous/slo-violation-detector https://review.opendev.org/c/openstack/project-config/+/896794 | 13:50 |
opendevreview | Merged openstack/project-config master: Add nebulous/slo-violation-detector https://review.opendev.org/c/openstack/project-config/+/896794 | 14:36 |
*** Guest1555 is now known as dasm | 14:48 | |
*** d34dh0r5- is now known as d34dh0r53 | 15: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 |
fungi | tinwood_: oh, i don't think we ever needed downtime to delete branches, only to rename projects | 16:32 |
fungi | tinwood_: 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 batches | 16:33 |
fungi | if they're not under release management, you can give a group (probably your release group) permission in acls to delete branches directly | 16:34 |
fungi | tinwood_: 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 changes | 16: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 |
fungi | tinwood_: 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 examples | 16:45 |
tinwood_ | fungi, awesome; will do. | 16:45 |
fungi | tinwood_: looks like ironic's acl has it, for the same reasons | 16:46 |
tinwood_ | fungi, +1 | 16:47 |
JayF | tinwood_: I can unearth the runbook I used to do this for the manually-managed bugfix branches, if that's useful | 16:57 |
JayF | tinwood_: 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 |
JayF | https://etherpad.opendev.org/p/IronicBugfixBranchCleanupRFR | 16:58 |
JayF | obviously you'll have to replace the tag strings/shas/messages as needed | 16:59 |
tinwood_ | yeah, that's good; it's always handy to have the options on the commands! | 17:06 |
clarkb | tinwood_: 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 |
clarkb | This is why we tag branches before we delete them. The tag acts a ref to keep the content around when we garbage collect | 18:48 |
fungi | though 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 themselves | 19:16 |
fungi | catcher in the rye conspiracy | 19:22 |
fungi | all garbage collection is, really | 19:23 |
fungi | (no not really) | 19:23 |
opendevreview | Ghanshyam proposed openstack/openstack-zuul-jobs master: Add stable/2023.2 to periodic-stable templates https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/896832 | 20:12 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!