<div dir="ltr">Sandro,<div>I think we need to add this to the formal release process of ovirt, i.e:</div><div> - when is a new branch added to gerrit hooks as stable branch</div><div> - what kind of verification / ci we want to run on it..</div><div><br></div><div>e</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Dec 2, 2015 at 2:49 PM, David Caro <span dir="ltr"><<a href="mailto:dcaro@redhat.com" target="_blank">dcaro@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 12/02 13:35, Sandro Bonazzola wrote:<br>
> On Tue, Dec 1, 2015 at 6:42 PM, David Caro <<a href="mailto:dcaro@redhat.com">dcaro@redhat.com</a>> wrote:<br>
><br>
> > On 12/01 19:37, Tal Nisan wrote:<br>
> > > Well, if the integration team is OK with that then I guess it's up to the<br>
> > > branch owner indeed and as said it will be very helpful for the backport<br>
> > > process if we indeed have CI jobs and gerrit hooks on the 3.6.1 branch<br>
> > for<br>
> > > the following weeks until 3.6.1 is released<br>
> ><br>
> > Ok, waiting for reply from the integration team, not sure if all of them<br>
> > are<br>
> > monitoring the infra list though, you might ask on devel too (or cc them)<br>
> ><br>
><br>
> ack on my side<br>
<br>
</span>Added the 3.6.1 branch to the stable branches list, about the jobs, feel free<br>
to send a patch following the indications below (from my previous email) and<br>
I'll review+merge<br>
<div class="HOEnZb"><div class="h5"><br>
><br>
><br>
><br>
> ><br>
> > ><br>
> > > On Tue, Dec 1, 2015 at 7:30 PM, David Caro <<a href="mailto:dcaro@redhat.com">dcaro@redhat.com</a>> wrote:<br>
> > ><br>
> > > > On 12/01 19:25, Tal Nisan wrote:<br>
> > > > > Well, it's not my decision but of the integration team, for me as the<br>
> > > > > stable branches maintainer it will help a lot to have the gerrit<br>
> > hooks<br>
> > > > (I'd<br>
> > > > > say even a must) and CI (not a must but definitely nice to have)<br>
> > > ><br>
> > > ><br>
> > > > I disagree on that, I think that it's the projects owners that should<br>
> > know<br>
> > > > what do they want to test and when, not the integration team.<br>
> > > ><br>
> > > > In any case, can you ask whoever makes the decision to ack?<br>
> > > ><br>
> > > > ><br>
> > > > > On Tue, Dec 1, 2015 at 7:23 PM, David Caro <<a href="mailto:dcaro@redhat.com">dcaro@redhat.com</a>> wrote:<br>
> > > > ><br>
> > > > > ><br>
> > > > > > So a couple questions here:<br>
> > > > > ><br>
> > > > > > * Who decides which branches are stable?<br>
> > > > > > * Who decides which branches have ci?<br>
> > > > > ><br>
> > > > > > In the latter, you can easily add the extra branch to the vdsm<br>
> > project<br>
> > > > > > yaml:<br>
> > > > > ><br>
> > > > > ><br>
> > > > > ><br>
> > > ><br>
> > <a href="https://gerrit.ovirt.org/gitweb?p=jenkins.git;a=blob;f=jobs/confs/projects/vdsm/vdsm_standard.yaml;hb=refs/heads/master" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/gitweb?p=jenkins.git;a=blob;f=jobs/confs/projects/vdsm/vdsm_standard.yaml;hb=refs/heads/master</a><br>
> > > > > ><br>
> > > > > > There on line 55, just define another version:<br>
> > > > > ><br>
> > > > > > - 3.6.1:<br>
> > > > > > branch: ovirt-3.6.1<br>
> > > > > ><br>
> > > > > > And the jobs will be created automatically once merged<br>
> > > > > ><br>
> > > > > ><br>
> > > > > > (I know, it needs a bit of a cleanup, and we are working on even<br>
> > not<br>
> > > > > > having to<br>
> > > > > > change any yaml, but for now it's what we have)<br>
> > > > > ><br>
> > > > > > On 12/01 10:52, Tal Nisan wrote:<br>
> > > > > > > The 3.6.1 branch is not treated by the hooks as a stable branch<br>
> > [1]<br>
> > > > > > > Aside for that, CI is not running on new patches there<br>
> > > > > > ><br>
> > > > > > > [1]<br>
> > > > > > ><br>
> > > > > > > - Check TM::SKIP, not in a monitored branch (ovirt-engine-3.6<br>
> > > > > > > ovirt-engine-3.6.0 ovirt-engine-3.5 ovirt-engine-3.4<br>
> > > > ovirt-engine-3.3<br>
> > > > > > > engine_3.2 engine_3.1 ovirt-engine-3.5.2 ovirt-engine-3.4.0<br>
> > > > > > > ovirt-engine-3.3.4 ovirt-engine-3.3.3 ovirt-engine-3.3.2<br>
> > > > > > ovirt-engine-3.3.1)<br>
> > > > > > > - Check merged to previous::IGNORE, Not in stable branch<br>
> > > > > > > (['ovirt-engine-3.6', 'ovirt-engine-3.6.0',<br>
> > 'ovirt-engine-3.5',<br>
> > > > > > > 'ovirt-engine-3.4', 'ovirt-engine-3.3', 'ovirt-engine-3.5.2',<br>
> > > > > > > 'ovirt-engine-3.4.0', 'ovirt-engine-3.3.4',<br>
> > 'ovirt-engine-3.3.3',<br>
> > > > > > > 'ovirt-engine-3.3.2', 'ovirt-engine-3.3.1'])<br>
> > > > > ><br>
> > > > > > > _______________________________________________<br>
> > > > > > > Infra mailing list<br>
> > > > > > > <a href="mailto:Infra@ovirt.org">Infra@ovirt.org</a><br>
> > > > > > > <a href="http://lists.ovirt.org/mailman/listinfo/infra" rel="noreferrer" target="_blank">http://lists.ovirt.org/mailman/listinfo/infra</a><br>
> > > > > ><br>
> > > > > ><br>
> > > > > > --<br>
> > > > > > David Caro<br>
> > > > > ><br>
> > > > > > Red Hat S.L.<br>
> > > > > > Continuous Integration Engineer - EMEA ENG Virtualization R&D<br>
> > > > > ><br>
> > > > > > Tel.: <a href="tel:%2B420%20532%20294%20605" value="+420532294605">+420 532 294 605</a><br>
> > > > > > Email: <a href="mailto:dcaro@redhat.com">dcaro@redhat.com</a><br>
> > > > > > IRC: dcaro|dcaroest@{freenode|oftc|redhat}<br>
> > > > > > Web: <a href="http://www.redhat.com" rel="noreferrer" target="_blank">www.redhat.com</a><br>
> > > > > > RHT Global #: 82-62605<br>
> > > > > ><br>
> > > ><br>
> > > > --<br>
> > > > David Caro<br>
> > > ><br>
> > > > Red Hat S.L.<br>
> > > > Continuous Integration Engineer - EMEA ENG Virtualization R&D<br>
> > > ><br>
> > > > Tel.: <a href="tel:%2B420%20532%20294%20605" value="+420532294605">+420 532 294 605</a><br>
> > > > Email: <a href="mailto:dcaro@redhat.com">dcaro@redhat.com</a><br>
> > > > IRC: dcaro|dcaroest@{freenode|oftc|redhat}<br>
> > > > Web: <a href="http://www.redhat.com" rel="noreferrer" target="_blank">www.redhat.com</a><br>
> > > > RHT Global #: 82-62605<br>
> > > ><br>
> ><br>
> > --<br>
> > David Caro<br>
> ><br>
> > Red Hat S.L.<br>
> > Continuous Integration Engineer - EMEA ENG Virtualization R&D<br>
> ><br>
> > Tel.: <a href="tel:%2B420%20532%20294%20605" value="+420532294605">+420 532 294 605</a><br>
> > Email: <a href="mailto:dcaro@redhat.com">dcaro@redhat.com</a><br>
> > IRC: dcaro|dcaroest@{freenode|oftc|redhat}<br>
> > Web: <a href="http://www.redhat.com" rel="noreferrer" target="_blank">www.redhat.com</a><br>
> > RHT Global #: 82-62605<br>
> ><br>
> > _______________________________________________<br>
> > Infra mailing list<br>
> > <a href="mailto:Infra@ovirt.org">Infra@ovirt.org</a><br>
> > <a href="http://lists.ovirt.org/mailman/listinfo/infra" rel="noreferrer" target="_blank">http://lists.ovirt.org/mailman/listinfo/infra</a><br>
> ><br>
> ><br>
><br>
><br>
> --<br>
> Sandro Bonazzola<br>
> Better technology. Faster innovation. Powered by community collaboration.<br>
> See how it works at <a href="http://redhat.com" rel="noreferrer" target="_blank">redhat.com</a><br>
<br>
--<br>
David Caro<br>
<br>
Red Hat S.L.<br>
Continuous Integration Engineer - EMEA ENG Virtualization R&D<br>
<br>
Tel.: <a href="tel:%2B420%20532%20294%20605" value="+420532294605">+420 532 294 605</a><br>
Email: <a href="mailto:dcaro@redhat.com">dcaro@redhat.com</a><br>
IRC: dcaro|dcaroest@{freenode|oftc|redhat}<br>
Web: <a href="http://www.redhat.com" rel="noreferrer" target="_blank">www.redhat.com</a><br>
RHT Global #: 82-62605<br>
</div></div><br>_______________________________________________<br>
Infra mailing list<br>
<a href="mailto:Infra@ovirt.org">Infra@ovirt.org</a><br>
<a href="http://lists.ovirt.org/mailman/listinfo/infra" rel="noreferrer" target="_blank">http://lists.ovirt.org/mailman/listinfo/infra</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">Eyal Edri<br>Supervisor, RHEV CI<br>EMEA ENG Virtualization R&D<br>Red Hat Israel<br><br>phone: +972-9-7692018<br>irc: eedri (on #tlv #rhev-dev #rhev-integ)</div>
</div>