New issue
Advanced search Search tips

Issue 846020 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Jul 6
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Feature

Blocked on:
issue 857598
issue 855799
issue 857595
issue 859123



Sign in to add a comment

Tracker: go/chromeos-skylab-special-tasks

Project Member Reported by pprabhu@chromium.org, May 23 2018

Issue description

Mega bug for the initial implementation effort for special tasks creation AE app.
 
Labels: -Type-Bug Type-Feature
Project Member

Comment 2 by bugdroid1@chromium.org, Jun 21 2018

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

commit f1d7c47657ae975a8d48d7000c01cdd4d32e8d98
Author: Prathmesh Prabhu <pprabhu@chromium.org>
Date: Thu Jun 21 18:34:35 2018

Seed crosskylabadmin

This CL bootstraps the Tracker service with simple rpcs
to refresh bots and query bots. Only the dut_id dimension
is supported yet.

BUG= chromium:846020 

Change-Id: I46a480a77600b6f71d8f44b262fef3521cb8587a
Reviewed-on: https://chromium-review.googlesource.com/1070687
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: Prathmesh Prabhu <pprabhu@chromium.org>

[add] https://crrev.com/f1d7c47657ae975a8d48d7000c01cdd4d32e8d98/go/src/infra/appengine/crosskylabadmin/app/tracker_test.go
[add] https://crrev.com/f1d7c47657ae975a8d48d7000c01cdd4d32e8d98/go/src/infra/appengine/crosskylabadmin/api/fleet/v1/v1.infra_testing
[add] https://crrev.com/f1d7c47657ae975a8d48d7000c01cdd4d32e8d98/go/src/infra/appengine/crosskylabadmin/app/entities.go
[add] https://crrev.com/f1d7c47657ae975a8d48d7000c01cdd4d32e8d98/go/src/infra/appengine/crosskylabadmin/api/fleet/v1/tracker.pb.go
[add] https://crrev.com/f1d7c47657ae975a8d48d7000c01cdd4d32e8d98/go/src/infra/appengine/crosskylabadmin/app/handlers.go
[add] https://crrev.com/f1d7c47657ae975a8d48d7000c01cdd4d32e8d98/go/src/infra/appengine/crosskylabadmin/app/app.yaml
[add] https://crrev.com/f1d7c47657ae975a8d48d7000c01cdd4d32e8d98/go/src/infra/appengine/crosskylabadmin/api/fleet/v1/gen.go
[add] https://crrev.com/f1d7c47657ae975a8d48d7000c01cdd4d32e8d98/go/src/infra/appengine/crosskylabadmin/api/fleet/v1/tracker.proto
[add] https://crrev.com/f1d7c47657ae975a8d48d7000c01cdd4d32e8d98/go/src/infra/appengine/crosskylabadmin/api/fleet/v1/trackerserver_dec.go
[add] https://crrev.com/f1d7c47657ae975a8d48d7000c01cdd4d32e8d98/go/src/infra/appengine/crosskylabadmin/app/swarming.go
[add] https://crrev.com/f1d7c47657ae975a8d48d7000c01cdd4d32e8d98/go/src/infra/appengine/crosskylabadmin/app/middleware.go
[add] https://crrev.com/f1d7c47657ae975a8d48d7000c01cdd4d32e8d98/go/src/infra/appengine/crosskylabadmin/api/fleet/v1/pb.discovery.go
[add] https://crrev.com/f1d7c47657ae975a8d48d7000c01cdd4d32e8d98/go/src/infra/appengine/crosskylabadmin/app/app.infra_testing
[add] https://crrev.com/f1d7c47657ae975a8d48d7000c01cdd4d32e8d98/go/src/infra/appengine/crosskylabadmin/app/tracker.go

Project Member

Comment 3 by bugdroid1@chromium.org, Jun 21 2018

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

commit b685eaabe1889448abe0603d36675ce24e937a40
Author: Prathmesh Prabhu <pprabhu@chromium.org>
Date: Thu Jun 21 23:03:30 2018

crosskylabadmin: Add tasker API, stub implementations

BUG= chromium:846020 

