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

Fail to get release-notes on MacOSX Big Sur #1862

Closed
stafot opened this issue Jan 21, 2021 · 6 comments
Closed

Fail to get release-notes on MacOSX Big Sur #1862

stafot opened this issue Jan 21, 2021 · 6 comments
Labels
area/release-eng Issues or PRs related to the Release Engineering subproject kind/bug Categorizes issue or PR as related to a bug. needs-priority sig/release Categorizes an issue or PR as relevant to SIG Release.

Comments

@stafot
Copy link

stafot commented Jan 21, 2021

What happened:

Running GO111MODULE=on go get k8s.io/release/cmd/release-notes failing and resulting the following errors

go: found k8s.io/release/cmd/release-notes in k8s.io/release v0.6.0
# github.com/shirou/gopsutil/disk
.go/pkg/mod/github.com/shirou/[email protected]+incompatible/disk/disk_darwin.go:64:52: cannot use stat.Mntfromname[:] (type []int8) as type []byte in argument to common.ByteToString
.go/pkg/mod/github.com/shirou/[email protected]+incompatible/disk/disk_darwin.go:65:50: cannot use stat.Mntonname[:] (type []int8) as type []byte in argument to common.ByteToString
.go/pkg/mod/github.com/shirou/[email protected]+incompatible/disk/disk_darwin.go:66:51: cannot use stat.Fstypename[:] (type []int8) as type []byte in argument to common.ByteToString
.go/pkg/mod/github.com/shirou/[email protected]+incompatible/disk/disk_darwin.go:77:44: cannot use stat.Fstypename[:] (type []int8) as type []byte in argument to common.ByteToString

What you expected to happen:

Successfully get release-notes

How to reproduce it (as minimally and precisely as possible):

Run GO111MODULE=on go get k8s.io/release/cmd/release-notes

Anything else we need to know?:

Other projects reported similar problems for macos and freebsd. ref1, ref2, ref3
There is a proposed workaround which I could raise PR for if no objections and if there is no better alternative/solution. ref

Environment:

  • Cloud provider or hardware configuration:
  • OS (e.g: cat /etc/os-release): MacOS Big Sur Version 11.1 (20C69)
  • Kernel (e.g. uname -a): Darwin Kernel Version 20.2.0
  • Others: go version go1.15.6 darwin/amd64
@stafot stafot added area/release-eng Issues or PRs related to the Release Engineering subproject kind/bug Categorizes issue or PR as related to a bug. sig/release Categorizes an issue or PR as relevant to SIG Release. labels Jan 21, 2021
@saschagrunert
Copy link
Member

Hey @stafot, thank you for opening the issue! 🙏

I think this issue is not valid for the master any more. Adding a few tests around cross compilation and releasing a new version of this library should already solve the issue.

@stafot
Copy link
Author

stafot commented Jan 21, 2021

@saschagrunert Ack and thanks. So my alternative is to get the latast master instead of use go get for the latest release and build it through the tooling provided for my platform? Is my understanding correct?

@saschagrunert
Copy link
Member

@saschagrunert Ack and thanks. So my alternative is to get the latast master instead of use go get for the latest release and build it through the tooling provided for my platform? Is my understanding correct?

Yep, can you verify that this works for you?

@stafot
Copy link
Author

stafot commented Jan 21, 2021

@saschagrunert Yes, building it from master on my local did the trick. Thanks for the advice. Before your PR got merged.

@saschagrunert
Copy link
Member

Thanks for the verification @stafot, lets close this for now. We can create a new release soon'ish.

/close

@k8s-ci-robot
Copy link
Contributor

@saschagrunert: Closing this issue.

In response to this:

Thanks for the verification @stafot, lets close this for now. We can create a new release soon'ish.

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/release-eng Issues or PRs related to the Release Engineering subproject kind/bug Categorizes issue or PR as related to a bug. needs-priority sig/release Categorizes an issue or PR as relevant to SIG Release.
Projects
None yet
Development

No branches or pull requests

3 participants