My favorites | Sign in
Project Home Downloads Wiki Issues Source
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 3733: 2.12 fails to start with symlinked logs directory
3 people starred this issue and may be notified of changes. Back to list
Status:  Submitted
Owner:  ----
Closed:  Jan 6


Sign in to add a comment
 
Reported by sfl...@gmail.com, Dec 22, 2015
Affected Version: 2.12

What steps will reproduce the problem?
1. stop gerrit
2. move logs folder to a different location and create a symlink named logs instead
3. try to start gerrit (~/site_dir/bin/gerrit.sh start)

What is the expected output? What do you see instead?
expected: Starting Gerrit Code Review: OK
instead: Starting Gerrit Code Review: FAILED

Further information:
Gerrit used to work with this setup, 2.11.5 still works. With 2.12 it fails to start. Looks like gerrit is trying to create a logs directory even though a symlink and the target directory exist. Running the jar file directly shows the error messages:

$ java -jar ~/site_dir/bin/gerrit.war daemon -d ~/site_dir
[main] ERROR com.google.gerrit.pgm.Daemon : Unable to start daemon
com.google.gerrit.common.Die: Cannot create log
directory: /home/gerrit/site_dir/logs
        at
com.google.gerrit.common.FileUtil.mkdirsOrDie(FileUtil.java:94)
        at
com.google.gerrit.pgm.util.ErrorLogFile.start(ErrorLogFile.java:57)
        at com.google.gerrit.pgm.Daemon.start(Daemon.java:290)
        at com.google.gerrit.pgm.Daemon.run(Daemon.java:203)
        at
com.google.gerrit.pgm.util.AbstractProgram.main(AbstractProgram.java:64)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
        at
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:606)
        at
com.google.gerrit.launcher.GerritLauncher.invokeProgram(GerritLauncher.java:159)
        at
com.google.gerrit.launcher.GerritLauncher.mainImpl(GerritLauncher.java:100)
        at
com.google.gerrit.launcher.GerritLauncher.main(GerritLauncher.java:56)
        at Main.main(Main.java:25)
Caused by:
java.nio.file.FileAlreadyExistsException: /home/gerrit/site_dir/logs
        at
sun.nio.fs.UnixException.translateToIOException(UnixException.java:88)
        at
sun.nio.fs.UnixException.rethrowAsIOException(UnixException.java:102)
        at
sun.nio.fs.UnixException.rethrowAsIOException(UnixException.java:107)
        at
sun.nio.fs.UnixFileSystemProvider.createDirectory(UnixFileSystemProvider.java:383)
        at java.nio.file.Files.createDirectory(Files.java:630)
        at java.nio.file.Files.createAndCheckIsDirectory(Files.java:734)
        at java.nio.file.Files.createDirectories(Files.java:683)
        at
com.google.gerrit.common.FileUtil.mkdirsOrDie(FileUtil.java:91)
        ... 12 more
Jan 5 (5 days ago)
#1 mark.g.e...@gmail.com
I have pushed a patch-set that fixes this issue at https://gerrit-review.googlesource.com/#/c/73612/. Hopefully the fix gets into the next Gerrit release.
Jan 6 (4 days ago)
Project Member #2 huga...@gmail.com
(No comment was entered for this change.)
Status: ChangeUnderReview
Jan 6 (4 days ago)
Project Member #3 david.pu...@sonymobile.com
(No comment was entered for this change.)
Status: Submitted
Labels: FixedIn-2.12.1
Sign in to add a comment

Powered by Google Project Hosting