-
Notifications
You must be signed in to change notification settings - Fork 6
deps: update dependencies to Kubernetes v1.26 #58
Conversation
@@ -87,7 +87,7 @@ setup: | |||
GOBIN=$(DEV_TOOL_PREFIX)/bin go install $(CODEGEN_PKG_NAME)/cmd/informer-gen@$(CODEGEN_PKG_VERSION) | |||
GOBIN=$(DEV_TOOL_PREFIX)/bin go install $(CODEGEN_PKG_NAME)/cmd/deepcopy-gen@$(CODEGEN_PKG_VERSION) | |||
GOBIN=$(DEV_TOOL_PREFIX)/bin go install sigs.k8s.io/kind@latest | |||
GOBIN=$(DEV_TOOL_PREFIX)/bin go install github.com/golangci/golangci-lint/cmd/golangci-lint@v1.50.1 | |||
GOBIN=$(DEV_TOOL_PREFIX)/bin go install github.com/golangci/golangci-lint/cmd/golangci-lint@v1.52.2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
update to be the latest version. (1.50.X doesn't support golang v1.20)
@@ -58,8 +57,6 @@ import ( | |||
) | |||
|
|||
func main() { | |||
rand.Seed(time.Now().UnixNano()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's deprecated in v1.20.
The math/rand package now automatically seeds the global random number generator (used by top-level functions like Float64 and Int) with a random value
https://tip.golang.org/doc/go1.20#math/rand
kubethrottler "github.com/everpeace/kube-throttler/pkg/scheduler_plugin" | ||
"k8s.io/component-base/logs" | ||
"k8s.io/kubernetes/cmd/kube-scheduler/app" | ||
) | ||
|
||
func kubeSchedulerCmd() *cobra.Command { | ||
rand.Seed(time.Now().UnixNano()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ditto
@@ -1,4 +1,4 @@ | |||
apiVersion: kubescheduler.config.k8s.io/v1beta2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
v1beta2 is already deprecated and will be removed soon:
kubernetes/kubernetes#117556
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! LGTM except for nits. PTAL.
.github/workflows/ci.yaml
Outdated
@@ -21,6 +21,7 @@ jobs: | |||
uses: actions/setup-go@v3 | |||
with: | |||
go-version-file: 'go.mod' | |||
go-version: "1.20" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
go.mod
now specifies 1.20
. Do we need this? ditto for other places.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Updated.
Thanks |
This PR updates dependencies to Kubernetes v1.26.
Also, it updates the Golang version to v1.20.