Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Update python-base submodule to pick up changes #562

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

Merged
merged 2 commits into from
Jul 11, 2018

Conversation

roycaihw
Copy link
Member

Remove required idp-certificate-authority-data in kubeconfig for oidc token refresh;
Watch properly decodes resourceVersion from custom object response.

And also update CHANGELOG about the upcoming breaking change for substituting the currently being-used async keyword, which is reserved in Python 3.7.

remove required idp-certificate-authority-data in kubeconfig for oidc
token refresh;
Watch properly decodes resourceVersion from custom object response.
@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Jul 10, 2018
@roycaihw
Copy link
Member Author

/cc @yliaog

@k8s-ci-robot k8s-ci-robot requested a review from yliaog July 10, 2018 19:25
# v7.0.0a1
**NOTE:**
- We are doing an Alpha release for v7.0.0 because there will be an **ACTION REQUIRED** API breaking change in this client to support Python 3.7, which substitutes the currently being-used `async` reserved keyword [kubernetes-client/gen#67](https://github.com/kubernetes-client/gen/pull/67)

Copy link
Contributor

Choose a reason for hiding this comment

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

so the PR has not been merged, do you want to wait for it?

Copy link
Member Author

@roycaihw roycaihw Jul 10, 2018

Choose a reason for hiding this comment

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

I'd like to release v7 alpha first, and we can release v7 stable afterwards to introduce the breaking change. v7 alpha won't include that change. What do you think?

@yliaog
Copy link
Contributor

yliaog commented Jul 10, 2018

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jul 10, 2018
@k8s-ci-robot k8s-ci-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jul 10, 2018
@k8s-ci-robot
Copy link
Contributor

New changes are detected. LGTM label has been removed.

@roycaihw
Copy link
Member Author

Tests are green after rebase on master. Merging

@roycaihw roycaihw merged commit 54eb120 into kubernetes-client:master Jul 11, 2018
@roycaihw roycaihw deleted the update-base branch August 14, 2018 00:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants