Issue 604426 link

Starred by 1 user

Issue metadata

Status:	Archived
Owner:	█ jrbarnette@chromium.org Last visit > 30 days ago
Closed:	Aug 2016
Components:	Infra>Client>ChromeOS
EstimatedDays:	----
NextAction:	----
OS:	Chrome
Pri:	2
Type:	Bug
VerifyIn-54 VerifyIn-56 VerifyIn-57 VerifyIn-58 VerifyIn-59 VerifyIn-60 VerifyIn-61

Lab inventory script fails if logdir doesn't exist

Reported by jrbarnette@chromium.org, Apr 18 2016

Issue description

The lab_inventory script fails if the selected logdir doesn't
exist.  The impact is that when bringing up a new autotest primary
server, the inventory script typically won't run, because autotest
doesn't create the directory, either.

We could change the installation procedure, but better, I think,
would be to change the inventory script to create a missing logdir.

Comment 1 by ccra...@apps.district833.org, Apr 18 2016

qwerty computer

Comment 2 by jrbarnette@chromium.org, Apr 18 2016

Owner: jrbarnette@chromium.org
Status: Started (was: Available)

Project Member

Comment 3 by bugdroid1@chromium.org, Apr 20 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/5af97406efacb4d9b0701aafc843f34f08e09cc2

commit 5af97406efacb4d9b0701aafc843f34f08e09cc2
Author: Richard Barnette <jrbarnette@chromium.org>
Date: Mon Apr 18 18:00:26 2016

[autotest] Create logdir in lab_inventory script.

When running the lab_inventory script, if the requested logs
directory doesn't exist, create it.

BUG= chromium:604426 
TEST=run the script locally, see the directory get created.

Change-Id: I48b85b7a83fed38e9e9059f7ce1bbac9900b4178
Reviewed-on: https://chromium-review.googlesource.com/339431
Commit-Ready: Richard Barnette <jrbarnette@chromium.org>
Tested-by: Richard Barnette <jrbarnette@chromium.org>
Reviewed-by: Paul Hobbs <phobbs@google.com>

[modify] https://crrev.com/5af97406efacb4d9b0701aafc843f34f08e09cc2/site_utils/lab_inventory.py