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

fix for golang 1.23 #91

Merged
merged 1 commit into from
Sep 3, 2024
Merged

Conversation

xtreme-nitin-ravindran
Copy link
Member

Replace KeepAlive with KeepAliveConfig
Fixes

• [FAILED] [0.001 seconds]
Linux-specific tests [It] enables TCP (socket) keepalive with an appropriate interval
/tmp/build/1944c6a2/gopath/src/github.com/cloudfoundry/bosh-utils/httpclient/keepalive_syscall_linux_test.go:17

  [FAILED] Expected
      <int>: 15
  to equal
      <int>: 30
  In [It] at: /tmp/build/1944c6a2/gopath/src/github.com/cloudfoundry/bosh-utils/httpclient/keepalive_syscall_linux_test.go:58 @ 09/03/24 14:42:33.858

  Full Stack Trace
    github.com/cloudfoundry/bosh-utils/httpclient_test.init.func3.1()
    	/tmp/build/1944c6a2/gopath/src/github.com/cloudfoundry/bosh-utils/httpclient/keepalive_syscall_linux_test.go:58 +0x81a

Replace KeepAlive with KeepAliveConfig
Fixes
```
• [FAILED] [0.001 seconds]
Linux-specific tests [It] enables TCP (socket) keepalive with an appropriate interval
/tmp/build/1944c6a2/gopath/src/github.com/cloudfoundry/bosh-utils/httpclient/keepalive_syscall_linux_test.go:17

  [FAILED] Expected
      <int>: 15
  to equal
      <int>: 30
  In [It] at: /tmp/build/1944c6a2/gopath/src/github.com/cloudfoundry/bosh-utils/httpclient/keepalive_syscall_linux_test.go:58 @ 09/03/24 14:42:33.858

  Full Stack Trace
    github.com/cloudfoundry/bosh-utils/httpclient_test.init.func3.1()
    	/tmp/build/1944c6a2/gopath/src/github.com/cloudfoundry/bosh-utils/httpclient/keepalive_syscall_linux_test.go:58 +0x81a
```
@xtreme-nitin-ravindran xtreme-nitin-ravindran merged commit a546b52 into master Sep 3, 2024
5 checks passed
@xtreme-nitin-ravindran xtreme-nitin-ravindran deleted the fix-for-golang-1.23 branch September 3, 2024 22:57
@xtreme-nitin-ravindran xtreme-nitin-ravindran restored the fix-for-golang-1.23 branch September 3, 2024 23:02
@xtreme-nitin-ravindran xtreme-nitin-ravindran deleted the fix-for-golang-1.23 branch September 3, 2024 23:26
mingxiao pushed a commit to mingxiao/bosh-utils that referenced this pull request Sep 4, 2024
…golang-1.23"

This reverts commit a546b52, reversing
changes made to 95a094e.
jpalermo pushed a commit that referenced this pull request Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants