diff options
author | Niels de Vos <ndevos@redhat.com> | 2015-11-19 15:57:54 +0100 |
---|---|---|
committer | Niels de Vos <ndevos@redhat.com> | 2015-11-21 01:18:25 -0800 |
commit | 9e096b42d9e6448175eecf0e9b979a644c6104b0 (patch) | |
tree | 3e3533c4ae306d73f2898c5ffe15b1bd6fcfbcd4 /rpc | |
parent | 5863be29f26e2ea53bee09c689c2103de9985fe7 (diff) |
tests: fix timeout in mount-nfs-auth.t
The mount timeout was too short. The normal configuration-change path
(construct graph, call reconfigure) and the auth-refresh path might in
effect run serially. Therefore we have to wait for the *sum* of those
two intervals. As with all too-short-timeout problems, the result was
that the test would run fine most of the time. However, it has caused
spurious failures on my own patches a half dozen times, and I have a
half dozen other emails about it nuking other people's as well (most
often but not always on NetBSD).
The fix, obviously, is to calculate and use the right timeout value for
NFS mount actions. Other actions and timeouts have been left alone.
Cherry picked from commit ad876d7a127cf56a3cca11c24ad2b20e1955f82b:
> Change-Id: Ic8f013c8c830e33c48bcc6d1b603d6d22a8ba3c5
> Signed-off-by: Jeff Darcy <jdarcy@redhat.com>
> Reviewed-on: http://review.gluster.org/12396
> Tested-by: NetBSD Build System <jenkins@build.gluster.org>
> Tested-by: Gluster Build System <jenkins@build.gluster.com>
> Reviewed-by: Kaleb KEITHLEY <kkeithle@redhat.com>
Change-Id: Ic8f013c8c830e33c48bcc6d1b603d6d22a8ba3c5
BUG: 1283679
Signed-off-by: Niels de Vos <ndevos@redhat.com>
Reviewed-on: http://review.gluster.org/12663
Tested-by: NetBSD Build System <jenkins@build.gluster.org>
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: jiffin tony Thottan <jthottan@redhat.com>
Diffstat (limited to 'rpc')
0 files changed, 0 insertions, 0 deletions