Skip to content
This repository has been archived by the owner on Nov 19, 2020. It is now read-only.

Set “name” path parameter for endpoint watch #107

Open
ekoome opened this issue Dec 10, 2018 · 3 comments · May be fixed by #108
Open

Set “name” path parameter for endpoint watch #107

ekoome opened this issue Dec 10, 2018 · 3 comments · May be fixed by #108

Comments

@ekoome
Copy link

ekoome commented Dec 10, 2018

How do I set “name” path parameter for endpoint watch in GET /api/v1/watch/namespaces/{namespace}/endpoints/{name} - https://v1-9.docs.kubernetes.io/docs/reference/generated/kubernetes-api/v1.9/#watch-145

The current api for watch does not have a “name” parameter for the service in this case.

func (c *Client) Watch(ctx context.Context, namespace string, r Resource, options ...Option) (*Watcher, error)
@ericchiang
Copy link
Owner

I wasn't aware that you were able to watch an individual instance of a resource. Does this work for things other than endpoints?

There's currently no way to do this with this client.

@ekoome
Copy link
Author

ekoome commented Dec 11, 2018

Ability to watch by name is also available on Services - https://v1-9.docs.kubernetes.io/docs/reference/generated/kubernetes-api/v1.9/#watch-174;
and Ingresses- https://v1-9.docs.kubernetes.io/docs/reference/generated/kubernetes-api/v1.9/#-strong-read-operations-strong--154

This simplifies watch operations. It would be great to have this in the client for completeness.

@ericchiang ericchiang linked a pull request Dec 12, 2018 that will close this issue
Open
@ericchiang
Copy link
Owner

I took a crack at this and couldn't get watch on an individual configmap working #108

The docs do say that this should work though... https://v1-9.docs.kubernetes.io/docs/reference/generated/kubernetes-api/v1.9/#watch-210

Going to try to take another go at this later.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

*: allow watches on individual resources ericchiang/k8s
2 participants
@ericchiang @ekoome