<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Le mer. 27 janv. 2021 à 11:23, Herve Beraud <<a href="mailto:hberaud@redhat.com">hberaud@redhat.com</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Le mar. 26 janv. 2021 à 18:35, Jeremy Stanley <<a href="mailto:fungi@yuggoth.org" target="_blank">fungi@yuggoth.org</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 2021-01-26 17:52:07 +0100 (+0100), Előd Illés wrote:<br>
[...]<br>
> 1. I've used the list_eol_stale_branches.sh [2] script to get the list of<br>
> such not-yet-deleted branches for Ocata [3]. They are all tagged with<br>
> 'ocata-eol', but stable/ocata branch still exists for them. Could you please<br>
> delete these? [3]<br>
<br>
I'm happy to, have you made sure any open reviews for those branches<br>
are abandoned first? Gerrit won't allow deletion of a branch with<br>
open reviews.<br></blockquote><div><br></div><div><div class="gmail_quote">I think we need a first round of inspection on these stale branches to see if opened patches exist and then if needed start discussion with teams to ask them to drop the patches those who have been found. <br></div><div class="gmail_quote"><br></div><div class="gmail_quote">I'll try to add this feature to check for opened patches within `list_eol_stale_branches.sh`.</div></div></div></div></blockquote><div> </div>I created a quick and dirty script [1] to inspect all these repos branches  (based on the list given previously) and only os-collect-config still contains unmerged patches [2].<div><br></div><div>[1] <a href="http://paste.openstack.org/show/802035/">http://paste.openstack.org/show/802035/</a></div><div>[2] <a href="https://review.opendev.org/q/project:openstack/os-collect-config+branch:stable/ocata">https://review.opendev.org/q/project:openstack/os-collect-config+branch:stable/ocata</a></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div> </div></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> 2. On the Release Team meeting [1] we were hinted that with the newer<br>
> version of gerrit (that was installed at the end of November) some<br>
> automation is possible through gerrit API in the future. Can I get some help<br>
> about where should I start with the automation? Which repository should I<br>
> look, where can the deletion being triggered ("similarly like branch<br>
> creation")?<br>
[...]<br>
<br>
The Gerrit REST API method for deleting branches is documented here:<br>
<br>
<a href="https://review.opendev.org/Documentation/rest-api-projects.html#delete-branch" rel="noreferrer" target="_blank">https://review.opendev.org/Documentation/rest-api-projects.html#delete-branch</a><br>
<br>
I'm not immediately sure where branch creation happens in the forest<br>
of our release automation, but I would expect deletion could be<br>
implemented similarly. Hopefully someone more intimately familiar<br>
with those jobs can chime in.<br>
<br>
The access control we'll need to grant to automation so that it can<br>
call that is documented here:<br>
<br>
<a href="https://review.opendev.org/Documentation/access-control.html#category_delete" rel="noreferrer" target="_blank">https://review.opendev.org/Documentation/access-control.html#category_delete</a><br>
<br>
It'll need to be added manually as a permission for the Release<br>
Managers group in our All-Projects global ACL which individual<br>
projects inherit, and this documentation updated accordingly:<br>
<br>
<a href="https://opendev.org/opendev/system-config/src/branch/master/doc/source/gerrit.rst" rel="noreferrer" target="_blank">https://opendev.org/opendev/system-config/src/branch/master/doc/source/gerrit.rst</a><br>
<br>
Happy to answer other questions as they arise.<br>
-- <br>
Jeremy Stanley<br>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div>Hervé Beraud</div><div>Senior Software Engineer at Red Hat</div><div>irc: hberaud</div><div><a href="https://github.com/4383/" target="_blank">https://github.com/4383/</a></div><div><a href="https://twitter.com/4383hberaud" target="_blank">https://twitter.com/4383hberaud</a><br></div><div>-----BEGIN PGP SIGNATURE-----<br><br>wsFcBAABCAAQBQJb4AwCCRAHwXRBNkGNegAALSkQAHrotwCiL3VMwDR0vcja10Q+<br>Kf31yCutl5bAlS7tOKpPQ9XN4oC0ZSThyNNFVrg8ail0SczHXsC4rOrsPblgGRN+<br>RQLoCm2eO1AkB0ubCYLaq0XqSaO+Uk81QxAPkyPCEGT6SRxXr2lhADK0T86kBnMP<br>F8RvGolu3EFjlqCVgeOZaR51PqwUlEhZXZuuNKrWZXg/oRiY4811GmnvzmUhgK5G<br>5+f8mUg74hfjDbR2VhjTeaLKp0PhskjOIKY3vqHXofLuaqFDD+WrAy/NgDGvN22g<br>glGfj472T3xyHnUzM8ILgAGSghfzZF5Skj2qEeci9cB6K3Hm3osj+PbvfsXE/7Kw<br>m/xtm+FjnaywZEv54uCmVIzQsRIm1qJscu20Qw6Q0UiPpDFqD7O6tWSRKdX11UTZ<br>hwVQTMh9AKQDBEh2W9nnFi9kzSSNu4OQ1dRMcYHWfd9BEkccezxHwUM4Xyov5Fe0<br>qnbfzTB1tYkjU78loMWFaLa00ftSxP/DtQ//iYVyfVNfcCwfDszXLOqlkvGmY1/Y<br>F1ON0ONekDZkGJsDoS6QdiUSn8RZ2mHArGEWMV00EV5DCIbCXRvywXV43ckx8Z+3<br>B8qUJhBqJ8RS2F+vTs3DTaXqcktgJ4UkhYC2c1gImcPRyGrK9VY0sCT+1iA+wp/O<br>v6rDpkeNksZ9fFSyoY2o<br>=ECSj<br>-----END PGP SIGNATURE-----<br><br></div></div></div></div></div></div></div></div></div></div></div></div></div></div>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div>Hervé Beraud</div><div>Senior Software Engineer at Red Hat</div><div>irc: hberaud</div><div><a href="https://github.com/4383/" target="_blank">https://github.com/4383/</a></div><div><a href="https://twitter.com/4383hberaud" target="_blank">https://twitter.com/4383hberaud</a><br></div><div>-----BEGIN PGP SIGNATURE-----<br><br>wsFcBAABCAAQBQJb4AwCCRAHwXRBNkGNegAALSkQAHrotwCiL3VMwDR0vcja10Q+<br>Kf31yCutl5bAlS7tOKpPQ9XN4oC0ZSThyNNFVrg8ail0SczHXsC4rOrsPblgGRN+<br>RQLoCm2eO1AkB0ubCYLaq0XqSaO+Uk81QxAPkyPCEGT6SRxXr2lhADK0T86kBnMP<br>F8RvGolu3EFjlqCVgeOZaR51PqwUlEhZXZuuNKrWZXg/oRiY4811GmnvzmUhgK5G<br>5+f8mUg74hfjDbR2VhjTeaLKp0PhskjOIKY3vqHXofLuaqFDD+WrAy/NgDGvN22g<br>glGfj472T3xyHnUzM8ILgAGSghfzZF5Skj2qEeci9cB6K3Hm3osj+PbvfsXE/7Kw<br>m/xtm+FjnaywZEv54uCmVIzQsRIm1qJscu20Qw6Q0UiPpDFqD7O6tWSRKdX11UTZ<br>hwVQTMh9AKQDBEh2W9nnFi9kzSSNu4OQ1dRMcYHWfd9BEkccezxHwUM4Xyov5Fe0<br>qnbfzTB1tYkjU78loMWFaLa00ftSxP/DtQ//iYVyfVNfcCwfDszXLOqlkvGmY1/Y<br>F1ON0ONekDZkGJsDoS6QdiUSn8RZ2mHArGEWMV00EV5DCIbCXRvywXV43ckx8Z+3<br>B8qUJhBqJ8RS2F+vTs3DTaXqcktgJ4UkhYC2c1gImcPRyGrK9VY0sCT+1iA+wp/O<br>v6rDpkeNksZ9fFSyoY2o<br>=ECSj<br>-----END PGP SIGNATURE-----<br><br></div></div></div></div></div></div></div></div></div></div></div></div></div></div>