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

update kubernetes-client version to 5.12.1 #1722

Merged
merged 6 commits into from
Jul 19, 2022

Conversation

kyoppii13
Copy link
Contributor

@kyoppii13 kyoppii13 commented Mar 9, 2022

Updated version of k8s client to 5.12.1.

The reason is as follows.

  • I want to use the backoff parameter (kubernetes.request.retry.backoffLimit).
  • Since the supported version of k8s cluster was old.

https://github.com/fabric8io/kubernetes-client

kubernetes-client migration document fromt 4.x to 5.x

@kyoppii13 kyoppii13 marked this pull request as ready for review March 9, 2022 12:34
@yoyama
Copy link
Contributor

yoyama commented Apr 15, 2022

LGTM. But it might be better to add some tests.

@kyoppii13
Copy link
Contributor Author

@yoyama
Thanks for your review. I added a test.
please review again.

@kyoppii13
Copy link
Contributor Author

@yoyama
I resolve conflicts.
Please review.

@yoyama
Copy link
Contributor

yoyama commented Jul 8, 2022

@kyoppii13 I left comments on minor things. Please check it.

@kyoppii13 kyoppii13 requested a review from yoyama July 12, 2022 09:40
@kyoppii13
Copy link
Contributor Author

kyoppii13 commented Jul 12, 2022

@yoyama
Thanks for your review. I fixed it.
Please review.

Copy link
Contributor

@yoyama yoyama left a comment

Choose a reason for hiding this comment

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

LGTM 👍
Thank you for your contribution.

@yoyama yoyama merged commit c3e6cea into treasure-data:master Jul 19, 2022
@szyn szyn added this to the v0.10.5 milestone Feb 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants