I've reverted the original patch entirely. Our policy is to either mark the test as bad or revert the entire patch. This seems to have caused multiple failures in the test system, so I've reverted the entire patch. Please re-land the patch with any fixes as a fresh review.
--
On Wed, Apr 18, 2018 at 8:25 AM, Atin Mukherjee <amukherj@xxxxxxxxxx> wrote:
I request Kinglong and the owner of trash feature to debug this issue and send a fix which can revert back my change.commit d206fab73f6815c927a84171ee9361The above commit has caused (thanks to Amar for bisect!) trash.t test in upstream CI to fail very frequently. As per fstat.gluster.org (refer :https://bit.ly/2qGcSP6) this test has failed 17 times in master branch in last 4 days. Given we're nearing GlusterFS 4.1 branching and there're few important patches blocked in the regression pipeline queue, I've sent a patch https://review.gluster.org/c9b31557b1
Author: Kinglong Mee <mijinlong@xxxxxxxxxxx>
Date: Mon Apr 9 08:33:51 2018 -0400
storage/posix: add pgfid in readdirp if needed
Change-Id: I6745428fd9d4e402bf2cad52cee8ab46b7fd822f
fixes: bz#1560319
Signed-off-by: Kinglong Mee <mijinlong@xxxxxxxxxxx>19894 to mark trash.t as bad for now as a temporary arrangement.
_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
http://lists.gluster.org/mailman/listinfo/gluster-devel
--
nigelb
_______________________________________________ Gluster-devel mailing list Gluster-devel@xxxxxxxxxxx http://lists.gluster.org/mailman/listinfo/gluster-devel
