Use of ssl_context= keyword raises DeprecationWarnings

[mrocklin]

I get the following deprecation warning when using this library.

lib/python3.6/site-packages/kubernetes_asyncio/client/rest.py:73: DeprecationWarning: ssl_context is deprecated, use ssl=context instead
    ssl_context=ssl_context

Swapping out ssl_context= for ssl= works fine when I apply the change locally. However I'm not sure how best to enact change in this repository, given that parts (all?) of it is generated.

cc @yuvipanda

[tomplus]

Yes, this part is generated by swagger-codegen. Now I'm working on switching to the openapi-generator (#58). It's fixed there in the beta release so this warning will be removed eventually.

[tomplus]

It's been fixed in the version v9.0.0.

[mrocklin]

Thanks!

…

On Sun, Mar 24, 2019 at 5:06 AM Tomasz Prus ***@***.***> wrote: It's been fixed in the version v9.0.0. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#59 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AASszB1xTeBeBVGSp87No6EMzKieTObdks5vZ0ASgaJpZM4afBNe> .

[mickours]

The problem is still present in the current version (10.0.1):

kubernetes_asyncio/client/rest.py:69: DeprecationWarning: ssl_context is deprecated, use ssl=context instead
    ssl_context=ssl_context

[tomplus]

@mickours could you check the version of aiohttp? The latest version of the library can print such warnings: #87 Thanks.

[mickours]

Thanks for the quick answer!

I'm using aiohttp 3.5.4

It seems to come from this line:
https://github.com/tomplus/kubernetes_asyncio/blob/master/kubernetes_asyncio/client/rest.py#L69

[tomplus]

Thanks, confirmed, reopen the issue.

[tomplus]

It's fixed in the latest version of the library.

[mrocklin]

I'm glad to hear it! Thank you for tracking this @tomplus .

[mickours]

Thanks a lot! 👍