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

Issue 770989 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

rpc flight recorder fails to send metrics due to missing pyasn1.codec.ber

Project Member Reported by akes...@chromium.org, Oct 3 2017

Issue description

INFO:requests.packages.urllib3.connectionpool:Starting new HTTP connection (1): metadata.google.internal
WARNING:root:Failed to configure ts_mon, monitoring is disabled: No module named pyasn1.codec.ber
Traceback (most recent call last):
  File "/usr/local/autotest/venv/autotest_lib/site-packages/chromite/lib/ts_mon_config.py", line 92, in _SetupTsMonFromOptions
    config.process_argparse_options(options)
  File "/usr/local/autotest/site-packages/chromite/third_party/infra_libs/ts_mon/config.py", line 239, in process_argparse_options
    ca_certs=args.ts_mon_ca_certs)
  File "/usr/local/autotest/site-packages/chromite/third_party/infra_libs/ts_mon/common/monitors.py", line 126, in __init__
    credentials = credential_factory.create(self._SCOPES)
  File "/usr/local/autotest/site-packages/chromite/third_party/infra_libs/ts_mon/common/monitors.py", line 75, in create
    credentials = GoogleCredentials.from_stream(self.path)
  File "/usr/local/autotest/site-packages/chromite/third_party/oauth2client/client.py", line 1346, in from_stream
    credential_filename)
  File "/usr/local/autotest/site-packages/chromite/third_party/oauth2client/client.py", line 1444, in _get_application_default_credential_from_file
    from oauth2client import service_account
  File "/usr/local/autotest/site-packages/chromite/third_party/oauth2client/service_account.py", line 23, in <module>
    from pyasn1.codec.ber import decoder
