123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302 |
- /*
- Copyright 2015 The Kubernetes Authors.
- Licensed under the Apache License, Version 2.0 (the "License");
- you may not use this file except in compliance with the License.
- You may obtain a copy of the License at
- http://www.apache.org/licenses/LICENSE-2.0
- Unless required by applicable law or agreed to in writing, software
- distributed under the License is distributed on an "AS IS" BASIS,
- WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- See the License for the specific language governing permissions and
- limitations under the License.
- */
- package podgc
- import (
- "sync"
- "testing"
- "time"
- "k8s.io/api/core/v1"
- metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
- "k8s.io/apimachinery/pkg/labels"
- "k8s.io/apimachinery/pkg/util/sets"
- "k8s.io/client-go/informers"
- coreinformers "k8s.io/client-go/informers/core/v1"
- clientset "k8s.io/client-go/kubernetes"
- "k8s.io/client-go/kubernetes/fake"
- "k8s.io/kubernetes/pkg/controller"
- "k8s.io/kubernetes/pkg/controller/testutil"
- )
- type FakeController struct{}
- func (*FakeController) Run(<-chan struct{}) {}
- func (*FakeController) HasSynced() bool {
- return true
- }
- func (*FakeController) LastSyncResourceVersion() string {
- return ""
- }
- func alwaysReady() bool { return true }
- func NewFromClient(kubeClient clientset.Interface, terminatedPodThreshold int) (*PodGCController, coreinformers.PodInformer) {
- informerFactory := informers.NewSharedInformerFactory(kubeClient, controller.NoResyncPeriodFunc())
- podInformer := informerFactory.Core().V1().Pods()
- controller := NewPodGC(kubeClient, podInformer, terminatedPodThreshold)
- controller.podListerSynced = alwaysReady
- return controller, podInformer
- }
- func TestGCTerminated(t *testing.T) {
- type nameToPhase struct {
- name string
- phase v1.PodPhase
- }
- testCases := []struct {
- pods []nameToPhase
- threshold int
- deletedPodNames sets.String
- }{
- {
- pods: []nameToPhase{
- {name: "a", phase: v1.PodFailed},
- {name: "b", phase: v1.PodSucceeded},
- },
- threshold: 0,
- // threshold = 0 disables terminated pod deletion
- deletedPodNames: sets.NewString(),
- },
- {
- pods: []nameToPhase{
- {name: "a", phase: v1.PodFailed},
- {name: "b", phase: v1.PodSucceeded},
- {name: "c", phase: v1.PodFailed},
- },
- threshold: 1,
- deletedPodNames: sets.NewString("a", "b"),
- },
- {
- pods: []nameToPhase{
- {name: "a", phase: v1.PodRunning},
- {name: "b", phase: v1.PodSucceeded},
- {name: "c", phase: v1.PodFailed},
- },
- threshold: 1,
- deletedPodNames: sets.NewString("b"),
- },
- {
- pods: []nameToPhase{
- {name: "a", phase: v1.PodFailed},
- {name: "b", phase: v1.PodSucceeded},
- },
- threshold: 1,
- deletedPodNames: sets.NewString("a"),
- },
- {
- pods: []nameToPhase{
- {name: "a", phase: v1.PodFailed},
- {name: "b", phase: v1.PodSucceeded},
- },
- threshold: 5,
- deletedPodNames: sets.NewString(),
- },
- }
- for i, test := range testCases {
- client := fake.NewSimpleClientset(&v1.NodeList{Items: []v1.Node{*testutil.NewNode("node")}})
- gcc, podInformer := NewFromClient(client, test.threshold)
- deletedPodNames := make([]string, 0)
- var lock sync.Mutex
- gcc.deletePod = func(_, name string) error {
- lock.Lock()
- defer lock.Unlock()
- deletedPodNames = append(deletedPodNames, name)
- return nil
- }
- creationTime := time.Unix(0, 0)
- for _, pod := range test.pods {
- creationTime = creationTime.Add(1 * time.Hour)
- podInformer.Informer().GetStore().Add(&v1.Pod{
- ObjectMeta: metav1.ObjectMeta{Name: pod.name, CreationTimestamp: metav1.Time{Time: creationTime}},
- Status: v1.PodStatus{Phase: pod.phase},
- Spec: v1.PodSpec{NodeName: "node"},
- })
- }
- gcc.gc()
- pass := true
- for _, pod := range deletedPodNames {
- if !test.deletedPodNames.Has(pod) {
- pass = false
- }
- }
- if len(deletedPodNames) != len(test.deletedPodNames) {
- pass = false
- }
- if !pass {
- t.Errorf("[%v]pod's deleted expected and actual did not match.\n\texpected: %v\n\tactual: %v", i, test.deletedPodNames, deletedPodNames)
- }
- }
- }
- func TestGCOrphaned(t *testing.T) {
- type nameToPhase struct {
- name string
- phase v1.PodPhase
- }
- testCases := []struct {
- pods []nameToPhase
- threshold int
- deletedPodNames sets.String
- }{
- {
- pods: []nameToPhase{
- {name: "a", phase: v1.PodFailed},
- {name: "b", phase: v1.PodSucceeded},
- },
- threshold: 0,
- deletedPodNames: sets.NewString("a", "b"),
- },
- {
- pods: []nameToPhase{
- {name: "a", phase: v1.PodRunning},
- },
- threshold: 1,
- deletedPodNames: sets.NewString("a"),
- },
- }
- for i, test := range testCases {
- client := fake.NewSimpleClientset()
- gcc, podInformer := NewFromClient(client, test.threshold)
- deletedPodNames := make([]string, 0)
- var lock sync.Mutex
- gcc.deletePod = func(_, name string) error {
- lock.Lock()
- defer lock.Unlock()
- deletedPodNames = append(deletedPodNames, name)
- return nil
- }
- creationTime := time.Unix(0, 0)
- for _, pod := range test.pods {
- creationTime = creationTime.Add(1 * time.Hour)
- podInformer.Informer().GetStore().Add(&v1.Pod{
- ObjectMeta: metav1.ObjectMeta{Name: pod.name, CreationTimestamp: metav1.Time{Time: creationTime}},
- Status: v1.PodStatus{Phase: pod.phase},
- Spec: v1.PodSpec{NodeName: "node"},
- })
- }
- pods, err := podInformer.Lister().List(labels.Everything())
- if err != nil {
- t.Errorf("Error while listing all Pods: %v", err)
- return
- }
- gcc.gcOrphaned(pods)
- pass := true
- for _, pod := range deletedPodNames {
- if !test.deletedPodNames.Has(pod) {
- pass = false
- }
- }
- if len(deletedPodNames) != len(test.deletedPodNames) {
- pass = false
- }
- if !pass {
- t.Errorf("[%v]pod's deleted expected and actual did not match.\n\texpected: %v\n\tactual: %v", i, test.deletedPodNames, deletedPodNames)
- }
- }
- }
- func TestGCUnscheduledTerminating(t *testing.T) {
- type nameToPhase struct {
- name string
- phase v1.PodPhase
- deletionTimeStamp *metav1.Time
- nodeName string
- }
- testCases := []struct {
- name string
- pods []nameToPhase
- deletedPodNames sets.String
- }{
- {
- name: "Unscheduled pod in any phase must be deleted",
- pods: []nameToPhase{
- {name: "a", phase: v1.PodFailed, deletionTimeStamp: &metav1.Time{}, nodeName: ""},
- {name: "b", phase: v1.PodSucceeded, deletionTimeStamp: &metav1.Time{}, nodeName: ""},
- {name: "c", phase: v1.PodRunning, deletionTimeStamp: &metav1.Time{}, nodeName: ""},
- },
- deletedPodNames: sets.NewString("a", "b", "c"),
- },
- {
- name: "Scheduled pod in any phase must not be deleted",
- pods: []nameToPhase{
- {name: "a", phase: v1.PodFailed, deletionTimeStamp: nil, nodeName: ""},
- {name: "b", phase: v1.PodSucceeded, deletionTimeStamp: nil, nodeName: "node"},
- {name: "c", phase: v1.PodRunning, deletionTimeStamp: &metav1.Time{}, nodeName: "node"},
- },
- deletedPodNames: sets.NewString(),
- },
- }
- for i, test := range testCases {
- client := fake.NewSimpleClientset()
- gcc, podInformer := NewFromClient(client, -1)
- deletedPodNames := make([]string, 0)
- var lock sync.Mutex
- gcc.deletePod = func(_, name string) error {
- lock.Lock()
- defer lock.Unlock()
- deletedPodNames = append(deletedPodNames, name)
- return nil
- }
- creationTime := time.Unix(0, 0)
- for _, pod := range test.pods {
- creationTime = creationTime.Add(1 * time.Hour)
- podInformer.Informer().GetStore().Add(&v1.Pod{
- ObjectMeta: metav1.ObjectMeta{Name: pod.name, CreationTimestamp: metav1.Time{Time: creationTime},
- DeletionTimestamp: pod.deletionTimeStamp},
- Status: v1.PodStatus{Phase: pod.phase},
- Spec: v1.PodSpec{NodeName: pod.nodeName},
- })
- }
- pods, err := podInformer.Lister().List(labels.Everything())
- if err != nil {
- t.Errorf("Error while listing all Pods: %v", err)
- return
- }
- gcc.gcUnscheduledTerminating(pods)
- pass := true
- for _, pod := range deletedPodNames {
- if !test.deletedPodNames.Has(pod) {
- pass = false
- }
- }
- if len(deletedPodNames) != len(test.deletedPodNames) {
- pass = false
- }
- if !pass {
- t.Errorf("[%v]pod's deleted expected and actual did not match.\n\texpected: %v\n\tactual: %v, test: %v", i, test.deletedPodNames, deletedPodNames, test.name)
- }
- }
- }
|