[PATCH v4] File Server Remote VSS Protocol server
Jeremy Allison
jra at samba.org
Mon Mar 30 12:18:29 MDT 2015
On Mon, Mar 30, 2015 at 04:09:29PM +0200, David Disseldorp wrote:
> On Fri, 27 Mar 2015 21:28:20 -0700, Jeremy Allison wrote:
>
> > On Fri, Mar 27, 2015 at 12:44:09PM +0100, David Disseldorp wrote:
> > > This patch-set adds a new File Server Remote VSS (Volume Shadow Copy
> > > Service) Protocol server, test infrastructure, documentation, and
> > > corresponding VFS layer changes to handle remote share snapshot
> > > requests.
> >
> > Pushed, with a minor cleanup in:
> >
> > fsrvp: prune shadow copies if associated path doesn't exist
>
> Thanks for the push. Looks like there's a build failure in the unit
> tests that I don't hit locally:
>
> In file included from ../source4/torture/local/local.c:25:
> default/source4/torture/rpc/proto.h:22: error: 'struct policy_handle' declared inside parameter list
> default/source4/torture/rpc/proto.h:22: error: its scope is only this definition or declaration, which is probably not what you want
> default/source4/torture/rpc/proto.h:22: error: 'struct dcerpc_binding_handle' declared inside parameter list
> ...
>
> The autogenerated torture/rpc/proto.h header isn't needed here anyway,
> as torture_local_fsrvp() is provided by torture/local/proto.h.
>
> Please squash this patch:
>
> diff --git a/source4/torture/local/local.c b/source4/torture/local/local.c
> index db729af..5d3b4e1 100644
> --- a/source4/torture/local/local.c
> +++ b/source4/torture/local/local.c
> @@ -22,7 +22,6 @@
> #include "torture/local/proto.h"
> #include "torture/ndr/proto.h"
> #include "torture/auth/proto.h"
> -#include "torture/rpc/proto.h"
> #include "../lib/crypto/test_proto.h"
> #include "lib/registry/tests/proto.h"
> #include "lib/replace/replace-testsuite.h"
>
> ...into "torture: add local FSRVP server state tests"
>
> Otherwise I can send the whole patch-set as a new round if you'd
> prefer.
Hmmm. Still failing with a missmatch on parameters:
[2(6)/1746 at 1s] samba.tests.docs
UNEXPECTED(failure): samba.tests.docs.samba.tests.docs.SmbDotConfTests.test_default_s3(none)
REASON: Exception: Exception: Traceback (most recent call last):
File "/memdisk/jra/a/b31007/samba/bin/python/samba/tests/__init__.py", line 113, in run
testMethod()
File "/memdisk/jra/a/b31007/samba/bin/python/samba/tests/docs.py", line 189, in test_default_s3
self._test_default(['bin/testparm'])
File "/memdisk/jra/a/b31007/samba/bin/python/samba/tests/docs.py", line 238, in _test_default
"Parameters that do not have matching defaults:"))
AssertionError: Parameters that do not have matching defaults:
fss: prune stale
Expected: no
Got:
fss: sequence timeout
Expected: 180 or 1800, depending on operation
Got:
Can you take a look ? Thanks,
Jeremy.
More information about the samba-technical
mailing list