mirror of
https://github.com/k3s-io/kubernetes.git
synced 2025-08-05 18:24:07 +00:00
Merge pull request #3194 from vishh/stats
Update "/stats" endpoint on kubelet.
This commit is contained in:
commit
78547d9a87
@ -325,7 +325,7 @@ func (s *Server) ServeHTTP(w http.ResponseWriter, req *http.Request) {
|
|||||||
|
|
||||||
// serveStats implements stats logic.
|
// serveStats implements stats logic.
|
||||||
func (s *Server) serveStats(w http.ResponseWriter, req *http.Request) {
|
func (s *Server) serveStats(w http.ResponseWriter, req *http.Request) {
|
||||||
// /stats/<podfullname>/<containerName> or /stats/<podfullname>/<uuid>/<containerName>
|
// /stats/<podfullname>/<containerName> or /stats/<namespace>/<podfullname>/<uuid>/<containerName>
|
||||||
components := strings.Split(strings.TrimPrefix(path.Clean(req.URL.Path), "/"), "/")
|
components := strings.Split(strings.TrimPrefix(path.Clean(req.URL.Path), "/"), "/")
|
||||||
var stats *info.ContainerInfo
|
var stats *info.ContainerInfo
|
||||||
var err error
|
var err error
|
||||||
@ -348,22 +348,20 @@ func (s *Server) serveStats(w http.ResponseWriter, req *http.Request) {
|
|||||||
podFullName := GetPodFullName(&api.BoundPod{
|
podFullName := GetPodFullName(&api.BoundPod{
|
||||||
ObjectMeta: api.ObjectMeta{
|
ObjectMeta: api.ObjectMeta{
|
||||||
Name: components[1],
|
Name: components[1],
|
||||||
// TODO: I am broken
|
|
||||||
Namespace: api.NamespaceDefault,
|
Namespace: api.NamespaceDefault,
|
||||||
Annotations: map[string]string{ConfigSourceAnnotationKey: "etcd"},
|
Annotations: map[string]string{ConfigSourceAnnotationKey: "etcd"},
|
||||||
},
|
},
|
||||||
})
|
})
|
||||||
stats, err = s.host.GetContainerInfo(podFullName, "", components[2], &query)
|
stats, err = s.host.GetContainerInfo(podFullName, "", components[2], &query)
|
||||||
case 4:
|
case 5:
|
||||||
podFullName := GetPodFullName(&api.BoundPod{
|
podFullName := GetPodFullName(&api.BoundPod{
|
||||||
ObjectMeta: api.ObjectMeta{
|
ObjectMeta: api.ObjectMeta{
|
||||||
Name: components[1],
|
Name: components[2],
|
||||||
// TODO: I am broken
|
Namespace: components[1],
|
||||||
Namespace: "",
|
|
||||||
Annotations: map[string]string{ConfigSourceAnnotationKey: "etcd"},
|
Annotations: map[string]string{ConfigSourceAnnotationKey: "etcd"},
|
||||||
},
|
},
|
||||||
})
|
})
|
||||||
stats, err = s.host.GetContainerInfo(podFullName, components[2], components[2], &query)
|
stats, err = s.host.GetContainerInfo(podFullName, components[3], components[4], &query)
|
||||||
default:
|
default:
|
||||||
http.Error(w, "unknown resource.", http.StatusNotFound)
|
http.Error(w, "unknown resource.", http.StatusNotFound)
|
||||||
return
|
return
|
||||||
|
@ -34,7 +34,7 @@ import (
|
|||||||
|
|
||||||
type fakeKubelet struct {
|
type fakeKubelet struct {
|
||||||
infoFunc func(name string) (api.PodInfo, error)
|
infoFunc func(name string) (api.PodInfo, error)
|
||||||
containerInfoFunc func(podFullName, containerName string, req *info.ContainerInfoRequest) (*info.ContainerInfo, error)
|
containerInfoFunc func(podFullName, uid, containerName string, req *info.ContainerInfoRequest) (*info.ContainerInfo, error)
|
||||||
rootInfoFunc func(query *info.ContainerInfoRequest) (*info.ContainerInfo, error)
|
rootInfoFunc func(query *info.ContainerInfoRequest) (*info.ContainerInfo, error)
|
||||||
machineInfoFunc func() (*info.MachineInfo, error)
|
machineInfoFunc func() (*info.MachineInfo, error)
|
||||||
boundPodsFunc func() ([]api.BoundPod, error)
|
boundPodsFunc func() ([]api.BoundPod, error)
|
||||||
@ -48,7 +48,7 @@ func (fk *fakeKubelet) GetPodInfo(name, uuid string) (api.PodInfo, error) {
|
|||||||
}
|
}
|
||||||
|
|
||||||
func (fk *fakeKubelet) GetContainerInfo(podFullName, uuid, containerName string, req *info.ContainerInfoRequest) (*info.ContainerInfo, error) {
|
func (fk *fakeKubelet) GetContainerInfo(podFullName, uuid, containerName string, req *info.ContainerInfoRequest) (*info.ContainerInfo, error) {
|
||||||
return fk.containerInfoFunc(podFullName, containerName, req)
|
return fk.containerInfoFunc(podFullName, uuid, containerName, req)
|
||||||
}
|
}
|
||||||
|
|
||||||
func (fk *fakeKubelet) GetRootInfo(req *info.ContainerInfoRequest) (*info.ContainerInfo, error) {
|
func (fk *fakeKubelet) GetRootInfo(req *info.ContainerInfoRequest) (*info.ContainerInfo, error) {
|
||||||
@ -144,7 +144,7 @@ func TestContainerInfo(t *testing.T) {
|
|||||||
podID := "somepod"
|
podID := "somepod"
|
||||||
expectedPodID := "somepod" + ".default.etcd"
|
expectedPodID := "somepod" + ".default.etcd"
|
||||||
expectedContainerName := "goodcontainer"
|
expectedContainerName := "goodcontainer"
|
||||||
fw.fakeKubelet.containerInfoFunc = func(podID, containerName string, req *info.ContainerInfoRequest) (*info.ContainerInfo, error) {
|
fw.fakeKubelet.containerInfoFunc = func(podID, uid, containerName string, req *info.ContainerInfoRequest) (*info.ContainerInfo, error) {
|
||||||
if podID != expectedPodID || containerName != expectedContainerName {
|
if podID != expectedPodID || containerName != expectedContainerName {
|
||||||
return nil, fmt.Errorf("bad podID or containerName: podID=%v; containerName=%v", podID, containerName)
|
return nil, fmt.Errorf("bad podID or containerName: podID=%v; containerName=%v", podID, containerName)
|
||||||
}
|
}
|
||||||
@ -166,6 +166,36 @@ func TestContainerInfo(t *testing.T) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func TestContainerInfoWithUidNamespace(t *testing.T) {
|
||||||
|
fw := newServerTest()
|
||||||
|
expectedInfo := &info.ContainerInfo{}
|
||||||
|
podID := "somepod"
|
||||||
|
expectedNamespace := "custom"
|
||||||
|
expectedPodID := "somepod" + "." + expectedNamespace + ".etcd"
|
||||||
|
expectedContainerName := "goodcontainer"
|
||||||
|
expectedUid := "9b01b80f-8fb4-11e4-95ab-4200af06647"
|
||||||
|
fw.fakeKubelet.containerInfoFunc = func(podID, uid, containerName string, req *info.ContainerInfoRequest) (*info.ContainerInfo, error) {
|
||||||
|
if podID != expectedPodID || uid != expectedUid || containerName != expectedContainerName {
|
||||||
|
return nil, fmt.Errorf("bad podID or uid or containerName: podID=%v; uid=%v; containerName=%v", podID, uid, containerName)
|
||||||
|
}
|
||||||
|
return expectedInfo, nil
|
||||||
|
}
|
||||||
|
|
||||||
|
resp, err := http.Get(fw.testHTTPServer.URL + fmt.Sprintf("/stats/%v/%v/%v/%v", expectedNamespace, podID, expectedUid, expectedContainerName))
|
||||||
|
if err != nil {
|
||||||
|
t.Fatalf("Got error GETing: %v", err)
|
||||||
|
}
|
||||||
|
defer resp.Body.Close()
|
||||||
|
var receivedInfo info.ContainerInfo
|
||||||
|
err = json.NewDecoder(resp.Body).Decode(&receivedInfo)
|
||||||
|
if err != nil {
|
||||||
|
t.Fatalf("received invalid json data: %v", err)
|
||||||
|
}
|
||||||
|
if !reflect.DeepEqual(&receivedInfo, expectedInfo) {
|
||||||
|
t.Errorf("received wrong data: %#v", receivedInfo)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
func TestRootInfo(t *testing.T) {
|
func TestRootInfo(t *testing.T) {
|
||||||
fw := newServerTest()
|
fw := newServerTest()
|
||||||
expectedInfo := &info.ContainerInfo{}
|
expectedInfo := &info.ContainerInfo{}
|
||||||
|
Loading…
Reference in New Issue
Block a user