V6r11 fix rms 141125

1. Test existence of job in finalization: if absent, terminate OK
2. Allow that permissions are OK for part of a list of LFNs (when testing write access to parent directory). Allows removal of replicas even if one cannot be removed
3. Test SE validity before removing replica
4. Typo fixed in ProxyManager
5. Fix request timeout computation
6. Fail requests for users who are no longer in the system
Hi Andrei,
the file is closed when using this statement, and I don’t think this is complicated to read… Of course I can write

f = open(inputFileName,’r’)
lines = f.read()
lines = content.splitlines()
lfns = []
for line in lines:
     line = line.strip()
     lfn = line.split()[0]
     lfns.append( lfn)

but… his is considerably longer and in the end not easier to understand… 

Most changes are “patch level” except the FTS stuff, as I told you before. This is indeed well tested, as I needed it for launching FTS transfers “by hand” and monitor them…

Anyway thanks for the merge ;-)
Cheers,
Philippe

Le 6 déc. 2014 à 00:35, Andrei Tsaregorodtsev <notifications@github.com> a écrit :

> In DataManagementSystem/scripts/dirac-dms-clean-directory.py:
> 
> >    Script.showHelp()
> >    DIRACExit( -1 )
> >  else:
> >    inputFileName = args[0]
> >  
> > -if os.path.exists(inputFileName):
> > -  inputFile = open(inputFileName, 'r')
> > -  string = inputFile.read()
> > -  lfns = sortList(string.splitlines(), True)
> > -  inputFile.close()
> > +if os.path.exists( inputFileName ):
> > +  lfns = [lfn.strip().split()[0] for lfn in sorted( open( inputFileName, 'r' ).read().splitlines() )]
> This is very smart, but can I ask you to better split such statements in several readable lines so that others can easier read the code ( not only here, you have a habit of such style of coding ). This would allow also to close the open file which is a good practice even in a command.
> 
> —
> Reply to this email directly or view it on GitHub.
> 

-- 
Dr Philippe Charpentier
J09210, Physics Department, CERN, CH-1211 Genève 23
LHCb Experiment Distributed Computing Coordinator
Office 2-R-007, Tel : +41 22 767 4244 ,  GSM : +41 76 487 0167
mailto:Philippe.Charpentier@cern.ch  http://cern.ch/Philippe.Charpentier