New issue
Advanced search Search tips

Issue 848787 link

Starred by 4 users

Issue metadata

Status: Fixed
Owner:
Closed: Sep 19
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Feature



Sign in to add a comment

Remove "LUCI" chip

Project Member Reported by no...@chromium.org, Jun 1 2018

Issue description

Now that most of chromium is on LUCI, replace LUCI chips in buildbucket-gerrit plugin with a buidlbot chip (on buildbot builds)
 

Comment 1 by efoo@chromium.org, Jun 1 2018

I am thinking maybe we can to swap it. As in, flip the tag to [Buildbot] instead of [LUCI] to highlight that Buildbot is not the norm. 

Thoughts?
I think that's what comment 0 suggests too.

Comment 3 by no...@chromium.org, Jun 1 2018

Cc: no...@chromium.org
Owner: efoo@chromium.org
efoo volunteered

Comment 4 by no...@chromium.org, Jun 13 2018

Summary: Remove "LUCI" chip (was: Replace "LUCI" chip with Buildbot chips)
I think we can remove it altogether. The problem it used to solve no longer exists. We were worried that if a build fails because of LUCI, users may want to know, e.g. correlated failure with LUCI chip presence and opt-out. This is no longer relevant. Removing this will reduce code complexity.

Comment 5 by no...@chromium.org, Jun 22 2018

Components: Infra>Platform>Buildbucket>Gerrit

Comment 6 by no...@chromium.org, Jun 22 2018

Components: -Infra>Platform>Buildbucket
Cc: efoo@chromium.org
Owner: no...@chromium.org
Status: Started (was: Assigned)
Status: Fixed (was: Started)
deployed
Thanks! Much less noisy :-)
Project Member

Comment 12 by bugdroid1@chromium.org, Sep 24

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/40721ae447dd3509445322c521df65885fd99279

commit 40721ae447dd3509445322c521df65885fd99279
Author: Eric Foo <efoo@chromium.org>
Date: Mon Sep 24 19:35:11 2018

Update info on gerrit tryjobs on LUCI/Buildbot

Bug:  848787 
Change-Id: I7283eea0de68d6f795b74b8159ca62588e226fb8
Reviewed-on: https://chromium-review.googlesource.com/1239568
Auto-Submit: Eric Foo <efoo@chromium.org>
Reviewed-by: Nodir Turakulov <nodir@chromium.org>
Commit-Queue: Nodir Turakulov <nodir@chromium.org>
Cr-Commit-Position: refs/heads/master@{#17833}
[modify] https://crrev.com/40721ae447dd3509445322c521df65885fd99279/doc/users/services/about_luci.md
[modify] https://crrev.com/40721ae447dd3509445322c521df65885fd99279/doc/images/luci_chip.png

Sign in to add a comment