diff options
author | 2021-07-29 21:27:25 -0600 | |
---|---|---|
committer | 2021-07-29 23:27:25 -0400 | |
commit | 9d5b8cd13d541cc3d2b7598ab255643d6298ba9d (patch) | |
tree | 5ac6f17f1d1a6b0e9033c759fb276a52c3ffab4e /plugin/kubernetes/handler_test.go | |
parent | 8ff7c4b83444c488051384cea0606f085e5b6959 (diff) | |
download | coredns-9d5b8cd13d541cc3d2b7598ab255643d6298ba9d.tar.gz coredns-9d5b8cd13d541cc3d2b7598ab255643d6298ba9d.tar.zst coredns-9d5b8cd13d541cc3d2b7598ab255643d6298ba9d.zip |
kubernetes: Improve namespace usage (#4767)
* Use GetByKey instead of List in GetNamespaceByName.
* Add ToNamespace to reduce memory for namespace cache.
Signed-off-by: Mat Lowery <mlowery@ebay.com>
Diffstat (limited to 'plugin/kubernetes/handler_test.go')
-rw-r--r-- | plugin/kubernetes/handler_test.go | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/plugin/kubernetes/handler_test.go b/plugin/kubernetes/handler_test.go index 19eed2d50..709d831b3 100644 --- a/plugin/kubernetes/handler_test.go +++ b/plugin/kubernetes/handler_test.go @@ -782,16 +782,14 @@ func (APIConnServeTest) GetNodeByName(ctx context.Context, name string) (*api.No }, nil } -func (APIConnServeTest) GetNamespaceByName(name string) (*api.Namespace, error) { +func (APIConnServeTest) GetNamespaceByName(name string) (*object.Namespace, error) { if name == "pod-nons" { // handler_pod_verified_test.go uses this for non-existent namespace. - return &api.Namespace{}, nil + return nil, fmt.Errorf("namespace not found") } if name == "nsnoexist" { return nil, fmt.Errorf("namespace not found") } - return &api.Namespace{ - ObjectMeta: meta.ObjectMeta{ - Name: name, - }, + return &object.Namespace{ + Name: name, }, nil } |