github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2025-07-24 12:15:52 +00:00
Code Issues Projects Releases Wiki Activity

Merge pull request #13815 from robertabbott/abbott/kubelet_get_instead_of_list

Browse Source 
Auto commit by PR queue bot
This commit is contained in:
k8s-merge-robot 2015-09-10 15:59:01 -07:00
commit ee7168d868
2 changed files with 12 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
pkg/kubelet/kubelet.go
View File

@ -207,7 +207,11 @@ func NewMainKubelet(
fieldSelector := fields.Set{client.ObjectNameField: nodeName}.AsSelector()
listWatch := &cache.ListWatch{
ListFunc: func() (runtime.Object, error) {
return kubeClient.Nodes().List(labels.Everything(), fieldSelector)
obj, err := kubeClient.Nodes().Get(nodeName)
if err != nil {
return nil, err
}
return &api.NodeList{Items: []api.Node{*obj}}, nil
},
WatchFunc: func(resourceVersion string) (watch.Interface, error) {
return kubeClient.Nodes().Watch(labels.Everything(), fieldSelector, resourceVersion)
@ -714,17 +718,7 @@ func (kl *Kubelet) GetNode() (*api.Node, error) {
if kl.standaloneMode {
return nil, errors.New("no node entry for kubelet in standalone mode")
}
l, err := kl.nodeLister.List()
if err != nil {
return nil, errors.New("cannot list nodes")
}
nodeName := kl.nodeName
for _, n := range l.Items {
if n.Name == nodeName {
return &n, nil
}
}
return nil, fmt.Errorf("node %v not found", nodeName)
return kl.nodeLister.GetNodeInfo(kl.nodeName)
}
// Starts garbage collection threads.

8
pkg/kubelet/kubelet_test.go
View File

@ -18,7 +18,6 @@ package kubelet
import (
"bytes"
"errors"
"fmt"
"io"
"io/ioutil"
@ -966,7 +965,12 @@ type testNodeLister struct {
}
func (ls testNodeLister) GetNodeInfo(id string) (*api.Node, error) {
return nil, errors.New("not implemented")
for _, node := range ls.nodes {
if node.Name == id {
return &node, nil
}
}
return nil, fmt.Errorf("Node with name: %s does not exist", id)
}
func (ls testNodeLister) List() (api.NodeList, error) {