New issue
Advanced search Search tips

Issue 816564 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: May 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocked on:
issue 756649



Sign in to add a comment

Moving amd64-generic-goma-canary-chromium-pfq-informational to swarming will require special recipe.

Project Member Reported by dgarr...@chromium.org, Feb 26 2018

Issue description

The goma canary builder relies on a special config flag being sent by the waterfall to request that the canary version of goma is used for the build.

Post-swarming, the waterfall won't be available to supply this flag.

Approaches I see:

A) A special build recipe.
B) Pass in flag via buildbucket request.
C) Move goma installation to cbuildbot and use cbuildbot_config.

My thinking is that it's best to create a new recipe, then add a new config value to specify which recipe to use. When we start auto-generating scheduler configs, this will be used there.
 
Blockedon: 756649
Components: Infra>Goma
Let me describe the current status (mainly for goma members)

goma canary bot on chromeos side is currently running on 
https://ci.chromium.org/buildbot/chromiumos.chromium/amd64-generic-goma-canary-chromium-pfq-informational/
(note: as of 1st Mar. goma canary version is not used yet)

Currently goma is installed in "ensure_goma" step, which runs before "cbuildbot" step. So, cbuildbot config cannot be utilized in ensure_goma step.

To install goma in ensure_step, in chrome (not chromeos), we have a config map (builder name -> config), and use it to specify "goma_canary" config.
So I'm considering to use similar structure in chromeos bots, too.

However, post-swarming, builder name won't be available. If cbuildbot runs in a VM that is different from the builder, canary version of goma might not be installed in such a swarming VM.

So we need to consider something dgarrett mentioned.

Status: Assigned (was: Untriaged)
Components: Infra>Client>ChromeOS>CI
Components: -Infra>Client>ChromeOS
Status: Started (was: Assigned)
I have migrated this to swarming, but forgot to tweak the recipe.

I believe this means the builder is currently running, but not using the Goma canary.
Project Member

Comment 8 by bugdroid1@chromium.org, May 28 2018

Labels: merge-merged-config
The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/manifest-internal/+/c7f93cd797bb2e8202beeb4800914d5679732e4c

commit c7f93cd797bb2e8202beeb4800914d5679732e4c
Author: Don Garrett <dgarrett@google.com>
Date: Mon May 28 02:58:16 2018

I cannot see any goma-canary bot in #Comment 7. Hmm, did it disappear...?
I can see

https://cros-goldeneye.corp.google.com/chromeos/legoland/builderHistory?buildConfig=amd64-generic-goma-canary-chromium-pfq-informational&buildBranch

and I confirmed the canary version was not used for a while.

If it continued from around Apr 25, no one didn't checked chrome version for goma version 155 and 156? sigh...
Status: Fixed (was: Started)
I'm sorry about that, it was a dumb oversite on my part.

But, it does appear to be fixed now.

Sign in to add a comment