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

Support Kubernetes v1.23 #431

Merged
merged 7 commits into from
Feb 4, 2022
Merged

Support Kubernetes v1.23 #431

merged 7 commits into from
Feb 4, 2022

Conversation

peng225
Copy link
Contributor

@peng225 peng225 commented Jan 28, 2022

issue: #418

EXTERNAL_PROVISIONER_VERSION = 3.0.0 → v3.1.0 (https://github.com/kubernetes-csi/external-provisioner)
EXTERNAL_RESIZER_VERSION = 1.3.0 → v1.4.0 (https://github.com/kubernetes-csi/external-resizer)
NODE_DRIVER_REGISTRAR_VERSION = 2.3.0 → v2.4.0 (https://github.com/kubernetes-csi/node-driver-registrar)
LIVENESSPROBE_VERSION = 2.4.0 → v2.6.0 (https://github.com/kubernetes-csi/livenessprobe)
helm: 3.7.1 → v3.8.0 (https://github.com/helm/helm)
helm-doc: 1.5.0 → v1.7.0 (https://github.com/norwoodj/helm-docs)
protoc: v3.15.0 → v3.19.3 (https://github.com/protocolbuffers/protobuf)
MINIKUBE_VERSION := v1.23.2 → v1.25.1 (https://github.com/kubernetes/minikube)
prometheus/common: v0.26.0 → v0.28.0 (https://github.com/kubernetes/kubernetes/blob/v1.23.3/go.mod#L71-L73)
grpc: v1.42.0 (https://github.com/grpc/grpc)
protobuf: v1.26.0 → v1.27.1 (https://github.com/kubernetes/kubernetes/blob/v1.23.3/go.mod#L102-L103)
YQ: 4.14.1 → 4.18.1 (https://github.com/mikefarah/yq)
golang: 1.16 → v1.17 (https://github.com/golang)
cert-manager: v1.3.1 → v1.7.0 (https://github.com/jetstack/cert-manager)
python: 3.7→3.10 (https://www.python.org/downloads/)
protoc-gen-doc: v1.4.1 → v1.5.0 (https://github.com/pseudomuto/protoc-gen-doc)
spf13/cobra: v1.2.1 → v1.3.0 (https://github.com/spf13/cobra)
spf13/viper v1.8.1 → v1.10.1 (https://github.com/spf13/viper)
protoc-gen-go-grpc: v1.1.0 → v1.2.0 (https://pkg.go.dev/google.golang.org/grpc/cmd/protoc-gen-go-grpc)
klog: v1.0.0 → v2.40.1 (https://github.com/kubernetes/klog)
mount-utils: 0.22.2 → 0.23.3 (https://github.com/kubernetes/mount-utils)

Signed-off-by: Shinya Hayashi shinya-hayashi@cybozu.co.jp

@peng225 peng225 self-assigned this Jan 28, 2022
@project-bot project-bot bot added this to To do in Development Jan 28, 2022
@toshipp toshipp moved this from To do to In progress in Development Jan 31, 2022
@peng225 peng225 force-pushed the support_kubernetes_v1.23 branch 2 times, most recently from 23a1ae6 to 44daf36 Compare February 2, 2022 06:09
Signed-off-by: Shinya Hayashi <shinya-hayashi@cybozu.co.jp>
Signed-off-by: Shinya Hayashi <shinya-hayashi@cybozu.co.jp>
Signed-off-by: Shinya Hayashi <shinya-hayashi@cybozu.co.jp>
Signed-off-by: Shinya Hayashi <shinya-hayashi@cybozu.co.jp>
Signed-off-by: Shinya Hayashi <shinya-hayashi@cybozu.co.jp>
Signed-off-by: Shinya Hayashi <shinya-hayashi@cybozu.co.jp>
@peng225 peng225 requested review from a team, toshipp, llamerada-jp and cupnes and removed request for a team and llamerada-jp February 2, 2022 07:44
@peng225 peng225 marked this pull request as ready for review February 2, 2022 07:45
@peng225 peng225 moved this from In progress to Review in progress in Development Feb 3, 2022
Copy link
Contributor

@llamerada-jp llamerada-jp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@llamerada-jp
Copy link
Contributor

Would you regenerate *.pg.go files using updated protoc command.

@@ -12,5 +12,5 @@ sources:
dependencies:
- name: cert-manager
repository: https://charts.jetstack.io
version: 1.3.1
version: 1.7.0
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you modify charts/topolvm/README.md.gotmpl and deploy/README.md to update version in documents.

@@ -12,7 +12,7 @@
- [How to expose free storage capacity of nodes](#how-to-expose-free-storage-capacity-of-nodes)
- [How to annotate resources](#how-to-annotate-resources)
- [Setting of divisors](#setting-of-divisors)
- [Ephemeral Inline Volume](#ephemeral-inline-volume)
- [Ephemeral Inline Volume (**deprecated**)](#ephemeral-inline-volume-deprecated)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would you revert the changes in this file? Because this document is a proposal.

@llamerada-jp llamerada-jp removed the request for review from cupnes February 3, 2022 04:41
@cupnes cupnes requested review from daichimukai and cupnes and removed request for daichimukai February 3, 2022 05:01
@cupnes cupnes removed the request for review from toshipp February 3, 2022 05:12
@llamerada-jp llamerada-jp self-requested a review February 3, 2022 07:38
llamerada-jp
llamerada-jp previously approved these changes Feb 3, 2022
Copy link
Contributor

@llamerada-jp llamerada-jp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

go.mod Outdated
github.com/prometheus/client_golang v1.11.0
github.com/prometheus/client_model v0.2.0
github.com/prometheus/common v0.26.0
github.com/prometheus/common v0.28.0
github.com/pseudomuto/protoc-gen-doc v1.4.1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you forget the following?

protoc-gen-doc: v1.4.1 → v1.5.0 (https://github.com/pseudomuto/protoc-gen-doc)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cupnes
Sorry, I change the protoc-gen-doc version to v1.5.0.

Signed-off-by: Shinya Hayashi <shinya-hayashi@cybozu.co.jp>
@llamerada-jp llamerada-jp mentioned this pull request Feb 4, 2022
@peng225 peng225 merged commit 9dab586 into main Feb 4, 2022
@peng225 peng225 deleted the support_kubernetes_v1.23 branch February 4, 2022 01:04
Development automation moved this from Review in progress to Done Feb 4, 2022
@toshipp toshipp mentioned this pull request Feb 7, 2022
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants