Hi Niels,

I think the changes look good and glad they are merged. Since your patch is merged, we can close the following: https://github.com/nfs-ganesha/ci-tests/pull/32 as that is tracking the same effort.

Also there are few more other pull requests waiting for merge. Could you please review and merge them?
https://github.com/nfs-ganesha/ci-tests/pulls

Once the patches are merged, then I can create separate Jenkins jobs for the new tests.

On Fri, May 11, 2018 at 4:57 PM, Niels de Vos <ndevos@redhat.com> wrote:
Hi Girjesh,

I just came across a mistake in the dbench test that runs in the CentOS
CI for each patch that gets sent to gerrithub. The test mount an
NFS-export over different versions of the NFS protocol, but the actual
run is using the local working directory instead of the mountpoint.

Could you please review the two patches I've posted here?
  https://github.com/nfs-ganesha/ci-tests/pull/36

There are currently quite a few patches in the queue for testing, so my
manual run with the changed client.sh script may need to wait a while
before it gets started. It is the expectation that
https://ci.centos.org/job/nfs_ganesha_dbench/344/console uses
https://raw.githubusercontent.com/nixpanic/nfs-ganesha_ci-tests/e9818ee888f3a7ca60571b1d7244a697703d782f/dbench/client.sh

Lets see if it indeed works well :)

Thanks,
Niels



--
Regards
Girjesh