New issue
Advanced search Search tips

Issue 747542 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

Gerrit error in PreCQ?

Project Member Reported by jorgelo@chromium.org, Jul 21 2017

Issue description

export_to_gcloud: Unhandled exception:
Traceback (most recent call last):
  File "/b/c/cbuild/repository/chromite/bin/export_to_gcloud", line 93, in <module>
    main()
  File "/b/c/cbuild/repository/chromite/bin/export_to_gcloud", line 29, in main
    wrapper.DoMain()
  File "/b/c/cbuild/repository/chromite/scripts/wrapper.py", line 164, in DoMain
    commandline.ScriptWrapperMain(FindTarget)
  File "/b/c/cbuild/repository/chromite/lib/commandline.py", line 837, in ScriptWrapperMain
    ret = target(argv[1:])
  File "/b/c/cbuild/repository/chromite/scripts/export_to_gcloud.py", line 100, in main
    dslib.ChunkedBatchWrite(entities, c)
  File "/b/c/cbuild/repository/chromite/lib/dslib.py", line 59, in ChunkedBatchWrite
    except gcloud.exceptions.BadRequest:
NameError: global name 'gcloud' is not defined

And

Failed in stage PreCQLauncher:

<type 'exceptions.TypeError'>: 'NoneType' object has no attribute '__getitem__'
Traceback (most recent call last):
  File "/b/c/cbuild/repository/chromite/lib/failures_lib.py", line 228, in wrapped_functor
    return functor(*args, **kwargs)
  File "/b/c/cbuild/repository/chromite/cbuildbot/stages/sync_stages.py", line 1893, in PerformStage
    builder_run=self._run)
  File "/b/c/cbuild/repository/chromite/cbuildbot/validation_pool.py", line 501, in AcquirePool
    if pool.AcquireChanges(gerrit_query, ready_fn, change_filter):
  File "/b/c/cbuild/repository/chromite/cbuildbot/validation_pool.py", line 418, in AcquireChanges
    self, self.candidates, self.non_manifest_changes)
  File "/b/c/cbuild/repository/chromite/cbuildbot/stages/sync_stages.py", line 1805, in ProcessChanges
    pool, launchable_progress_map):
  File "/b/c/cbuild/repository/chromite/cbuildbot/stages/sync_stages.py", line 1485, in GetDisjointTransactionsToTest
    max_txn_length=self.MAX_PATCHES_PER_TRYBOT_RUN)
  File "/b/c/cbuild/repository/chromite/cbuildbot/validation_pool.py", line 1963, in CreateDisjointTransactions
    changes, max_txn_length=max_txn_length)
  File "/b/c/cbuild/repository/chromite/cbuildbot/patch_series.py", line 505, in CreateDisjointTransactions
    for change, plan, ex in self.CreateTransactions(changes, limit_to=changes):
  File "/b/c/cbuild/repository/chromite/cbuildbot/patch_series.py", line 474, in CreateTransactions
    plan = self.CreateTransaction(change, limit_to=limit_to)
  File "/b/c/cbuild/repository/chromite/cbuildbot/patch_series.py", line 454, in CreateTransaction
    cq_deps_seen, limit_to=limit_to)
  File "/b/c/cbuild/repository/chromite/cbuildbot/patch_series.py", line 109, in f
    return functor(self, parent, *args, **kwargs)
  File "/b/c/cbuild/repository/chromite/cbuildbot/patch_series.py", line 597, in _AddChangeToPlanWithDeps
    gerrit_deps, limit_to=limit_to)
  File "/b/c/cbuild/repository/chromite/cbuildbot/patch_series.py", line 413, in _LookupUncommittedChanges
    dep_change = self._GetGerritPatch(dep)
  File "/b/c/cbuild/repository/chromite/cbuildbot/patch_series.py", line 360, in _GetGerritPatch
    query_text, must_match=not git.IsSHA1(query_text))
  File "/b/c/cbuild/repository/chromite/lib/gerrit.py", line 181, in QuerySingleRecord
    results = self.Query(change, **query_kwds)
  File "/b/c/cbuild/repository/chromite/lib/gerrit.py", line 287, in Query
    x, self.host) for x in result]
  File "/b/c/cbuild/repository/chromite/lib/patch.py", line 1716, in ConvertQueryResults
    change_id = change['change_id'].split('~')[-1]
TypeError: 'NoneType' object has no attribute '__getitem__'
 
Cc: dgarr...@chromium.org
Owner: davidri...@chromium.org
gcloud is a command tool for google cloud. Is it default for the builder to have gcloud setup? 
No, it's not, mostly for versioning reasons.

1) It's not packaged as Ubuntu packages, which makes it require 'special' handling.
2) We need a way to manage versioning, that stays in track with branches other than TOT.
3) gcloud command line, gsutil commandline, and languages APIs used differently, and we sometimes need different versions of each.
Cc: -dgarr...@chromium.org ayatane@chromium.org
It IS available inside virtualenv though....

./infra_virtualenv/pip_packages/gcloud-0.18.1-cp27-none-any.whl
Components: Infra>Client>ChromeOS>CI
Components: -Infra>Client>ChromeOS
Status: Assigned (was: Untriaged)
This bug has an owner, thus, it's been triaged. Changing status to "assigned".

Sign in to add a comment