-
Notifications
You must be signed in to change notification settings - Fork 17.8k
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 NetBSD 9.0 builder #37244
Comments
You beat me to it :) Should it be in addition to the 8.0 builder, or
instead of it?
--
Benny
|
I think we should replace the 8.0 builder with 9.0. NetBSD is not a first-class port, so I don't think we should expand our coverage to older versions unless there is some compelling reason. |
Works for me. |
Change https://golang.org/cl/226977 mentions this issue: |
(Our current I agree with @toothrot that there isn't a need for 8.0 once 9.0 is working. However, I think it would be easier to add a new 9.0 builder, letting it run for some time to make sure it works better than 8.0, then remove the 8.0 builder. We should also increase the disk size while building a new image. Its current one is very small (4 GB) and causes issues like #30839, and inability to install large dependencies needed for some tests. /cc @randall77 |
I've investigated what it would take to get a 9.0 image built in CL 226977. The image was built and boots up, buildlet runs, but it seems something went wrong with installing CA certificates on it. It fails when attempting to fetch an HTTPS URL:
In contrast, using the current netbsd-amd64-8-0-2018q1 image results in:
@bsiegert Do you have ideas what would cause that? Are you able to help with investigating this? Thanks. |
Hello Dmitri,
Dmitri Shuralyov writes:
@bsiegert Do you have ideas what would cause that? Are you able to help with investigating this? Thanks.
Probably no rootcerts are found.
Installing mozilla-rootcerts-openssl package (or manually populating them) will
probably fix it.
|
@iamleot Note that the But thanks for the suggestion, I can explore how the |
You might need to add mozilla-rootcerts-openssl in addition to mozilla-rootcerts. I admit that I don't know why the latter alone is not enough. |
`mozilla-rootcerts' package installs certificates under /usr/pkg and - at
least openssl - will not use them. In order to populate /etc/openssl
with the certificates `mozilla-rootcerts install' (a script provided by
mozilla-rootcerts package) should be manually invoked.
`mozilla-rootcerts-openssl' package just automatically take care of
also doing the last step.
|
@iamleot Thanks for providing context. However, note that the
If adding the |
Adding the
I'll go with that approach in the CL. Thanks for the suggestion. |
FYI, I can allocate a netbsd 9.0 builder, but I can't do anything with it.
None of the other commands I tried ( |
In fact, I can't even |
@randall77 The NetBSD 9.0 builder is still in development, so you'll need to re-build I'll update this thread when the new builder is operational. |
Update NetBSD amd64 and 386 builders to the latest NetBSD 9.0 release, using the latest versions of anita (2.3) and pkgsrc packages (2019Q4). Also increase the image disk size from 4 GB to 16 GB. The motivation is to help tests that require more disk space and large dependencies. Additional adjustments to the image building script will need to be made to make use of increased disk space for the /tmp volume, as it is currently being mounted in RAM with size equal to 25% of memory. Remove the manual invocation of the mozilla-rootcerts install command. It no longer works as is in NetBSD 9.0. The mozilla-rootcerts-openssl package is added instead, which successfully performs that same task. Hopefully this will be a more robust approach for the future. Continue to use an explicit URL for the NetBSD release download, and remove unused find_latest_release. The plan is to roll out these builders first (without trybots), gain more confidence, enable trybots, and then remove NetBSD 8.0 builders. For golang/go#37244. For golang/go#30839. For golang/go#31858. For golang/go#31726. For golang/go#38283. Change-Id: Id41278eaebe64cfbcf7b1ab9bfd64e550ee0bbf1 Reviewed-on: https://go-review.googlesource.com/c/build/+/226977 Reviewed-by: Benny Siegert <[email protected]> Reviewed-by: Alexander Rakoczy <[email protected]>
Change https://golang.org/cl/227538 mentions this issue: |
The /tmp partition was being mounted as tmpfs -s=ram%25 by default. On the n1-highcpu-4 machine type with 3.6 GB RAM, this resulted in there being less than 1 GB of available disk space, which is not sufficient to build Go and run its tests. CL 226977 has increased the image disk size from 4 GB to 16 GB, so using the disk for /tmp resolves issues with insufficient disk space. I also considered using an alternative machine type like n1-standard-4 with 15 GB RAM instead, but didn't pursue that option for now because it was not clear if it could be made to work for the 32-bit builder. Remove the replacement of wd0 with sd0 in /etc/fstab. It is no longer required because the NetBSD 9.0 installer defaults to installing on GPT rather than MBR as was done in earlier versions. For golang/go#37244. For golang/go#30839. For golang/go#31726. Change-Id: Icbd0f909ab7768d775c96a6b606b5279a600f1bd Reviewed-on: https://go-review.googlesource.com/c/build/+/227538 Reviewed-by: Benny Siegert <[email protected]> Run-TryBot: Benny Siegert <[email protected]> TryBot-Result: Gobot Gobot <[email protected]>
Change https://golang.org/cl/227545 mentions this issue: |
The NetBSD 9.0 builders are operational as of CL 227538. They've been running on a few commits in all repositories without issues. The netbsd/386 builder is passing, no longer hanging in the runtime tests as the NetBSD 8.0 version did, so keep it enabled. If anything unexpected comes up, we should investigate. For golang/go#37244. For golang/go#31726. Change-Id: Ie3167fa3eabb7f707487561ca2cd4505572df741 Reviewed-on: https://go-review.googlesource.com/c/build/+/227545 Run-TryBot: Dmitri Shuralyov <[email protected]> TryBot-Result: Gobot Gobot <[email protected]> Reviewed-by: Andrew Bonventre <[email protected]>
Change https://golang.org/cl/227738 mentions this issue: |
NetBSD 9.0 is out:
https://netbsd.org/releases/formal-9/NetBSD-9.0.html
Tracking bug for adding a builder.
/cc @bsiegert
The text was updated successfully, but these errors were encountered: