Thanks for the patch Poornima.
On Mon, Jul 25, 2016 at 10:40 AM, Poornima Gurusiddaiah <pgurusid@xxxxxxxxxx> wrote:
Hi,
Thanks Pranith. Here is the patch for the same: http://review.gluster.org/14997
The failure is not because there was a hang(the logs show that glfs_set_volfile_server was still executing), but because the EXPECT_WITHIN was not really waiting. And hence there was a race between the execution of the process gfapi_hang and the kill.
Regards,
Poornima
----- Original Message -----
> From: "Pranith Kumar Karampuri" <pkarampu@xxxxxxxxxx>
> To: "Soumya Koduri" <skoduri@xxxxxxxxxx>, "Poornima Gurusiddaiah"
> <pgurusid@xxxxxxxxxx>
> Cc: "Gluster Devel" <gluster-devel@xxxxxxxxxxx>
> Sent: Saturday, July 23, 2016 8:20:36 AM
> Subject: spurious failure in tests/basic/gfapi/libgfapi-fini-hang.t
> I see both of your names in git blame output.
> https://build.gluster.org/job/rackspace-regression-2GB-triggered/22439/console
> has more information about the failure. This failure happened on
> http://review.gluster.org/#/c/14985/ which changes only .t files so I
> believe the reason for the failure to be something else. Could you please
> take a look to find if there is a problem with the test or if it caught a
> race in some recent code change?
> --
> Pranith
--
Pranith
_______________________________________________ Gluster-devel mailing list Gluster-devel@xxxxxxxxxxx http://www.gluster.org/mailman/listinfo/gluster-devel
