Release branch trybots are failing on "no such option: --git-cache-dir" |
|||||||
Issue descriptionOS: Chrome When attempting to run a release trybot on R53 or R54 the builder fails when it tries to call cbuildbot with --git-cache-dir. For example `./cbuildbot --remote --buildbot --branch=release-R54-8743.B lulu-cheets-release` resulted in https://uberchromegw.corp.google.com/i/chromiumos.tryserver/builders/cheets_release/builds/1/ which fails in bootstrap ... 10:00:25: INFO: RunCommand: chromite/bin/cbuildbot lulu-cheets-release --buildroot /b/cbuild/internal_master --buildnumber 1 --git-cache-dir /b/cros_git_cache --buildbot --branch release-R54-8743.B --remote-version 4 --sourceroot /b/build/slave/cheets_release/build --nobootstrap in /tmp/cbuildbot-tmpQgG8QD/tmpDg1PXP Usage: cbuildbot [options] buildbot_config [buildbot_config ...] cbuildbot: error: no such option: --git-cache-dir ... I suspect this is rooted in https://chromium-review.googlesource.com/387106 ?
,
Sep 27 2016
I think it depends on what comes up, for the particular case I was trying to get this trybot for we can probably get by with a ToT build. It is quite possible others will hit this eventually (e.g. someone building say an old firmware or factory branch manually) which may up the priority, the problem is when this does come up it will probably get urgent for the impacted project quickly. On the other hand it may be no one sees a problem for weeks.
,
Sep 27 2016
we do check for branch version before appending this option. I'll take a look what's going wrong.
,
Sep 27 2016
I'm working on a recipe patch to fix this.
,
Sep 27 2016
made a quick fix in remote_try. Looks like it's running. but it's remote try with --debug. https://uberchromegw.corp.google.com/i/chromiumos.tryserver/builders/cheets_release/builds/3
,
Sep 27 2016
removed --debug. Please find the job at https://uberchromegw.corp.google.com/i/chromiumos.tryserver/builders/cheets_release/builds/4 We'll work on the fix.
,
Sep 27 2016
,
Sep 28 2016
,
Sep 28 2016
,
Sep 29 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/chromite/+/bb79be9871a84063bdae21500debbfe76520afd6 commit bb79be9871a84063bdae21500debbfe76520afd6 Author: Don Garrett <dgarrett@google.com> Date: Tue Sep 27 18:14:07 2016 cbuildbot: Add api version to --git-cache-dir. Also, use the same version for the deprecated --repo-cache option. BUG= chromium:650713 TEST=run_tests Change-Id: I03c4636b87f93fe786079f59c7200bf22383e5c4 Reviewed-on: https://chromium-review.googlesource.com/389873 Reviewed-by: Don Garrett <dgarrett@chromium.org> Commit-Queue: Don Garrett <dgarrett@chromium.org> Tested-by: Don Garrett <dgarrett@chromium.org> [modify] https://crrev.com/bb79be9871a84063bdae21500debbfe76520afd6/scripts/cbuildbot.py [modify] https://crrev.com/bb79be9871a84063bdae21500debbfe76520afd6/cbuildbot/constants.py
,
Sep 29 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/tools/build.git/+/fae60ba54c83abbbfbe84b5533f6a3e8f9b55178 commit fae60ba54c83abbbfbe84b5533f6a3e8f9b55178 Author: dnj <dnj@chromium.org> Date: Thu Sep 29 19:41:03 2016 Base flags on bootstrapped Chromite branch. Currently, Chromite flags are chosen based on the checked-out Chromite barnch. However, this can be overridden by the "--branch" parameter. Update the recipe to check for this parameter and, if present, base its flag choices off that branch instaed of the bootstrapping Chromite branch. BUG= chromium:650713 TEST=expectations Review-Url: https://codereview.chromium.org/2374543006 [modify] https://crrev.com/fae60ba54c83abbbfbe84b5533f6a3e8f9b55178/scripts/slave/recipe_modules/chromite/api.py [modify] https://crrev.com/fae60ba54c83abbbfbe84b5533f6a3e8f9b55178/scripts/slave/recipe_modules/chromite/config.py [add] https://crrev.com/fae60ba54c83abbbfbe84b5533f6a3e8f9b55178/scripts/slave/recipes/cros/cbuildbot_tryjob.expected/release_branch_one_param.json [add] https://crrev.com/fae60ba54c83abbbfbe84b5533f6a3e8f9b55178/scripts/slave/recipes/cros/cbuildbot_tryjob.expected/release_branch_two_params.json [modify] https://crrev.com/fae60ba54c83abbbfbe84b5533f6a3e8f9b55178/scripts/slave/recipes/cros/cbuildbot_tryjob.py
,
Sep 29 2016
The following revision refers to this bug: https://chrome-internal.googlesource.com/chrome/tools/build_limited/scripts/slave/+/1059d421c173a18f95afc205fbbccfe8ea8e0d59 commit 1059d421c173a18f95afc205fbbccfe8ea8e0d59 Author: recipe-roller <recipe-roller@chromium.org> Date: Thu Sep 29 19:45:08 2016
,
Sep 29 2016
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/6c7de18dafd100a1ab1e014f29a4b9987b1c8bd0 commit 6c7de18dafd100a1ab1e014f29a4b9987b1c8bd0 Author: recipe-roller <recipe-roller@chromium.org> Date: Thu Sep 29 20:05:03 2016 Roll recipe dependencies (trivial). This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) build: https://crrev.com/fae60ba54c83abbbfbe84b5533f6a3e8f9b55178 Base flags on bootstrapped Chromite branch. (dnj@chromium.org) TBR=martiniss@chromium.org,phajdan.jr@chromium.org BUG= chromium:650713 Recipe-Tryjob-Bypass-Reason: Autoroller Bugdroid-Send-Email: False Review-Url: https://codereview.chromium.org/2380113003 [modify] https://crrev.com/6c7de18dafd100a1ab1e014f29a4b9987b1c8bd0/infra/config/recipes.cfg
,
Jun 7 2018
|
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by dgarr...@chromium.org
, Sep 27 2016