nfs4: log abandoned movers with WARN

to correlate with later on coming BAD_STATEID acked-by: Karsten Schwank Target: master, 2.12, 2.11, 2.10 Require-notes: no Require-book: no
dCache · Mar 3, 2015 · f096b76 · f096b76
1 parent 6fc4132
commit f096b76
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/modules/dcache-nfs/src/main/java/org/dcache/chimera/nfsv41/mover/EDSOperationREAD.java b/modules/dcache-nfs/src/main/java/org/dcache/chimera/nfsv41/mover/EDSOperationREAD.java
@@ -42,7 +42,7 @@ public void process(CompoundContext context, nfs_resop4 result) {
 
             NfsMover mover = _activeIO.get(_args.opread.stateid);
             if(mover == null) {
-                throw new BadStateidException("No mover associated with given stateid");
+                throw new BadStateidException("No mover associated with given stateid: " + _args.opread.stateid);
             }
             mover.attachSession(context.getSession());
 

diff --git a/modules/dcache-nfs/src/main/java/org/dcache/chimera/nfsv41/mover/EDSOperationWRITE.java b/modules/dcache-nfs/src/main/java/org/dcache/chimera/nfsv41/mover/EDSOperationWRITE.java
@@ -46,7 +46,7 @@ public void process(CompoundContext context, nfs_resop4 result) {
 
             NfsMover mover = _activeIO.get( _args.opwrite.stateid);
             if (mover == null) {
-                throw new BadStateidException("No mover associated with given stateid");
+                throw new BadStateidException("No mover associated with given stateid: " + _args.opwrite.stateid);
             }
 
             mover.attachSession(context.getSession());

diff --git a/.../dcache-nfs/src/main/java/org/dcache/chimera/nfsv41/mover/NfsMoverValidationCallback.java b/.../dcache-nfs/src/main/java/org/dcache/chimera/nfsv41/mover/NfsMoverValidationCallback.java
@@ -48,7 +48,7 @@ public void noroute(CellPath path) {
     private void kill() {
         NfsMover mover = moverRef.get();
         if (mover != null) {
-            LOGGER.info("Killing abandoned mover: {}", mover);
+            LOGGER.warn("Killing abandoned mover: {}", mover);
             mover.disable(new CacheException(CacheException.THIRD_PARTY_TRANSFER_FAILED, "Abandoned mover"));
         }
     }