Opened 7 years ago

Closed 7 years ago

#885 closed defect (fixed)

The ProjectArchive permission checker should use smaller batches with chmod commands

Reported by: Nicklas Nordborg Owned by: Nicklas Nordborg
Priority: major Milestone: Reggie v4.4
Component: net.sf.basedb.reggie Keywords:


The current implementation will search for all files with incorrect file permission (see #864). All changes are batched into a single command. We have noted that some files (running a manual find returns 19041 files with read permission for everyone) still have incorrect permissions and in the log file there are some error messages:

net.schmizz.sshj.transport.TransportException: Broken pipe
	at net.schmizz.sshj.transport.TransportImpl.write(
	at net.sf.basedb.reggie.ssh.SshHost.executeCmd(
	at net.sf.basedb.reggie.projectarchive.ProjectArchiveService.permissionCheck(
	at net.sf.basedb.reggie.projectarchive.ProjectArchiveService$
	at net.sf.basedb.util.timer.ThreadTimerTask$
Caused by: Broken pipe
	at Method)
	at net.schmizz.sshj.transport.TransportImpl.write(
	... 7 more

I think this could be related to that the service is trying to fix the permission of all 19k files at the same time. I think we should try to use smaller batches and see if the problem goes away.

Change History (1)

comment:1 Changed 7 years ago by Nicklas Nordborg

Resolution: fixed
Status: newclosed

(In [3918]) Fixes #885: The ProjectArchive? permission checker should use smaller batches with chmod commands

The chmod commands are now batched at most 100 at a time.

Note: See TracTickets for help on using tickets.