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

[RFC] Major breaking changes in the upcoming release #974

Closed
roycaihw opened this issue Oct 1, 2019 · 17 comments
Closed

[RFC] Major breaking changes in the upcoming release #974

roycaihw opened this issue Oct 1, 2019 · 17 comments
Assignees
Labels
api-review Categorizes an issue or PR as actively needing an API review. kind/bug Categorizes issue or PR as related to a bug. lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed. priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. release-note-action-required Denotes a PR that introduces potentially breaking changes that require user action.

Comments

@roycaihw
Copy link
Member

roycaihw commented Oct 1, 2019

Hi Kubernetes Python client developers, this is a request for comment issue. Please leave comments if you have any opinion/suggestion. Thanks!

What happened:
A majority of the code base is generated code. This repo is switching from using swagger-codegen to using openapi-generator. For more backgrounds:

Major breaking changes were detected when we released 11.0.0a1 with openapi-generator:

The breaking changes are:

  1. [Major] kubernetes.client.apis package is renamed to kubernetes.client.api
  2. [Major] kubernetes package code now uses absolute import instead of relative import
  3. [Minor] The swagger_types attribute in all models is renamed to openapi_types

the major breaking changes come from swagger-api/swagger-codegen#6839 (comment), which is also in latest swagger-codegen. This repo didn't have these changes because we were depending on a old version of swagger-codegen.

What versions are affected?:

  • All 8.0.x, 9.0.x, 10.0.x versions are not affected
  • 11.0.0a1 (corresponding to Kubernetes 1.15) is the only version that's affected currently
  • The next release 11.0.0b1 and future releases (corresponding to Kubernetes 1.15, 1.16+) will be affected.

Impact and fix:

  • All clients that import kubernetes.client.apis must change the import to kubernetes.client.api when upgrading to 11.0.0b1+, to avoid import error.
  • Clients that use relative import and patch the kubernetes package locally may experience unexpected import error. Clients should install kubernetes package properly following the instructions in the Readme or the release page.
@roycaihw roycaihw added kind/bug Categorizes issue or PR as related to a bug. priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. release-note-action-required Denotes a PR that introduces potentially breaking changes that require user action. api-review Categorizes an issue or PR as actively needing an API review. labels Oct 1, 2019
@roycaihw
Copy link
Member Author

roycaihw commented Oct 1, 2019

cc @dims @fabianvf @mbohlool @micw523 @oz123 @scottilee @tomplus @yliaog (apologies if i have missed anyone)

@oz123
Copy link
Contributor

oz123 commented Oct 1, 2019

@roycaihw my 2 cents (appologies for being so scrooge, it's late here):
I consider this "impolite" to remove existing functionality without warnings. Maybe we can add an empty package called api which only imports everything from apis.
However, when one uses stuff from api we can issue a deprecation warning. This package can stay along for 2 versions giving people enough time to update their code.

I'm willing to work a PR for this.
If someone else feels the urgency to do this before me here is a reference:
https://www.lesinskis.com/python_deprecation_tutorial.html

@tomplus
Copy link
Member

tomplus commented Oct 2, 2019

@oz123 there are more changes than renaming modules so it'd be still important to watch out when upgrading to the proposed version.

@oz123
Copy link
Contributor

oz123 commented Oct 2, 2019

@tomplus , thanks for the hint. I will definitely try to make aliases for all the breaking changes, so breaking isn't breaking, rather warning.

@fejta-bot
Copy link

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle stale

@k8s-ci-robot k8s-ci-robot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Dec 31, 2019
@avarf
Copy link

avarf commented Jan 17, 2020

Hi,
Any news regarding 1.16 support? Or the the estimated release time?

/remove-lifecycle stale

@k8s-ci-robot k8s-ci-robot removed the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Jan 17, 2020
@roycaihw
Copy link
Member Author

Hi @avarf, we are actively working on 11.0.0 stable release. After that we will start 12.0.0a1 release, which corresponds to Kubernetes 1.16

@avarf
Copy link

avarf commented Feb 20, 2020

Hi @roycaihw and thanks for your works. Is there any rough timeline for these releases?

@roycaihw
Copy link
Member Author

@palnabarun Nabarun is working on the 11.0.0 release and I think a PR will be created in the next two weeks.

@palnabarun
Copy link
Member

/assign

@palnabarun
Copy link
Member

@roycaihw I think all of the points here are taken care of and 11.0.0 had been released sometime back.

Shall this issue be closed?

Ref: #1097

palnabarun added a commit to palnabarun/python that referenced this issue Jun 22, 2020
palnabarun added a commit to palnabarun/python that referenced this issue Jun 22, 2020
palnabarun added a commit to palnabarun/python that referenced this issue Jul 16, 2020
@fejta-bot
Copy link

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle stale

@k8s-ci-robot k8s-ci-robot added lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. and removed lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. labels Aug 3, 2020
palnabarun added a commit to palnabarun/python that referenced this issue Aug 19, 2020
@palnabarun
Copy link
Member

/remove-lifecycle stale

palnabarun added a commit to palnabarun/python that referenced this issue Oct 12, 2020
palnabarun added a commit to palnabarun/python that referenced this issue Oct 14, 2020
palnabarun added a commit to palnabarun/python that referenced this issue Nov 7, 2020
palnabarun added a commit to palnabarun/python that referenced this issue Nov 7, 2020
palnabarun added a commit to palnabarun/python that referenced this issue Nov 13, 2020
palnabarun added a commit to palnabarun/python that referenced this issue Nov 13, 2020
palnabarun added a commit to palnabarun/python that referenced this issue Nov 13, 2020
palnabarun added a commit to palnabarun/python that referenced this issue Nov 16, 2020
palnabarun added a commit to palnabarun/python that referenced this issue Nov 16, 2020
palnabarun added a commit to palnabarun/python that referenced this issue Nov 16, 2020
palnabarun added a commit to palnabarun/python that referenced this issue Nov 16, 2020
@fejta-bot
Copy link

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle stale

@k8s-ci-robot k8s-ci-robot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Nov 30, 2020
@fejta-bot
Copy link

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle rotten

@k8s-ci-robot k8s-ci-robot added lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed. and removed lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. labels Dec 30, 2020
@fejta-bot
Copy link

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/close

@k8s-ci-robot
Copy link
Contributor

@fejta-bot: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

roycaihw pushed a commit to roycaihw/client-python that referenced this issue Apr 11, 2021
roycaihw pushed a commit to roycaihw/client-python that referenced this issue Apr 12, 2021
bh717 pushed a commit to bh717/python-dapp that referenced this issue Apr 1, 2024
bh717 pushed a commit to bh717/python-dapp that referenced this issue Apr 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api-review Categorizes an issue or PR as actively needing an API review. kind/bug Categorizes issue or PR as related to a bug. lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed. priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. release-note-action-required Denotes a PR that introduces potentially breaking changes that require user action.
Projects
None yet
Development

No branches or pull requests

7 participants