-
Notifications
You must be signed in to change notification settings - Fork 614
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
Add /pods HTTP endpoint #825
Conversation
ac28b8d
to
21e8f10
Compare
node/api/server.go
Outdated
@@ -36,7 +36,8 @@ type ServeMux interface { | |||
type PodHandlerConfig struct { | |||
RunInContainer ContainerExecHandlerFunc | |||
GetContainerLogs ContainerLogsHandlerFunc | |||
GetPods PodListerFunc | |||
GetPodsFromProvider PodListerFunc |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think we should change the name of this field. Maybe add a doc string?
b02e587
to
73b7568
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Still working on this PR. I'll ask for re-review once things are ready. |
bf3c528
to
b0f7704
Compare
@cpuguy83 I made it so we just re-use the podlister that the resource manager has. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
No description provided.