Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

x/build: separate LUCI swarming bits into its own server app #61912

Closed
Tracked by #61772
cagedmantis opened this issue Aug 9, 2023 · 10 comments
Closed
Tracked by #61772

x/build: separate LUCI swarming bits into its own server app #61912

cagedmantis opened this issue Aug 9, 2023 · 10 comments
Assignees
Labels
Builders x/build issues (builders, bots, dashboards) NeedsFix The path to resolution is known, but the work has not been done.
Milestone

Comments

@cagedmantis
Copy link
Contributor

No description provided.

@gopherbot gopherbot added the Builders x/build issues (builders, bots, dashboards) label Aug 9, 2023
@gopherbot gopherbot added this to the Unreleased milestone Aug 9, 2023
@cagedmantis cagedmantis added the NeedsFix The path to resolution is known, but the work has not been done. label Aug 9, 2023
@cagedmantis
Copy link
Contributor Author

@heschi created the new DNS for this.

@gopherbot
Copy link

Change https://go.dev/cl/519475 mentions this issue: cmd/gomoteserver: create gomote server

gopherbot pushed a commit to golang/build that referenced this issue Aug 16, 2023
This change copies the gomote server functionality from the coordinator
into a new server. This will require refactoring after this initial CL.

Updates golang/go#61912
For golang/go#61772

Change-Id: I25ef657f4512d034bdc3e98bf2718e5e768be1c4
Reviewed-on: https://go-review.googlesource.com/c/build/+/519475
Run-TryBot: Carlos Amedee <carlos@golang.org>
Auto-Submit: Carlos Amedee <carlos@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
@gopherbot
Copy link

Change https://go.dev/cl/520239 mentions this issue: cmd/gomoteserver: add files to build gomote server

gopherbot pushed a commit to golang/build that referenced this issue Aug 17, 2023
This change adds files to build the gomote server. There
is a small fix on a flag description in the actual gomote
server itself.

Updates golang/go#61912
For golang/go#61772

Change-Id: If424342975bf210cd27d8b18879210eac53ea8fb
Reviewed-on: https://go-review.googlesource.com/c/build/+/520239
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Run-TryBot: Carlos Amedee <carlos@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Auto-Submit: Carlos Amedee <carlos@golang.org>
@gopherbot
Copy link

Change https://go.dev/cl/520655 mentions this issue: cmd/gomoteserver: add Makefile and associated cloud build config

@gopherbot
Copy link

Change https://go.dev/cl/520695 mentions this issue: deploy, cmd/gomoteserver: add the Kubernetes configuration

gopherbot pushed a commit to golang/build that referenced this issue Aug 17, 2023
This change adds the last required configuration files
required for building the gomote server.

Updates golang/go#61912
For golang/go#61772

Change-Id: I625a4d265260eb95f1a37de8c5b3596b1c2fd7c8
Reviewed-on: https://go-review.googlesource.com/c/build/+/520655
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Auto-Submit: Carlos Amedee <carlos@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Run-TryBot: Carlos Amedee <carlos@golang.org>
@gopherbot
Copy link

Change https://go.dev/cl/525980 mentions this issue: cmd/gomoteserver: fix gomoteserver deployments

gopherbot pushed a commit to golang/build that referenced this issue Sep 6, 2023
This change adds the Kubernetes configuration changes necessary
for the gomote server to be deployed in the cluster. It creates
a new ingress for the gomote server. It also adds an external
IP address for the gomote ssh functionality.

Updates golang/go#61912
For golang/go#61772

Change-Id: If821ff368bbd025a566b04f2d9779f2a12ff4cb2
Reviewed-on: https://go-review.googlesource.com/c/build/+/520695
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Heschi Kreinick <heschi@google.com>
Run-TryBot: Carlos Amedee <carlos@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
gopherbot pushed a commit to golang/build that referenced this issue Sep 7, 2023
This change:
- Fixes an issue where the generated version had an extra space
  appended to the end of it.
- Initializes the storage client.
- Adds some flags to be registered.

For golang/go#61912

Change-Id: I7b07fa9a406cc2858991005148b96f3b1ecc3a6d
Reviewed-on: https://go-review.googlesource.com/c/build/+/525980
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
@cagedmantis
Copy link
Contributor Author

gomote.golang.org and gomotessh.golang.org have been created. The ingress and load balancer has been created. Gomoteserver has been separated into it's own service. The swarming gomote implementation is being created and is being implemented.

@gopherbot
Copy link

Change https://go.dev/cl/526259 mentions this issue: internal/gomote: create gomote swarming server implementation

@gopherbot
Copy link

Change https://go.dev/cl/526260 mentions this issue: cmd/gomoteserver: use the gomote swarming GRPC server

gopherbot pushed a commit to golang/build that referenced this issue Sep 7, 2023
This change creates a new gomote GRPC server implementation which only
supports swarming bots.

For golang/go#61912

Change-Id: Ib813b5190e0826bcb008c67cacf11b95b9250823
Reviewed-on: https://go-review.googlesource.com/c/build/+/526259
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Auto-Submit: Carlos Amedee <carlos@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
@gopherbot
Copy link

Change https://go.dev/cl/529356 mentions this issue: internal/gomote, cmd/coordinator: remove LUCI from coordinator

gopherbot pushed a commit to golang/build that referenced this issue Sep 19, 2023
This change removes any LUCI integration from the gomote
implementation used by the coordinator.

For golang/go#61912

Change-Id: I735b8ec40196eeac245c1cac43cd3d9e6b50646d
Reviewed-on: https://go-review.googlesource.com/c/build/+/529356
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Builders x/build issues (builders, bots, dashboards) NeedsFix The path to resolution is known, but the work has not been done.
Projects
Archived in project
Development

No branches or pull requests

3 participants