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 the /runningpods/ api endpoint #611
Add the /runningpods/ api endpoint #611
Conversation
This adds tracing to the testing to get the kubelet's logs upon failure. In addition, it keeps track of the pods, and the node statuses throughout the test.
c613524
to
cceb51d
Compare
vkubelet/api/pods.go
Outdated
ctx = log.WithLogger(ctx, log.L) | ||
pods, err := p.GetPods(ctx) | ||
if err != nil { | ||
return strongerrors.System(err) |
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.
@cpuguy83 No idea if this is the right one to use.
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.
The provider should give the correct error rather than try to figure it out here.
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.
Will handlError handle it correctly if they don't return a strongerror?
cceb51d
to
0ba2881
Compare
This adds an API endpoint from the kubelet (/runningpods/). It is an endpoint on kubelet which is considered a "debug" endpoint, so it might be worth exposing through the options, but by default it is exposed in most k8s configs AFAICT.
0ba2881
to
10c44e7
Compare
@cpuguy83 any opinions? |
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
* Add the /runningpods/ api endpoint This adds an API endpoint from the kubelet (/runningpods/). It is an endpoint on kubelet which is considered a "debug" endpoint, so it might be worth exposing through the options, but by default it is exposed in most k8s configs AFAICT.
This adds an API endpoint from the kubelet (/runningpods/). It is
an endpoint on kubelet which is considered a "debug" endpoint, so
it might be worth exposing through the options, but by default
it is exposed in most k8s configs AFAICT.