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

Issue 837463 link

Starred by 4 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Jun 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Feature



Sign in to add a comment

Mobmonitor should provide information about the base container and provide a button to force update

Project Member Reported by haddowk@chromium.org, Apr 27 2018

Issue description


On bootup sometimes network connections cause the base container not to be downloaded

There should be a way to check the size / correctness of the lxc containers and force update if a problem is suspected.

DEBUG:root:Running 'sudo test -e "/mnt/moblab/containers/test_4_1524789929_31556/rootfs"'
DEBUG:root:Running 'sudo test -e "/mnt/moblab/containers/test_3_1524789863_29369/rootfs"'
DEBUG:root:Running 'sudo test -e "/mnt/moblab/containers/base_05/rootfs"'
DEBUG:root:Running 'sudo test -e "/mnt/moblab/containers/test_2_1524789789_26450/rootfs"'
ERROR:root:Base container already exists. Set force_delete to True to force to re-stage base container. Note that this action will destroy all running test containers
DEBUG:root:Running 'sudo chown -R root "/usr/local/autotest/containers/base_05"'
DEBUG:root:Running 'sudo chgrp -R root "/usr/local/autotest/containers/base_05"'
+ sudo chmod -R +r /usr/local/autotest/containers/base_05 /usr/local/autotest/containers/base_05.tar.xz
+ logger -t moblab-base-container-init Ending.

 
Are these the commands to test the size/correctness of lxc containers?

What is the criteria for correctness?

How do you force an update of the container?
Status: Started (was: Assigned)
Project Member

Comment 3 by bugdroid1@chromium.org, May 31 2018

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

commit d6e0c978da39d91766e0ea23874d0c7e6274dca1
Author: Matt Mallett <mattmallett@chromium.org>
Date: Thu May 31 03:38:54 2018

[moblab] create check for lxc base container

Mobmonitor check that attempts to clone and start a new image
off of the base container. If it's unable to do this, tests
that rely on the base container will also likely not be able
to. Give the ability to repair the base container on a failed
mobmonitor check by forced redownloading it.

Refactored config reading that is duplicated in several places
into a util/config module.

BUG= chromium:837463 
TEST=base_container_check_unittest.py cloud_storage_speedtest_unittest.py
	config_unittest.py

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

[add] https://crrev.com/d6e0c978da39d91766e0ea23874d0c7e6274dca1/src/mobmonitor/checkfiles/moblab/base_container_check.py
[modify] https://crrev.com/d6e0c978da39d91766e0ea23874d0c7e6274dca1/src/mobmonitor/diagnostic_checks/disk_info.py
[add] https://crrev.com/d6e0c978da39d91766e0ea23874d0c7e6274dca1/src/mobmonitor/util/config.py
[modify] https://crrev.com/d6e0c978da39d91766e0ea23874d0c7e6274dca1/src/mobmonitor/diagnostic_checks/cloud_storage_speedtest_unittest.py
[add] https://crrev.com/d6e0c978da39d91766e0ea23874d0c7e6274dca1/src/mobmonitor/checkfiles/moblab/base_container_check_unittest.py
[add] https://crrev.com/d6e0c978da39d91766e0ea23874d0c7e6274dca1/src/mobmonitor/diagnostic_checks/repair_base_container_unittest.py
[add] https://crrev.com/d6e0c978da39d91766e0ea23874d0c7e6274dca1/src/mobmonitor/diagnostic_checks/base_container_clone_unittest.py
[add] https://crrev.com/d6e0c978da39d91766e0ea23874d0c7e6274dca1/src/mobmonitor/diagnostic_checks/repair_base_container.py
[modify] https://crrev.com/d6e0c978da39d91766e0ea23874d0c7e6274dca1/src/mobmonitor/checkfiles/devserver/gs_check.py
[modify] https://crrev.com/d6e0c978da39d91766e0ea23874d0c7e6274dca1/src/mobmonitor/checkfiles/moblab/moblab_actions.py
[modify] https://crrev.com/d6e0c978da39d91766e0ea23874d0c7e6274dca1/src/mobmonitor/diagnostic_checks/cloud_storage_speedtest.py
[add] https://crrev.com/d6e0c978da39d91766e0ea23874d0c7e6274dca1/src/mobmonitor/checkfiles/__init__.py
[add] https://crrev.com/d6e0c978da39d91766e0ea23874d0c7e6274dca1/src/mobmonitor/util/config_unittest.py
[add] https://crrev.com/d6e0c978da39d91766e0ea23874d0c7e6274dca1/src/mobmonitor/diagnostic_checks/base_container_clone.py
[modify] https://crrev.com/d6e0c978da39d91766e0ea23874d0c7e6274dca1/src/mobmonitor/README
[modify] https://crrev.com/d6e0c978da39d91766e0ea23874d0c7e6274dca1/src/mobmonitor/diagnostic_checks/manager.py

Status: Fixed (was: Started)

Sign in to add a comment