1234567891011121314151617181920212223242526272829303132333435363738394041424344454647484950515253545556575859606162636465666768697071727374757677787980818283848586878889909192939495969798991001011021031041051061071081091101111121131141151161171181191201211221231241251261271281291301311321331341351361371381391401411421431441451461471481491501511521531541551561571581591601611621631641651661671681691701711721731741751761771781791801811821831841851861871881891901911921931941951961971981992002012022032042052062072082092102112122132142152162172182192202212222232242252262272282292302312322332342352362372382392402412422432442452462472482492502512522532542552562572582592602612622632642652662672682692702712722732742752762772782792802812822832842852862872882892902912922932942952962972982993003013023033043053063073083093103113123133143153163173183193203213223233243253263273283293303313323333343353363373383393403413423433443453463473483493503513523533543553563573583593603613623633643653663673683693703713723733743753763773783793803813823833843853863873883893903913923933943953963973983994004014024034044054064074084094104114124134144154164174184194204214224234244254264274284294304314324334344354364374384394404414424434444454464474484494504514524534544554564574584594604614624634644654664674684694704714724734744754764774784794804814824834844854864874884894904914924934944954964974984995005015025035045055065075085095105115125135145155165175185195205215225235245255265275285295305315325335345355365375385395405415425435445455465475485495505515525535545555565575585595605615625635645655665675685695705715725735745755765775785795805815825835845855865875885895905915925935945955965975985996006016026036046056066076086096106116126136146156166176186196206216226236246256266276286296306316326336346356366376386396406416426436446456466476486496506516526536546556566576586596606616626636646656666676686696706716726736746756766776786796806816826836846856866876886896906916926936946956966976986997007017027037047057067077087097107117127137147157167177187197207217227237247257267277287297307317327337347357367377387397407417427437447457467477487497507517527537547557567577587597607617627637647657667677687697707717727737747757767777787797807817827837847857867877887897907917927937947957967977987998008018028038048058068078088098108118128138148158168178188198208218228238248258268278288298308318328338348358368378388398408418428438448458468478488498508518528538548558568578588598608618628638648658668678688698708718728738748758768778788798808818828838848858868878888898908918928938948958968978988999009019029039049059069079089099109119129139149159169179189199209219229239249259269279289299309319329339349359369379389399409419429439449459469479489499509519529539549559569579589599609619629639649659669679689699709719729739749759769779789799809819829839849859869879889899909919929939949959969979989991000100110021003100410051006100710081009101010111012101310141015101610171018101910201021102210231024102510261027102810291030103110321033103410351036103710381039104010411042104310441045104610471048104910501051105210531054105510561057105810591060106110621063106410651066106710681069107010711072107310741075107610771078107910801081108210831084108510861087108810891090109110921093109410951096109710981099110011011102110311041105110611071108110911101111111211131114111511161117111811191120112111221123112411251126112711281129113011311132113311341135113611371138113911401141114211431144114511461147114811491150115111521153115411551156115711581159116011611162116311641165116611671168116911701171117211731174117511761177117811791180118111821183118411851186118711881189119011911192119311941195119611971198119912001201120212031204120512061207120812091210121112121213121412151216121712181219122012211222122312241225122612271228122912301231123212331234123512361237123812391240124112421243124412451246124712481249125012511252125312541255125612571258125912601261126212631264126512661267126812691270127112721273127412751276127712781279128012811282128312841285128612871288128912901291129212931294129512961297129812991300130113021303130413051306130713081309131013111312131313141315131613171318131913201321132213231324132513261327132813291330133113321333133413351336133713381339134013411342134313441345134613471348134913501351135213531354135513561357135813591360136113621363136413651366136713681369137013711372137313741375137613771378137913801381138213831384138513861387138813891390139113921393139413951396139713981399140014011402140314041405140614071408140914101411141214131414141514161417141814191420142114221423142414251426142714281429143014311432143314341435143614371438143914401441144214431444144514461447144814491450145114521453145414551456145714581459146014611462146314641465146614671468146914701471147214731474147514761477147814791480148114821483148414851486148714881489149014911492149314941495149614971498149915001501150215031504150515061507150815091510151115121513151415151516151715181519152015211522152315241525152615271528152915301531153215331534153515361537153815391540154115421543154415451546154715481549155015511552155315541555155615571558155915601561156215631564156515661567156815691570157115721573157415751576157715781579158015811582158315841585158615871588158915901591159215931594159515961597159815991600160116021603160416051606160716081609161016111612161316141615161616171618161916201621162216231624162516261627162816291630163116321633163416351636163716381639164016411642164316441645164616471648164916501651165216531654165516561657165816591660166116621663166416651666166716681669167016711672167316741675167616771678167916801681168216831684168516861687168816891690169116921693169416951696169716981699170017011702170317041705 |
- /*
- 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 history
- import (
- "bytes"
- "context"
- "encoding/json"
- "fmt"
- "reflect"
- "testing"
- apps "k8s.io/api/apps/v1"
- "k8s.io/api/core/v1"
- "k8s.io/client-go/informers"
- "k8s.io/client-go/kubernetes/fake"
- clientscheme "k8s.io/client-go/kubernetes/scheme"
- "k8s.io/kubernetes/pkg/api/legacyscheme"
- "k8s.io/kubernetes/pkg/controller"
- "k8s.io/apimachinery/pkg/api/errors"
- "k8s.io/apimachinery/pkg/api/resource"
- metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
- "k8s.io/apimachinery/pkg/labels"
- "k8s.io/apimachinery/pkg/runtime"
- "k8s.io/apimachinery/pkg/types"
- "k8s.io/apimachinery/pkg/util/strategicpatch"
- core "k8s.io/client-go/testing"
- "time"
- )
- func TestRealHistory_ListControllerRevisions(t *testing.T) {
- type testcase struct {
- name string
- parent metav1.Object
- selector labels.Selector
- revisions []*apps.ControllerRevision
- want map[string]bool
- }
- testFn := func(test *testcase, t *testing.T) {
- client := fake.NewSimpleClientset()
- informerFactory := informers.NewSharedInformerFactory(client, controller.NoResyncPeriodFunc())
- stop := make(chan struct{})
- defer close(stop)
- informerFactory.Start(stop)
- informer := informerFactory.Apps().V1().ControllerRevisions()
- informerFactory.WaitForCacheSync(stop)
- for i := range test.revisions {
- informer.Informer().GetIndexer().Add(test.revisions[i])
- }
- history := NewHistory(client, informer.Lister())
- revisions, err := history.ListControllerRevisions(test.parent, test.selector)
- if err != nil {
- t.Errorf("%s: %s", test.name, err)
- }
- got := make(map[string]bool)
- for i := range revisions {
- got[revisions[i].Name] = true
- }
- if !reflect.DeepEqual(test.want, got) {
- t.Errorf("%s: want %v got %v", test.name, test.want, got)
- }
- }
- ss1 := newStatefulSet(3, "ss1", types.UID("ss1"), map[string]string{"foo": "bar"})
- ss2 := newStatefulSet(3, "ss2", types.UID("ss2"), map[string]string{"goo": "car"})
- sel1, err := metav1.LabelSelectorAsSelector(ss1.Spec.Selector)
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev1, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(&ss1.Spec.Template), 1, nil)
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev1.Namespace = ss1.Namespace
- ss1Rev2, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(&ss1.Spec.Template), 2, nil)
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev2.Namespace = ss1.Namespace
- ss2Rev1, err := NewControllerRevision(ss2, parentKind, ss2.Spec.Template.Labels, rawTemplate(&ss2.Spec.Template), 1, nil)
- if err != nil {
- t.Fatal(err)
- }
- ss2Rev1.Namespace = ss2.Namespace
- ss1Orphan, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(&ss1.Spec.Template), 3, nil)
- if err != nil {
- t.Fatal(err)
- }
- ss1Orphan.Namespace = ss1.Namespace
- ss1Orphan.OwnerReferences = nil
- tests := []testcase{
- {
- name: "selects none",
- parent: &ss1.ObjectMeta,
- selector: sel1,
- revisions: nil,
- want: map[string]bool{},
- },
- {
- name: "selects all",
- parent: &ss1.ObjectMeta,
- selector: sel1,
- revisions: []*apps.ControllerRevision{ss1Rev1, ss1Rev2},
- want: map[string]bool{ss1Rev1.Name: true, ss1Rev2.Name: true},
- },
- {
- name: "doesn't select another Objects history",
- parent: &ss1.ObjectMeta,
- selector: sel1,
- revisions: []*apps.ControllerRevision{ss1Rev1, ss1Rev2, ss2Rev1},
- want: map[string]bool{ss1Rev1.Name: true, ss1Rev2.Name: true},
- },
- {
- name: "selects orphans",
- parent: &ss1.ObjectMeta,
- selector: sel1,
- revisions: []*apps.ControllerRevision{ss1Rev1, ss1Rev2, ss1Orphan},
- want: map[string]bool{ss1Rev1.Name: true, ss1Rev2.Name: true, ss1Orphan.Name: true},
- },
- }
- for i := range tests {
- testFn(&tests[i], t)
- }
- }
- func TestFakeHistory_ListControllerRevisions(t *testing.T) {
- type testcase struct {
- name string
- parent metav1.Object
- selector labels.Selector
- revisions []*apps.ControllerRevision
- want map[string]bool
- }
- testFn := func(test *testcase, t *testing.T) {
- client := fake.NewSimpleClientset()
- informerFactory := informers.NewSharedInformerFactory(client, controller.NoResyncPeriodFunc())
- stop := make(chan struct{})
- defer close(stop)
- informerFactory.Start(stop)
- informer := informerFactory.Apps().V1().ControllerRevisions()
- informerFactory.WaitForCacheSync(stop)
- for i := range test.revisions {
- informer.Informer().GetIndexer().Add(test.revisions[i])
- }
- history := NewFakeHistory(informer)
- revisions, err := history.ListControllerRevisions(test.parent, test.selector)
- if err != nil {
- t.Errorf("%s: %s", test.name, err)
- }
- got := make(map[string]bool)
- for i := range revisions {
- got[revisions[i].Name] = true
- }
- if !reflect.DeepEqual(test.want, got) {
- t.Errorf("%s: want %v got %v", test.name, test.want, got)
- }
- }
- ss1 := newStatefulSet(3, "ss1", types.UID("ss1"), map[string]string{"foo": "bar"})
- ss2 := newStatefulSet(3, "ss2", types.UID("ss2"), map[string]string{"goo": "car"})
- sel1, err := metav1.LabelSelectorAsSelector(ss1.Spec.Selector)
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev1, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(&ss1.Spec.Template), 1, nil)
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev1.Namespace = ss1.Namespace
- ss1Rev2, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(&ss1.Spec.Template), 2, nil)
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev2.Namespace = ss1.Namespace
- ss2Rev1, err := NewControllerRevision(ss2, parentKind, ss2.Spec.Template.Labels, rawTemplate(&ss2.Spec.Template), 1, nil)
- if err != nil {
- t.Fatal(err)
- }
- ss2Rev1.Namespace = ss2.Namespace
- ss1Orphan, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(&ss1.Spec.Template), 3, nil)
- if err != nil {
- t.Fatal(err)
- }
- ss1Orphan.Namespace = ss1.Namespace
- ss1Orphan.OwnerReferences = nil
- tests := []testcase{
- {
- name: "selects none",
- parent: &ss1.ObjectMeta,
- selector: sel1,
- revisions: nil,
- want: map[string]bool{},
- },
- {
- name: "selects all",
- parent: &ss1.ObjectMeta,
- selector: sel1,
- revisions: []*apps.ControllerRevision{ss1Rev1, ss1Rev2},
- want: map[string]bool{ss1Rev1.Name: true, ss1Rev2.Name: true},
- },
- {
- name: "doesn't select another Objects history",
- parent: &ss1.ObjectMeta,
- selector: sel1,
- revisions: []*apps.ControllerRevision{ss1Rev1, ss1Rev2, ss2Rev1},
- want: map[string]bool{ss1Rev1.Name: true, ss1Rev2.Name: true},
- },
- {
- name: "selects orphans",
- parent: &ss1.ObjectMeta,
- selector: sel1,
- revisions: []*apps.ControllerRevision{ss1Rev1, ss1Rev2, ss1Orphan},
- want: map[string]bool{ss1Rev1.Name: true, ss1Rev2.Name: true, ss1Orphan.Name: true},
- },
- }
- for i := range tests {
- testFn(&tests[i], t)
- }
- }
- func TestRealHistory_CreateControllerRevision(t *testing.T) {
- type testcase struct {
- name string
- parent metav1.Object
- revision *apps.ControllerRevision
- existing []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }
- rename bool
- }
- testFn := func(test *testcase, t *testing.T) {
- client := fake.NewSimpleClientset()
- informerFactory := informers.NewSharedInformerFactory(client, controller.NoResyncPeriodFunc())
- stop := make(chan struct{})
- defer close(stop)
- informerFactory.Start(stop)
- informer := informerFactory.Apps().V1().ControllerRevisions()
- informerFactory.WaitForCacheSync(stop)
- history := NewHistory(client, informer.Lister())
- var collisionCount int32
- for _, item := range test.existing {
- _, err := client.AppsV1().ControllerRevisions(item.parent.GetNamespace()).Create(context.TODO(), item.revision, metav1.CreateOptions{})
- if err != nil {
- t.Fatal(err)
- }
- }
- // Clear collisionCount before creating the test revision
- collisionCount = 0
- created, err := history.CreateControllerRevision(test.parent, test.revision, &collisionCount)
- if err != nil {
- t.Errorf("%s: %s", test.name, err)
- }
- if test.rename {
- if created.Name == test.revision.Name {
- t.Errorf("%s: wanted rename got %s %s", test.name, created.Name, test.revision.Name)
- }
- expectedName := ControllerRevisionName(test.parent.GetName(), HashControllerRevision(test.revision, &collisionCount))
- if created.Name != expectedName {
- t.Errorf("%s: on name collision wanted new name %s got %s", test.name, expectedName, created.Name)
- }
- // Second name collision will be caused by an identical revision, so no need to do anything
- _, err = history.CreateControllerRevision(test.parent, test.revision, &collisionCount)
- if err != nil {
- t.Errorf("%s: %s", test.name, err)
- }
- if collisionCount != 1 {
- t.Errorf("%s: on second name collision wanted collisionCount 1 got %d", test.name, collisionCount)
- }
- }
- if !test.rename && created.Name != test.revision.Name {
- t.Errorf("%s: wanted %s got %s", test.name, test.revision.Name, created.Name)
- }
- }
- ss1 := newStatefulSet(3, "ss1", types.UID("ss1"), map[string]string{"foo": "bar"})
- ss1.Status.CollisionCount = new(int32)
- ss2 := newStatefulSet(3, "ss2", types.UID("ss2"), map[string]string{"goo": "car"})
- ss2.Status.CollisionCount = new(int32)
- ss1Rev1, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(&ss1.Spec.Template), 1, ss1.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev1.Namespace = ss1.Namespace
- // Create a new revision with the same name and hash label as an existing revision, but with
- // a different template. This could happen as a result of a hash collision, but in this test
- // this situation is created by setting name and hash label to values known to be in use by
- // an existing revision.
- modTemplate := ss1.Spec.Template.DeepCopy()
- modTemplate.Labels["foo"] = "not_bar"
- ss1Rev2, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(modTemplate), 2, ss1.Status.CollisionCount)
- ss1Rev2.Name = ss1Rev1.Name
- ss1Rev2.Labels[ControllerRevisionHashLabel] = ss1Rev1.Labels[ControllerRevisionHashLabel]
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev2.Namespace = ss1.Namespace
- ss2Rev1, err := NewControllerRevision(ss2, parentKind, ss2.Spec.Template.Labels, rawTemplate(&ss2.Spec.Template), 1, ss2.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss2Rev1.Namespace = ss2.Namespace
- tests := []testcase{
- {
- name: "creates new",
- parent: &ss1.ObjectMeta,
- revision: ss1Rev1,
- existing: nil,
- rename: false,
- },
- {
- name: "create doesn't conflict when parents differ",
- parent: &ss2.ObjectMeta,
- revision: ss2Rev1,
- existing: []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }{
- {
- parent: ss1,
- revision: ss1Rev1,
- },
- },
- rename: false,
- },
- {
- name: "create renames on conflict",
- parent: &ss1.ObjectMeta,
- revision: ss1Rev1,
- existing: []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }{
- {
- parent: ss1,
- revision: ss1Rev2,
- },
- },
- rename: true,
- },
- }
- for i := range tests {
- testFn(&tests[i], t)
- }
- }
- func TestFakeHistory_CreateControllerRevision(t *testing.T) {
- type testcase struct {
- name string
- parent metav1.Object
- revision *apps.ControllerRevision
- existing []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }
- rename bool
- }
- testFn := func(test *testcase, t *testing.T) {
- client := fake.NewSimpleClientset()
- informerFactory := informers.NewSharedInformerFactory(client, controller.NoResyncPeriodFunc())
- stop := make(chan struct{})
- defer close(stop)
- informerFactory.Start(stop)
- informer := informerFactory.Apps().V1().ControllerRevisions()
- informerFactory.WaitForCacheSync(stop)
- history := NewFakeHistory(informer)
- var collisionCount int32
- for i := range test.existing {
- _, err := history.CreateControllerRevision(test.existing[i].parent, test.existing[i].revision, &collisionCount)
- if err != nil {
- t.Fatal(err)
- }
- }
- // Clear collisionCount before creating the test revision
- collisionCount = 0
- created, err := history.CreateControllerRevision(test.parent, test.revision, &collisionCount)
- if err != nil {
- t.Errorf("%s: %s", test.name, err)
- }
- if test.rename {
- if created.Name == test.revision.Name {
- t.Errorf("%s: wanted rename got %s %s", test.name, created.Name, test.revision.Name)
- }
- expectedName := ControllerRevisionName(test.parent.GetName(), HashControllerRevision(test.revision, &collisionCount))
- if created.Name != expectedName {
- t.Errorf("%s: on name collision wanted new name %s got %s", test.name, expectedName, created.Name)
- }
- // Second name collision should have incremented collisionCount to 2
- _, err = history.CreateControllerRevision(test.parent, test.revision, &collisionCount)
- if err != nil {
- t.Errorf("%s: %s", test.name, err)
- }
- if collisionCount != 2 {
- t.Errorf("%s: on second name collision wanted collisionCount 1 got %d", test.name, collisionCount)
- }
- }
- if !test.rename && created.Name != test.revision.Name {
- t.Errorf("%s: wanted %s got %s", test.name, test.revision.Name, created.Name)
- }
- }
- ss1 := newStatefulSet(3, "ss1", types.UID("ss1"), map[string]string{"foo": "bar"})
- ss1.Status.CollisionCount = new(int32)
- ss2 := newStatefulSet(3, "ss2", types.UID("ss2"), map[string]string{"goo": "car"})
- ss2.Status.CollisionCount = new(int32)
- ss1Rev1, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(&ss1.Spec.Template), 1, ss1.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev1.Namespace = ss1.Namespace
- ss1Rev2, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(&ss1.Spec.Template), 2, ss1.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev2.Namespace = ss1.Namespace
- ss2Rev1, err := NewControllerRevision(ss2, parentKind, ss2.Spec.Template.Labels, rawTemplate(&ss2.Spec.Template), 1, ss2.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss2Rev1.Namespace = ss2.Namespace
- tests := []testcase{
- {
- name: "creates new",
- parent: &ss1.ObjectMeta,
- revision: ss1Rev1,
- existing: nil,
- rename: false,
- },
- {
- name: "create doesn't conflict when parents differ",
- parent: &ss2.ObjectMeta,
- revision: ss2Rev1,
- existing: []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }{
- {
- parent: ss1,
- revision: ss1Rev1,
- },
- },
- rename: false,
- },
- {
- name: "create renames on conflict",
- parent: &ss1.ObjectMeta,
- revision: ss1Rev1,
- existing: []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }{
- {
- parent: ss1,
- revision: ss1Rev1,
- },
- },
- rename: true,
- },
- }
- for i := range tests {
- testFn(&tests[i], t)
- }
- }
- func TestRealHistory_UpdateControllerRevision(t *testing.T) {
- conflictAttempts := 0
- type testcase struct {
- name string
- revision *apps.ControllerRevision
- newRevision int64
- existing []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }
- reactor core.ReactionFunc
- err bool
- }
- conflictSuccess := func(action core.Action) (bool, runtime.Object, error) {
- defer func() {
- conflictAttempts++
- }()
- switch action.(type) {
- case core.UpdateActionImpl:
- update := action.(core.UpdateAction)
- if conflictAttempts < 2 {
- return true, update.GetObject(), errors.NewConflict(update.GetResource().GroupResource(), "", fmt.Errorf("conflict"))
- }
- return true, update.GetObject(), nil
- default:
- return false, nil, nil
- }
- }
- internalError := func(action core.Action) (bool, runtime.Object, error) {
- switch action.(type) {
- case core.UpdateActionImpl:
- return true, nil, errors.NewInternalError(fmt.Errorf("internal error"))
- default:
- return false, nil, nil
- }
- }
- testFn := func(test *testcase, t *testing.T) {
- client := fake.NewSimpleClientset()
- informerFactory := informers.NewSharedInformerFactory(client, controller.NoResyncPeriodFunc())
- stop := make(chan struct{})
- defer close(stop)
- informerFactory.Start(stop)
- informer := informerFactory.Apps().V1().ControllerRevisions()
- informerFactory.WaitForCacheSync(stop)
- history := NewHistory(client, informer.Lister())
- var collisionCount int32
- for i := range test.existing {
- _, err := history.CreateControllerRevision(test.existing[i].parent, test.existing[i].revision, &collisionCount)
- if err != nil {
- t.Fatal(err)
- }
- }
- if test.reactor != nil {
- client.PrependReactor("*", "*", test.reactor)
- }
- updated, err := history.UpdateControllerRevision(test.revision, test.newRevision)
- if !test.err && err != nil {
- t.Errorf("%s: %s", test.name, err)
- }
- if !test.err && updated.Revision != test.newRevision {
- t.Errorf("%s: got %d want %d", test.name, updated.Revision, test.newRevision)
- }
- if test.err && err == nil {
- t.Errorf("%s: expected error", test.name)
- }
- }
- ss1 := newStatefulSet(3, "ss1", types.UID("ss1"), map[string]string{"foo": "bar"})
- ss1.Status.CollisionCount = new(int32)
- ss1Rev1, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(&ss1.Spec.Template), 1, ss1.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev1.Namespace = ss1.Namespace
- ss1Rev2, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(&ss1.Spec.Template), 2, ss1.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev2.Namespace = ss1.Namespace
- tests := []testcase{
- {
- name: "update succeeds",
- revision: ss1Rev1,
- newRevision: ss1Rev1.Revision + 1,
- existing: []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }{
- {
- parent: ss1,
- revision: ss1Rev1,
- },
- },
- reactor: nil,
- err: false,
- },
- {
- name: "update succeeds no noop",
- revision: ss1Rev1,
- newRevision: ss1Rev1.Revision,
- existing: []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }{
- {
- parent: ss1,
- revision: ss1Rev1,
- },
- },
- reactor: nil,
- err: false,
- }, {
- name: "update fails on error",
- revision: ss1Rev1,
- newRevision: ss1Rev1.Revision + 10,
- existing: []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }{
- {
- parent: ss1,
- revision: ss1Rev1,
- },
- },
- reactor: internalError,
- err: true,
- },
- {
- name: "update on succeeds on conflict",
- revision: ss1Rev1,
- newRevision: ss1Rev1.Revision + 1,
- existing: []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }{
- {
- parent: ss1,
- revision: ss1Rev1,
- },
- },
- reactor: conflictSuccess,
- err: false,
- },
- }
- for i := range tests {
- conflictAttempts = 0
- testFn(&tests[i], t)
- }
- }
- func TestFakeHistory_UpdateControllerRevision(t *testing.T) {
- type testcase struct {
- name string
- revision *apps.ControllerRevision
- newRevision int64
- existing []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }
- err bool
- }
- testFn := func(test *testcase, t *testing.T) {
- client := fake.NewSimpleClientset()
- informerFactory := informers.NewSharedInformerFactory(client, controller.NoResyncPeriodFunc())
- stop := make(chan struct{})
- defer close(stop)
- informerFactory.Start(stop)
- informer := informerFactory.Apps().V1().ControllerRevisions()
- informerFactory.WaitForCacheSync(stop)
- history := NewFakeHistory(informer)
- var collisionCount int32
- for i := range test.existing {
- _, err := history.CreateControllerRevision(test.existing[i].parent, test.existing[i].revision, &collisionCount)
- if err != nil {
- t.Fatal(err)
- }
- }
- updated, err := history.UpdateControllerRevision(test.revision, test.newRevision)
- if !test.err && err != nil {
- t.Errorf("%s: %s", test.name, err)
- }
- if !test.err && updated.Revision != test.newRevision {
- t.Errorf("%s: got %d want %d", test.name, updated.Revision, test.newRevision)
- }
- if test.err && err == nil {
- t.Errorf("%s: expected error", test.name)
- }
- }
- ss1 := newStatefulSet(3, "ss1", types.UID("ss1"), map[string]string{"foo": "bar"})
- ss1.Status.CollisionCount = new(int32)
- ss1Rev1, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(&ss1.Spec.Template), 1, ss1.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev1.Namespace = ss1.Namespace
- ss1Rev2, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(&ss1.Spec.Template), 2, ss1.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev2.Namespace = ss1.Namespace
- tests := []testcase{
- {
- name: "update succeeds",
- revision: ss1Rev1,
- newRevision: ss1Rev1.Revision + 1,
- existing: []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }{
- {
- parent: ss1,
- revision: ss1Rev1,
- },
- },
- err: false,
- },
- {
- name: "update succeeds no noop",
- revision: ss1Rev1,
- newRevision: ss1Rev1.Revision,
- existing: []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }{
- {
- parent: ss1,
- revision: ss1Rev1,
- },
- },
- err: false,
- },
- }
- for i := range tests {
- testFn(&tests[i], t)
- }
- }
- func TestRealHistory_DeleteControllerRevision(t *testing.T) {
- type testcase struct {
- name string
- revision *apps.ControllerRevision
- existing []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }
- err bool
- }
- testFn := func(test *testcase, t *testing.T) {
- client := fake.NewSimpleClientset()
- informerFactory := informers.NewSharedInformerFactory(client, controller.NoResyncPeriodFunc())
- stop := make(chan struct{})
- defer close(stop)
- informerFactory.Start(stop)
- informer := informerFactory.Apps().V1().ControllerRevisions()
- informerFactory.WaitForCacheSync(stop)
- history := NewHistory(client, informer.Lister())
- var collisionCount int32
- for i := range test.existing {
- _, err := history.CreateControllerRevision(test.existing[i].parent, test.existing[i].revision, &collisionCount)
- if err != nil {
- t.Fatal(err)
- }
- }
- err := history.DeleteControllerRevision(test.revision)
- if !test.err && err != nil {
- t.Errorf("%s: %s", test.name, err)
- }
- if test.err && err == nil {
- t.Errorf("%s: expected error", test.name)
- }
- }
- ss1 := newStatefulSet(3, "ss1", types.UID("ss1"), map[string]string{"foo": "bar"})
- ss1.Status.CollisionCount = new(int32)
- ss2 := newStatefulSet(3, "ss2", types.UID("ss2"), map[string]string{"goo": "car"})
- ss2.Status.CollisionCount = new(int32)
- ss1Rev1, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(&ss1.Spec.Template), 1, ss1.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev1.Namespace = ss1.Namespace
- ss1Rev2, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(&ss1.Spec.Template), 2, ss1.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev2.Namespace = ss1.Namespace
- ss2Rev1, err := NewControllerRevision(ss2, parentKind, ss2.Spec.Template.Labels, rawTemplate(&ss2.Spec.Template), 1, ss2.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss2Rev1.Namespace = ss2.Namespace
- ss2Rev2, err := NewControllerRevision(ss2, parentKind, ss2.Spec.Template.Labels, rawTemplate(&ss2.Spec.Template), 2, ss2.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss2Rev2.Namespace = ss2.Namespace
- tests := []testcase{
- {
- name: "delete empty fails",
- revision: ss1Rev1,
- existing: nil,
- err: true,
- },
- {
- name: "delete existing succeeds",
- revision: ss1Rev1,
- existing: []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }{
- {
- parent: ss1,
- revision: ss1Rev1,
- },
- },
- err: false,
- }, {
- name: "delete non-existing fails",
- revision: ss1Rev1,
- existing: []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }{
- {
- parent: ss2,
- revision: ss2Rev1,
- },
- {
- parent: ss2,
- revision: ss2Rev2,
- },
- },
- err: true,
- },
- }
- for i := range tests {
- testFn(&tests[i], t)
- }
- }
- func TestFakeHistory_DeleteControllerRevision(t *testing.T) {
- type testcase struct {
- name string
- revision *apps.ControllerRevision
- existing []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }
- err bool
- }
- testFn := func(test *testcase, t *testing.T) {
- client := fake.NewSimpleClientset()
- informerFactory := informers.NewSharedInformerFactory(client, controller.NoResyncPeriodFunc())
- stop := make(chan struct{})
- defer close(stop)
- informerFactory.Start(stop)
- informer := informerFactory.Apps().V1().ControllerRevisions()
- informerFactory.WaitForCacheSync(stop)
- history := NewFakeHistory(informer)
- var collisionCount int32
- for i := range test.existing {
- _, err := history.CreateControllerRevision(test.existing[i].parent, test.existing[i].revision, &collisionCount)
- if err != nil {
- t.Fatal(err)
- }
- }
- err := history.DeleteControllerRevision(test.revision)
- if !test.err && err != nil {
- t.Errorf("%s: %s", test.name, err)
- }
- if test.err && err == nil {
- t.Errorf("%s: expected error", test.name)
- }
- }
- ss1 := newStatefulSet(3, "ss1", types.UID("ss1"), map[string]string{"foo": "bar"})
- ss1.Status.CollisionCount = new(int32)
- ss2 := newStatefulSet(3, "ss2", types.UID("ss2"), map[string]string{"goo": "car"})
- ss2.Status.CollisionCount = new(int32)
- ss1Rev1, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(&ss1.Spec.Template), 1, ss1.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev1.Namespace = ss1.Namespace
- ss1Rev2, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(&ss1.Spec.Template), 2, ss1.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev2.Namespace = ss1.Namespace
- ss2Rev1, err := NewControllerRevision(ss2, parentKind, ss2.Spec.Template.Labels, rawTemplate(&ss2.Spec.Template), 1, ss2.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss2Rev1.Namespace = ss2.Namespace
- ss2Rev2, err := NewControllerRevision(ss2, parentKind, ss2.Spec.Template.Labels, rawTemplate(&ss2.Spec.Template), 2, ss2.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss2Rev2.Namespace = ss2.Namespace
- tests := []testcase{
- {
- name: "delete empty fails",
- revision: ss1Rev1,
- existing: nil,
- err: true,
- },
- {
- name: "delete existing succeeds",
- revision: ss1Rev1,
- existing: []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }{
- {
- parent: ss1,
- revision: ss1Rev1,
- },
- },
- err: false,
- }, {
- name: "delete non-existing fails",
- revision: ss1Rev1,
- existing: []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }{
- {
- parent: ss2,
- revision: ss2Rev1,
- },
- {
- parent: ss2,
- revision: ss2Rev2,
- },
- },
- err: true,
- },
- }
- for i := range tests {
- testFn(&tests[i], t)
- }
- }
- func TestRealHistory_AdoptControllerRevision(t *testing.T) {
- type testcase struct {
- name string
- parent metav1.Object
- revision *apps.ControllerRevision
- existing []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }
- err bool
- }
- testFn := func(test *testcase, t *testing.T) {
- client := fake.NewSimpleClientset()
- client.AddReactor("*", "*", func(action core.Action) (bool, runtime.Object, error) {
- switch action := action.(type) {
- case core.PatchActionImpl:
- var found *apps.ControllerRevision
- for i := range test.existing {
- if test.revision.Name == test.existing[i].revision.Name &&
- test.revision.Namespace == test.existing[i].revision.Namespace {
- found = test.existing[i].revision
- break
- }
- }
- if found == nil {
- return true, nil, errors.NewNotFound(apps.Resource("controllerrevisions"), test.revision.Name)
- }
- b, err := strategicpatch.StrategicMergePatch(
- []byte(runtime.EncodeOrDie(clientscheme.Codecs.LegacyCodec(apps.SchemeGroupVersion), test.revision)),
- action.GetPatch(), test.revision)
- if err != nil {
- return true, nil, err
- }
- obj, err := runtime.Decode(clientscheme.Codecs.LegacyCodec(apps.SchemeGroupVersion), b)
- if err != nil {
- return true, nil, err
- }
- patched, err := legacyscheme.Scheme.ConvertToVersion(obj, apps.SchemeGroupVersion)
- if err != nil {
- return true, nil, err
- }
- return true, patched, err
- default:
- return false, nil, nil
- }
- })
- informerFactory := informers.NewSharedInformerFactory(client, controller.NoResyncPeriodFunc())
- stop := make(chan struct{})
- defer close(stop)
- informerFactory.Start(stop)
- informer := informerFactory.Apps().V1().ControllerRevisions()
- informerFactory.WaitForCacheSync(stop)
- history := NewHistory(client, informer.Lister())
- var collisionCount int32
- for i := range test.existing {
- _, err := history.CreateControllerRevision(test.existing[i].parent, test.existing[i].revision, &collisionCount)
- if err != nil {
- t.Fatal(err)
- }
- }
- adopted, err := history.AdoptControllerRevision(test.parent, parentKind, test.revision)
- if !test.err && err != nil {
- t.Errorf("%s: %s", test.name, err)
- }
- if !test.err && !metav1.IsControlledBy(adopted, test.parent) {
- t.Errorf("%s: adoption failed", test.name)
- }
- if test.err && err == nil {
- t.Errorf("%s: expected error", test.name)
- }
- }
- ss1 := newStatefulSet(3, "ss1", types.UID("ss1"), map[string]string{"foo": "bar"})
- ss1.Status.CollisionCount = new(int32)
- ss2 := newStatefulSet(3, "ss2", types.UID("ss2"), map[string]string{"goo": "car"})
- ss2.Status.CollisionCount = new(int32)
- ss1Rev1, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(&ss1.Spec.Template), 1, ss1.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev1.Namespace = ss1.Namespace
- ss1Rev2, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(&ss1.Spec.Template), 2, ss1.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev2.Namespace = ss1.Namespace
- ss1Rev2.OwnerReferences = []metav1.OwnerReference{}
- ss2Rev1, err := NewControllerRevision(ss2, parentKind, ss2.Spec.Template.Labels, rawTemplate(&ss2.Spec.Template), 1, ss2.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss2Rev1.Namespace = ss2.Namespace
- tests := []testcase{
- {
- name: "adopting an orphan succeeds",
- parent: ss1,
- revision: ss1Rev2,
- existing: []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }{
- {
- parent: ss1,
- revision: ss1Rev2,
- },
- },
- err: false,
- },
- {
- name: "adopting an owned revision fails",
- parent: ss1,
- revision: ss2Rev1,
- existing: []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }{
- {
- parent: ss2,
- revision: ss2Rev1,
- },
- },
- err: true,
- },
- {
- name: "adopting a non-existent revision fails",
- parent: ss1,
- revision: ss1Rev2,
- existing: nil,
- err: true,
- },
- }
- for i := range tests {
- testFn(&tests[i], t)
- }
- }
- func TestFakeHistory_AdoptControllerRevision(t *testing.T) {
- type testcase struct {
- name string
- parent metav1.Object
- parentType *metav1.TypeMeta
- revision *apps.ControllerRevision
- existing []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }
- err bool
- }
- testFn := func(test *testcase, t *testing.T) {
- client := fake.NewSimpleClientset()
- informerFactory := informers.NewSharedInformerFactory(client, controller.NoResyncPeriodFunc())
- stop := make(chan struct{})
- defer close(stop)
- informerFactory.Start(stop)
- informer := informerFactory.Apps().V1().ControllerRevisions()
- informerFactory.WaitForCacheSync(stop)
- history := NewFakeHistory(informer)
- var collisionCount int32
- for i := range test.existing {
- _, err := history.CreateControllerRevision(test.existing[i].parent, test.existing[i].revision, &collisionCount)
- if err != nil {
- t.Fatal(err)
- }
- }
- adopted, err := history.AdoptControllerRevision(test.parent, parentKind, test.revision)
- if !test.err && err != nil {
- t.Errorf("%s: %s", test.name, err)
- }
- if !test.err && !metav1.IsControlledBy(adopted, test.parent) {
- t.Errorf("%s: adoption failed", test.name)
- }
- if test.err && err == nil {
- t.Errorf("%s: expected error", test.name)
- }
- }
- ss1 := newStatefulSet(3, "ss1", types.UID("ss1"), map[string]string{"foo": "bar"})
- ss1.Status.CollisionCount = new(int32)
- ss2 := newStatefulSet(3, "ss2", types.UID("ss2"), map[string]string{"goo": "car"})
- ss2.Status.CollisionCount = new(int32)
- ss1Rev1, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(&ss1.Spec.Template), 1, ss1.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev1.Namespace = ss1.Namespace
- ss1Rev2, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(&ss1.Spec.Template), 2, ss1.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev2.Namespace = ss1.Namespace
- ss1Rev2.OwnerReferences = []metav1.OwnerReference{}
- ss2Rev1, err := NewControllerRevision(ss2, parentKind, ss2.Spec.Template.Labels, rawTemplate(&ss2.Spec.Template), 1, ss2.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss2Rev1.Namespace = ss2.Namespace
- tests := []testcase{
- {
- name: "adopting an orphan succeeds",
- parent: ss1,
- parentType: &ss1.TypeMeta,
- revision: ss1Rev2,
- existing: []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }{
- {
- parent: ss1,
- revision: ss1Rev2,
- },
- },
- err: false,
- },
- {
- name: "adopting an owned revision fails",
- parent: ss1,
- parentType: &ss1.TypeMeta,
- revision: ss2Rev1,
- existing: []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }{
- {
- parent: ss2,
- revision: ss2Rev1,
- },
- },
- err: true,
- },
- {
- name: "adopting a non-existent revision fails",
- parent: ss1,
- parentType: &ss1.TypeMeta,
- revision: ss1Rev2,
- existing: nil,
- err: true,
- },
- }
- for i := range tests {
- testFn(&tests[i], t)
- }
- }
- func TestRealHistory_ReleaseControllerRevision(t *testing.T) {
- type testcase struct {
- name string
- parent metav1.Object
- revision *apps.ControllerRevision
- existing []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }
- err bool
- }
- testFn := func(test *testcase, t *testing.T) {
- client := fake.NewSimpleClientset()
- client.AddReactor("*", "*", func(action core.Action) (bool, runtime.Object, error) {
- switch action := action.(type) {
- case core.PatchActionImpl:
- var found *apps.ControllerRevision
- for i := range test.existing {
- if test.revision.Name == test.existing[i].revision.Name &&
- test.revision.Namespace == test.existing[i].revision.Namespace {
- found = test.existing[i].revision
- break
- }
- }
- if found == nil {
- return true, nil, errors.NewNotFound(apps.Resource("controllerrevisions"), test.revision.Name)
- }
- if !metav1.IsControlledBy(test.revision, test.parent) {
- return true, nil, errors.NewInvalid(
- test.revision.GroupVersionKind().GroupKind(), test.revision.Name, nil)
- }
- b, err := strategicpatch.StrategicMergePatch(
- []byte(runtime.EncodeOrDie(clientscheme.Codecs.LegacyCodec(apps.SchemeGroupVersion), test.revision)),
- action.GetPatch(), test.revision)
- if err != nil {
- return true, nil, err
- }
- obj, err := runtime.Decode(clientscheme.Codecs.LegacyCodec(apps.SchemeGroupVersion), b)
- if err != nil {
- return true, nil, err
- }
- patched, err := legacyscheme.Scheme.ConvertToVersion(obj, apps.SchemeGroupVersion)
- if err != nil {
- return true, nil, err
- }
- return true, patched, err
- default:
- return false, nil, nil
- }
- })
- informerFactory := informers.NewSharedInformerFactory(client, controller.NoResyncPeriodFunc())
- stop := make(chan struct{})
- defer close(stop)
- informerFactory.Start(stop)
- informer := informerFactory.Apps().V1().ControllerRevisions()
- informerFactory.WaitForCacheSync(stop)
- history := NewHistory(client, informer.Lister())
- var collisionCount int32
- for i := range test.existing {
- _, err := history.CreateControllerRevision(test.existing[i].parent, test.existing[i].revision, &collisionCount)
- if err != nil {
- t.Fatal(err)
- }
- }
- adopted, err := history.ReleaseControllerRevision(test.parent, test.revision)
- if !test.err {
- if err != nil {
- t.Errorf("%s: %s", test.name, err)
- }
- if adopted == nil {
- return
- }
- if metav1.IsControlledBy(adopted, test.parent) {
- t.Errorf("%s: release failed", test.name)
- }
- }
- if test.err && err == nil {
- t.Errorf("%s: expected error", test.name)
- }
- }
- ss1 := newStatefulSet(3, "ss1", types.UID("ss1"), map[string]string{"foo": "bar"})
- ss2 := newStatefulSet(3, "ss2", types.UID("ss2"), map[string]string{"goo": "car"})
- ss1Rev1, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(&ss1.Spec.Template), 1, nil)
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev1.Namespace = ss1.Namespace
- ss1Rev2, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(&ss1.Spec.Template), 2, nil)
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev2.Namespace = ss1.Namespace
- ss1Rev2.OwnerReferences = []metav1.OwnerReference{}
- ss2Rev1, err := NewControllerRevision(ss2, parentKind, ss2.Spec.Template.Labels, rawTemplate(&ss2.Spec.Template), 1, nil)
- if err != nil {
- t.Fatal(err)
- }
- ss2Rev1.Namespace = ss2.Namespace
- tests := []testcase{
- {
- name: "releasing an owned revision succeeds",
- parent: ss1,
- revision: ss1Rev1,
- existing: []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }{
- {
- parent: ss1,
- revision: ss1Rev1,
- },
- },
- err: false,
- },
- {
- name: "releasing an orphan succeeds",
- parent: ss1,
- revision: ss1Rev2,
- existing: []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }{
- {
- parent: ss1,
- revision: ss1Rev2,
- },
- },
- err: false,
- },
- {
- name: "releasing a revision owned by another controller succeeds",
- parent: ss1,
- revision: ss2Rev1,
- existing: []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }{
- {
- parent: ss2,
- revision: ss2Rev1,
- },
- },
- err: false,
- },
- {
- name: "releasing a non-existent revision succeeds",
- parent: ss1,
- revision: ss1Rev1,
- existing: nil,
- err: false,
- },
- }
- for i := range tests {
- testFn(&tests[i], t)
- }
- }
- func TestFakeHistory_ReleaseControllerRevision(t *testing.T) {
- type testcase struct {
- name string
- parent metav1.Object
- revision *apps.ControllerRevision
- existing []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }
- err bool
- }
- testFn := func(test *testcase, t *testing.T) {
- client := fake.NewSimpleClientset()
- informerFactory := informers.NewSharedInformerFactory(client, controller.NoResyncPeriodFunc())
- stop := make(chan struct{})
- defer close(stop)
- informerFactory.Start(stop)
- informer := informerFactory.Apps().V1().ControllerRevisions()
- informerFactory.WaitForCacheSync(stop)
- history := NewFakeHistory(informer)
- var collisionCount int32
- for i := range test.existing {
- _, err := history.CreateControllerRevision(test.existing[i].parent, test.existing[i].revision, &collisionCount)
- if err != nil {
- t.Fatal(err)
- }
- }
- adopted, err := history.ReleaseControllerRevision(test.parent, test.revision)
- if !test.err {
- if err != nil {
- t.Errorf("%s: %s", test.name, err)
- }
- if adopted == nil {
- return
- }
- if metav1.IsControlledBy(adopted, test.parent) {
- t.Errorf("%s: release failed", test.name)
- }
- }
- if test.err && err == nil {
- t.Errorf("%s: expected error", test.name)
- }
- }
- ss1 := newStatefulSet(3, "ss1", types.UID("ss1"), map[string]string{"foo": "bar"})
- ss1.Status.CollisionCount = new(int32)
- ss2 := newStatefulSet(3, "ss2", types.UID("ss2"), map[string]string{"goo": "car"})
- ss2.Status.CollisionCount = new(int32)
- ss1Rev1, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(&ss1.Spec.Template), 1, ss1.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev1.Namespace = ss1.Namespace
- ss1Rev2, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(&ss1.Spec.Template), 2, ss1.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev2.Namespace = ss1.Namespace
- ss1Rev2.OwnerReferences = []metav1.OwnerReference{}
- ss2Rev1, err := NewControllerRevision(ss2, parentKind, ss2.Spec.Template.Labels, rawTemplate(&ss2.Spec.Template), 1, ss2.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss2Rev1.Namespace = ss2.Namespace
- tests := []testcase{
- {
- name: "releasing an owned revision succeeds",
- parent: ss1,
- revision: ss1Rev1,
- existing: []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }{
- {
- parent: ss1,
- revision: ss1Rev1,
- },
- },
- err: false,
- },
- {
- name: "releasing an orphan succeeds",
- parent: ss1,
- revision: ss1Rev2,
- existing: []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }{
- {
- parent: ss1,
- revision: ss1Rev2,
- },
- },
- err: false,
- },
- {
- name: "releasing a revision owned by another controller succeeds",
- parent: ss1,
- revision: ss2Rev1,
- existing: []struct {
- parent metav1.Object
- revision *apps.ControllerRevision
- }{
- {
- parent: ss2,
- revision: ss2Rev1,
- },
- },
- err: false,
- },
- {
- name: "releasing a non-existent revision succeeds",
- parent: ss1,
- revision: ss1Rev1,
- existing: nil,
- err: false,
- },
- }
- for i := range tests {
- testFn(&tests[i], t)
- }
- }
- func TestFindEqualRevisions(t *testing.T) {
- type testcase struct {
- name string
- revision *apps.ControllerRevision
- revisions []*apps.ControllerRevision
- want map[string]bool
- }
- testFn := func(test *testcase, t *testing.T) {
- found := FindEqualRevisions(test.revisions, test.revision)
- if len(found) != len(test.want) {
- t.Errorf("%s: want %d revisions found %d", test.name, len(test.want), len(found))
- }
- for i := range found {
- if !test.want[found[i].Name] {
- t.Errorf("%s: wanted %s not found", test.name, found[i].Name)
- }
- }
- }
- ss1 := newStatefulSet(3, "ss1", types.UID("ss1"), map[string]string{"foo": "bar"})
- ss1.Status.CollisionCount = new(int32)
- ss2 := newStatefulSet(3, "ss2", types.UID("ss2"), map[string]string{"goo": "car"})
- ss2.Status.CollisionCount = new(int32)
- ss1Rev1, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(&ss1.Spec.Template), 1, ss1.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev1.Namespace = ss1.Namespace
- ss1Rev2, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(&ss1.Spec.Template), 2, ss1.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev2.Namespace = ss1.Namespace
- ss1Rev2.OwnerReferences = []metav1.OwnerReference{}
- ss2Rev1, err := NewControllerRevision(ss2, parentKind, ss2.Spec.Template.Labels, rawTemplate(&ss2.Spec.Template), 1, ss2.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss2Rev1.Namespace = ss2.Namespace
- ss2Rev2, err := NewControllerRevision(ss2, parentKind, ss2.Spec.Template.Labels, rawTemplate(&ss2.Spec.Template), 2, ss2.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss2Rev2.Namespace = ss2.Namespace
- tests := []testcase{
- {
- name: "finds equivalent",
- revision: ss1Rev1,
- revisions: []*apps.ControllerRevision{ss1Rev1, ss2Rev1, ss2Rev2},
- want: map[string]bool{ss1Rev1.Name: true},
- },
- {
- name: "finds nothing when empty",
- revision: ss1Rev1,
- revisions: nil,
- want: map[string]bool{},
- },
- {
- name: "finds nothing with no matches",
- revision: ss1Rev1,
- revisions: []*apps.ControllerRevision{ss2Rev2, ss2Rev1},
- want: map[string]bool{},
- },
- }
- for i := range tests {
- testFn(&tests[i], t)
- }
- }
- func TestSortControllerRevisions(t *testing.T) {
- type testcase struct {
- name string
- revisions []*apps.ControllerRevision
- want []string
- }
- testFn := func(test *testcase, t *testing.T) {
- t.Run(test.name, func(t *testing.T) {
- SortControllerRevisions(test.revisions)
- for i := range test.revisions {
- if test.revisions[i].Name != test.want[i] {
- t.Errorf("%s: want %s at %d got %s", test.name, test.want[i], i, test.revisions[i].Name)
- }
- }
- })
- }
- ss1 := newStatefulSet(3, "ss1", types.UID("ss1"), map[string]string{"foo": "bar"})
- ss1.Status.CollisionCount = new(int32)
- ss1Rev1, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(&ss1.Spec.Template), 1, ss1.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev1.Namespace = ss1.Namespace
- ss1Rev2, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(&ss1.Spec.Template), 2, ss1.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev2.Namespace = ss1.Namespace
- ss1Rev3, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(&ss1.Spec.Template), 3, ss1.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev3Time2, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(&ss1.Spec.Template), 3, ss1.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev3Time2.Namespace = ss1.Namespace
- ss1Rev3Time2.CreationTimestamp = metav1.Time{Time: ss1Rev3.CreationTimestamp.Add(time.Second)}
- ss1Rev3Time2Name2, err := NewControllerRevision(ss1, parentKind, ss1.Spec.Template.Labels, rawTemplate(&ss1.Spec.Template), 3, ss1.Status.CollisionCount)
- if err != nil {
- t.Fatal(err)
- }
- ss1Rev3Time2Name2.Namespace = ss1.Namespace
- ss1Rev3Time2Name2.CreationTimestamp = metav1.Time{Time: ss1Rev3.CreationTimestamp.Add(time.Second)}
- tests := []testcase{
- {
- name: "out of order",
- revisions: []*apps.ControllerRevision{ss1Rev2, ss1Rev1, ss1Rev3},
- want: []string{ss1Rev1.Name, ss1Rev2.Name, ss1Rev3.Name},
- },
- {
- name: "sorted",
- revisions: []*apps.ControllerRevision{ss1Rev1, ss1Rev2, ss1Rev3},
- want: []string{ss1Rev1.Name, ss1Rev2.Name, ss1Rev3.Name},
- },
- {
- name: "reversed",
- revisions: []*apps.ControllerRevision{ss1Rev3, ss1Rev2, ss1Rev1},
- want: []string{ss1Rev1.Name, ss1Rev2.Name, ss1Rev3.Name},
- },
- {
- name: "with ties",
- revisions: []*apps.ControllerRevision{ss1Rev3, ss1Rev3Time2, ss1Rev2, ss1Rev1},
- want: []string{ss1Rev1.Name, ss1Rev2.Name, ss1Rev3.Name, ss1Rev3Time2.Name, ss1Rev3Time2Name2.Name},
- },
- {
- name: "empty",
- revisions: nil,
- want: nil,
- },
- }
- for i := range tests {
- testFn(&tests[i], t)
- }
- }
- func newStatefulSet(replicas int, name string, uid types.UID, labels map[string]string) *apps.StatefulSet {
- // Converting all the map-only selectors to set-based selectors.
- var testMatchExpressions []metav1.LabelSelectorRequirement
- for key, value := range labels {
- sel := metav1.LabelSelectorRequirement{
- Key: key,
- Operator: metav1.LabelSelectorOpIn,
- Values: []string{value},
- }
- testMatchExpressions = append(testMatchExpressions, sel)
- }
- return &apps.StatefulSet{
- TypeMeta: metav1.TypeMeta{
- Kind: "StatefulSet",
- APIVersion: "apps/v1",
- },
- ObjectMeta: metav1.ObjectMeta{
- Name: name,
- Namespace: v1.NamespaceDefault,
- UID: uid,
- },
- Spec: apps.StatefulSetSpec{
- Selector: &metav1.LabelSelector{
- // Purposely leaving MatchLabels nil, so to ensure it will break if any link
- // in the chain ignores the set-based MatchExpressions.
- MatchLabels: nil,
- MatchExpressions: testMatchExpressions,
- },
- Replicas: func() *int32 { i := int32(replicas); return &i }(),
- Template: v1.PodTemplateSpec{
- ObjectMeta: metav1.ObjectMeta{
- Labels: labels,
- },
- Spec: v1.PodSpec{
- Containers: []v1.Container{
- {
- Name: "nginx",
- Image: "nginx",
- VolumeMounts: []v1.VolumeMount{
- {Name: "datadir", MountPath: "/tmp/"},
- {Name: "home", MountPath: "/home"},
- },
- },
- },
- Volumes: []v1.Volume{{
- Name: "home",
- VolumeSource: v1.VolumeSource{
- HostPath: &v1.HostPathVolumeSource{
- Path: fmt.Sprintf("/tmp/%v", "home"),
- },
- }}},
- },
- },
- VolumeClaimTemplates: []v1.PersistentVolumeClaim{
- {
- ObjectMeta: metav1.ObjectMeta{Name: "datadir"},
- Spec: v1.PersistentVolumeClaimSpec{
- Resources: v1.ResourceRequirements{
- Requests: v1.ResourceList{
- v1.ResourceStorage: *resource.NewQuantity(1, resource.BinarySI),
- },
- },
- },
- },
- },
- ServiceName: "governingsvc",
- },
- }
- }
- var parentKind = apps.SchemeGroupVersion.WithKind("StatefulSet")
- func rawTemplate(template *v1.PodTemplateSpec) runtime.RawExtension {
- buf := new(bytes.Buffer)
- enc := json.NewEncoder(buf)
- if err := enc.Encode(template); err != nil {
- panic(err)
- }
- return runtime.RawExtension{Raw: buf.Bytes()}
- }
|