Change-Id: I034a71c90c94c8f47bc08298adeadd9de84f6e1d
Reviewed-on: https://chromium-review.googlesource.com/1102168
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Reviewed-by: Aviv Keshet <akeshet@chromium.org>
Commit-Queue: Prathmesh Prabhu <pprabhu@chromium.org>

[modify] https://crrev.com/b685eaabe1889448abe0603d36675ce24e937a40/go/src/infra/appengine/crosskylabadmin/app/tracker_test.go
[add] https://crrev.com/b685eaabe1889448abe0603d36675ce24e937a40/go/src/infra/appengine/crosskylabadmin/api/fleet/v1/common.pb.go
[modify] https://crrev.com/b685eaabe1889448abe0603d36675ce24e937a40/go/src/infra/appengine/crosskylabadmin/api/fleet/v1/tracker.pb.go
[add] https://crrev.com/b685eaabe1889448abe0603d36675ce24e937a40/go/src/infra/appengine/crosskylabadmin/api/fleet/v1/tasker.pb.go
[modify] https://crrev.com/b685eaabe1889448abe0603d36675ce24e937a40/go/src/infra/appengine/crosskylabadmin/app/handlers.go
[add] https://crrev.com/b685eaabe1889448abe0603d36675ce24e937a40/go/src/infra/appengine/crosskylabadmin/app/tasker_test.go
[modify] https://crrev.com/b685eaabe1889448abe0603d36675ce24e937a40/go/src/infra/appengine/crosskylabadmin/app/swarming.go
[add] https://crrev.com/b685eaabe1889448abe0603d36675ce24e937a40/go/src/infra/appengine/crosskylabadmin/api/fleet/v1/tasker.proto
[modify] https://crrev.com/b685eaabe1889448abe0603d36675ce24e937a40/go/src/infra/appengine/crosskylabadmin/api/fleet/v1/gen.go
[modify] https://crrev.com/b685eaabe1889448abe0603d36675ce24e937a40/go/src/infra/appengine/crosskylabadmin/api/fleet/v1/tracker.proto
[add] https://crrev.com/b685eaabe1889448abe0603d36675ce24e937a40/go/src/infra/appengine/crosskylabadmin/app/test_common.go
[add] https://crrev.com/b685eaabe1889448abe0603d36675ce24e937a40/go/src/infra/appengine/crosskylabadmin/api/fleet/v1/taskerserver_dec.go
[add] https://crrev.com/b685eaabe1889448abe0603d36675ce24e937a40/go/src/infra/appengine/crosskylabadmin/api/fleet/v1/common.proto
[add] https://crrev.com/b685eaabe1889448abe0603d36675ce24e937a40/go/src/infra/appengine/crosskylabadmin/app/tasker.go
[modify] https://crrev.com/b685eaabe1889448abe0603d36675ce24e937a40/go/src/infra/appengine/crosskylabadmin/api/fleet/v1/pb.discovery.go
[modify] https://crrev.com/b685eaabe1889448abe0603d36675ce24e937a40/go/src/infra/appengine/crosskylabadmin/app/tracker.go

Project Member

Comment 4 by bugdroid1@chromium.org, Jun 22 2018

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

commit 42810727aaf4f3a4eed94afc07f9c74d9981d75c
Author: Prathmesh Prabhu <pprabhu@chromium.org>
Date: Fri Jun 22 18:49:15 2018

crosskylabadmin: implement EnsureBackgroundTasks [1/3]

This CL implements EnsureBackgroundTasks in a simplistic way.
Requested number of tasks are created but without checking existing
Swarming tasks.

BUG= chromium:846020 

Change-Id: I3dfb72073a5c8ab50cb79fc5143ac5a87b824181
Reviewed-on: https://chromium-review.googlesource.com/1105471
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: Prathmesh Prabhu <pprabhu@chromium.org>

[modify] https://crrev.com/42810727aaf4f3a4eed94afc07f9c74d9981d75c/go/src/infra/appengine/crosskylabadmin/app/entities.go
[modify] https://crrev.com/42810727aaf4f3a4eed94afc07f9c74d9981d75c/go/src/infra/appengine/crosskylabadmin/app/tasker_test.go
[modify] https://crrev.com/42810727aaf4f3a4eed94afc07f9c74d9981d75c/go/src/infra/appengine/crosskylabadmin/app/swarming.go
[modify] https://crrev.com/42810727aaf4f3a4eed94afc07f9c74d9981d75c/go/src/infra/appengine/crosskylabadmin/app/test_common.go
[modify] https://crrev.com/42810727aaf4f3a4eed94afc07f9c74d9981d75c/go/src/infra/appengine/crosskylabadmin/app/tasker.go
[modify] https://crrev.com/42810727aaf4f3a4eed94afc07f9c74d9981d75c/go/src/infra/appengine/crosskylabadmin/app/handlers.go
[modify] https://crrev.com/42810727aaf4f3a4eed94afc07f9c74d9981d75c/go/src/infra/appengine/crosskylabadmin/app/tracker.go

Project Member

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

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

commit 3cb176b3f616b959b5d086ef4e3634525337318c
Author: Prathmesh Prabhu <pprabhu@chromium.org>
Date: Fri Jun 22 23:51:38 2018

crosskylabadmin: Run 'go generate'.

Last CL failed to include generated files in the final version. FixIt.

BUG= chromium:846020 

Change-Id: Ieb2da0bd6f76bff3ceb5b928d3ca8cbfe6c0ff99
Reviewed-on: https://chromium-review.googlesource.com/1112658
Commit-Queue: Prathmesh Prabhu <pprabhu@chromium.org>
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>

[modify] https://crrev.com/3cb176b3f616b959b5d086ef4e3634525337318c/go/src/infra/appengine/crosskylabadmin/api/fleet/v1/tasker.pb.go
[modify] https://crrev.com/3cb176b3f616b959b5d086ef4e3634525337318c/go/src/infra/appengine/crosskylabadmin/api/fleet/v1/pb.discovery.go

Blockedon: 855799
Project Member

Comment 8 by bugdroid1@chromium.org, Jun 23 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/464cd988865f813d6ba4f731a93393184ab8a086

commit 464cd988865f813d6ba4f731a93393184ab8a086
Author: Allen Li <ayatane@chromium.org>
Date: Sat Jun 23 05:51:16 2018

lucifer: Add new binary

BUG= chromium:846020 
TEST=None

Change-Id: I8544b33f910cb3961cef4b831a015005d2235c03
Reviewed-on: https://chromium-review.googlesource.com/1112720
Commit-Ready: Prathmesh Prabhu <pprabhu@chromium.org>
Tested-by: Prathmesh Prabhu <pprabhu@chromium.org>
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>

[modify] https://crrev.com/464cd988865f813d6ba4f731a93393184ab8a086/chromeos-base/lucifer/lucifer-9999.ebuild

Project Member

Comment 9 by bugdroid1@chromium.org, Jun 26 2018

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

commit 7fefea948aa1606290143f843020afcdb4afa995
Author: Prathmesh Prabhu <pprabhu@chromium.org>
Date: Tue Jun 26 18:08:32 2018

crosskylabadmin: Fix skylab command line for admin tasks

BUG= chromium:846020 

Change-Id: Ib6146384891475e0f0b58495a21d7c1794804f27
Reviewed-on: https://chromium-review.googlesource.com/1112725
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: Prathmesh Prabhu <pprabhu@chromium.org>

[modify] https://crrev.com/7fefea948aa1606290143f843020afcdb4afa995/go/src/infra/appengine/crosskylabadmin/app/tasker_test.go
[modify] https://crrev.com/7fefea948aa1606290143f843020afcdb4afa995/go/src/infra/appengine/crosskylabadmin/app/tasker.go

Project Member

Comment 10 by bugdroid1@chromium.org, Jun 26 2018

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

commit 4e4480b5f6f1b15d54f1acc80462806e23685847
Author: Prathmesh Prabhu <pprabhu@chromium.org>
Date: Tue Jun 26 18:25:15 2018

crosskylabadmin: Address outstanding review comments

BUG= chromium:846020 

Change-Id: If20b01ebda39bab56d376555b2a2bb005b8a7dbe
Reviewed-on: https://chromium-review.googlesource.com/1112751
Commit-Queue: Prathmesh Prabhu <pprabhu@chromium.org>
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>

