New issue
Advanced search Search tips

Issue 736947 link

Starred by 2 users

Issue metadata

Status: Duplicate
Owner:
Closed: Jul 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Errors talking to statsd

Project Member Reported by dgarr...@chromium.org, Jun 27 2017

Issue description

The current PreCQ launcher is failing with errors like this:

https://uberchromegw.corp.google.com/i/chromeos/builders/pre-cq-launcher/builds/9457

14:40:22: ERROR: unexpected error error(111, 'Connection refused') while sending data
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/statsd/connection.py", line 77, in send
    self.udp_sock.send(send_data)
error: [Errno 111] Connection refused



 
 Issue 737140  has been merged into this issue.
Owner: pprabhu@chromium.org
Status: Started (was: Untriaged)
This shouldn't ben an error, just a warning.

https://chromium-review.googlesource.com/c/546696/
removes this unused dependency.
Cc: pprabhu@chromium.org
Owner: akes...@chromium.org
https://chromium-review.googlesource.com/c/549188/ does more cleanup
You win :D

I'll drop my CL.
> https://chromium-review.googlesource.com/c/549188/ does more cleanup

That's not all the cleanup that's needed:

: jrbarnette $REPOS/cros.base/chromite; find * -name '*.py' | xgrep graphite | grep import        
cbuildbot/stages/sync_stages.py:from chromite.lib import graphite
cbuildbot/stages/report_stages.py:from chromite.lib import graphite
lib/cidb.py:from chromite.lib import graphite
lib/cros_test_lib.py:from chromite.lib import graphite
lib/graphite_lib/statsd_mock.py:from chromite.lib.graphite_lib import stats_es_mock
lib/graphite_lib/es_utils.py:  from chromite.lib.graphite_lib import elasticsearch_mock as elasticsearch
lib/graphite_lib/elasticsearch_mock_unittest.py:from chromite.lib.graphite_lib import elasticsearch_mock as elasticsearch
lib/graphite_lib/stats.py:  from chromite.lib.graphite_lib import statsd_mock as statsd
lib/graphite_lib/statsd_mock_unittest.py:from chromite.lib.graphite_lib import statsd_mock as statsd
lib/graphite_lib/elasticsearch_mock.py:from chromite.lib.graphite_lib import stats_es_mock
lib/graphite.py:from chromite.lib.graphite_lib import es_utils
lib/graphite.py:from chromite.lib.graphite_lib import stats
lib/graphite.py:from chromite.lib.graphite_lib import stats_es_mock
scripts/cbuildbot.py:from chromite.lib import graphite

I reckon that it's a new bug...

Project Member

Comment 6 by bugdroid1@chromium.org, Jun 28 2017

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

commit 9d79260557f371d08e19c8df52aba425032993e0
Author: Aviv Keshet <akeshet@chromium.org>
Date: Wed Jun 28 00:56:10 2017

cbuildbot: stop sending stats to statsd

statsd is so very deprecated.

BUG= chromium:736947 
TEST=None

Change-Id: I7cdde2947bd2d5d4f888d57c2ce281daabb02213
Reviewed-on: https://chromium-review.googlesource.com/549188
Commit-Ready: Aviv Keshet <akeshet@chromium.org>
Tested-by: Aviv Keshet <akeshet@chromium.org>
Reviewed-by: Don Garrett <dgarrett@chromium.org>

[modify] https://crrev.com/9d79260557f371d08e19c8df52aba425032993e0/lib/cidb.py
[modify] https://crrev.com/9d79260557f371d08e19c8df52aba425032993e0/cbuildbot/stages/sync_stages.py

Mergedinto: 667171
Status: Duplicate (was: Started)

Sign in to add a comment