New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 828193 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Apr 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Apache fails to restart after a reboot

Project Member Reported by mattmallett@chromium.org, Apr 2 2018

Issue description

Apache service fails to start on reboot. Logged cause is this

```
+ [ ! -e /var/www/.boto ]
+ ln -s /home/moblab/.boto /var/www/.boto
ln: failed to access '/var/www/.boto': Permission denied
```

The addition of `set -e` causes the script to exit on error, before starting apache.

It seems that `test -e` doesn't actually detect a symlink, and we should use `test -L` instead.
 
Status: Started (was: Untriaged)
Project Member

Comment 2 by bugdroid1@chromium.org, Apr 3 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/board-overlays/+/921b9e7506373502552474ac464f0dc40ce7d298

commit 921b9e7506373502552474ac464f0dc40ce7d298
Author: Matt Mallett <mattmallett@chromium.org>
Date: Tue Apr 03 19:50:49 2018

[moblab] Fix apache failing to start on reboot

Change upstart script to check for an existing symlink "-L"
instead of an existing file "-e". "-e" doesn't detect a
symlink, apparently, causing the ln -s command to fail.

BUG= chromium:828193 
TEST=build and test on local device that apache starts after reboot

Change-Id: Id0e3a7f3214938c99bf81167773b0626c741f1c9
Reviewed-on: https://chromium-review.googlesource.com/991122
Commit-Ready: Matt Mallett <mattmallett@chromium.org>
Tested-by: Matt Mallett <mattmallett@chromium.org>
Tested-by: 佳燕 林 <lv10121007@gmail.com>
Reviewed-by: Keith Haddow <haddowk@chromium.org>

[modify] https://crrev.com/921b9e7506373502552474ac464f0dc40ce7d298/project-moblab/chromeos-base/chromeos-bsp-moblab/files/init/moblab-apache-init.conf

Status: Fixed (was: Started)

Sign in to add a comment