Skip to content

Commit

Permalink
PoolManager : set return code to CacheException.PERMISSION_DENIED if …
Browse files Browse the repository at this point in the history
…staging is not allowed due to stage protection.

Motivation:

If stage protection is enabled the return code
CacheException.FILE_NOT_ONLINE is used in reply to the doors
causing them to retry multiple times. A fail fast is desired in
these cases.

Modification:

Replace CacheException.FILE_NOT_ONLINE w/ CacheException.PERMISSION_DENIED.

Result:

Clients fail fast w/ permission denied on attemot to access NEARLINE
file if stage protection does not allow it.

Target: master
Request: 3.0
Request: 2.16
Acjed-by: Tigran Mkrtchyan
Patch: https://rb.dcache.org/r/9907/
  • Loading branch information
DmitryLitvintsev committed Nov 22, 2016
1 parent 0c0ee5b commit f70172a
Showing 1 changed file with 1 addition and 1 deletion.
Expand Up @@ -1204,7 +1204,7 @@ private void nextStep(RequestState state, int shouldContinue ){
_forceContinue = true;
_status = "Failed";
_log.debug("Subject is not authorized to stage");
_currentRc = CacheException.FILE_NOT_ONLINE;
_currentRc = CacheException.PERMISSION_DENIED;
_currentRm = "File not online. Staging not allowed.";
sendInfoMessage(
_currentRc , "Permission denied." + _currentRm);
Expand Down

0 comments on commit f70172a

Please sign in to comment.