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

Issue 651301 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Feb 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

failure to configure ts_mon

Project Member Reported by kevcheng@chromium.org, Sep 29 2016

Issue description

http://chromeos-server70.hot.corp.google.com/afe/#tab_id=view_job&object_id=78515012

http://chromeos-server70.hot.corp.google.com/results/78515012-kevcheng/chromeos6-row1-rack2-host5/debug/autoserv.DEBUG

09/28 21:29:23.542 WARNI|     ts_mon_config:0095| Failed to configure ts_mon, monitoring is disabled: 'int' object has no attribute '__getitem__'
Traceback (most recent call last):
  File "/usr/local/autotest/site-packages/chromite/lib/ts_mon_config.py", line 90, in SetupTsMonGlobalState
    config.process_argparse_options(parser.parse_args(args=args))
  File "/usr/local/autotest/site-packages/chromite/third_party/argparse.py", line 1747, in parse_args
    args, argv = self.parse_known_args(args, namespace)
  File "/usr/local/autotest/site-packages/chromite/third_party/argparse.py", line 1779, in parse_known_args
    namespace, args = self._parse_known_args(args, namespace)
  File "/usr/local/autotest/site-packages/chromite/third_party/argparse.py", line 1820, in _parse_known_args
    option_tuple = self._parse_optional(arg_string)
  File "/usr/local/autotest/site-packages/chromite/third_party/argparse.py", line 2096, in _parse_optional
    if not arg_string[0] in self.prefix_chars:
TypeError: 'int' object has no attribute '__getitem__'
 

Comment 1 by pho...@chromium.org, Sep 29 2016

Status: Started (was: Untriaged)

Comment 3 by pho...@chromium.org, Sep 30 2016

It's just a warning - here's a fix https://chromium-review.googlesource.com/#/c/390438
Project Member

Comment 4 by bugdroid1@chromium.org, Oct 1 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/a4f6f9178ec80a4fbf1688f783a2996dac3ebb1f

commit a4f6f9178ec80a4fbf1688f783a2996dac3ebb1f
Author: Paul Hobbs <phobbs@google.com>
Date: Thu Sep 29 07:22:06 2016

ts_mon_config: Fix argument for short-lived processes.

Arguments need to be strings to be parsed correctly.

BUG= chromium:651301 
TEST=added a unittest, which fails until the change is made

Change-Id: Id6c8a1f79dde19467e0f616f151075ddc041c554
Reviewed-on: https://chromium-review.googlesource.com/390438
Commit-Ready: Paul Hobbs <phobbs@google.com>
Tested-by: Paul Hobbs <phobbs@google.com>
Reviewed-by: Paul Hobbs <phobbs@google.com>
Reviewed-by: Aviv Keshet <akeshet@chromium.org>

[modify] https://crrev.com/a4f6f9178ec80a4fbf1688f783a2996dac3ebb1f/lib/ts_mon_config.py
[modify] https://crrev.com/a4f6f9178ec80a4fbf1688f783a2996dac3ebb1f/lib/metrics_unittest.py

Comment 5 by pho...@chromium.org, Feb 21 2017

Status: Fixed (was: Started)

Sign in to add a comment