controller-runtime icon indicating copy to clipboard operation
controller-runtime copied to clipboard

Support non-CRUD subresources (like pod logs)

Open gaocegege opened this issue 6 years ago • 23 comments

Hi I am one of the maintainers @kubeflow

We are using kubebuilder to implement some CRDs for Machine Learning/Deep Learning workloads. We meet some problems with the client in controller-runtime.

We need to get the logs of pods using the client, But It seems that controller-runtime client has a higher level abstraction on top of Kubernetes client. Do we have a plan to support expansion (e.g. https://github.com/kubernetes/client-go/blob/master/kubernetes/typed/core/v1/pod_expansion.go) for the client? Or is there any suggestion to do it with controller-runtime client?

Now we have to create a Kubernetes client and use this to get pod logs.

I'd appreciate it if anyone could help me.

Thanks :clinking_glasses: :beers:

gaocegege avatar May 29 '19 09:05 gaocegege

/kind feature

DirectXMan12 avatar May 31 '19 21:05 DirectXMan12

We had some plans for supporting subresources, but haven't gotten around to them yet. If you put forth a proposed design, I'd be happy to take a look. We'd have to add support for the wonky streaming that logs uses, since right now the client just deals with CRUD.

DirectXMan12 avatar May 31 '19 22:05 DirectXMan12

/priority backlog

DirectXMan12 avatar May 31 '19 22:05 DirectXMan12

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 Aug 29 '19 22:08 fejta-bot

So, we'll get generic subresource support eventually, but non-CRUD subresources need to be handled specially

DirectXMan12 avatar Sep 03 '19 19:09 DirectXMan12

cc #172 (which is mainly for CRUD subresources ATM)

DirectXMan12 avatar Sep 03 '19 19:09 DirectXMan12

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 Oct 03 '19 20:10 fejta-bot

/remove-lifecycle rotten

gaocegege avatar Oct 06 '19 01:10 gaocegege

/lifecycle frozen

DirectXMan12 avatar Oct 07 '19 19:10 DirectXMan12

/help

vincepri avatar Feb 20 '20 19:02 vincepri

@vincepri: This request has been marked as needing help from a contributor.

Please ensure the request meets the requirements listed here.

If this request no longer meets these requirements, the label can be removed by commenting with the /remove-help command.

In response to this:

/help

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.

k8s-ci-robot avatar Feb 20 '20 19:02 k8s-ci-robot

/kind design

vincepri avatar Feb 20 '20 19:02 vincepri

/help

Listson avatar Oct 26 '20 10:10 Listson

I am also interested in this

imilchev avatar Jan 04 '21 10:01 imilchev

is there any workarouds?

27149chen avatar Mar 06 '21 15:03 27149chen

is there any workarouds?

You can use client-go instead.

coderanger avatar Mar 07 '21 12:03 coderanger

You can use client-go instead.

It is hard to keep two clients, I think. It will be better if we support it in controller-runtime.

gaocegege avatar Aug 31 '21 09:08 gaocegege

Is there any progress on this issue?

acechef avatar Oct 20 '22 01:10 acechef

👀

benjaminjb avatar Nov 14 '22 18:11 benjaminjb

Is it ready or still in the design phase?

jasiu001 avatar Dec 01 '23 11:12 jasiu001

Nothing changed.

sbueringer avatar Dec 05 '23 08:12 sbueringer