git-cl-try -b does not work for infra tryjobs |
|||||||
Issue description`git cl try -b "Infra Presubmit"` fails because it tries schedules the build on master.tryserver.infra, because apparently git-cl-try talks to experimental builders-map.appspot.com (source: https://cs.chromium.org/chromium/infra/appengine/experimental/builders_map/builders_map.py?sq=package:chromium) which thinks that "Infra Presubmit" is defined on master.tryserver.ifnra `git cl try -master luci.infra.try -b "Infra Presubmit"` does not work because luci.infra.try is not a master and there is no way to specify a bucket `git cl try` works because it just checks "CQ dry run" on rietveld
,
Aug 3 2016
,
Aug 17 2016
,
Oct 4 2016
,
Oct 4 2016
,
Oct 28 2016
-B flag was added to git-cl-try which specifies bucket. Now specifying `-B luci.infra.try` is a workaround. Not specifying -B does not fail, but schedules a build in a wrong bucket that never gets executed and expires (purple) in 48 hours. We need to fix git-cl-try without -m/-B
,
Dec 4 2017
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra/+/b706523bde92978ec4a5013d1ea9a6131b46e29c commit b706523bde92978ec4a5013d1ea9a6131b46e29c Author: Nodir Turakulov <nodir@google.com> Date: Mon Dec 04 20:58:44 2017 [luci-migration] allow access to public masters git-cl-try needs to load luci_is_prod on @chromium.org accounts for non-googelers for public masters. Allow public access to public masters Bug: 632835 Change-Id: I8da6c828ebbb7509a3b1d7fdc4f34f5a050bfc52 Reviewed-on: https://chromium-review.googlesource.com/804746 Reviewed-by: Robbie Iannucci <iannucci@chromium.org> Commit-Queue: Nodir Turakulov <nodir@chromium.org> [modify] https://crrev.com/b706523bde92978ec4a5013d1ea9a6131b46e29c/go/src/infra/appengine/luci-migration/app/common_test.go [modify] https://crrev.com/b706523bde92978ec4a5013d1ea9a6131b46e29c/go/src/infra/appengine/luci-migration/app/index.go [modify] https://crrev.com/b706523bde92978ec4a5013d1ea9a6131b46e29c/go/src/infra/appengine/luci-migration/app/handlers.go [modify] https://crrev.com/b706523bde92978ec4a5013d1ea9a6131b46e29c/go/src/infra/appengine/luci-migration/app/index_test.go [modify] https://crrev.com/b706523bde92978ec4a5013d1ea9a6131b46e29c/go/src/infra/appengine/luci-migration/config/config.pb.go [modify] https://crrev.com/b706523bde92978ec4a5013d1ea9a6131b46e29c/go/src/infra/appengine/luci-migration/config/config.proto
,
Jan 9 2018
builders-map.appspot.com should be killed: issue 570733 use -B flag to specify bucket |
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by sergeybe...@chromium.org
, Jul 29 2016