[add] https://crrev.com/4e4480b5f6f1b15d54f1acc80462806e23685847/go/src/infra/appengine/crosskylabadmin/app/internal/internal.infra_testing
[add] https://crrev.com/4e4480b5f6f1b15d54f1acc80462806e23685847/go/src/infra/appengine/crosskylabadmin/app/internal/cron/cron.go
[modify] https://crrev.com/4e4480b5f6f1b15d54f1acc80462806e23685847/go/src/infra/appengine/crosskylabadmin/app/entities.go
[modify] https://crrev.com/4e4480b5f6f1b15d54f1acc80462806e23685847/go/src/infra/appengine/crosskylabadmin/app/tasker_test.go
[modify] https://crrev.com/4e4480b5f6f1b15d54f1acc80462806e23685847/go/src/infra/appengine/crosskylabadmin/app/swarming.go
[add] https://crrev.com/4e4480b5f6f1b15d54f1acc80462806e23685847/go/src/infra/appengine/crosskylabadmin/app/internal/cron/cron.infra_testing
[modify] https://crrev.com/4e4480b5f6f1b15d54f1acc80462806e23685847/go/src/infra/appengine/crosskylabadmin/app/test_common.go
[modify] https://crrev.com/4e4480b5f6f1b15d54f1acc80462806e23685847/go/src/infra/appengine/crosskylabadmin/app/tracker_test.go
[modify] https://crrev.com/4e4480b5f6f1b15d54f1acc80462806e23685847/go/src/infra/appengine/crosskylabadmin/app/middleware.go
[modify] https://crrev.com/4e4480b5f6f1b15d54f1acc80462806e23685847/go/src/infra/appengine/crosskylabadmin/app/tasker.go
[modify] https://crrev.com/4e4480b5f6f1b15d54f1acc80462806e23685847/go/src/infra/appengine/crosskylabadmin/app/handlers.go
[modify] https://crrev.com/4e4480b5f6f1b15d54f1acc80462806e23685847/go/src/infra/appengine/crosskylabadmin/app/tracker.go

Project Member

Comment 11 by bugdroid1@chromium.org, Jun 26 2018

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

commit 8fae7faad46efe397f5e606d85dc865a27c4a637
Author: Prathmesh Prabhu <pprabhu@chromium.org>
Date: Tue Jun 26 18:44:22 2018

crosskylabadmin: Simplify Tracker.RefreshBots

BUG= chromium:846020 

Change-Id: I965fb1b247d2e1eef77c686775238d129541f647
Reviewed-on: https://chromium-review.googlesource.com/1112763
Commit-Queue: Prathmesh Prabhu <pprabhu@chromium.org>
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>

[modify] https://crrev.com/8fae7faad46efe397f5e606d85dc865a27c4a637/go/src/infra/appengine/crosskylabadmin/app/tracker.go

Validated that reset tasks are created and succeed: https://chrome-swarming.appspot.com/task?id=3e56eed7d120d710
Project Member

Comment 14 by bugdroid1@chromium.org, Jun 27 2018

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

commit a9d8343f695cb8d833e04c38469b935fa13ad8dd
Author: Prathmesh Prabhu <pprabhu@chromium.org>
Date: Wed Jun 27 17:48:23 2018

crosskylabadmin: Move swarming client to separate package.

The swarming client adaptor is now moved to a clients/ package. This
package is not named swarming because users need to reference the
generated swarming bindings along with this package. Any client side
adaptors that allow using stubs for unittests should live in this
package.

BUG= chromium:846020 

Change-Id: Ie3a56e5597beb77eec9b4d62bb331e96bf30a33c
Reviewed-on: https://chromium-review.googlesource.com/1116063
Commit-Queue: Prathmesh Prabhu <pprabhu@chromium.org>
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>

