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