cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Will Stevens <wstev...@cloudops.com>
Subject Re: PRs for 4.9 Release
Date Thu, 07 Apr 2016 17:47:31 GMT
Yes, if the PR is ONLY a test and does not touch any other code then we can
only run that test.  I agree with you.

If any code is changed outside the test, I always run the full suite.  Make
sense?

*Will STEVENS*
Lead Developer

*CloudOps* *| *Cloud Solutions Experts
420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6
w cloudops.com *|* tw @CloudOps_

On Thu, Apr 7, 2016 at 1:41 PM, Daan Hoogland <daan.hoogland@gmail.com>
wrote:

> running a new test in an environment should pass but should not require all
> other test being re-validated. SO what is the point of running all others?
> I am not saying we shouldn't regularly run all tests but in this case it
> adds no value AFAICT.
>
>
> On Thu, Apr 7, 2016 at 7:35 PM, Will Stevens <wstevens@cloudops.com>
> wrote:
>
> > Yes, I want to run tests against the tests.  I have found issues in some
> of
> > the tests not working as expected, so I add the test to the run and run
> > them.  I think it is worth it because then we know the test is valid
> > later.  I would rather have the author fix the test now if there are
> > problems than having to work through it later.
> >
> > I am going to be doing a big push on getting testing cleaned up, so when
> I
> > start going through all the tests and validating them, I want to reduce
> the
> > amount of work I have to do, so validating the tests at source makes
> sense.
> >  :)
> >
> > *Will STEVENS*
> > Lead Developer
> >
> > *CloudOps* *| *Cloud Solutions Experts
> > 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6
> > w cloudops.com *|* tw @CloudOps_
> >
> > On Thu, Apr 7, 2016 at 1:30 PM, Daan Hoogland <daan.hoogland@gmail.com>
> > wrote:
> >
> > > 1326 is just a test. it does not touch production code. Do we run
> > > regression tests against such PRs. seems a waste to me.
> > >
> > > On Thu, Apr 7, 2016 at 7:27 PM, Daan Hoogland <daan.hoogland@gmail.com
> >
> > > wrote:
> > >
> > > >
> > > > On Thu, Apr 7, 2016 at 7:26 PM, Daan Hoogland <
> daan.hoogland@gmail.com
> > >
> > > > wrote:
> > > >
> > > >> 1326 - master (*pending CI)
> > > >
> > > >
> > > > ​starting​
> > > >
> > > >
> > > >
> > > > --
> > > > Daan
> > > >
> > >
> > >
> > >
> > > --
> > > Daan
> > >
> >
>
>
>
> --
> Daan
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message