[modify] https://crrev.com/a9d8343f695cb8d833e04c38469b935fa13ad8dd/go/src/infra/appengine/crosskylabadmin/app/tracker_test.go
[rename] https://crrev.com/a9d8343f695cb8d833e04c38469b935fa13ad8dd/go/src/infra/appengine/crosskylabadmin/app/clients/swarming.go
[modify] https://crrev.com/a9d8343f695cb8d833e04c38469b935fa13ad8dd/go/src/infra/appengine/crosskylabadmin/app/tasker.go
[modify] https://crrev.com/a9d8343f695cb8d833e04c38469b935fa13ad8dd/go/src/infra/appengine/crosskylabadmin/app/test_common.go
[modify] https://crrev.com/a9d8343f695cb8d833e04c38469b935fa13ad8dd/go/src/infra/appengine/crosskylabadmin/app/tasker_test.go
[add] https://crrev.com/a9d8343f695cb8d833e04c38469b935fa13ad8dd/go/src/infra/appengine/crosskylabadmin/app/clients/clients.infra_testing
[modify] https://crrev.com/a9d8343f695cb8d833e04c38469b935fa13ad8dd/go/src/infra/appengine/crosskylabadmin/app/tracker.go

Project Member

Comment 15 by bugdroid1@chromium.org, Jun 27 2018

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

commit bc94bd064fe648e48c771f8dbc1a3c32d28fb82d
Author: Prathmesh Prabhu <pprabhu@chromium.org>
Date: Wed Jun 27 18:37:43 2018

crosskylabadmin: Move frontend APIs to a package

BUG= chromium:846020 

Change-Id: I6460bf591d37bde2e0032b8a347390f0861b7e17
Reviewed-on: https://chromium-review.googlesource.com/1116064
Commit-Queue: Prathmesh Prabhu <pprabhu@chromium.org>
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>

[rename] https://crrev.com/bc94bd064fe648e48c771f8dbc1a3c32d28fb82d/go/src/infra/appengine/crosskylabadmin/app/frontend/tasker.go
[rename] https://crrev.com/bc94bd064fe648e48c771f8dbc1a3c32d28fb82d/go/src/infra/appengine/crosskylabadmin/app/frontend/tracker.go
[rename] https://crrev.com/bc94bd064fe648e48c771f8dbc1a3c32d28fb82d/go/src/infra/appengine/crosskylabadmin/app/frontend/tracker_test.go
[add] https://crrev.com/bc94bd064fe648e48c771f8dbc1a3c32d28fb82d/go/src/infra/appengine/crosskylabadmin/app/frontend/frontend.go
[rename] https://crrev.com/bc94bd064fe648e48c771f8dbc1a3c32d28fb82d/go/src/infra/appengine/crosskylabadmin/app/frontend/entities.go
[rename] https://crrev.com/bc94bd064fe648e48c771f8dbc1a3c32d28fb82d/go/src/infra/appengine/crosskylabadmin/app/frontend/tasker_test.go
[add] https://crrev.com/bc94bd064fe648e48c771f8dbc1a3c32d28fb82d/go/src/infra/appengine/crosskylabadmin/app/frontend/frontend.infra_testing
[rename] https://crrev.com/bc94bd064fe648e48c771f8dbc1a3c32d28fb82d/go/src/infra/appengine/crosskylabadmin/app/frontend/test_common.go
[delete] https://crrev.com/3377251b7ca6eb3c8037ad39fa5ee7baa65e35bb/go/src/infra/appengine/crosskylabadmin/app/middleware.go
[modify] https://crrev.com/bc94bd064fe648e48c771f8dbc1a3c32d28fb82d/go/src/infra/appengine/crosskylabadmin/app/handlers.go
[modify] https://crrev.com/bc94bd064fe648e48c771f8dbc1a3c32d28fb82d/go/src/infra/appengine/crosskylabadmin/app/app.infra_testing

Project Member

Comment 16 by bugdroid1@chromium.org, Jun 27 2018

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

commit 20e3d31ed17c23689a3bce08bc546687023d2c11
Author: Prathmesh Prabhu <pprabhu@chromium.org>
Date: Wed Jun 27 19:43:06 2018

crosskylabadmin: Add a unique tag to all tasks

This will make it possible to easily list all tasks created by the app.

BUG= chromium:846020 

