9fans - fans of the OS Plan 9 from Bell Labs
 help / color / mirror / Atom feed
* Re: [9fans] Building go1.12.14
@ 2020-01-03 23:38 Steven Stallion
  0 siblings, 0 replies; 8+ messages in thread
From: Steven Stallion @ 2020-01-03 23:38 UTC (permalink / raw)
  To: 9fans

>> You may want to set GO_TEST_TIMEOUT_SCALE=2, for example,
>> before running all.rc, to increase the timeout to 18 minutes.
> 
> Gotcha.  I'll crank this up and rebuild to see if that clears the
> issue.  With repsect to the failure in exec_test.go, I came across
> this failure on the amd64 builder but haven't found a fix yet:
> https://20191016t122939-dot-build-dot-golang-org.appspot.com/log/5cb857c582e994206fe624231401985967694144

It turns out that Richard posted a patch (exec-postnote-race) some
time ago that I was missing, which resolves the issue in os/exec.
Once applied, tests are now passing normally.  I'll update the wiki
later this evening.

Cheers,
Steve


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [9fans] Building go1.12.14
  2020-01-01 22:08           ` David du Colombier
@ 2020-01-01 22:58             ` Steven Stallion
  0 siblings, 0 replies; 8+ messages in thread
From: Steven Stallion @ 2020-01-01 22:58 UTC (permalink / raw)
  To: 9fans

> You may want to set GO_TEST_TIMEOUT_SCALE=2, for example,
> before running all.rc, to increase the timeout to 18 minutes.

Gotcha.  I'll crank this up and rebuild to see if that clears the
issue.  With repsect to the failure in exec_test.go, I came across
this failure on the amd64 builder but haven't found a fix yet:
https://20191016t122939-dot-build-dot-golang-org.appspot.com/log/5cb857c582e994206fe624231401985967694144

Cheers,
Steve


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [9fans] Building go1.12.14
  2020-01-01 21:36         ` Steven Stallion
@ 2020-01-01 22:08           ` David du Colombier
  2020-01-01 22:58             ` Steven Stallion
  0 siblings, 1 reply; 8+ messages in thread
From: David du Colombier @ 2020-01-01 22:08 UTC (permalink / raw)
  To: 9fans

In the error message you posted, I can see the cmd/go tests are timing out.

It may just be too slow to finish in less that 9 minutes on your machine.
There are a lot of cmd/go tests, especially in the recent releases,
and they are quite slow to run on Plan 9.

You may want to set GO_TEST_TIMEOUT_SCALE=2, for example,
before running all.rc, to increase the timeout to 18 minutes.

-- 
David du Colombier

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [9fans] Building go1.12.14
  2020-01-01 19:10       ` David du Colombier
@ 2020-01-01 21:36         ` Steven Stallion
  2020-01-01 22:08           ` David du Colombier
  0 siblings, 1 reply; 8+ messages in thread
From: Steven Stallion @ 2020-01-01 21:36 UTC (permalink / raw)
  To: 9fans

> Also keep in mind that Go doesn't use any of the Plan 9 libraries
> and uses its own TLS implementation. So fixing the Plan 9
> tools to support TLS 1.2 will have no impact on Go.

Makes sense.  It looks like those patches at least got things moving
again on the Plan 9 side.

I updated my bootstrap compiler and rebuilt the latest release
(1.13.5); I'm still seeing the same failures.  Some digging around
online shows that the same issues are being seen on the 9front builder
for amd64 periodically.  These don't appear to be show-stoppers, so
I'll swing back around later and see if I can suss out what's going
on.

Cheers,
Steve


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [9fans] Building go1.12.14
  2020-01-01 19:04     ` David du Colombier
@ 2020-01-01 19:10       ` David du Colombier
  2020-01-01 21:36         ` Steven Stallion
  0 siblings, 1 reply; 8+ messages in thread
From: David du Colombier @ 2020-01-01 19:10 UTC (permalink / raw)
  To: 9fans

Also keep in mind that Go doesn't use any of the Plan 9 libraries
and uses its own TLS implementation. So fixing the Plan 9
tools to support TLS 1.2 will have no impact on Go.

-- 
David du Colombier

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [9fans] Building go1.12.14
  2020-01-01 18:08   ` Steven Stallion
@ 2020-01-01 19:04     ` David du Colombier
  2020-01-01 19:10       ` David du Colombier
  0 siblings, 1 reply; 8+ messages in thread
From: David du Colombier @ 2020-01-01 19:04 UTC (permalink / raw)
  To: 9fans

> Thanks.  I'm beginning to suspect there may be some additional patches
> needed that I may not have applied.  I've noticed when attempting to
> hget files from GitHub that I'm running into issues with TLS:
> 
>         tlsClient: protocol version not supported
>         
> I've double checked, and libsec-x509-sha2.diff is indeed applied.  Do
> you know of any additional patches that are needed to update TLS
> support?

It seems that GitHub now rejects TLS version < 1.2.

You could try to apply these patches to add TLS 1.2 support to devtls and tlshand:

http://9legacy.org/9legacy/patch/tls-devtls12.diff
http://9legacy.org/9legacy/patch/tls-tlshand12.diff

-- 
David du Colombier

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [9fans] Building go1.12.14
  2020-01-01 17:12 ` [9fans] " David du Colombier
@ 2020-01-01 18:08   ` Steven Stallion
  2020-01-01 19:04     ` David du Colombier
  0 siblings, 1 reply; 8+ messages in thread
From: Steven Stallion @ 2020-01-01 18:08 UTC (permalink / raw)
  To: 9fans

> I think the second issue has been fixed in Go 1.13.
> 
> You can follow the issues their resolution on https://github.com/golang/go/labels/OS-Plan9.

Thanks.  I'm beginning to suspect there may be some additional patches
needed that I may not have applied.  I've noticed when attempting to
hget files from GitHub that I'm running into issues with TLS:

	tlsClient: protocol version not supported
	
I've double checked, and libsec-x509-sha2.diff is indeed applied.  Do
you know of any additional patches that are needed to update TLS
support?

Cheers,
Steve


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [9fans] Building go1.12.14
  2020-01-01 16:20 Steven Stallion
@ 2020-01-01 17:12 ` David du Colombier
  2020-01-01 18:08   ` Steven Stallion
  0 siblings, 1 reply; 8+ messages in thread
From: David du Colombier @ 2020-01-01 17:12 UTC (permalink / raw)
  To: 9fans

> Assuming this is a new issue with the plan9 port, is there a dashboard
> available of working release builds that could be used to avoid
> playing update-whackamole?

Generally speaking, your best bet is running the master branch
on Plan 9, or at least the latest release.

I think the second issue has been fixed in Go 1.13.

You can follow the issues their resolution on https://github.com/golang/go/labels/OS-Plan9.

-- 
David du Colombier

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2020-01-03 23:38 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-01-03 23:38 [9fans] Building go1.12.14 Steven Stallion
  -- strict thread matches above, loose matches on Subject: below --
2020-01-01 16:20 Steven Stallion
2020-01-01 17:12 ` [9fans] " David du Colombier
2020-01-01 18:08   ` Steven Stallion
2020-01-01 19:04     ` David du Colombier
2020-01-01 19:10       ` David du Colombier
2020-01-01 21:36         ` Steven Stallion
2020-01-01 22:08           ` David du Colombier
2020-01-01 22:58             ` Steven Stallion

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).