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

Issue 838605 link

Starred by 2 users

Issue metadata

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

Blocking:
issue 838612



Sign in to add a comment

Mobmonitor not starting

Project Member Reported by haddowk@chromium.org, May 1 2018

Issue description


In M66 the mobmonitor-init tries to start mobmonitor from

/usr/local/moblab 

However that directory does not exist.
 
Blocking: 838612
Status: Started (was: Assigned)
What version are you trying this on?

I just flashed R66-10452.78.0 and mobmonitor is running, directory /usr/local/moblab exists.

I checked if there were any missing cherry picks. All the work to move mobmonitor code to platform/moblab went straight into R66.
Problem only appears on a recovery image. Confirmed that the entire /usr/local/moblab directory does not exist when using a system recovery image.
Project Member

Comment 4 by bugdroid1@chromium.org, May 1 2018

Labels: merge-merged-release-R66-10452.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/moblab/+/a64b978b676c2e590a4f7f9a2e6c56b9f421aae8

commit a64b978b676c2e590a4f7f9a2e6c56b9f421aae8
Author: Matt Mallett <mattmallett@chromium.org>
Date: Tue May 01 19:53:00 2018

[moblab] Move mobmonitor to /etc/moblab/mobmonitor

/usr/local/moblab does not get built into a recovery
image, so we need to move the source onto something on
the root fs.

BUG= chromium:838605 
TEST=mobmonitor_unittest.py checkfile/manager_unittest.py

Change-Id: I5d0102fb41debfe33cf54dd957fc7b434663c7ff

[modify] https://crrev.com/a64b978b676c2e590a4f7f9a2e6c56b9f421aae8/src/mobmonitor/mobmonitor.py
[modify] https://crrev.com/a64b978b676c2e590a4f7f9a2e6c56b9f421aae8/src/mobmonitor/checkfile/manager.py

Project Member

Comment 5 by bugdroid1@chromium.org, May 1 2018

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

commit de0166d35afa5a049ada7c6186d588852161eef1
Author: Matt Mallett <mattmallett@chromium.org>
Date: Tue May 01 21:24:57 2018

[moblab] Move mobmonitor to /etc/moblab/mobmonitor

/usr/local/moblab does not get built into a recovery
image, so we need to move the source onto something on
the root fs.

BUG= chromium:838605 
TEST=build on local device, verify mobmonitor runs

Change-Id: I00f981ed8b7fab6f5a42ab595ad7f370e73e7bd1
Reviewed-on: https://chromium-review.googlesource.com/1038428
Reviewed-by: Keith Haddow <haddowk@chromium.org>
Commit-Queue: Keith Haddow <haddowk@chromium.org>
Tested-by: Keith Haddow <haddowk@chromium.org>

[modify] https://crrev.com/de0166d35afa5a049ada7c6186d588852161eef1/project-moblab/sys-apps/mobmonitor/mobmonitor-9999.ebuild
[modify] https://crrev.com/de0166d35afa5a049ada7c6186d588852161eef1/project-moblab/chromeos-base/chromeos-bsp-moblab/files/init/moblab-mobmonitor-init.conf

Project Member

Comment 6 by bugdroid1@chromium.org, May 2 2018

Labels: merge-merged-release-R67-10575.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/board-overlays/+/0ec9622150aa01f65261b62b0c64ca6a303675c2

commit 0ec9622150aa01f65261b62b0c64ca6a303675c2
Author: Matt Mallett <mattmallett@chromium.org>
Date: Wed May 02 18:16:59 2018

[moblab] Move mobmonitor to /etc/moblab/mobmonitor

/usr/local/moblab does not get built into a recovery
image, so we need to move the source onto something on
the root fs.

BUG= chromium:838605 
TEST=build on local device, verify mobmonitor runs

Change-Id: I00f981ed8b7fab6f5a42ab595ad7f370e73e7bd1
Reviewed-on: https://chromium-review.googlesource.com/1038307
Tested-by: Matt Mallett <mattmallett@chromium.org>
Tested-by: Keith Haddow <haddowk@chromium.org>
Reviewed-by: Keith Haddow <haddowk@chromium.org>
Commit-Queue: Keith Haddow <haddowk@chromium.org>
Trybot-Ready: Keith Haddow <haddowk@chromium.org>