Change-Id: Ieb5edf7a50bc2bfaa271b7e2eaf52f32a7a99e9c
Reviewed-on: https://chromium-review.googlesource.com/1116196
Commit-Queue: Prathmesh Prabhu <pprabhu@chromium.org>
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>

[modify] https://crrev.com/20e3d31ed17c23689a3bce08bc546687023d2c11/go/src/infra/appengine/crosskylabadmin/app/frontend/tasker.go
[modify] https://crrev.com/20e3d31ed17c23689a3bce08bc546687023d2c11/go/src/infra/appengine/crosskylabadmin/app/frontend/frontend.go

Project Member

Comment 17 by bugdroid1@chromium.org, Jun 28 2018

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

commit 10a77d837481bafd33181fbd4edee04ef98d88bb
Author: Prathmesh Prabhu <pprabhu@chromium.org>
Date: Thu Jun 28 19:39:33 2018

crosskylabadmin: Add cron handlers

BUG= chromium:846020 

Change-Id: I8b82b856e67e62915925573554415ad4bb100640
Reviewed-on: https://chromium-review.googlesource.com/1111618
Commit-Queue: Prathmesh Prabhu <pprabhu@chromium.org>
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>

[modify] https://crrev.com/10a77d837481bafd33181fbd4edee04ef98d88bb/go/src/infra/appengine/crosskylabadmin/app/frontend/tasker.go
[modify] https://crrev.com/10a77d837481bafd33181fbd4edee04ef98d88bb/go/src/infra/appengine/crosskylabadmin/app/frontend/tasker_test.go
[add] https://crrev.com/10a77d837481bafd33181fbd4edee04ef98d88bb/go/src/infra/appengine/crosskylabadmin/app/cron/cron.infra_testing
[modify] https://crrev.com/10a77d837481bafd33181fbd4edee04ef98d88bb/go/src/infra/appengine/crosskylabadmin/app/frontend/tracker_test.go
[modify] https://crrev.com/10a77d837481bafd33181fbd4edee04ef98d88bb/go/src/infra/appengine/crosskylabadmin/app/frontend/frontend.go
[modify] https://crrev.com/10a77d837481bafd33181fbd4edee04ef98d88bb/go/src/infra/appengine/crosskylabadmin/app/handlers.go
[add] https://crrev.com/10a77d837481bafd33181fbd4edee04ef98d88bb/go/src/infra/appengine/crosskylabadmin/app/cron/cron.go
[modify] https://crrev.com/10a77d837481bafd33181fbd4edee04ef98d88bb/go/src/infra/appengine/crosskylabadmin/app/frontend/tracker.go
[add] https://crrev.com/10a77d837481bafd33181fbd4edee04ef98d88bb/go/src/infra/appengine/crosskylabadmin/app/cron.yaml

Blockedon: 857595 857598
Blockedon: 859123
Project Member

Comment 20 by bugdroid1@chromium.org, Jun 30 2018

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

commit 871331bd8aae9cc6f5ff7a4b699a185197ac12be
Author: Prathmesh Prabhu <pprabhu@chromium.org>
Date: Sat Jun 30 00:02:01 2018

crosskylabadmin: Summarize dut_state and bot idle time

BUG= chromium:846020 

Change-Id: I0e293593cb3dd290a2a75d7b512320e76457fd60
Reviewed-on: https://chromium-review.googlesource.com/1117773
Commit-Queue: Prathmesh Prabhu <pprabhu@chromium.org>
Reviewed-by: Marc-Antoine Ruel <maruel@chromium.org>

[modify] https://crrev.com/871331bd8aae9cc6f5ff7a4b699a185197ac12be/go/src/infra/appengine/crosskylabadmin/app/frontend/tracker_test.go
[modify] https://crrev.com/871331bd8aae9cc6f5ff7a4b699a185197ac12be/go/src/infra/appengine/crosskylabadmin/app/clients/swarming.go
[modify] https://crrev.com/871331bd8aae9cc6f5ff7a4b699a185197ac12be/go/src/infra/appengine/crosskylabadmin/api/fleet/v1/tracker.pb.go
[modify] https://crrev.com/871331bd8aae9cc6f5ff7a4b699a185197ac12be/go/src/infra/appengine/crosskylabadmin/app/frontend/test_common.go
[modify] https://crrev.com/871331bd8aae9cc6f5ff7a4b699a185197ac12be/go/src/infra/appengine/crosskylabadmin/app/frontend/tracker.go
[modify] https://crrev.com/871331bd8aae9cc6f5ff7a4b699a185197ac12be/go/src/infra/appengine/crosskylabadmin/api/fleet/v1/tracker.proto
[modify] https://crrev.com/871331bd8aae9cc6f5ff7a4b699a185197ac12be/go/src/infra/appengine/crosskylabadmin/api/fleet/v1/pb.discovery.go

