<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class=""><br class=""><div><br class=""><blockquote type="cite" class=""><div class="">On Jul 15, 2019, at 6:30 AM, Thierry Carrez <<a href="mailto:thierry@openstack.org" class="">thierry@openstack.org</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><span style="caret-color: rgb(0, 0, 0); font-family: BitstreamVeraSansMono-Roman; font-size: 13px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; float: none; display: inline !important;" class="">Ghanshyam Mann wrote:</span><br style="caret-color: rgb(0, 0, 0); font-family: BitstreamVeraSansMono-Roman; font-size: 13px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><blockquote type="cite" style="font-family: BitstreamVeraSansMono-Roman; font-size: 13px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""> ---- On Sun, 14 Jul 2019 05:38:28 +0900 Sean McGinnis <<a href="mailto:sean.mcginnis@gmx.com" class="">sean.mcginnis@gmx.com</a>> wrote ----<br class=""> > On Sat, Jul 13, 2019 at 09:19:35PM +0900, Ghanshyam Mann wrote:<br class=""> > > ---- On Sat, 13 Jul 2019 21:16:32 +0900 Ghanshyam Mann <<a href="mailto:gmann@ghanshyammann.com" class="">gmann@ghanshyammann.com</a>> wrote ----<br class=""> > > > Hi Release team,<br class=""> > > ><br class=""> > > > Today I noticed while doing patrole review that stable/stain has been created for patrole which is wrong.<br class=""> > > > Patrole is branchless[1] and I remember I have not requested the stable branch while releasing the patrole.<br class=""> > > ><br class=""> > > > Is it created by mistakenly ? or intentional?<br class=""> > ><br class=""> > > I found the patch which created this - <a href="https://review.opendev.org/#/c/650173/1" class="">https://review.opendev.org/#/c/650173/1</a>.<br class=""> > ><br class=""> > > Can we revert that but that patch has more changes?<br class=""> > ><br class=""> > > -gmann<br class=""> > ><br class=""> ><br class=""> > We can't really just revert it. If you would like to change this, please update<br class=""> > the patrole release type to tempest-plugin or move it to be an independent<br class=""> > release deliverable if it is not actually cycle based.<br class=""> ><br class=""> > You can also remove the branching information with that change, then after that<br class=""> > is gone and there isn't a risk of recreating it, the infra team may be able to<br class=""> > assist in deleting the existing branch.<br class="">Release model for patrole is 'cycle-with-intermediary' which is right, we do not need<br class="">to change that. We just need to remove the stable/stein and branch information which<br class="">was added for the only stein. I will push the change.<br class="">Can we have a tag which clearly says the branchless and no-stable branch nature for deliverables?<br class="">'tempest-plugins' was introduced for a different purpose. new tag can be applicable for other<br class="">deliverables also (current or in future). That can help to avoid these errrors.<br class=""></blockquote><br style="caret-color: rgb(0, 0, 0); font-family: BitstreamVeraSansMono-Roman; font-size: 13px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><span style="caret-color: rgb(0, 0, 0); font-family: BitstreamVeraSansMono-Roman; font-size: 13px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; float: none; display: inline !important;" class="">Creating a new release model or a new deliverable type sounds a bit overkill.</span><br style="caret-color: rgb(0, 0, 0); font-family: BitstreamVeraSansMono-Roman; font-size: 13px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><br style="caret-color: rgb(0, 0, 0); font-family: BitstreamVeraSansMono-Roman; font-size: 13px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><span style="caret-color: rgb(0, 0, 0); font-family: BitstreamVeraSansMono-Roman; font-size: 13px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; float: none; display: inline !important;" class="">I think the simpler would be to add a new value to the existing "stable-branch-type" key. Like "stable-branch-type: none" and then set that value for all tempest plugins, tempest itself and patrole.</span><br style="caret-color: rgb(0, 0, 0); font-family: BitstreamVeraSansMono-Roman; font-size: 13px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><br style="caret-color: rgb(0, 0, 0); font-family: BitstreamVeraSansMono-Roman; font-size: 13px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><span style="caret-color: rgb(0, 0, 0); font-family: BitstreamVeraSansMono-Roman; font-size: 13px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; float: none; display: inline !important;" class="">See<span class="Apple-converted-space"> </span></span><a href="https://review.opendev.org/670808" style="font-family: BitstreamVeraSansMono-Roman; font-size: 13px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px;" class="">https://review.opendev.org/670808</a><span style="caret-color: rgb(0, 0, 0); font-family: BitstreamVeraSansMono-Roman; font-size: 13px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; float: none; display: inline !important;" class=""><span class="Apple-converted-space"> </span>as a strawman.</span><br style="caret-color: rgb(0, 0, 0); font-family: BitstreamVeraSansMono-Roman; font-size: 13px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><br style="caret-color: rgb(0, 0, 0); font-family: BitstreamVeraSansMono-Roman; font-size: 13px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><span style="caret-color: rgb(0, 0, 0); font-family: BitstreamVeraSansMono-Roman; font-size: 13px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; float: none; display: inline !important;" class="">--<span class="Apple-converted-space"> </span></span><br style="caret-color: rgb(0, 0, 0); font-family: BitstreamVeraSansMono-Roman; font-size: 13px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><span style="caret-color: rgb(0, 0, 0); font-family: BitstreamVeraSansMono-Roman; font-size: 13px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; float: none; display: inline !important;" class="">Thierry Carrez (ttx)</span></div></blockquote></div><br class=""><div class="">If the project isn’t creating stable branches each cycle, how is it cycle-based? What’s wrong with moving it to use the independent release model, which was created for cases like this?</div><div class=""><br class=""></div><div class="">Doug</div><div class=""><br class=""></div></body></html>