[modify] https://crrev.com/0ec9622150aa01f65261b62b0c64ca6a303675c2/project-moblab/sys-apps/mobmonitor-ui/mobmonitor-ui-9999.ebuild
[modify] https://crrev.com/0ec9622150aa01f65261b62b0c64ca6a303675c2/project-moblab/sys-apps/mobmonitor/mobmonitor-9999.ebuild
[modify] https://crrev.com/0ec9622150aa01f65261b62b0c64ca6a303675c2/project-moblab/chromeos-base/chromeos-bsp-moblab/files/init/moblab-mobmonitor-init.conf

Project Member

Comment 7 by bugdroid1@chromium.org, May 2 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/moblab/+/7e5c803854141ba0910135bba0889f205abc2836

commit 7e5c803854141ba0910135bba0889f205abc2836
Author: Matt Mallett <mattmallett@chromium.org>
Date: Tue May 01 21:18:05 2018

[moblab] Move mobmonitor to /etc/moblab/mobmonitor

/usr/local/moblab does not get built into a recovery
image, so we need to move the source onto something on
the root fs.

BUG= chromium:838605 
TEST=mobmonitor_unittest.py checkfile/manager_unittest.py

Change-Id: I5d0102fb41debfe33cf54dd957fc7b434663c7ff

[modify] https://crrev.com/7e5c803854141ba0910135bba0889f205abc2836/src/mobmonitor/mobmonitor.py
[modify] https://crrev.com/7e5c803854141ba0910135bba0889f205abc2836/src/mobmonitor/checkfile/manager.py

Project Member

Comment 8 by bugdroid1@chromium.org, May 4 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/moblab/+/9fb3403bff2446bf7ee217dec63b972d970db85e

commit 9fb3403bff2446bf7ee217dec63b972d970db85e
Author: Matt Mallett <mattmallett@chromium.org>
Date: Fri May 04 01:32:19 2018

[moblab] Move mobmonitor to /etc/moblab/mobmonitor

/usr/local/moblab does not get built into a recovery
image, so we need to move the source onto something on
the root fs.

BUG= chromium:838605 
TEST=mobmonitor_unittest.py checkfile/manager_unittest.py

Change-Id: I5d0102fb41debfe33cf54dd957fc7b434663c7ff
Reviewed-on: https://chromium-review.googlesource.com/1038056
Commit-Ready: Matt Mallett <mattmallett@chromium.org>
Tested-by: Keith Haddow <haddowk@chromium.org>
Tested-by: Matt Mallett <mattmallett@chromium.org>
Reviewed-by: Keith Haddow <haddowk@chromium.org>

[modify] https://crrev.com/9fb3403bff2446bf7ee217dec63b972d970db85e/src/mobmonitor/mobmonitor.py
[modify] https://crrev.com/9fb3403bff2446bf7ee217dec63b972d970db85e/src/mobmonitor/checkfile/manager.py

Project Member

Comment 9 by bugdroid1@chromium.org, May 4 2018

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

commit 88ddb15c3a3aa212c9eb7589f74b5490c7cfee92
Author: Matt Mallett <mattmallett@chromium.org>
Date: Fri May 04 07:13:45 2018

[moblab] Move mobmonitor to /etc/moblab/mobmonitor

/usr/local/moblab does not get built into a recovery
image, so we need to move the source onto something on
the root fs.

BUG= chromium:838605 
TEST=build on local device, verify mobmonitor runs

Change-Id: I00f981ed8b7fab6f5a42ab595ad7f370e73e7bd1
Reviewed-on: https://chromium-review.googlesource.com/1038057
Commit-Ready: Matt Mallett <mattmallett@chromium.org>
Tested-by: Keith Haddow <haddowk@chromium.org>
Tested-by: Matt Mallett <mattmallett@chromium.org>
Reviewed-by: Keith Haddow <haddowk@chromium.org>

[modify] https://crrev.com/88ddb15c3a3aa212c9eb7589f74b5490c7cfee92/project-moblab/sys-apps/mobmonitor-ui/mobmonitor-ui-9999.ebuild
[modify] https://crrev.com/88ddb15c3a3aa212c9eb7589f74b5490c7cfee92/project-moblab/sys-apps/mobmonitor/mobmonitor-9999.ebuild
[modify] https://crrev.com/88ddb15c3a3aa212c9eb7589f74b5490c7cfee92/project-moblab/chromeos-base/chromeos-bsp-moblab/files/init/moblab-mobmonitor-init.conf

Status: Fixed (was: Started)

Sign in to add a comment