Project Member

Comment 21 by bugdroid1@chromium.org, Jun 30 2018

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

commit d35ee831146af49c8912d5b8443cfe67c92f550d
Author: Prathmesh Prabhu <pprabhu@chromium.org>
Date: Sat Jun 30 00:11:33 2018

crosskylabadmin: Persist BotID of bots in the datastore

This is needed in the tasker.Trigger* RPCs to lookup tasks corresponding
to the known bots.

BUG= chromium:846020 

Change-Id: I95f2861ac4c74b970ad6ef87e6d4c5f8a83ade1d
Reviewed-on: https://chromium-review.googlesource.com/1119533
Commit-Queue: Prathmesh Prabhu <pprabhu@chromium.org>
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>

[modify] https://crrev.com/d35ee831146af49c8912d5b8443cfe67c92f550d/go/src/infra/appengine/crosskylabadmin/app/frontend/tasker_test.go
[modify] https://crrev.com/d35ee831146af49c8912d5b8443cfe67c92f550d/go/src/infra/appengine/crosskylabadmin/app/frontend/tracker.go
[modify] https://crrev.com/d35ee831146af49c8912d5b8443cfe67c92f550d/go/src/infra/appengine/crosskylabadmin/app/frontend/entities.go

All tasks are now being triggered: https://pantheon.corp.google.com/appengine/cronjobs?folder=&organizationId=433637338589&project=chromeos-skylab-bot-fleet

Once the last few in-flight CLs land, this bug is all done.
Project Member

Comment 27 by bugdroid1@chromium.org, Jul 6

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

commit a2bf76f9696d4d460f2ba10da0ce892c6af75411
Author: Prathmesh Prabhu <pprabhu@chromium.org>
Date: Fri Jul 06 19:45:34 2018

crosskylabadmin: trigger repair tasks from cron

BUG= chromium:846020 

Change-Id: I2285fb0b6502fb15dd2d6d5e839bda6fae76994f
Reviewed-on: https://chromium-review.googlesource.com/1124665
Commit-Queue: Prathmesh Prabhu <pprabhu@chromium.org>
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>

[modify] https://crrev.com/a2bf76f9696d4d460f2ba10da0ce892c6af75411/go/src/infra/appengine/crosskylabadmin/app/cron/cron.go
[modify] https://crrev.com/a2bf76f9696d4d460f2ba10da0ce892c6af75411/go/src/infra/appengine/crosskylabadmin/app/cron.yaml

Status: Fixed (was: Started)
All done!
Project Member

Comment 29 by bugdroid1@chromium.org, Jul 6

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

commit dcfa207f085bf3072a5cb65b6ed3d03b5408ef5c
Author: Prathmesh Prabhu <pprabhu@chromium.org>
Date: Fri Jul 06 22:22:25 2018

crosskylabadmin: Rename local variables in frontend tests

Cosmetic changes to improve readability.

BUG= chromium:846020 

Change-Id: I66cd50c6e8b10d6e91cc395c91718d1fd17c76c3
Reviewed-on: https://chromium-review.googlesource.com/1128205
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: Prathmesh Prabhu <pprabhu@chromium.org>

[modify] https://crrev.com/dcfa207f085bf3072a5cb65b6ed3d03b5408ef5c/go/src/infra/appengine/crosskylabadmin/app/frontend/tasker_test.go
[modify] https://crrev.com/dcfa207f085bf3072a5cb65b6ed3d03b5408ef5c/go/src/infra/appengine/crosskylabadmin/app/frontend/tracker_test.go

Sign in to add a comment