ImportError: No module named pyasn1.codec.ber
INFO:root:Servers entered production: set([u'chromeos-server33.cbf.corp.google.com', u'chromeos-server14.mtv.corp.google.com', u'chromeos-server98.mtv.corp.google.com', u'chromeos-server104.mtv.corp.google.com', u'chromeos-server105.mtv.corp.google.com', u'chromeos-server102.mtv.co
rp.google.com', u'chromeos-server126.hot.corp.google.com', u'chromeos-server37.cbf.corp.google.com', u'chromeos-server133.hot.corp.google.com', u'chromeos-server109.mtv.corp.google.com', u'chromeos-server120.mtv.corp.google.com', u'chromeos-server36.cbf.corp.google.com', u'chromeos
-server13.cbf.corp.google.com', u'chromeos-server108.mtv.corp.google.com', u'chromeos-skunk-3.mtv.corp.google.com', u'chromeos-server5.hot.corp.google.com', u'chromeos-server32.cbf.corp.google.com', u'chromeos-skunk-5.mtv.corp.google.com', u'chromeos-server132.hot.corp.google.com',
 u'chromeos-server10.hot.corp.google.com', u'chromeos-server4.cbf.corp.google.com', u'chromeos-server128.hot.corp.google.com', u'chromeos-skunk-2.mtv.corp.google.com', u'chromeos-server26.mtv.corp.google.com', u'chromeos-server12.cbf.corp.google.com', u'chromeos-server129.hot.corp.
google.com', u'chromeos-skunk1.mtv.corp.google.com', u'chromeos-server130.hot.corp.google.com', u'chromeos-skunk-4.mtv.corp.google.com', u'chromeos-server100.mtv.corp.google.com', u'chromeos-server97.mtv.corp.google.com', u'chromeos-server27.mtv.corp.google.com', u'chromeos-server1
01.mtv.corp.google.com', u'chromeos-server110.mtv.corp.google.com', u'chromeos-server111.mtv.corp.google.com', u'chromeos-server125.hot.corp.google.com', u'chromeos-server118.mtv.corp.google.com', u'chromeos-server119.mtv.corp.google.com', u'chromeos-server35.cbf.corp.google.com', 
u'chromeos-server153.cbf.corp.google.com', u'chromeos-server134.hot.corp.google.com', u'chromeos-server135.hot.corp.google.com', u'chromeos-server154.cbf.corp.google.com', u'chromeos-server9.cbf.corp.google.com', u'chromeos-server131.hot.corp.google.com', u'chromeos-server155.cbf.c
orp.google.com', u'chromeos-server127.hot.corp.google.com', u'chromeos-server112.mtv.corp.google.com'])
INFO:root:cautotest:get_server_time success
WARNING:root:Flushing process has been closed (exit code 0), skipped sending metric 'SecondsDistribution'
INFO:root:cautotest:ping_db success
WARNING:root:Flushing process has been closed (exit code 0), skipped sending metric 'SecondsDistribution'
WARNING:root:Flushing process has been closed (exit code 0), skipped sending metric 'CounterMetric'
INFO:root:chromeos-server33.cbf.corp.google.com:get_server_time success

 
Cc: ayatane@chromium.org
Need to add pyasn to the autotest virualenv requirements
Cc: pho...@chromium.org
Owner: akes...@chromium.org
Status: Started (was: Untriaged)
Project Member

Comment 4 by bugdroid1@chromium.org, Oct 5 2017

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

commit 332a5e9fe74631057f3543073654384b6e4bd16c
Author: Aviv Keshet <akeshet@chromium.org>
Date: Thu Oct 05 12:16:56 2017

autotest: add pyasn1 requirements

BUG= chromium:770989 
TEST=bin/python_venv -c "from pyasn1.codec.ber import decoder"

Change-Id: I786670e823c8b165b3d4ae6bd0f2616a6cd12de1
Reviewed-on: https://chromium-review.googlesource.com/696668
Commit-Ready: Aviv Keshet <akeshet@chromium.org>
Tested-by: Aviv Keshet <akeshet@chromium.org>
Reviewed-by: Allen Li <ayatane@chromium.org>

[modify] https://crrev.com/332a5e9fe74631057f3543073654384b6e4bd16c/venv/requirements.txt

Does this need a push to prod to fix?
Owner: dgarr...@chromium.org
Yes.
-> secondary for push to prod.
Push tests are currently broken.  https://crbug.com/772097 

I have a fix I'll chump after review. Then I'll look to see if there is a process to force a push test run.
Owner: akes...@chromium.org
Status: Assigned (was: Started)
Waaah. Well, that dep is fixed, revealing another neceesary one:

INFO:requests.packages.urllib3.connectionpool:Starting new HTTP connection (1): metadata.google.internal
WARNING:root:Failed to configure ts_mon, monitoring is disabled: No module named rsa
Traceback (most recent call last):
  File "/usr/local/autotest/venv/autotest_lib/site-packages/chromite/lib/ts_mon_config.py", line 92, in _SetupTsMonFromOptions
    config.process_argparse_options(options)
  File "/usr/local/autotest/site-packages/chromite/third_party/infra_libs/ts_mon/config.py", line 239, in process_argparse_options
    ca_certs=args.ts_mon_ca_certs)
  File "/usr/local/autotest/site-packages/chromite/third_party/infra_libs/ts_mon/common/monitors.py", line 126, in __init__
    credentials = credential_factory.create(self._SCOPES)
  File "/usr/local/autotest/site-packages/chromite/third_party/infra_libs/ts_mon/common/monitors.py", line 75, in create
    credentials = GoogleCredentials.from_stream(self.path)
  File "/usr/local/autotest/site-packages/chromite/third_party/oauth2client/client.py", line 1346, in from_stream
    credential_filename)
  File "/usr/local/autotest/site-packages/chromite/third_party/oauth2client/client.py", line 1444, in _get_application_default_credential_from_file
    from oauth2client import service_account
  File "/usr/local/autotest/site-packages/chromite/third_party/oauth2client/service_account.py", line 25, in <module>
    import rsa
ImportError: No module named rsa
Status: Started (was: Assigned)
Project Member

Comment 10 by bugdroid1@chromium.org, Oct 6 2017

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

commit 4f14dec98140a2a669ee8b1756de997d8bd7b852
Author: Aviv Keshet <akeshet@chromium.org>
Date: Fri Oct 06 22:51:14 2017

autotest: add rsa and oauth2client to venv requirements

BUG= chromium:770989 
TEST=bin/python_venv -c 'import rsa'
TEST=bin/python_venv -c 'from oauth2client import service_account'

Change-Id: If38e208f95bcb118670ff9eb46ae1d25669e2a6f
Reviewed-on: https://chromium-review.googlesource.com/705874
Commit-Ready: Aviv Keshet <akeshet@chromium.org>
Tested-by: Aviv Keshet <akeshet@chromium.org>
Reviewed-by: Allen Li <ayatane@chromium.org>

[modify] https://crrev.com/4f14dec98140a2a669ee8b1756de997d8bd7b852/venv/requirements.txt

Status: Fixed (was: Started)

Sign in to add a comment