Uh oh!
There was an error while loading. Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork 81
Bump the k8s-dependencies group across 1 directory with 5 updates#498
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
Uh oh!
There was an error while loading. Please reload this page.
Changes from all commits
File filter
Filter by extension
Conversations
Uh oh!
There was an error while loading. Please reload this page.
Jump to
Uh oh!
There was an error while loading. Please reload this page.
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -11,3 +11,4 @@ | ||
| !variables.env | ||
| *tmp.mod | ||
| *tmp.sum | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,14 +1,13 @@ | ||
| # Project Development Dependencies. | ||
| This is directory which stores Go modules with pinned buildable package that is used within this repository, managed by https://github.com/bwplotka/bingo. | ||
| This is directory which stores Go modules with pinned buildable package that is used within this repository, managed by <https://github.com/bwplotka/bingo>. | ||
| * Run `bingo get` to install all tools having each own module file in this directory. | ||
| * Run `bingo get <tool>` to install <tool> that have own module file in this directory. | ||
| * For Makefile: Make sure to put `include .bingo/Variables.mk` in your Makefile, then use $(<upper case tool name>) variable where <tool> is the .bingo/<tool>.mod. | ||
| * Run `bingo get <tool>` to install `<tool>` that have own module file in this directory. | ||
| * For Makefile: Make sure to put `include .bingo/Variables.mk` in your Makefile, then use `\$(<UPPER_CASE_TOOL_NAME>)` variable where `<tool>` is the .bingo/`<tool>.mod`. | ||
| * For shell: Run `source .bingo/variables.env` to source all environment variable for each tool. | ||
| * For go: Import `.bingo/variables.go` to for variable names. | ||
| * See https://github.com/bwplotka/bingo or -h on how to add, remove or change binaries dependencies. | ||
| * See <https://github.com/bwplotka/bingo> or -h on how to add, remove or change binaries dependencies. | ||
| ## Requirements | ||
| * Go 1.14+ | ||
| * Go 1.24.x or 1.25.x |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -118,11 +118,9 @@ verify: manifests generate format tidy | ||
| GO_INSTALL_OPTS ?= "-mod=mod" | ||
| # Not guaranteed to have patch releases available and node image tags are full versions (i.e v1.28.0 - no v1.28, v1.29, etc.) | ||
| # The KIND_NODE_VERSION is set by getting the version of the k8s.io/client-go dependency from the go.mod | ||
| # and sets major version to "1" and the patch version to "0". For example, a client-go version of v0.28.5 | ||
| # will map to a KIND_NODE_VERSION of 1.28.0 | ||
| KIND_NODE_VERSION := $(shell go list -m k8s.io/client-go | cut -d" " -f2 | sed 's/^v0\.\([[:digit:]]\{1,\}\)\.[[:digit:]]\{1,\}$$/1.\1.0/') | ||
| # Determine the latest available kindest/node image for the Kubernetes minor version | ||
| # matching the k8s.io/client-go dependency (e.g., client-go v0.36.1 → kindest/node:v1.36.1) | ||
Contributor There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Nit: non-ASCII in Makefile | ||
| KIND_NODE_VERSION := $(shell ./hack/get-kind-node-version.sh) | ||
| KIND_CLUSTER_IMAGE := kindest/node:v$(KIND_NODE_VERSION) | ||
| .PHONY: kind-cluster | ||
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
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.
Why remove this? Does the logic no longer hold true?
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.
yeah, it's not guaranteed to have .0 releases either XD