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

[WIP] Release 12.0 #1088

Closed
wants to merge 10 commits into from
Closed

[WIP] Release 12.0 #1088

wants to merge 10 commits into from

Conversation

scottilee
Copy link
Contributor

@scottilee scottilee commented Feb 23, 2020

Release 12.0 introduces several changes and is compatible with Kubernetes v1.16.

  1. Updated the constants version
  2. Generated client changes
  3. Applied hotfixes
  4. Updated python-base submodules
  5. Updated the CHANGELOG (WORK-IN-PROGRESS)

Fixes #1052

micw523 and others added 10 commits December 23, 2019 17:15
This removes the __del__ function from the generated Python client,
and replaces it with a cleanup function. When a ThreadPool is created,
the cleanup function is registered with the atexit module.

This PR also allows the client to be used as a context manager, which
will automatically clean up after itself rather than having to wait til
process exit.

This fixes issue #1037, where the API client could hang indefinitely at
garbage collection.
Cleanup ThreadPool with atexit rather than __del__
Document hot issues in release process
Grammar fixes to contributing doc
@k8s-ci-robot k8s-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Feb 23, 2020
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: scottilee
To complete the pull request process, please assign roycaihw
You can assign the PR to them by writing /assign @roycaihw in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@roycaihw
Copy link
Member

It looks like this PR is still WIP (it merges the master branch to the release branch). Please let me know when you have more updates and need review

@palnabarun
Copy link
Member

/cc

@roycaihw
Copy link
Member

roycaihw commented Mar 2, 2020

Could you also track upgrading the upstream code generator in the description?

@scottilee
Copy link
Contributor Author

Closing this in favor of a new PR due to many merge conflicts and issues.

@scottilee scottilee closed this Mar 27, 2020
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. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants