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

【feature】adapt openyurt for k8s version v1.18 #288

Merged
merged 1 commit into from
May 18, 2021

Conversation

rambohe-ch
Copy link
Member

Ⅰ. Describe what this PR does

adapt openyurt for k8s version v1.18, and the main work as following:

  1. the version of k8s.io libs(like client-go) were upgraded to v1.18.8
  2. add context parameter in api for calling client-go v1.18
  3. refactor serializer of cache manager to adapt ClientNegotiator in k8s.io/apimachinery v1.18

Ⅱ. Does this pull request fix one issue?

Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.

Ⅳ. Describe how to verify it

make test
make build
and make sure all components can work correctly with k8s v1.18 cluster

Ⅴ. Special notes for reviews

@rambohe-ch rambohe-ch requested review from kadisi and Fei-Guo May 2, 2021 07:31
@rambohe-ch rambohe-ch changed the title 【feature】adapt openyurt for k8s version v1.18 WIP: 【feature】adapt openyurt for k8s version v1.18 May 2, 2021
@rambohe-ch rambohe-ch changed the title WIP: 【feature】adapt openyurt for k8s version v1.18 [WIP]: 【feature】adapt openyurt for k8s version v1.18 May 2, 2021
@rambohe-ch rambohe-ch force-pushed the adapt-v1.18 branch 2 times, most recently from 776ec54 to f0531d0 Compare May 5, 2021 08:48
@rambohe-ch rambohe-ch force-pushed the adapt-v1.18 branch 2 times, most recently from 79d6034 to f557ebd Compare May 15, 2021 12:46
@rambohe-ch rambohe-ch changed the title [WIP]: 【feature】adapt openyurt for k8s version v1.18 【feature】adapt openyurt for k8s version v1.18 May 17, 2021
@Fei-Guo
Copy link
Member

Fei-Guo commented May 17, 2021

LGTM, leave to @kadisi for another pass.

@kadisi
Copy link
Member

kadisi commented May 18, 2021

/LGTM thanks

@kadisi kadisi merged commit 68a7ecb into openyurtio:master May 18, 2021
@rambohe-ch rambohe-ch deleted the adapt-v1.18 branch May 27, 2021 06:26
MrGirl pushed a commit to MrGirl/openyurt that referenced this pull request Mar 29, 2022
【feature】adapt openyurt for k8s version v1.18
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.

3 participants