-
Notifications
You must be signed in to change notification settings - Fork 17.3k
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: add LUCI illumos-amd64 builder #67302
Comments
G'day! I am creating the new builder with hostname illumos-amd64-jclulow. The CSR is attached: illumos-amd64-jclulow.csr.txt Please, Mr. Music, will you play? |
Thank you for starting to work on this builder. Here's the certificate: Edit: I'll also note that if the previous coordinator-based builder starts to get in the way of you bringing up the new LUCI builder, you should feel free to stop the previous builder and focus on the new one. It's certainly fine to keep the old one running if it doesn't get in the way. Thanks. |
I put the new one in a new VM, so they don't interact. The host is definitely getting LUCI tokens, and the swarm thing is definitely doing something inscrutable and then rebooting the instance every now and again. I had a peek in some logs, and I think it's a bit confused about the platform: it says "solaris-amd64" in a bunch of log messages, which is obviously not quite right, it should be "illumos-amd64" presumably. Please let me know what to do next! |
Thanks for getting to this point. The bot is visible at https://chromium-swarm.appspot.com/bot?id=illumos-amd64-jclulow. As you noted, it's identifying itself as having a cipd_platform value of "solaris-amd64", but we indeed need it to be "illumos-amd64" in order to match the builder definition at https://ci.chromium.org/ui/p/golang/builders/ci/gotip-illumos-amd64. I suggest stopping this builder for now to avoid it providing noise for the https://ci.chromium.org/ui/p/golang/builders/ci/gotip-solaris-amd64 builder. The next steps here will be to make a change in the swarming bot code to report its OS as illumos rather than solaris. The relevant code is here: I suggest taking a look at a previous example of a similar change being done in #64660 (comment). |
There currently isn't a LUCI builder that tests the illumos/amd64 port (other than the misc-compile builder, which tests only that the port compiles). This is the tracking issue for it.
The next steps that a builder owner will need to follow to make progress here are documented https://go.dev/wiki/DashboardBuilders#luci-builders.
The text was updated successfully, but these errors were encountered: