New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 884908 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Oct 5
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Auto-update luci-scheduler.cfg

Project Member Reported by dgarr...@chromium.org, Sep 17

Issue description

We auto-generate luci-scheduler.cfg, and submit it into chromite/config to ease code reviews.

However, updates to that file don't take effect until they are also submitted into the ChromeOS project LUCI config repo.

We should auto mirror changes to this config file from chromite into the official config, just to make update management simpler.
 
Status: Started (was: Untriaged)
We are about to have more people making changes to swarming scheduling. Making this automatic will reduce the learning curve (and thus the mistakes) for these users.
I believe this is now working. Waiting to make sure the scheduling is correct.

https://cros-goldeneye.corp.google.com/chromeos/legoland/builderHistory?buildConfig=luci-scheduler-updater&buildBranch=master
Seems to be working. I need to update documentation.
Project Member

Comment 5 by bugdroid1@chromium.org, Oct 5

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/ff45f4b0da47314a109af9814956b0f1bf8d80f8

commit ff45f4b0da47314a109af9814956b0f1bf8d80f8
Author: Don Garrett <dgarrett@google.com>
Date: Fri Oct 05 22:43:57 2018

gen_luci_scheduler: Update deployment docs.

Changes to luci-scheduler.cfg are now autodeployed, so remove the
manual deployment docs. Instead add links the to deployment builder.

BUG= chromium:884908 
TEST=run_tests

Change-Id: I2c5686527f0d089daf34db5ee5439061508a7473
Reviewed-on: https://chromium-review.googlesource.com/1262175
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Tested-by: Don Garrett <dgarrett@chromium.org>
Reviewed-by: Bernie Thompson <bhthompson@chromium.org>

[modify] https://crrev.com/ff45f4b0da47314a109af9814956b0f1bf8d80f8/scripts/gen_luci_scheduler.py
[modify] https://crrev.com/ff45f4b0da47314a109af9814956b0f1bf8d80f8/scripts/gen_luci_scheduler_unittest.py
[modify] https://crrev.com/ff45f4b0da47314a109af9814956b0f1bf8d80f8/config/luci-scheduler.cfg

Status: Fixed (was: Started)

Sign in to add a comment