python icon indicating copy to clipboard operation
python copied to clipboard

Add missing supports for kubernetes client stable and gold

Open mbohlool opened this issue 8 years ago • 16 comments

according to new kubernetes client guidelines python client is beta support and has silver capabilities. We are almost there to have gold capabilities and stable support. missing parts are:

Gold:

  • [ ] Support attach, port-forward calls
  • [ ] Proto encoding
  • [ ] Generated examples #252 #219

Stable:

  • [ ] N documented users of the library
  • [ ] Support level documented per-platform
  • [ ] Deprecation policy (backwards compatibility guarantees documented)
  • [ ] Release process well documented

mbohlool avatar Mar 31 '17 09:03 mbohlool

@mbohlool what does "proto encoding" mean?

dims avatar Mar 31 '17 11:03 dims

@lavalamp should know more about this, but I assume it means we encode the request body with proto instead of json.

mbohlool avatar Mar 31 '17 21:03 mbohlool

Correct, request & response bodies. Here's an example proto file: https://github.com/kubernetes/kubernetes/blob/master/pkg/apis/apps/v1beta1/generated.proto

You may not be able to determine the proto encoding from the OpenAPI spec--it'll be interesting to see you try to support these protos in a non-go language...

Also the levels were chosen so that Silver was a perfectly acceptable sort of client to have, so it's not bad to be Silver :)

It's just that if someone is going to write a high-bandwidth controller of some sort, proto is a lot more efficient.

lavalamp avatar Mar 31 '17 21:03 lavalamp

@lavalamp what does "Generated Examples" mean in that doc? Does that mean that you have example code for the library? or that it is generated from something?

brendandburns avatar Apr 27 '17 23:04 brendandburns

Sorry if it's too obious, but does proto mean using this library: https://github.com/google/protobuf/tree/master/python ?

pokoli avatar Jun 07 '17 12:06 pokoli

I looked briefly. The library needs protoc to be installed using other means than pip. If prefer other python only solutions but if that is only option, we should take it. Basically we want to be able to talk proto with api server instead of json. That could include using generated proto messages in k8s main repo. This needs some investigation before implementation.

On Jun 7, 2017 5:09 AM, "Sergi Almacellas Abellana" < [email protected]> wrote:

Sorry if it's too obious, but does proto mean using this library: https://github.com/google/protobuf/tree/master/python ?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/kubernetes-incubator/client-python/issues/166#issuecomment-306775399, or mute the thread https://github.com/notifications/unsubscribe-auth/ABic4IcM8NiAlEK3zjc7lq-MWdjDBlAeks5sBpL6gaJpZM4MvaP_ .

mbohlool avatar Jun 08 '17 06:06 mbohlool

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

fejta-bot avatar Apr 26 '19 20:04 fejta-bot

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

fejta-bot avatar May 26 '19 21:05 fejta-bot

Just to clarify, does this mean port forwarding is not currently supported?

BenElgar avatar Jun 20 '19 21:06 BenElgar

/remove-lifecycle rotten

micw523 avatar Jun 20 '19 22:06 micw523

That is correct. Port forwarding is not yet supported on this client.

micw523 avatar Jun 20 '19 22:06 micw523

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

fejta-bot avatar Sep 18 '19 22:09 fejta-bot

/lifecycle frozen

micw523 avatar Sep 18 '19 23:09 micw523

port-forward call still not supported in python-client now?

mayu2010 avatar Feb 16 '20 14:02 mayu2010

Is there a record of documented users of the library? i.e. A markdown file or something. I'd like to add to it if so.

ghost avatar Feb 28 '20 18:02 ghost

Hey guys, any idea on how the attach / port-forward item is looking?

KnifeyMoloko avatar May 19 '20 10:05 KnifeyMoloko