Mailing-List: contact dev-help@cloudstack.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cloudstack.apache.org
Received-SPF: error (nike.apache.org: local policy)
MIME-Version: 1.0
In-Reply-To: <20CF38CB4385CE4D9D1558D52A0FC058056C53@SJCPEX01CL03.citrite.net>
References: <20CF38CB4385CE4D9D1558D52A0FC058056C53@SJCPEX01CL03.citrite.net>
From: David Nalley <david@gnsa.us>
Date: Fri, 28 Jun 2013 23:43:36 -0400
Message-ID: 
 <CAKprHVat6zeNtxF9hFfqgTaVr54sPi4+ZPNnVPk9uTfKNigHXw@mail.gmail.com>
Subject: Re: [DISCUSS] If BVT breaks, revert the commits...
To: dev@cloudstack.apache.org
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

On Fri, Jun 28, 2013 at 8:18 PM, Alex Huang <Alex.Huang@citrix.com> wrote:
> After Dave's complain in the vmsync [MERGE] thread about BVT in horrible =
shape on master, I went around to figure out what exactly happened.  The be=
st I can figure is that after a certain merge (I will leave out which merge=
 as that's not important), BVT no longer runs automatically.  It was promis=
ed to be fixed and there are people who are actively fixing it but it's bee=
n in this way for about two weeks.  People running BVTs are working around =
the problem but it's not automated anymore and so it's no longer running on=
 master.  I understand people are nice and tried to be accommodating to oth=
er people by working around the problem but sometimes we just have to be an=
 arse.  So let me be that arse...
>
> New Rule....
> If BVT or automated regression tests break on master or any release branc=
h, we revert all commits that broke it.  It doesn't matter if they promise =
to fix it within the next hour.  If it's broken, the release manager will r=
evert the commits and developers must resubmit.  It sounds mean but it's th=
e only way this problem can be fixed.
>
> To avoid having a bunch of reverts and resubmits, the developers should b=
e able to request that BVT run on their branch and don't merge until BVT on=
 their branch is at 100%.  We will work on figuring out how to do that.
>
> Comments?
>
> --Alex

+100 - not only +100 but I will increment ASFBots $beverage counter a
few in your favor for suggesting this.

--David