123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652653654655656657658659660661662663664665666667668669670671672673674675676677678679680681682683684685686687688689690691692693694695696697698699700701702703704705706707708709710711712713714715716717718719720721722723724725726727728729730731732733734735736737738739740741742743744745746747748749750751752753754755756757758759760761762763764765766767768769770771772773774775776777778779780781782783784785786787788789790791792793794795796797798799800801802803804805806807808809810811812813814815816817818819820821822823824825826827828829830831832833834835836837838839840841842843844845846847848849850851852853854855856857858859860861862863864865866867868869870871872873874875876877878879880881882883884885886887888889890891892893894895896897898899900901902903904905906907908909910911912913914915916917918919920921922923924925926927928929930931932933934935936937938939940941942943944945946947948949950951952953954955956957958959960961962963964965966967968969970971972973974975976977978979980981982983984985986987988989990991992993994995996997998999100010011002100310041005100610071008100910101011101210131014101510161017101810191020102110221023102410251026102710281029 |
- /*
- Copyright 2016 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 disruption
- import (
- "fmt"
- "runtime/debug"
- "testing"
- "time"
- apps "k8s.io/api/apps/v1"
- autoscalingapi "k8s.io/api/autoscaling/v1"
- "k8s.io/api/core/v1"
- policy "k8s.io/api/policy/v1beta1"
- apiequality "k8s.io/apimachinery/pkg/api/equality"
- "k8s.io/apimachinery/pkg/api/meta/testrestmapper"
- metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
- "k8s.io/apimachinery/pkg/runtime"
- "k8s.io/apimachinery/pkg/runtime/schema"
- "k8s.io/apimachinery/pkg/types"
- "k8s.io/apimachinery/pkg/util/intstr"
- "k8s.io/apimachinery/pkg/util/uuid"
- "k8s.io/client-go/informers"
- scalefake "k8s.io/client-go/scale/fake"
- core "k8s.io/client-go/testing"
- "k8s.io/client-go/tools/cache"
- "k8s.io/client-go/util/workqueue"
- _ "k8s.io/kubernetes/pkg/apis/core/install"
- "k8s.io/kubernetes/pkg/controller"
- "github.com/Azure/go-autorest/autorest/to"
- )
- type pdbStates map[string]policy.PodDisruptionBudget
- var alwaysReady = func() bool { return true }
- func (ps *pdbStates) Set(pdb *policy.PodDisruptionBudget) error {
- key, err := controller.KeyFunc(pdb)
- if err != nil {
- return err
- }
- (*ps)[key] = *pdb.DeepCopy()
- return nil
- }
- func (ps *pdbStates) Get(key string) policy.PodDisruptionBudget {
- return (*ps)[key]
- }
- func (ps *pdbStates) VerifyPdbStatus(t *testing.T, key string, disruptionsAllowed, currentHealthy, desiredHealthy, expectedPods int32,
- disruptedPodMap map[string]metav1.Time) {
- actualPDB := ps.Get(key)
- expectedStatus := policy.PodDisruptionBudgetStatus{
- PodDisruptionsAllowed: disruptionsAllowed,
- CurrentHealthy: currentHealthy,
- DesiredHealthy: desiredHealthy,
- ExpectedPods: expectedPods,
- DisruptedPods: disruptedPodMap,
- ObservedGeneration: actualPDB.Generation,
- }
- actualStatus := actualPDB.Status
- if !apiequality.Semantic.DeepEqual(actualStatus, expectedStatus) {
- debug.PrintStack()
- t.Fatalf("PDB %q status mismatch. Expected %+v but got %+v.", key, expectedStatus, actualStatus)
- }
- }
- func (ps *pdbStates) VerifyDisruptionAllowed(t *testing.T, key string, disruptionsAllowed int32) {
- pdb := ps.Get(key)
- if pdb.Status.PodDisruptionsAllowed != disruptionsAllowed {
- debug.PrintStack()
- t.Fatalf("PodDisruptionAllowed mismatch for PDB %q. Expected %v but got %v.", key, disruptionsAllowed, pdb.Status.PodDisruptionsAllowed)
- }
- }
- type disruptionController struct {
- *DisruptionController
- podStore cache.Store
- pdbStore cache.Store
- rcStore cache.Store
- rsStore cache.Store
- dStore cache.Store
- ssStore cache.Store
- scaleClient *scalefake.FakeScaleClient
- }
- var customGVK = schema.GroupVersionKind{
- Group: "custom.k8s.io",
- Version: "v1",
- Kind: "customresource",
- }
- func newFakeDisruptionController() (*disruptionController, *pdbStates) {
- ps := &pdbStates{}
- informerFactory := informers.NewSharedInformerFactory(nil, controller.NoResyncPeriodFunc())
- scheme := runtime.NewScheme()
- scheme.AddKnownTypeWithName(customGVK, &v1.Service{})
- fakeScaleClient := &scalefake.FakeScaleClient{}
- dc := NewDisruptionController(
- informerFactory.Core().V1().Pods(),
- informerFactory.Policy().V1beta1().PodDisruptionBudgets(),
- informerFactory.Core().V1().ReplicationControllers(),
- informerFactory.Apps().V1().ReplicaSets(),
- informerFactory.Apps().V1().Deployments(),
- informerFactory.Apps().V1().StatefulSets(),
- nil,
- testrestmapper.TestOnlyStaticRESTMapper(scheme),
- fakeScaleClient,
- )
- dc.getUpdater = func() updater { return ps.Set }
- dc.podListerSynced = alwaysReady
- dc.pdbListerSynced = alwaysReady
- dc.rcListerSynced = alwaysReady
- dc.rsListerSynced = alwaysReady
- dc.dListerSynced = alwaysReady
- dc.ssListerSynced = alwaysReady
- return &disruptionController{
- dc,
- informerFactory.Core().V1().Pods().Informer().GetStore(),
- informerFactory.Policy().V1beta1().PodDisruptionBudgets().Informer().GetStore(),
- informerFactory.Core().V1().ReplicationControllers().Informer().GetStore(),
- informerFactory.Apps().V1().ReplicaSets().Informer().GetStore(),
- informerFactory.Apps().V1().Deployments().Informer().GetStore(),
- informerFactory.Apps().V1().StatefulSets().Informer().GetStore(),
- fakeScaleClient,
- }, ps
- }
- func fooBar() map[string]string {
- return map[string]string{"foo": "bar"}
- }
- func newSel(labels map[string]string) *metav1.LabelSelector {
- return &metav1.LabelSelector{MatchLabels: labels}
- }
- func newSelFooBar() *metav1.LabelSelector {
- return newSel(map[string]string{"foo": "bar"})
- }
- func newMinAvailablePodDisruptionBudget(t *testing.T, minAvailable intstr.IntOrString) (*policy.PodDisruptionBudget, string) {
- pdb := &policy.PodDisruptionBudget{
- TypeMeta: metav1.TypeMeta{APIVersion: "v1"},
- ObjectMeta: metav1.ObjectMeta{
- UID: uuid.NewUUID(),
- Name: "foobar",
- Namespace: metav1.NamespaceDefault,
- ResourceVersion: "18",
- },
- Spec: policy.PodDisruptionBudgetSpec{
- MinAvailable: &minAvailable,
- Selector: newSelFooBar(),
- },
- }
- pdbName, err := controller.KeyFunc(pdb)
- if err != nil {
- t.Fatalf("Unexpected error naming pdb %q: %v", pdb.Name, err)
- }
- return pdb, pdbName
- }
- func newMaxUnavailablePodDisruptionBudget(t *testing.T, maxUnavailable intstr.IntOrString) (*policy.PodDisruptionBudget, string) {
- pdb := &policy.PodDisruptionBudget{
- TypeMeta: metav1.TypeMeta{APIVersion: "v1"},
- ObjectMeta: metav1.ObjectMeta{
- UID: uuid.NewUUID(),
- Name: "foobar",
- Namespace: metav1.NamespaceDefault,
- ResourceVersion: "18",
- },
- Spec: policy.PodDisruptionBudgetSpec{
- MaxUnavailable: &maxUnavailable,
- Selector: newSelFooBar(),
- },
- }
- pdbName, err := controller.KeyFunc(pdb)
- if err != nil {
- t.Fatalf("Unexpected error naming pdb %q: %v", pdb.Name, err)
- }
- return pdb, pdbName
- }
- func updatePodOwnerToRc(t *testing.T, pod *v1.Pod, rc *v1.ReplicationController) {
- var controllerReference metav1.OwnerReference
- var trueVar = true
- controllerReference = metav1.OwnerReference{UID: rc.UID, APIVersion: controllerKindRC.GroupVersion().String(), Kind: controllerKindRC.Kind, Name: rc.Name, Controller: &trueVar}
- pod.OwnerReferences = append(pod.OwnerReferences, controllerReference)
- }
- func updatePodOwnerToRs(t *testing.T, pod *v1.Pod, rs *apps.ReplicaSet) {
- var controllerReference metav1.OwnerReference
- var trueVar = true
- controllerReference = metav1.OwnerReference{UID: rs.UID, APIVersion: controllerKindRS.GroupVersion().String(), Kind: controllerKindRS.Kind, Name: rs.Name, Controller: &trueVar}
- pod.OwnerReferences = append(pod.OwnerReferences, controllerReference)
- }
- // pod, podName := newPod(t, name)
- func updatePodOwnerToSs(t *testing.T, pod *v1.Pod, ss *apps.StatefulSet) {
- var controllerReference metav1.OwnerReference
- var trueVar = true
- controllerReference = metav1.OwnerReference{UID: ss.UID, APIVersion: controllerKindSS.GroupVersion().String(), Kind: controllerKindSS.Kind, Name: ss.Name, Controller: &trueVar}
- pod.OwnerReferences = append(pod.OwnerReferences, controllerReference)
- }
- func newPod(t *testing.T, name string) (*v1.Pod, string) {
- pod := &v1.Pod{
- TypeMeta: metav1.TypeMeta{APIVersion: "v1"},
- ObjectMeta: metav1.ObjectMeta{
- UID: uuid.NewUUID(),
- Annotations: make(map[string]string),
- Name: name,
- Namespace: metav1.NamespaceDefault,
- ResourceVersion: "18",
- Labels: fooBar(),
- },
- Spec: v1.PodSpec{},
- Status: v1.PodStatus{
- Conditions: []v1.PodCondition{
- {Type: v1.PodReady, Status: v1.ConditionTrue},
- },
- },
- }
- podName, err := controller.KeyFunc(pod)
- if err != nil {
- t.Fatalf("Unexpected error naming pod %q: %v", pod.Name, err)
- }
- return pod, podName
- }
- func newReplicationController(t *testing.T, size int32) (*v1.ReplicationController, string) {
- rc := &v1.ReplicationController{
- TypeMeta: metav1.TypeMeta{APIVersion: "v1"},
- ObjectMeta: metav1.ObjectMeta{
- UID: uuid.NewUUID(),
- Name: "foobar",
- Namespace: metav1.NamespaceDefault,
- ResourceVersion: "18",
- Labels: fooBar(),
- },
- Spec: v1.ReplicationControllerSpec{
- Replicas: &size,
- Selector: fooBar(),
- },
- }
- rcName, err := controller.KeyFunc(rc)
- if err != nil {
- t.Fatalf("Unexpected error naming RC %q", rc.Name)
- }
- return rc, rcName
- }
- func newDeployment(t *testing.T, size int32) (*apps.Deployment, string) {
- d := &apps.Deployment{
- TypeMeta: metav1.TypeMeta{APIVersion: "v1"},
- ObjectMeta: metav1.ObjectMeta{
- UID: uuid.NewUUID(),
- Name: "foobar",
- Namespace: metav1.NamespaceDefault,
- ResourceVersion: "18",
- Labels: fooBar(),
- },
- Spec: apps.DeploymentSpec{
- Replicas: &size,
- Selector: newSelFooBar(),
- },
- }
- dName, err := controller.KeyFunc(d)
- if err != nil {
- t.Fatalf("Unexpected error naming Deployment %q: %v", d.Name, err)
- }
- return d, dName
- }
- func newReplicaSet(t *testing.T, size int32) (*apps.ReplicaSet, string) {
- rs := &apps.ReplicaSet{
- TypeMeta: metav1.TypeMeta{APIVersion: "v1"},
- ObjectMeta: metav1.ObjectMeta{
- UID: uuid.NewUUID(),
- Name: "foobar",
- Namespace: metav1.NamespaceDefault,
- ResourceVersion: "18",
- Labels: fooBar(),
- },
- Spec: apps.ReplicaSetSpec{
- Replicas: &size,
- Selector: newSelFooBar(),
- },
- }
- rsName, err := controller.KeyFunc(rs)
- if err != nil {
- t.Fatalf("Unexpected error naming ReplicaSet %q: %v", rs.Name, err)
- }
- return rs, rsName
- }
- func newStatefulSet(t *testing.T, size int32) (*apps.StatefulSet, string) {
- ss := &apps.StatefulSet{
- TypeMeta: metav1.TypeMeta{APIVersion: "v1"},
- ObjectMeta: metav1.ObjectMeta{
- UID: uuid.NewUUID(),
- Name: "foobar",
- Namespace: metav1.NamespaceDefault,
- ResourceVersion: "18",
- Labels: fooBar(),
- },
- Spec: apps.StatefulSetSpec{
- Replicas: &size,
- Selector: newSelFooBar(),
- },
- }
- ssName, err := controller.KeyFunc(ss)
- if err != nil {
- t.Fatalf("Unexpected error naming StatefulSet %q: %v", ss.Name, err)
- }
- return ss, ssName
- }
- func update(t *testing.T, store cache.Store, obj interface{}) {
- if err := store.Update(obj); err != nil {
- t.Fatalf("Could not add %+v to %+v: %v", obj, store, err)
- }
- }
- func add(t *testing.T, store cache.Store, obj interface{}) {
- if err := store.Add(obj); err != nil {
- t.Fatalf("Could not add %+v to %+v: %v", obj, store, err)
- }
- }
- // Create one with no selector. Verify it matches 0 pods.
- func TestNoSelector(t *testing.T) {
- dc, ps := newFakeDisruptionController()
- pdb, pdbName := newMinAvailablePodDisruptionBudget(t, intstr.FromInt(3))
- pdb.Spec.Selector = &metav1.LabelSelector{}
- pod, _ := newPod(t, "yo-yo-yo")
- add(t, dc.pdbStore, pdb)
- dc.sync(pdbName)
- ps.VerifyPdbStatus(t, pdbName, 0, 0, 3, 0, map[string]metav1.Time{})
- add(t, dc.podStore, pod)
- dc.sync(pdbName)
- ps.VerifyPdbStatus(t, pdbName, 0, 0, 3, 0, map[string]metav1.Time{})
- }
- // Verify that available/expected counts go up as we add pods, then verify that
- // available count goes down when we make a pod unavailable.
- func TestUnavailable(t *testing.T) {
- dc, ps := newFakeDisruptionController()
- pdb, pdbName := newMinAvailablePodDisruptionBudget(t, intstr.FromInt(3))
- add(t, dc.pdbStore, pdb)
- dc.sync(pdbName)
- // Add three pods, verifying that the counts go up at each step.
- pods := []*v1.Pod{}
- for i := int32(0); i < 4; i++ {
- ps.VerifyPdbStatus(t, pdbName, 0, i, 3, i, map[string]metav1.Time{})
- pod, _ := newPod(t, fmt.Sprintf("yo-yo-yo %d", i))
- pods = append(pods, pod)
- add(t, dc.podStore, pod)
- dc.sync(pdbName)
- }
- ps.VerifyPdbStatus(t, pdbName, 1, 4, 3, 4, map[string]metav1.Time{})
- // Now set one pod as unavailable
- pods[0].Status.Conditions = []v1.PodCondition{}
- update(t, dc.podStore, pods[0])
- dc.sync(pdbName)
- // Verify expected update
- ps.VerifyPdbStatus(t, pdbName, 0, 3, 3, 4, map[string]metav1.Time{})
- }
- // Verify that an integer MaxUnavailable won't
- // allow a disruption for pods with no controller.
- func TestIntegerMaxUnavailable(t *testing.T) {
- dc, ps := newFakeDisruptionController()
- pdb, pdbName := newMaxUnavailablePodDisruptionBudget(t, intstr.FromInt(1))
- add(t, dc.pdbStore, pdb)
- dc.sync(pdbName)
- // This verifies that when a PDB has 0 pods, disruptions are not allowed.
- ps.VerifyDisruptionAllowed(t, pdbName, 0)
- pod, _ := newPod(t, "naked")
- add(t, dc.podStore, pod)
- dc.sync(pdbName)
- ps.VerifyDisruptionAllowed(t, pdbName, 0)
- }
- // Verify that an integer MaxUnavailable will recompute allowed disruptions when the scale of
- // the selected pod's controller is modified.
- func TestIntegerMaxUnavailableWithScaling(t *testing.T) {
- dc, ps := newFakeDisruptionController()
- pdb, pdbName := newMaxUnavailablePodDisruptionBudget(t, intstr.FromInt(2))
- add(t, dc.pdbStore, pdb)
- rs, _ := newReplicaSet(t, 7)
- add(t, dc.rsStore, rs)
- pod, _ := newPod(t, "pod")
- updatePodOwnerToRs(t, pod, rs)
- add(t, dc.podStore, pod)
- dc.sync(pdbName)
- ps.VerifyPdbStatus(t, pdbName, 0, 1, 5, 7, map[string]metav1.Time{})
- // Update scale of ReplicaSet and check PDB
- rs.Spec.Replicas = to.Int32Ptr(5)
- update(t, dc.rsStore, rs)
- dc.sync(pdbName)
- ps.VerifyPdbStatus(t, pdbName, 0, 1, 3, 5, map[string]metav1.Time{})
- }
- // Verify that an percentage MaxUnavailable will recompute allowed disruptions when the scale of
- // the selected pod's controller is modified.
- func TestPercentageMaxUnavailableWithScaling(t *testing.T) {
- dc, ps := newFakeDisruptionController()
- pdb, pdbName := newMaxUnavailablePodDisruptionBudget(t, intstr.FromString("30%"))
- add(t, dc.pdbStore, pdb)
- rs, _ := newReplicaSet(t, 7)
- add(t, dc.rsStore, rs)
- pod, _ := newPod(t, "pod")
- updatePodOwnerToRs(t, pod, rs)
- add(t, dc.podStore, pod)
- dc.sync(pdbName)
- ps.VerifyPdbStatus(t, pdbName, 0, 1, 4, 7, map[string]metav1.Time{})
- // Update scale of ReplicaSet and check PDB
- rs.Spec.Replicas = to.Int32Ptr(3)
- update(t, dc.rsStore, rs)
- dc.sync(pdbName)
- ps.VerifyPdbStatus(t, pdbName, 0, 1, 2, 3, map[string]metav1.Time{})
- }
- // Create a pod with no controller, and verify that a PDB with a percentage
- // specified won't allow a disruption.
- func TestNakedPod(t *testing.T) {
- dc, ps := newFakeDisruptionController()
- pdb, pdbName := newMinAvailablePodDisruptionBudget(t, intstr.FromString("28%"))
- add(t, dc.pdbStore, pdb)
- dc.sync(pdbName)
- // This verifies that when a PDB has 0 pods, disruptions are not allowed.
- ps.VerifyDisruptionAllowed(t, pdbName, 0)
- pod, _ := newPod(t, "naked")
- add(t, dc.podStore, pod)
- dc.sync(pdbName)
- ps.VerifyDisruptionAllowed(t, pdbName, 0)
- }
- // Verify that we count the scale of a ReplicaSet even when it has no Deployment.
- func TestReplicaSet(t *testing.T) {
- dc, ps := newFakeDisruptionController()
- pdb, pdbName := newMinAvailablePodDisruptionBudget(t, intstr.FromString("20%"))
- add(t, dc.pdbStore, pdb)
- rs, _ := newReplicaSet(t, 10)
- add(t, dc.rsStore, rs)
- pod, _ := newPod(t, "pod")
- updatePodOwnerToRs(t, pod, rs)
- add(t, dc.podStore, pod)
- dc.sync(pdbName)
- ps.VerifyPdbStatus(t, pdbName, 0, 1, 2, 10, map[string]metav1.Time{})
- }
- func TestScaleResource(t *testing.T) {
- customResourceUID := uuid.NewUUID()
- replicas := int32(10)
- pods := int32(4)
- maxUnavailable := int32(5)
- dc, ps := newFakeDisruptionController()
- dc.scaleClient.AddReactor("get", "customresources", func(action core.Action) (handled bool, ret runtime.Object, err error) {
- obj := &autoscalingapi.Scale{
- ObjectMeta: metav1.ObjectMeta{
- Namespace: metav1.NamespaceDefault,
- UID: customResourceUID,
- },
- Spec: autoscalingapi.ScaleSpec{
- Replicas: replicas,
- },
- }
- return true, obj, nil
- })
- pdb, pdbName := newMaxUnavailablePodDisruptionBudget(t, intstr.FromInt(int(maxUnavailable)))
- add(t, dc.pdbStore, pdb)
- trueVal := true
- for i := 0; i < int(pods); i++ {
- pod, _ := newPod(t, fmt.Sprintf("pod-%d", i))
- pod.SetOwnerReferences([]metav1.OwnerReference{
- {
- Kind: customGVK.Kind,
- APIVersion: customGVK.GroupVersion().String(),
- Controller: &trueVal,
- UID: customResourceUID,
- },
- })
- add(t, dc.podStore, pod)
- }
- dc.sync(pdbName)
- disruptionsAllowed := int32(0)
- if replicas-pods < maxUnavailable {
- disruptionsAllowed = maxUnavailable - (replicas - pods)
- }
- ps.VerifyPdbStatus(t, pdbName, disruptionsAllowed, pods, replicas-maxUnavailable, replicas, map[string]metav1.Time{})
- }
- // Verify that multiple controllers doesn't allow the PDB to be set true.
- func TestMultipleControllers(t *testing.T) {
- const podCount = 2
- dc, ps := newFakeDisruptionController()
- pdb, pdbName := newMinAvailablePodDisruptionBudget(t, intstr.FromString("1%"))
- add(t, dc.pdbStore, pdb)
- pods := []*v1.Pod{}
- for i := 0; i < podCount; i++ {
- pod, _ := newPod(t, fmt.Sprintf("pod %d", i))
- pods = append(pods, pod)
- add(t, dc.podStore, pod)
- }
- dc.sync(pdbName)
- // No controllers yet => no disruption allowed
- ps.VerifyDisruptionAllowed(t, pdbName, 0)
- rc, _ := newReplicationController(t, 1)
- rc.Name = "rc 1"
- for i := 0; i < podCount; i++ {
- updatePodOwnerToRc(t, pods[i], rc)
- }
- add(t, dc.rcStore, rc)
- dc.sync(pdbName)
- // One RC and 200%>1% healthy => disruption allowed
- ps.VerifyDisruptionAllowed(t, pdbName, 1)
- rc, _ = newReplicationController(t, 1)
- rc.Name = "rc 2"
- for i := 0; i < podCount; i++ {
- updatePodOwnerToRc(t, pods[i], rc)
- }
- add(t, dc.rcStore, rc)
- dc.sync(pdbName)
- // 100%>1% healthy BUT two RCs => no disruption allowed
- // TODO: Find out if this assert is still needed
- //ps.VerifyDisruptionAllowed(t, pdbName, 0)
- }
- func TestReplicationController(t *testing.T) {
- // The budget in this test matches foo=bar, but the RC and its pods match
- // {foo=bar, baz=quux}. Later, when we add a rogue pod with only a foo=bar
- // label, it will match the budget but have no controllers, which should
- // trigger the controller to set PodDisruptionAllowed to false.
- labels := map[string]string{
- "foo": "bar",
- "baz": "quux",
- }
- dc, ps := newFakeDisruptionController()
- // 34% should round up to 2
- pdb, pdbName := newMinAvailablePodDisruptionBudget(t, intstr.FromString("34%"))
- add(t, dc.pdbStore, pdb)
- rc, _ := newReplicationController(t, 3)
- rc.Spec.Selector = labels
- add(t, dc.rcStore, rc)
- dc.sync(pdbName)
- // It starts out at 0 expected because, with no pods, the PDB doesn't know
- // about the RC. This is a known bug. TODO(mml): file issue
- ps.VerifyPdbStatus(t, pdbName, 0, 0, 0, 0, map[string]metav1.Time{})
- pods := []*v1.Pod{}
- for i := int32(0); i < 3; i++ {
- pod, _ := newPod(t, fmt.Sprintf("foobar %d", i))
- updatePodOwnerToRc(t, pod, rc)
- pods = append(pods, pod)
- pod.Labels = labels
- add(t, dc.podStore, pod)
- dc.sync(pdbName)
- if i < 2 {
- ps.VerifyPdbStatus(t, pdbName, 0, i+1, 2, 3, map[string]metav1.Time{})
- } else {
- ps.VerifyPdbStatus(t, pdbName, 1, 3, 2, 3, map[string]metav1.Time{})
- }
- }
- rogue, _ := newPod(t, "rogue")
- add(t, dc.podStore, rogue)
- dc.sync(pdbName)
- ps.VerifyDisruptionAllowed(t, pdbName, 0)
- }
- func TestStatefulSetController(t *testing.T) {
- labels := map[string]string{
- "foo": "bar",
- "baz": "quux",
- }
- dc, ps := newFakeDisruptionController()
- // 34% should round up to 2
- pdb, pdbName := newMinAvailablePodDisruptionBudget(t, intstr.FromString("34%"))
- add(t, dc.pdbStore, pdb)
- ss, _ := newStatefulSet(t, 3)
- add(t, dc.ssStore, ss)
- dc.sync(pdbName)
- // It starts out at 0 expected because, with no pods, the PDB doesn't know
- // about the SS. This is a known bug. TODO(mml): file issue
- ps.VerifyPdbStatus(t, pdbName, 0, 0, 0, 0, map[string]metav1.Time{})
- pods := []*v1.Pod{}
- for i := int32(0); i < 3; i++ {
- pod, _ := newPod(t, fmt.Sprintf("foobar %d", i))
- updatePodOwnerToSs(t, pod, ss)
- pods = append(pods, pod)
- pod.Labels = labels
- add(t, dc.podStore, pod)
- dc.sync(pdbName)
- if i < 2 {
- ps.VerifyPdbStatus(t, pdbName, 0, i+1, 2, 3, map[string]metav1.Time{})
- } else {
- ps.VerifyPdbStatus(t, pdbName, 1, 3, 2, 3, map[string]metav1.Time{})
- }
- }
- }
- func TestTwoControllers(t *testing.T) {
- // Most of this test is in verifying intermediate cases as we define the
- // three controllers and create the pods.
- rcLabels := map[string]string{
- "foo": "bar",
- "baz": "quux",
- }
- dLabels := map[string]string{
- "foo": "bar",
- "baz": "quuux",
- }
- dc, ps := newFakeDisruptionController()
- // These constants are related, but I avoid calculating the correct values in
- // code. If you update a parameter here, recalculate the correct values for
- // all of them. Further down in the test, we use these to control loops, and
- // that level of logic is enough complexity for me.
- const collectionSize int32 = 11 // How big each collection is
- const minimumOne int32 = 4 // integer minimum with one controller
- const minimumTwo int32 = 7 // integer minimum with two controllers
- pdb, pdbName := newMinAvailablePodDisruptionBudget(t, intstr.FromString("28%"))
- add(t, dc.pdbStore, pdb)
- rc, _ := newReplicationController(t, collectionSize)
- rc.Spec.Selector = rcLabels
- add(t, dc.rcStore, rc)
- dc.sync(pdbName)
- ps.VerifyPdbStatus(t, pdbName, 0, 0, 0, 0, map[string]metav1.Time{})
- pods := []*v1.Pod{}
- unavailablePods := collectionSize - minimumOne - 1
- for i := int32(1); i <= collectionSize; i++ {
- pod, _ := newPod(t, fmt.Sprintf("quux %d", i))
- updatePodOwnerToRc(t, pod, rc)
- pods = append(pods, pod)
- pod.Labels = rcLabels
- if i <= unavailablePods {
- pod.Status.Conditions = []v1.PodCondition{}
- }
- add(t, dc.podStore, pod)
- dc.sync(pdbName)
- if i <= unavailablePods {
- ps.VerifyPdbStatus(t, pdbName, 0, 0, minimumOne, collectionSize, map[string]metav1.Time{})
- } else if i-unavailablePods <= minimumOne {
- ps.VerifyPdbStatus(t, pdbName, 0, i-unavailablePods, minimumOne, collectionSize, map[string]metav1.Time{})
- } else {
- ps.VerifyPdbStatus(t, pdbName, 1, i-unavailablePods, minimumOne, collectionSize, map[string]metav1.Time{})
- }
- }
- d, _ := newDeployment(t, collectionSize)
- d.Spec.Selector = newSel(dLabels)
- add(t, dc.dStore, d)
- dc.sync(pdbName)
- ps.VerifyPdbStatus(t, pdbName, 1, minimumOne+1, minimumOne, collectionSize, map[string]metav1.Time{})
- rs, _ := newReplicaSet(t, collectionSize)
- rs.Spec.Selector = newSel(dLabels)
- rs.Labels = dLabels
- add(t, dc.rsStore, rs)
- dc.sync(pdbName)
- ps.VerifyPdbStatus(t, pdbName, 1, minimumOne+1, minimumOne, collectionSize, map[string]metav1.Time{})
- // By the end of this loop, the number of ready pods should be N+2 (hence minimumTwo+2).
- unavailablePods = 2*collectionSize - (minimumTwo + 2) - unavailablePods
- for i := int32(1); i <= collectionSize; i++ {
- pod, _ := newPod(t, fmt.Sprintf("quuux %d", i))
- updatePodOwnerToRs(t, pod, rs)
- pods = append(pods, pod)
- pod.Labels = dLabels
- if i <= unavailablePods {
- pod.Status.Conditions = []v1.PodCondition{}
- }
- add(t, dc.podStore, pod)
- dc.sync(pdbName)
- if i <= unavailablePods {
- ps.VerifyPdbStatus(t, pdbName, 0, minimumOne+1, minimumTwo, 2*collectionSize, map[string]metav1.Time{})
- } else if i-unavailablePods <= minimumTwo-(minimumOne+1) {
- ps.VerifyPdbStatus(t, pdbName, 0, (minimumOne+1)+(i-unavailablePods), minimumTwo, 2*collectionSize, map[string]metav1.Time{})
- } else {
- ps.VerifyPdbStatus(t, pdbName, i-unavailablePods-(minimumTwo-(minimumOne+1)),
- (minimumOne+1)+(i-unavailablePods), minimumTwo, 2*collectionSize, map[string]metav1.Time{})
- }
- }
- // Now we verify we can bring down 1 pod and a disruption is still permitted,
- // but if we bring down two, it's not. Then we make the pod ready again and
- // verify that a disruption is permitted again.
- ps.VerifyPdbStatus(t, pdbName, 2, 2+minimumTwo, minimumTwo, 2*collectionSize, map[string]metav1.Time{})
- pods[collectionSize-1].Status.Conditions = []v1.PodCondition{}
- update(t, dc.podStore, pods[collectionSize-1])
- dc.sync(pdbName)
- ps.VerifyPdbStatus(t, pdbName, 1, 1+minimumTwo, minimumTwo, 2*collectionSize, map[string]metav1.Time{})
- pods[collectionSize-2].Status.Conditions = []v1.PodCondition{}
- update(t, dc.podStore, pods[collectionSize-2])
- dc.sync(pdbName)
- ps.VerifyPdbStatus(t, pdbName, 0, minimumTwo, minimumTwo, 2*collectionSize, map[string]metav1.Time{})
- pods[collectionSize-1].Status.Conditions = []v1.PodCondition{{Type: v1.PodReady, Status: v1.ConditionTrue}}
- update(t, dc.podStore, pods[collectionSize-1])
- dc.sync(pdbName)
- ps.VerifyPdbStatus(t, pdbName, 1, 1+minimumTwo, minimumTwo, 2*collectionSize, map[string]metav1.Time{})
- }
- // Test pdb doesn't exist
- func TestPDBNotExist(t *testing.T) {
- dc, _ := newFakeDisruptionController()
- pdb, _ := newMinAvailablePodDisruptionBudget(t, intstr.FromString("67%"))
- add(t, dc.pdbStore, pdb)
- if err := dc.sync("notExist"); err != nil {
- t.Errorf("Unexpected error: %v, expect nil", err)
- }
- }
- func TestUpdateDisruptedPods(t *testing.T) {
- dc, ps := newFakeDisruptionController()
- dc.recheckQueue = workqueue.NewNamedDelayingQueue("pdb_queue")
- pdb, pdbName := newMinAvailablePodDisruptionBudget(t, intstr.FromInt(1))
- currentTime := time.Now()
- pdb.Status.DisruptedPods = map[string]metav1.Time{
- "p1": {Time: currentTime}, // Should be removed, pod deletion started.
- "p2": {Time: currentTime.Add(-5 * time.Minute)}, // Should be removed, expired.
- "p3": {Time: currentTime}, // Should remain, pod untouched.
- "notthere": {Time: currentTime}, // Should be removed, pod deleted.
- }
- add(t, dc.pdbStore, pdb)
- pod1, _ := newPod(t, "p1")
- pod1.DeletionTimestamp = &metav1.Time{Time: time.Now()}
- pod2, _ := newPod(t, "p2")
- pod3, _ := newPod(t, "p3")
- add(t, dc.podStore, pod1)
- add(t, dc.podStore, pod2)
- add(t, dc.podStore, pod3)
- dc.sync(pdbName)
- ps.VerifyPdbStatus(t, pdbName, 0, 1, 1, 3, map[string]metav1.Time{"p3": {Time: currentTime}})
- }
- func TestBasicFinderFunctions(t *testing.T) {
- dc, _ := newFakeDisruptionController()
- rs, _ := newReplicaSet(t, 10)
- add(t, dc.rsStore, rs)
- rc, _ := newReplicationController(t, 12)
- add(t, dc.rcStore, rc)
- ss, _ := newStatefulSet(t, 14)
- add(t, dc.ssStore, ss)
- testCases := map[string]struct {
- finderFunc podControllerFinder
- apiVersion string
- kind string
- name string
- uid types.UID
- findsScale bool
- expectedScale int32
- }{
- "replicaset controller with apps group": {
- finderFunc: dc.getPodReplicaSet,
- apiVersion: "apps/v1",
- kind: controllerKindRS.Kind,
- name: rs.Name,
- uid: rs.UID,
- findsScale: true,
- expectedScale: 10,
- },
- "replicaset controller with invalid group": {
- finderFunc: dc.getPodReplicaSet,
- apiVersion: "invalid/v1",
- kind: controllerKindRS.Kind,
- name: rs.Name,
- uid: rs.UID,
- findsScale: false,
- },
- "replicationcontroller with empty group": {
- finderFunc: dc.getPodReplicationController,
- apiVersion: "/v1",
- kind: controllerKindRC.Kind,
- name: rc.Name,
- uid: rc.UID,
- findsScale: true,
- expectedScale: 12,
- },
- "replicationcontroller with invalid group": {
- finderFunc: dc.getPodReplicationController,
- apiVersion: "apps/v1",
- kind: controllerKindRC.Kind,
- name: rc.Name,
- uid: rc.UID,
- findsScale: false,
- },
- "statefulset controller with extensions group": {
- finderFunc: dc.getPodStatefulSet,
- apiVersion: "apps/v1",
- kind: controllerKindSS.Kind,
- name: ss.Name,
- uid: ss.UID,
- findsScale: true,
- expectedScale: 14,
- },
- "statefulset controller with invalid kind": {
- finderFunc: dc.getPodStatefulSet,
- apiVersion: "apps/v1",
- kind: controllerKindRS.Kind,
- name: ss.Name,
- uid: ss.UID,
- findsScale: false,
- },
- }
- for tn, tc := range testCases {
- t.Run(tn, func(t *testing.T) {
- controllerRef := &metav1.OwnerReference{
- APIVersion: tc.apiVersion,
- Kind: tc.kind,
- Name: tc.name,
- UID: tc.uid,
- }
- controllerAndScale, _ := tc.finderFunc(controllerRef, metav1.NamespaceDefault)
- if controllerAndScale == nil {
- if tc.findsScale {
- t.Error("Expected scale, but got nil")
- }
- return
- }
- if got, want := controllerAndScale.scale, tc.expectedScale; got != want {
- t.Errorf("Expected scale %d, but got %d", want, got)
- }
- if got, want := controllerAndScale.UID, tc.uid; got != want {
- t.Errorf("Expected uid %s, but got %s", want, got)
- }
- })
- }
- }
- func TestDeploymentFinderFunction(t *testing.T) {
- labels := map[string]string{
- "foo": "bar",
- }
- testCases := map[string]struct {
- rsApiVersion string
- rsKind string
- depApiVersion string
- depKind string
- findsScale bool
- expectedScale int32
- }{
- "happy path": {
- rsApiVersion: "apps/v1",
- rsKind: controllerKindRS.Kind,
- depApiVersion: "extensions/v1",
- depKind: controllerKindDep.Kind,
- findsScale: true,
- expectedScale: 10,
- },
- "invalid rs apiVersion": {
- rsApiVersion: "invalid/v1",
- rsKind: controllerKindRS.Kind,
- depApiVersion: "apps/v1",
- depKind: controllerKindDep.Kind,
- findsScale: false,
- },
- "invalid rs kind": {
- rsApiVersion: "apps/v1",
- rsKind: "InvalidKind",
- depApiVersion: "apps/v1",
- depKind: controllerKindDep.Kind,
- findsScale: false,
- },
- "invalid deployment apiVersion": {
- rsApiVersion: "extensions/v1",
- rsKind: controllerKindRS.Kind,
- depApiVersion: "deployment/v1",
- depKind: controllerKindDep.Kind,
- findsScale: false,
- },
- "invalid deployment kind": {
- rsApiVersion: "apps/v1",
- rsKind: controllerKindRS.Kind,
- depApiVersion: "extensions/v1",
- depKind: "InvalidKind",
- findsScale: false,
- },
- }
- for tn, tc := range testCases {
- t.Run(tn, func(t *testing.T) {
- dc, _ := newFakeDisruptionController()
- dep, _ := newDeployment(t, 10)
- dep.Spec.Selector = newSel(labels)
- add(t, dc.dStore, dep)
- rs, _ := newReplicaSet(t, 5)
- rs.Labels = labels
- trueVal := true
- rs.OwnerReferences = append(rs.OwnerReferences, metav1.OwnerReference{
- APIVersion: tc.depApiVersion,
- Kind: tc.depKind,
- Name: dep.Name,
- UID: dep.UID,
- Controller: &trueVal,
- })
- add(t, dc.rsStore, rs)
- controllerRef := &metav1.OwnerReference{
- APIVersion: tc.rsApiVersion,
- Kind: tc.rsKind,
- Name: rs.Name,
- UID: rs.UID,
- }
- controllerAndScale, _ := dc.getPodDeployment(controllerRef, metav1.NamespaceDefault)
- if controllerAndScale == nil {
- if tc.findsScale {
- t.Error("Expected scale, but got nil")
- }
- return
- }
- if got, want := controllerAndScale.scale, tc.expectedScale; got != want {
- t.Errorf("Expected scale %d, but got %d", want, got)
- }
- if got, want := controllerAndScale.UID, dep.UID; got != want {
- t.Errorf("Expected uid %s, but got %s", want, got)
- }
- })
- }
- }
|