diff options
| author | Soumya Koduri <skoduri@redhat.com> | 2018-10-29 14:41:26 +0530 |
|---|---|---|
| committer | Soumya Koduri <skoduri@redhat.com> | 2019-01-03 15:08:16 +0530 |
| commit | 1651c8c3546777ef3072ae83127f5aa47bb160b5 (patch) | |
| tree | d54c49158e7caa5409c632666ad6a6a1748f6390 /tests | |
| parent | e33a97ea1aa23222c5f9e5ccbd52d4cb682e9c09 (diff) | |
lease: Treat unlk request as noop if lease not found
When the glusterfs server recalls the lease, it expects
client to flush data and unlock the lease. If not it sets
a timer (starting from the time it sends RECALL request) and post
timeout, it revokes it.
Here we could have a race where in client did send UNLK
lease request but because of network delay it may have reached
after server revokes it. To handle such situations, treat
such requests as noop and return sucesss.
Change-Id: I166402d10273f4f115ff04030ecbc14676a01663
updates: bz#1655532
Signed-off-by: Soumya Koduri <skoduri@redhat.com>
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/features/glfs-lease.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/features/glfs-lease.c b/tests/features/glfs-lease.c index 1a375ecbeb3..7255770912f 100644 --- a/tests/features/glfs-lease.c +++ b/tests/features/glfs-lease.c @@ -247,7 +247,7 @@ testcase1_rd_lease () VERIFY_RESULT (6, ret, NONE); ret = unlk_read_lease (fd1, lid1); - VERIFY_RESULT (7, ret, SHUD_FAIL); + VERIFY_RESULT (7, ret, SHUD_PASS); ret = glfs_close (fd1); VERIFY_RESULT (8, ret, SHUD_PASS); @@ -270,7 +270,7 @@ testcase2_wr_lease () VERIFY_RESULT (1, ret, SHUD_FAIL); ret = unlk_write_lease (fd1, lid1); - VERIFY_RESULT (2, ret, SHUD_FAIL); + VERIFY_RESULT (2, ret, SHUD_PASS); ret = set_write_lease (fd1, lid1); VERIFY_RESULT (3, ret, SHUD_PASS); |