On Wed, Mar 16, 2016 at 11:31 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.ttar: Removing leading `/' from member names./tests/bugs/distribute/bug-860663.t ..1..15ok 1, LINENUM:23ok 2, LINENUM:24ok 3, LINENUM:26ok 4, LINENUM:27ok 5, LINENUM:30ok 6, LINENUM:32ok 7, LINENUM:35not ok 8 , LINENUM:40FAILED COMMAND: ! gluster --mode=script --wignore volume rebalance patchy fix-layout startok 9, LINENUM:42ok 10, LINENUM:43ok 11, LINENUM:45ok 12, LINENUM:47ok 13, LINENUM:50ok 14, LINENUM:51ok 15, LINENUM:55Failed 1/15 subtestsGiven 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.
In this particular case, the fix has already been posted on 3.7 branch and just not merged yet. Link http://review.gluster.org/#/c/13537/3
Thanks Sakshi .
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.Thanks,Raghavendra Talur
_______________________________________________ Gluster-devel mailing list Gluster-devel@xxxxxxxxxxx http://www.gluster.org/mailman/listinfo/gluster-devel