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

[5130] [.*:] Upgrade Minio used for local development and e2e tests - Second Attempt #5401

Closed
wants to merge 4 commits into from

Conversation

B0go
Copy link
Contributor

@B0go B0go commented May 31, 2022

  • Change is not relevant to the end-user.

Changes

  • Upgrade Minio used for local development and e2e tests to minio/minio@91130e8 because this version upgrades github.com/shirou/gopsutil from version v2.20.3 to v2.20.9, adding support for Apple Silicon

Verification

If you try to run make test on a Apple Silicon based MacBook using main you get a few undefined: KinfoProc erros:

bogo@Bogos-MacBook-Pro ~/D/thanos (main)> make test
(re)installing /Users/bogo/Developer/Go/bin/alertmanager-v0.20.0
(re)installing /Users/bogo/Developer/Go/bin/minio-v0.0.0-20200527010300-cccf2de129da
# github.com/shirou/gopsutil/cpu
../../Go/pkg/mod/github.com/shirou/gopsutil@v2.20.3-0.20200314133625-53cec6b37e6a+incompatible/cpu/cpu_darwin_cgo.go:13:5: warning: 'TARGET_OS_MAC' is not defined, evaluates to 0 [-Wundef-prefix=TARGET_OS_]
# github.com/shirou/gopsutil/disk
disk_darwin.c:28:2: warning: 'IOMasterPort' is deprecated: first deprecated in macOS 12.0 [-Wdeprecated-declarations]
/Library/Developer/CommandLineTools/SDKs/MacOSX12.3.sdk/System/Library/Frameworks/IOKit.framework/Headers/IOKitLib.h:132:1: note: 'IOMasterPort' has been explicitly marked deprecated here
# github.com/shirou/gopsutil/process
../../Go/pkg/mod/github.com/shirou/gopsutil@v2.20.3-0.20200314133625-53cec6b37e6a+incompatible/process/process_darwin.go:576:34: undefined: KinfoProc
../../Go/pkg/mod/github.com/shirou/gopsutil@v2.20.3-0.20200314133625-53cec6b37e6a+incompatible/process/process_darwin.go:577:8: undefined: KinfoProc
../../Go/pkg/mod/github.com/shirou/gopsutil@v2.20.3-0.20200314133625-53cec6b37e6a+incompatible/process/process_darwin.go:590:32: undefined: KinfoProc
../../Go/pkg/mod/github.com/shirou/gopsutil@v2.20.3-0.20200314133625-53cec6b37e6a+incompatible/process/process_darwin.go:594:62: undefined: KinfoProc
make: *** [/Users/bogo/Developer/Go/bin/minio-v0.0.0-20200527010300-cccf2de129da] Error 2

After the upgrade, the errors stop happening:

bogo@Bogos-MacBook-Pro ~/D/thanos (upgrade-bingo-minio)> make test
(re)installing /Users/bogo/Developer/Go/bin/minio-v0.0.0-20201125204248-91130e884b5d
# github.com/shirou/gopsutil/cpu
../../Go/pkg/mod/github.com/shirou/gopsutil@v2.20.9+incompatible/cpu/cpu_darwin_cgo.go:13:5: warning: 'TARGET_OS_MAC' is not defined, evaluates to 0 [-Wundef-prefix=TARGET_OS_]
# github.com/shirou/gopsutil/disk
iostat_darwin.c:28:2: warning: 'IOMasterPort' is deprecated: first deprecated in macOS 12.0 [-Wdeprecated-declarations]
/Library/Developer/CommandLineTools/SDKs/MacOSX12.3.sdk/System/Library/Frameworks/IOKit.framework/Headers/IOKitLib.h:132:1: note: 'IOMasterPort' has been explicitly marked deprecated here
# github.com/shirou/gopsutil/host
smc_darwin.c:75:41: warning: 'kIOMasterPortDefault' is deprecated: first deprecated in macOS 12.0 [-Wdeprecated-declarations]
/Library/Developer/CommandLineTools/SDKs/MacOSX12.3.sdk/System/Library/Frameworks/IOKit.framework/Headers/IOKitLib.h:123:19: note: 'kIOMasterPortDefault' has been explicitly marked deprecated here
(re)installing /Users/bogo/Developer/Go/bin/prometheus-v2.4.3+incompatible
go: finding module for package github.com/oklog/oklog/pkg/group
go: finding module for package github.com/go-kit/kit/log/level
go: finding module for package github.com/mwitkow/go-conntrack
go: finding module for package github.com/pkg/errors
go: finding module for package github.com/go-kit/kit/log
go: finding module for package gopkg.in/alecthomas/kingpin.v2
go: finding module for package gopkg.in/yaml.v2
go: finding module for package golang.org/x/net/context
go: finding module for package golang.org/x/net/context/ctxhttp
go: finding module for package github.com/opentracing/opentracing-go
go: finding module for package github.com/gogo/protobuf/proto
go: finding module for package github.com/golang/snappy
go: finding module for package golang.org/x/time/rate
go: finding module for package github.com/opentracing-contrib/go-stdlib/nethttp
go: finding module for package github.com/prometheus/client_model/go
go: finding module for package golang.org/x/net/netutil
go: downloading golang.org/x/net v0.0.0-20220526153639-5463443f8c37
go: finding module for package google.golang.org/grpc
go: finding module for package github.com/hashicorp/consul/api
go: finding module for package github.com/aws/aws-sdk-go/aws
go: finding module for package github.com/aws/aws-sdk-go/aws/credentials
go: finding module for package github.com/aws/aws-sdk-go/aws/credentials/stscreds
go: finding module for package github.com/aws/aws-sdk-go/aws/ec2metadata
go: finding module for package github.com/aws/aws-sdk-go/aws/session
go: finding module for package github.com/aws/aws-sdk-go/service/ec2
go: finding module for package gopkg.in/fsnotify/fsnotify.v1
go: downloading github.com/hashicorp/consul v1.12.1
go: finding module for package golang.org/x/oauth2
go: downloading github.com/aws/aws-sdk-go v1.44.24
...

References

Issue #5130

B0go added 4 commits May 29, 2022 16:26
Signed-off-by: B0go <victorbogo@icloud.com>
…88f5ca63

Signed-off-by: B0go <victorbogo@icloud.com>
Signed-off-by: B0go <victorbogo@icloud.com>
@B0go
Copy link
Contributor Author

B0go commented May 31, 2022

Same problem :/
Screen Shot 2022-05-31 at 11 08 14

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant