123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652653654655656657658659660661662663664665666667668669670671672673674675676677678679680681682683684685686687688689690691692693694695696697698699700701702703704705706707708709710711712713714715716717718719720721722723724725726727728729730731732733734735736737738739740741742743744745746747748749750751752753754755756757758759760761762763764765766767768769770771772773774775776777778779780781782783784785786787788789790791792793794795796797798799800801802803804805806807808809810811812813814815816817818819820821822823824825826827828829830831832833834835836837838839840841842843844845846847848849850851852853854855856857858859860861862863864865866867868869870871872873874875876877878879880881882883884885886887888889890891892893894895896897898899900901902903904905906907908909910911912913914915916917918919920921922923924925926927928929930931932933934935936937938939940941942943944945946947948949950951952953954955956957958959960961962963964965966967968969970971972973974975976977978979980981982983984985986987988989990991992993994995996997998999100010011002100310041005100610071008100910101011101210131014101510161017101810191020102110221023102410251026102710281029103010311032103310341035103610371038103910401041104210431044104510461047104810491050105110521053105410551056105710581059106010611062106310641065106610671068106910701071107210731074107510761077107810791080108110821083108410851086108710881089109010911092109310941095109610971098109911001101110211031104110511061107110811091110111111121113111411151116111711181119112011211122112311241125112611271128112911301131113211331134113511361137113811391140114111421143114411451146114711481149115011511152115311541155115611571158115911601161116211631164116511661167116811691170117111721173117411751176117711781179118011811182118311841185118611871188118911901191119211931194119511961197119811991200120112021203120412051206120712081209121012111212121312141215121612171218121912201221122212231224122512261227122812291230123112321233123412351236123712381239124012411242124312441245124612471248124912501251125212531254125512561257125812591260126112621263126412651266126712681269127012711272127312741275127612771278127912801281128212831284128512861287128812891290129112921293129412951296129712981299130013011302130313041305130613071308130913101311131213131314131513161317131813191320132113221323132413251326132713281329133013311332133313341335133613371338133913401341134213431344134513461347134813491350135113521353135413551356135713581359136013611362136313641365136613671368136913701371137213731374137513761377137813791380138113821383138413851386138713881389139013911392139313941395139613971398139914001401140214031404140514061407140814091410141114121413141414151416141714181419142014211422142314241425142614271428142914301431143214331434143514361437143814391440144114421443144414451446144714481449145014511452145314541455145614571458145914601461146214631464146514661467146814691470147114721473147414751476147714781479148014811482148314841485148614871488148914901491149214931494149514961497149814991500150115021503150415051506150715081509151015111512151315141515151615171518151915201521152215231524152515261527152815291530153115321533153415351536153715381539154015411542154315441545154615471548154915501551155215531554155515561557155815591560156115621563156415651566156715681569157015711572157315741575157615771578157915801581158215831584158515861587158815891590159115921593159415951596159715981599160016011602160316041605160616071608160916101611161216131614161516161617161816191620162116221623162416251626162716281629163016311632163316341635163616371638163916401641164216431644164516461647164816491650165116521653165416551656165716581659166016611662166316641665166616671668166916701671167216731674167516761677167816791680168116821683168416851686168716881689169016911692169316941695169616971698169917001701170217031704 |
- /*
- 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"
- "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"
- "k8s.io/kubernetes/pkg/api/legacyscheme"
- "k8s.io/kubernetes/pkg/api/testapi"
- "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(item.revision)
- 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(testapi.Apps.Codec(), test.revision)),
- action.GetPatch(), test.revision)
- if err != nil {
- return true, nil, err
- }
- obj, err := runtime.Decode(testapi.Apps.Codec(), 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(testapi.Apps.Codec(), test.revision)),
- action.GetPatch(), test.revision)
- if err != nil {
- return true, nil, err
- }
- obj, err := runtime.Decode(testapi.Apps.Codec(), 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()}
- }
|