Re: On backporting fixes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On Wed, Mar 16, 2016 at 11:59 PM, Atin Mukherjee <atin.mukherjee83@xxxxxxxxx> wrote:

-Atin
Sent from one plus one


On 16-Mar-2016 11:32 am, "Raghavendra Talur" <rtalur@xxxxxxxxxx> wrote:
>
> Hi,
>
> Lot many fixes to tests were found to be not back ported to 3.7 and other release branches.
> This causes tests to fail only in those branches and leaves the maintainers puzzled.
>
> Also, this seems to be the case with back porting code fixes too.
>
> I copied all the changes to tests/ dir on master to tests/ dir on 3.7 branch and posted a patch at http://review.gluster.org/#/c/13683/
>
> This is failing for ./tests/bugs/distribute/bug-860663.t test :
> [10:39:11] Running tests in file ./tests/bugs/distribute/bug-860663.t
> tar: Removing leading `/' from member names
> ./tests/bugs/distribute/bug-860663.t .. 
> 1..15
> ok 1, LINENUM:23
> ok 2, LINENUM:24
> ok 3, LINENUM:26
> ok 4, LINENUM:27
> ok 5, LINENUM:30
> ok 6, LINENUM:32
> ok 7, LINENUM:35
> not ok 8 , LINENUM:40
> FAILED COMMAND: ! gluster --mode=script --wignore volume rebalance patchy fix-layout start
> ok 9, LINENUM:42
> ok 10, LINENUM:43
> ok 11, LINENUM:45
> ok 12, LINENUM:47
> ok 13, LINENUM:50
> ok 14, LINENUM:51
> ok 15, LINENUM:55
> Failed 1/15 subtests 
>
> Given that it is a simple rebalance command that is failing I am assuming that a critical patch has not been back ported to 3.7, correct me if I am wrong.
>
> I request every developer to take responsibility of back porting patches.
>
>
> Corollary question: Our test-framework is now capable of disabling tests for certain OS, certain branch etc. I would like to propose that we stop having tests in main git repo. This will remove need to back port test only fixes.
Where are you going to host these .t files then?

Proposal was a separate git repo for tests. It seems very unlikely to maintain it separately though.
Basically what I am looking for is making tests/ dir independent of git branch but still part of the git repo.
 

>
>
> Thanks,
> Raghavendra Talur
>
>
> _______________________________________________
> Gluster-devel mailing list
> Gluster-devel@xxxxxxxxxxx
> http://www.gluster.org/mailman/listinfo/gluster-devel


_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
http://www.gluster.org/mailman/listinfo/gluster-devel

[Index of Archives]     [Gluster Users]     [Ceph Users]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux