Motivation:
The are many reasons why we should sign our artifacts. The main two are:

  - allow library users to verify who have release the jars
  - maven central accepts only signed artifacts (and this is the goal)

Modification:
Add profile that will be activated by maven release plugin or manually
with `-P sign-artifacts` option.

Result:
produced artifacts will be released.

Acked-by: Jürgen Starek
Target: master, 0.19

Motivation:
typically, every time when file's open/lock/layout state id is used to
change file's state, the sequence number is incremented. Thus newly
created stateid should have sequence zero, as it will be bumped during
processing. As currently sequence number initialized with one (1), fresh
created lock and layout state ids have sequence 2 when sent to the
clients.

Modification:
Initialize sequence number of newly created state ids with zero.
Increment open state id's sequence number for all state ids.

Result:
better spec compliance, as lock and layout state ids will start with
sequence 1.

Acked-by: Jürgen Starek
Target: master, 0.19

keep maven happy

Acked-by: Olufemi Adeyemi
Target: master

Motivation:
as nfs4j api is not stable yet, breaking changes have to be documented.

Modification:
add API-changes.md as well as warning in the README.md

Result:
library users are better informed.

Acked-by: Paul Millar
Acked-by: Lea Morschel
Acked-by: Olufemi Adeyemi
Target: master

Motivation:
When client want's to inform metadata server about errors or statistics
in its interactions with the layout, then LAYOUTERROR and LAYOUTSTATS
operations are called. This information have to be propagated to the pnfs
device manager.

Modification:
Add NFSv41DeviceManager#layoutError and NFSv41DeviceManager#layoutStats
methods. Update corresponding operations to pass requests to device manager.

Result:
Layout usage errors and statistics available to the pnfs device manager.

Acked-by: Paul Millar
Target: master

Acked-by: Paul Millar
Acked-by: Lea Morschel
Target: master

Acked-by: Paul Millar
Acked-by: Lea Morschel
Target: master

Acked-by: Paul Millar
Target: master

Acked-by: Paul Millar
Target: master

stats mode field must contain file system object type

Acked-by: Paul Millar
Target: master

Motivation:
improve test coverage

Modification:
Add OperationLAYOUTCOMMITTest to test leyout commit with read-only open
state.

Result:
better code coverage.

Acked-by: Paul Millar
Target: master

Acked-by: Paul Millar
Target: master

Motivation:
Current implementation of layout commit operation assumes that
only action that have to be done is updating file size. However,
the back-end system may perform other actions as well if required.

Modification:
Introduce NFSv41DeviceManager#layoutCommit method. The driver
implementation should take care to update file size, if required. The
sanity check of a valid state it is sill performed as a part of genetic
request processing.

Result:
The layout driver gets possibility to handle commit operation.

Acked-by: Albert Rossi
Acked-by: Paul Millar
Target: master

Motivation:
as findbugs is dead move to the successor

https://mailman.cs.umd.edu/pipermail/findbugs-discuss/2017-September/004383.html

Modification:
enable spotbugs plugin that can be activated by:
```
mvn spotbugs:spotbugs
```

Result:
use up-to-date static code analysis tool

Target: master
Acked-By: Paul Millar

Acked-by: Paul Millar
Acked-by: Lea Morschel
Target: master

Motivation:
pNFS operations have usually have multiple arguments. This require that
corresponding methods as well accept multiple arguments, which makes
code looks a bit messy.

Modification:
re-factor NFSv41DeviceManager interface to accept corresponding XDR
arguments without extracting each field.

Result:
simple code.

Acked-by: Paul Millar
Target: master

Acked-by: Lea Morschel
Target: master, 0.18

Motivation:
The NFSv4OperationFactory interface provides only a method to retrieve
nfs operation for a given request. The request execution is done by nfs
server itself. Though a different factories can be provided, the way
how requests are executed can't be changed. The example of such desired
change can be execution metrics, fault injection, notifications.

Modification:
Introduce OperationExecutor interface. Introduce AbstractOperationExecutor
that provides a generic implementation of request execution, but sill
allows subclasses to control which requests are supported. Removed
RequestExecutionTimeGauges and OperationFactoryMXBeanImpl as they can be
added by subclassing/wrapping an instance of OperationExecutor.

Though this is a breaking change, code migration should be easy:

MDSOperationFactory -> MDSOperationExecutor

        NFSServerV41 nfs4 = new NFSServerV41.Builder()
                .withExportTable(exportFile)
                .withVfs(vfs)
                .withOperationExecutor(new MDSOperationExecutor())
                .build();

Result:
A way to inject different strategies for request execution

Acked-by: Paul Millar
Target: master

deprecate NFSServerV41Factory#setExportFile in favor for new method

Acked-by: Paul Millar
Target: master

Modified V3 Components to Use ExportTable

Signed-off-by: Adam Marcionek <amarcionek@seven10storage.com>

Signed-off-by: Adam Marcionek <amarcionek@seven10storage.com>

Signed-off-by: Adam Marcionek <amarcionek@seven10storage.com>

isPesudoInode -> isPseudoInode

Reported-by: David Kocher <dkocher@iterate.ch>
Acked-by: Lea Morschel
Acked-by: Paul Millar
Target: master

Acked-by: Paul Millar
Target: master

improve code coverage

Acked-by: Paul Millar
Target: master

Motivation:
current code have some issues:

  - the pseudo file systems directory cookie starts with zero,
    which is not allowed.

  - listing in pseudo file system ignores provided cookie.

Modification:

update listPseudoDirectory to generate cookies stargin from 3,
as 0, 1 and 2 are reserved values. Return tail of listing started from
cookie, if provided.

Result:
fixed directory listing in pseudo file system with non zero initial
cookie.

Acked-by: Paul Millar
Target: master, 0.18

Acked-by: Paul Millar
Target: master

Motivation:
To build more complicated test cases without mocking all world a timple
file system suitable test is required.

Modification:
Adjust local file system based vfs implementation

https://github.com/kofemann/simple-nfs/blob/master/src/main/java/org/dcache/simplenfs/LocalFileSystem.java

to work with jimfs (Google's file system for testing).

Update existing PseudoFsTest to to use DummyVFS.

Result:
More complicated tests can be created.

Acked-by: Paul Millar
Target: master

Motivation:
NFSv4.2 have introduced some new operations including LAYOUTSTATS and
LAYOUTERROR. Tough those operations are not supported, we return error
illegal.

Modification:
Add dummy OperationLAYOUTERROR and OperationLAYOUTSTATS that returns
NFSERR_NOTSUPP.

Result:
client receives NFSERR_NOTSUPP instead of NFSERR_ILLEGAL. First building
block to add support for LAYOUTSTATS and LAYOUTERROR.

Acked-by: Paul Millar
Target: master

ensure, that readdir behind last entry returns empty directory listing.

Acked-by: Paul Millar
Target: master

Motivation:
latest version of hamcrest test suite to write more sophisticated test
cases.

Modification:
Update dependency in the pom file. Update test to avoid usage of
deprecated API.

Result:
up-to-date test infrastructure to enhance tests.

Acked-by: Paul Millar
Target: master

up-to-date version with java11 support

Acked-by: Paul Millar
Target: master

Acked-by: Paul Millar
Target: master