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

Issue 786728 link

Starred by 1 user
Status: Untriaged
Owner: ----
Cc:
█ robert...@chromium.org
chanli@chromium.org
lijeffrey@chromium.org
liaoyuke@chromium.org
st...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

infra_api_clients/ should not depend on common/findit_http_client.py

Project Member Reported by st...@chromium.org, Nov 18 2017 
infra_api_clients/ is expected to be self-contained and only depends on python standard lib, libs/ and gae_libs/.
If there is other dependency, we'd better remove.

Traceback (most recent call last):
  File "util_scripts/remote_queries/test_data.py", line 18, in <module>
    from infra_api_clients.codereview.gerrit import Gerrit
  File "util_scripts/remote_queries/../../infra_api_clients/codereview/gerrit.py", line 9, in <module>
    from common.findit_http_client import FinditHttpClient
  File "util_scripts/remote_queries/../../common/findit_http_client.py", line 8, in <module>
    from common import monitoring
  File "util_scripts/remote_queries/../../common/monitoring.py", line 5, in <module>
    import gae_ts_mon
ImportError: No module named gae_ts_mon

Comment 1 by robert...@chromium.org, Jan 5 2018

Cc: robert...@chromium.org
Labels: Type-Bug
Owner: ----
Status: Available (was: Assigned)
Project Member

Comment 2 by sheriffbot@chromium.org, Jan 7

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
►

Sign in to add a comment