[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