New issue
Advanced search Search tips

Issue 825366 link

Starred by 4 users

Issue metadata

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

Blocked on:
issue 800447



Sign in to add a comment

Remove ChromeOS --git-cache code.

Project Member Reported by dgarr...@chromium.org, Mar 23 2018

Issue description

Our builders are now using the Chrome Infra git wrapper which invisibly uses git cache.

So, we should probably remove git cache support from our recipe, cbuildbot_launch, cbuildbot, and unfork our copy of repo.

 
Blockedon: 800447
er wait. I don't think that's true!

The git wrapper does exist, but doesn't have transparent caching yet!
(the git wrapper does do transparent retries of transient failures though).
Oh... I thought the cache support was already in place. Good to know.
Components: Infra>Client>ChromeOS>CI
Components: -Infra>Client>ChromeOS
Owner: dgarr...@chromium.org
Hi Don,

Is this change still valid?  It seems that we need to leave the git cache in place until the wrapper supports it transparently.  Is that true?  If so, should we go ahead and close this?

Thanks,
Mike

Comment 7 by nxia@chromium.org, Jun 8 2018

Cc: -nxia@chromium.org
I think we should find out the plan for adding transparent caching first.

Robbie?
Owner: iannucci@chromium.org
Owner: ----
Status: Available (was: Untriaged)
The transparent git cache has (unfortunately) been made much lower priority. We still want to do it, but I created a workaround for our P1 use case last quarter (namely; recipes in LUCI are now distributed via CIPD and not via Git).

I think the thing to do is to watch/work on issue 800447 and then revisit this.
Cc: iannu...@google.com
Cc: -iannucci@chromium.org

Sign in to add a comment