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

Issue 643589 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner: ----
Closed: Sep 14
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Feature



Sign in to add a comment

Allow golang in infra scripts/slave

Project Member Reported by shinyak@chromium.org, Sep 2 2016

Issue description

Came up from https://codereview.chromium.org/2306523002/ and https://codereview.chromium.org/2307743002/

infra python script is sometimes fragile. No type check, sometimes hard to test, etc.

When updating compile.py, we're always worried about breaking bots.
I feel it is really good we can use statically typed language like golang.

 
Components: -Infra Infra>Platform>Buildbot
Status: Available (was: Untriaged)
+Infra>Platform since they are probably the right people to talk to.

Personally, I totally support the idea of using Go for various helper scripts on slaves!

However, this also requires thinking through compilation and installing the right binaries on the right hosts. We have CIPD pipeline for doing a lot of that stuff, and I believe there is some work being done to allow CIPD packages being installed by recipes. Maybe that's how compile.py --> compile.go could be installed?

This might warrant a design doc rather than a bug, so more specific and detailed discussion can happen. Thanks!
yeah, as I sent a mail to ML, it would be good to use CIPD pipeline.
Currently I don't have any concrete idea to achieve this. I need to investigate how we can utilize golang more. Then let me share the design later. Thanks!
Project Member

Comment 3 by sheriffbot@chromium.org, Sep 8 2017

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Cc: iannucci@chromium.org
Components: -Infra>Platform>Buildbot Infra>Platform>Recipes
Labels: -Hotlist-Recharge-Cold
Status: Available (was: Untriaged)
Still valid FR. I think the plan is that a recipe module can spec CIPD package it needs to run.

+iannucci@ to confirm/deny.
Project Member

Comment 5 by sheriffbot@chromium.org, Sep 12

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Status: WontFix (was: Untriaged)
So, the idea is to add Go code to infra/go corner, get a CIPD package out of it, then install CIPD package in your recipe before running steps depending on the binary.

Go code itself won't be allowed in scripts/slave.

Sign in to add a comment