tokens_controller_test.go 29 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652653654655656657658659660661662663664665666667668669670671672673674675676677678679680681682683684685686687688689690691692
  1. /*
  2. Copyright 2014 The Kubernetes Authors.
  3. Licensed under the Apache License, Version 2.0 (the "License");
  4. you may not use this file except in compliance with the License.
  5. You may obtain a copy of the License at
  6. http://www.apache.org/licenses/LICENSE-2.0
  7. Unless required by applicable law or agreed to in writing, software
  8. distributed under the License is distributed on an "AS IS" BASIS,
  9. WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
  10. See the License for the specific language governing permissions and
  11. limitations under the License.
  12. */
  13. package serviceaccount
  14. import (
  15. "errors"
  16. "reflect"
  17. "testing"
  18. "time"
  19. "github.com/davecgh/go-spew/spew"
  20. "gopkg.in/square/go-jose.v2/jwt"
  21. "k8s.io/klog"
  22. "k8s.io/api/core/v1"
  23. apierrors "k8s.io/apimachinery/pkg/api/errors"
  24. metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
  25. "k8s.io/apimachinery/pkg/runtime"
  26. "k8s.io/apimachinery/pkg/runtime/schema"
  27. utilrand "k8s.io/apimachinery/pkg/util/rand"
  28. "k8s.io/client-go/informers"
  29. "k8s.io/client-go/kubernetes/fake"
  30. core "k8s.io/client-go/testing"
  31. api "k8s.io/kubernetes/pkg/apis/core"
  32. "k8s.io/kubernetes/pkg/controller"
  33. )
  34. type testGenerator struct {
  35. Token string
  36. Err error
  37. }
  38. func (t *testGenerator) GenerateToken(sc *jwt.Claims, pc interface{}) (string, error) {
  39. return t.Token, t.Err
  40. }
  41. // emptySecretReferences is used by a service account without any secrets
  42. func emptySecretReferences() []v1.ObjectReference {
  43. return []v1.ObjectReference{}
  44. }
  45. // missingSecretReferences is used by a service account that references secrets which do no exist
  46. func missingSecretReferences() []v1.ObjectReference {
  47. return []v1.ObjectReference{{Name: "missing-secret-1"}}
  48. }
  49. // regularSecretReferences is used by a service account that references secrets which are not ServiceAccountTokens
  50. func regularSecretReferences() []v1.ObjectReference {
  51. return []v1.ObjectReference{{Name: "regular-secret-1"}}
  52. }
  53. // tokenSecretReferences is used by a service account that references a ServiceAccountToken secret
  54. func tokenSecretReferences() []v1.ObjectReference {
  55. return []v1.ObjectReference{{Name: "token-secret-1"}}
  56. }
  57. // addTokenSecretReference adds a reference to the ServiceAccountToken that will be created
  58. func addTokenSecretReference(refs []v1.ObjectReference) []v1.ObjectReference {
  59. return addNamedTokenSecretReference(refs, "default-token-xn8fg")
  60. }
  61. // addNamedTokenSecretReference adds a reference to the named ServiceAccountToken
  62. func addNamedTokenSecretReference(refs []v1.ObjectReference, name string) []v1.ObjectReference {
  63. return append(refs, v1.ObjectReference{Name: name})
  64. }
  65. // serviceAccount returns a service account with the given secret refs
  66. func serviceAccount(secretRefs []v1.ObjectReference) *v1.ServiceAccount {
  67. return &v1.ServiceAccount{
  68. ObjectMeta: metav1.ObjectMeta{
  69. Name: "default",
  70. UID: "12345",
  71. Namespace: "default",
  72. ResourceVersion: "1",
  73. },
  74. Secrets: secretRefs,
  75. }
  76. }
  77. // updatedServiceAccount returns a service account with the resource version modified
  78. func updatedServiceAccount(secretRefs []v1.ObjectReference) *v1.ServiceAccount {
  79. sa := serviceAccount(secretRefs)
  80. sa.ResourceVersion = "2"
  81. return sa
  82. }
  83. // opaqueSecret returns a persisted non-ServiceAccountToken secret named "regular-secret-1"
  84. func opaqueSecret() *v1.Secret {
  85. return &v1.Secret{
  86. ObjectMeta: metav1.ObjectMeta{
  87. Name: "regular-secret-1",
  88. Namespace: "default",
  89. UID: "23456",
  90. ResourceVersion: "1",
  91. },
  92. Type: "Opaque",
  93. Data: map[string][]byte{
  94. "mykey": []byte("mydata"),
  95. },
  96. }
  97. }
  98. // createdTokenSecret returns the ServiceAccountToken secret posted when creating a new token secret.
  99. // Named "default-token-xn8fg", since that is the first generated name after rand.Seed(1)
  100. func createdTokenSecret(overrideName ...string) *v1.Secret {
  101. return namedCreatedTokenSecret("default-token-xn8fg")
  102. }
  103. // namedTokenSecret returns the ServiceAccountToken secret posted when creating a new token secret with the given name.
  104. func namedCreatedTokenSecret(name string) *v1.Secret {
  105. return &v1.Secret{
  106. ObjectMeta: metav1.ObjectMeta{
  107. Name: name,
  108. Namespace: "default",
  109. Annotations: map[string]string{
  110. v1.ServiceAccountNameKey: "default",
  111. v1.ServiceAccountUIDKey: "12345",
  112. },
  113. },
  114. Type: v1.SecretTypeServiceAccountToken,
  115. Data: map[string][]byte{
  116. "token": []byte("ABC"),
  117. "ca.crt": []byte("CA Data"),
  118. "namespace": []byte("default"),
  119. },
  120. }
  121. }
  122. // serviceAccountTokenSecret returns an existing ServiceAccountToken secret named "token-secret-1"
  123. func serviceAccountTokenSecret() *v1.Secret {
  124. return &v1.Secret{
  125. ObjectMeta: metav1.ObjectMeta{
  126. Name: "token-secret-1",
  127. Namespace: "default",
  128. UID: "23456",
  129. ResourceVersion: "1",
  130. Annotations: map[string]string{
  131. v1.ServiceAccountNameKey: "default",
  132. v1.ServiceAccountUIDKey: "12345",
  133. },
  134. },
  135. Type: v1.SecretTypeServiceAccountToken,
  136. Data: map[string][]byte{
  137. "token": []byte("ABC"),
  138. "ca.crt": []byte("CA Data"),
  139. "namespace": []byte("default"),
  140. },
  141. }
  142. }
  143. // serviceAccountTokenSecretWithoutTokenData returns an existing ServiceAccountToken secret that lacks token data
  144. func serviceAccountTokenSecretWithoutTokenData() *v1.Secret {
  145. secret := serviceAccountTokenSecret()
  146. delete(secret.Data, v1.ServiceAccountTokenKey)
  147. return secret
  148. }
  149. // serviceAccountTokenSecretWithoutCAData returns an existing ServiceAccountToken secret that lacks ca data
  150. func serviceAccountTokenSecretWithoutCAData() *v1.Secret {
  151. secret := serviceAccountTokenSecret()
  152. delete(secret.Data, v1.ServiceAccountRootCAKey)
  153. return secret
  154. }
  155. // serviceAccountTokenSecretWithCAData returns an existing ServiceAccountToken secret with the specified ca data
  156. func serviceAccountTokenSecretWithCAData(data []byte) *v1.Secret {
  157. secret := serviceAccountTokenSecret()
  158. secret.Data[v1.ServiceAccountRootCAKey] = data
  159. return secret
  160. }
  161. // serviceAccountTokenSecretWithoutNamespaceData returns an existing ServiceAccountToken secret that lacks namespace data
  162. func serviceAccountTokenSecretWithoutNamespaceData() *v1.Secret {
  163. secret := serviceAccountTokenSecret()
  164. delete(secret.Data, v1.ServiceAccountNamespaceKey)
  165. return secret
  166. }
  167. // serviceAccountTokenSecretWithNamespaceData returns an existing ServiceAccountToken secret with the specified namespace data
  168. func serviceAccountTokenSecretWithNamespaceData(data []byte) *v1.Secret {
  169. secret := serviceAccountTokenSecret()
  170. secret.Data[v1.ServiceAccountNamespaceKey] = data
  171. return secret
  172. }
  173. type reaction struct {
  174. verb string
  175. resource string
  176. reactor func(t *testing.T) core.ReactionFunc
  177. }
  178. func TestTokenCreation(t *testing.T) {
  179. testcases := map[string]struct {
  180. ClientObjects []runtime.Object
  181. IsAsync bool
  182. MaxRetries int
  183. Reactors []reaction
  184. ExistingServiceAccount *v1.ServiceAccount
  185. ExistingSecrets []*v1.Secret
  186. AddedServiceAccount *v1.ServiceAccount
  187. UpdatedServiceAccount *v1.ServiceAccount
  188. DeletedServiceAccount *v1.ServiceAccount
  189. AddedSecret *v1.Secret
  190. AddedSecretLocal *v1.Secret
  191. UpdatedSecret *v1.Secret
  192. DeletedSecret *v1.Secret
  193. ExpectedActions []core.Action
  194. }{
  195. "new serviceaccount with no secrets": {
  196. ClientObjects: []runtime.Object{serviceAccount(emptySecretReferences())},
  197. AddedServiceAccount: serviceAccount(emptySecretReferences()),
  198. ExpectedActions: []core.Action{
  199. core.NewGetAction(schema.GroupVersionResource{Version: "v1", Resource: "serviceaccounts"}, metav1.NamespaceDefault, "default"),
  200. core.NewCreateAction(schema.GroupVersionResource{Version: "v1", Resource: "secrets"}, metav1.NamespaceDefault, createdTokenSecret()),
  201. core.NewUpdateAction(schema.GroupVersionResource{Version: "v1", Resource: "serviceaccounts"}, metav1.NamespaceDefault, serviceAccount(addTokenSecretReference(emptySecretReferences()))),
  202. },
  203. },
  204. "new serviceaccount with no secrets encountering create error": {
  205. ClientObjects: []runtime.Object{serviceAccount(emptySecretReferences())},
  206. MaxRetries: 10,
  207. IsAsync: true,
  208. Reactors: []reaction{{
  209. verb: "create",
  210. resource: "secrets",
  211. reactor: func(t *testing.T) core.ReactionFunc {
  212. i := 0
  213. return func(core.Action) (bool, runtime.Object, error) {
  214. i++
  215. if i < 3 {
  216. return true, nil, apierrors.NewForbidden(api.Resource("secrets"), "foo", errors.New("no can do"))
  217. }
  218. return false, nil, nil
  219. }
  220. },
  221. }},
  222. AddedServiceAccount: serviceAccount(emptySecretReferences()),
  223. ExpectedActions: []core.Action{
  224. // Attempt 1
  225. core.NewGetAction(schema.GroupVersionResource{Version: "v1", Resource: "serviceaccounts"}, metav1.NamespaceDefault, "default"),
  226. core.NewCreateAction(schema.GroupVersionResource{Version: "v1", Resource: "secrets"}, metav1.NamespaceDefault, createdTokenSecret()),
  227. // Attempt 2
  228. core.NewGetAction(schema.GroupVersionResource{Version: "v1", Resource: "serviceaccounts"}, metav1.NamespaceDefault, "default"),
  229. core.NewCreateAction(schema.GroupVersionResource{Version: "v1", Resource: "secrets"}, metav1.NamespaceDefault, namedCreatedTokenSecret("default-token-txhzt")),
  230. // Attempt 3
  231. core.NewGetAction(schema.GroupVersionResource{Version: "v1", Resource: "serviceaccounts"}, metav1.NamespaceDefault, "default"),
  232. core.NewCreateAction(schema.GroupVersionResource{Version: "v1", Resource: "secrets"}, metav1.NamespaceDefault, namedCreatedTokenSecret("default-token-vnmz7")),
  233. core.NewUpdateAction(schema.GroupVersionResource{Version: "v1", Resource: "serviceaccounts"}, metav1.NamespaceDefault, serviceAccount(addNamedTokenSecretReference(emptySecretReferences(), "default-token-vnmz7"))),
  234. },
  235. },
  236. "new serviceaccount with no secrets encountering unending create error": {
  237. ClientObjects: []runtime.Object{serviceAccount(emptySecretReferences()), createdTokenSecret()},
  238. MaxRetries: 2,
  239. IsAsync: true,
  240. Reactors: []reaction{{
  241. verb: "create",
  242. resource: "secrets",
  243. reactor: func(t *testing.T) core.ReactionFunc {
  244. return func(core.Action) (bool, runtime.Object, error) {
  245. return true, nil, apierrors.NewForbidden(api.Resource("secrets"), "foo", errors.New("no can do"))
  246. }
  247. },
  248. }},
  249. AddedServiceAccount: serviceAccount(emptySecretReferences()),
  250. ExpectedActions: []core.Action{
  251. // Attempt
  252. core.NewGetAction(schema.GroupVersionResource{Version: "v1", Resource: "serviceaccounts"}, metav1.NamespaceDefault, "default"),
  253. core.NewCreateAction(schema.GroupVersionResource{Version: "v1", Resource: "secrets"}, metav1.NamespaceDefault, createdTokenSecret()),
  254. // Retry 1
  255. core.NewGetAction(schema.GroupVersionResource{Version: "v1", Resource: "serviceaccounts"}, metav1.NamespaceDefault, "default"),
  256. core.NewCreateAction(schema.GroupVersionResource{Version: "v1", Resource: "secrets"}, metav1.NamespaceDefault, namedCreatedTokenSecret("default-token-txhzt")),
  257. // Retry 2
  258. core.NewGetAction(schema.GroupVersionResource{Version: "v1", Resource: "serviceaccounts"}, metav1.NamespaceDefault, "default"),
  259. core.NewCreateAction(schema.GroupVersionResource{Version: "v1", Resource: "secrets"}, metav1.NamespaceDefault, namedCreatedTokenSecret("default-token-vnmz7")),
  260. },
  261. },
  262. "new serviceaccount with missing secrets": {
  263. ClientObjects: []runtime.Object{serviceAccount(missingSecretReferences())},
  264. AddedServiceAccount: serviceAccount(missingSecretReferences()),
  265. ExpectedActions: []core.Action{
  266. core.NewGetAction(schema.GroupVersionResource{Version: "v1", Resource: "serviceaccounts"}, metav1.NamespaceDefault, "default"),
  267. core.NewCreateAction(schema.GroupVersionResource{Version: "v1", Resource: "secrets"}, metav1.NamespaceDefault, createdTokenSecret()),
  268. core.NewUpdateAction(schema.GroupVersionResource{Version: "v1", Resource: "serviceaccounts"}, metav1.NamespaceDefault, serviceAccount(addTokenSecretReference(missingSecretReferences()))),
  269. },
  270. },
  271. "new serviceaccount with missing secrets and a local secret in the cache": {
  272. ClientObjects: []runtime.Object{serviceAccount(missingSecretReferences())},
  273. AddedServiceAccount: serviceAccount(tokenSecretReferences()),
  274. AddedSecretLocal: serviceAccountTokenSecret(),
  275. ExpectedActions: []core.Action{},
  276. },
  277. "new serviceaccount with non-token secrets": {
  278. ClientObjects: []runtime.Object{serviceAccount(regularSecretReferences()), opaqueSecret()},
  279. AddedServiceAccount: serviceAccount(regularSecretReferences()),
  280. ExpectedActions: []core.Action{
  281. core.NewGetAction(schema.GroupVersionResource{Version: "v1", Resource: "serviceaccounts"}, metav1.NamespaceDefault, "default"),
  282. core.NewCreateAction(schema.GroupVersionResource{Version: "v1", Resource: "secrets"}, metav1.NamespaceDefault, createdTokenSecret()),
  283. core.NewUpdateAction(schema.GroupVersionResource{Version: "v1", Resource: "serviceaccounts"}, metav1.NamespaceDefault, serviceAccount(addTokenSecretReference(regularSecretReferences()))),
  284. },
  285. },
  286. "new serviceaccount with token secrets": {
  287. ClientObjects: []runtime.Object{serviceAccount(tokenSecretReferences()), serviceAccountTokenSecret()},
  288. ExistingSecrets: []*v1.Secret{serviceAccountTokenSecret()},
  289. AddedServiceAccount: serviceAccount(tokenSecretReferences()),
  290. ExpectedActions: []core.Action{},
  291. },
  292. "new serviceaccount with no secrets with resource conflict": {
  293. ClientObjects: []runtime.Object{updatedServiceAccount(emptySecretReferences()), createdTokenSecret()},
  294. IsAsync: true,
  295. MaxRetries: 1,
  296. AddedServiceAccount: serviceAccount(emptySecretReferences()),
  297. ExpectedActions: []core.Action{
  298. core.NewGetAction(schema.GroupVersionResource{Version: "v1", Resource: "serviceaccounts"}, metav1.NamespaceDefault, "default"),
  299. core.NewGetAction(schema.GroupVersionResource{Version: "v1", Resource: "serviceaccounts"}, metav1.NamespaceDefault, "default"),
  300. },
  301. },
  302. "updated serviceaccount with no secrets": {
  303. ClientObjects: []runtime.Object{serviceAccount(emptySecretReferences())},
  304. UpdatedServiceAccount: serviceAccount(emptySecretReferences()),
  305. ExpectedActions: []core.Action{
  306. core.NewGetAction(schema.GroupVersionResource{Version: "v1", Resource: "serviceaccounts"}, metav1.NamespaceDefault, "default"),
  307. core.NewCreateAction(schema.GroupVersionResource{Version: "v1", Resource: "secrets"}, metav1.NamespaceDefault, createdTokenSecret()),
  308. core.NewUpdateAction(schema.GroupVersionResource{Version: "v1", Resource: "serviceaccounts"}, metav1.NamespaceDefault, serviceAccount(addTokenSecretReference(emptySecretReferences()))),
  309. },
  310. },
  311. "updated serviceaccount with missing secrets": {
  312. ClientObjects: []runtime.Object{serviceAccount(missingSecretReferences())},
  313. UpdatedServiceAccount: serviceAccount(missingSecretReferences()),
  314. ExpectedActions: []core.Action{
  315. core.NewGetAction(schema.GroupVersionResource{Version: "v1", Resource: "serviceaccounts"}, metav1.NamespaceDefault, "default"),
  316. core.NewCreateAction(schema.GroupVersionResource{Version: "v1", Resource: "secrets"}, metav1.NamespaceDefault, createdTokenSecret()),
  317. core.NewUpdateAction(schema.GroupVersionResource{Version: "v1", Resource: "serviceaccounts"}, metav1.NamespaceDefault, serviceAccount(addTokenSecretReference(missingSecretReferences()))),
  318. },
  319. },
  320. "updated serviceaccount with non-token secrets": {
  321. ClientObjects: []runtime.Object{serviceAccount(regularSecretReferences()), opaqueSecret()},
  322. UpdatedServiceAccount: serviceAccount(regularSecretReferences()),
  323. ExpectedActions: []core.Action{
  324. core.NewGetAction(schema.GroupVersionResource{Version: "v1", Resource: "serviceaccounts"}, metav1.NamespaceDefault, "default"),
  325. core.NewCreateAction(schema.GroupVersionResource{Version: "v1", Resource: "secrets"}, metav1.NamespaceDefault, createdTokenSecret()),
  326. core.NewUpdateAction(schema.GroupVersionResource{Version: "v1", Resource: "serviceaccounts"}, metav1.NamespaceDefault, serviceAccount(addTokenSecretReference(regularSecretReferences()))),
  327. },
  328. },
  329. "updated serviceaccount with token secrets": {
  330. ExistingSecrets: []*v1.Secret{serviceAccountTokenSecret()},
  331. UpdatedServiceAccount: serviceAccount(tokenSecretReferences()),
  332. ExpectedActions: []core.Action{},
  333. },
  334. "updated serviceaccount with no secrets with resource conflict": {
  335. ClientObjects: []runtime.Object{updatedServiceAccount(emptySecretReferences())},
  336. IsAsync: true,
  337. MaxRetries: 1,
  338. UpdatedServiceAccount: serviceAccount(emptySecretReferences()),
  339. ExpectedActions: []core.Action{
  340. core.NewGetAction(schema.GroupVersionResource{Version: "v1", Resource: "serviceaccounts"}, metav1.NamespaceDefault, "default"),
  341. core.NewGetAction(schema.GroupVersionResource{Version: "v1", Resource: "serviceaccounts"}, metav1.NamespaceDefault, "default"),
  342. },
  343. },
  344. "deleted serviceaccount with no secrets": {
  345. DeletedServiceAccount: serviceAccount(emptySecretReferences()),
  346. ExpectedActions: []core.Action{},
  347. },
  348. "deleted serviceaccount with missing secrets": {
  349. DeletedServiceAccount: serviceAccount(missingSecretReferences()),
  350. ExpectedActions: []core.Action{},
  351. },
  352. "deleted serviceaccount with non-token secrets": {
  353. ClientObjects: []runtime.Object{opaqueSecret()},
  354. DeletedServiceAccount: serviceAccount(regularSecretReferences()),
  355. ExpectedActions: []core.Action{},
  356. },
  357. "deleted serviceaccount with token secrets": {
  358. ClientObjects: []runtime.Object{serviceAccountTokenSecret()},
  359. ExistingSecrets: []*v1.Secret{serviceAccountTokenSecret()},
  360. DeletedServiceAccount: serviceAccount(tokenSecretReferences()),
  361. ExpectedActions: []core.Action{
  362. core.NewDeleteAction(schema.GroupVersionResource{Version: "v1", Resource: "secrets"}, metav1.NamespaceDefault, "token-secret-1"),
  363. },
  364. },
  365. "added secret without serviceaccount": {
  366. ClientObjects: []runtime.Object{serviceAccountTokenSecret()},
  367. AddedSecret: serviceAccountTokenSecret(),
  368. ExpectedActions: []core.Action{
  369. core.NewGetAction(schema.GroupVersionResource{Version: "v1", Resource: "serviceaccounts"}, metav1.NamespaceDefault, "default"),
  370. core.NewDeleteAction(schema.GroupVersionResource{Version: "v1", Resource: "secrets"}, metav1.NamespaceDefault, "token-secret-1"),
  371. },
  372. },
  373. "added secret with serviceaccount": {
  374. ExistingServiceAccount: serviceAccount(tokenSecretReferences()),
  375. AddedSecret: serviceAccountTokenSecret(),
  376. ExpectedActions: []core.Action{},
  377. },
  378. "added token secret without token data": {
  379. ClientObjects: []runtime.Object{serviceAccountTokenSecretWithoutTokenData()},
  380. ExistingServiceAccount: serviceAccount(tokenSecretReferences()),
  381. AddedSecret: serviceAccountTokenSecretWithoutTokenData(),
  382. ExpectedActions: []core.Action{
  383. core.NewGetAction(schema.GroupVersionResource{Version: "v1", Resource: "secrets"}, metav1.NamespaceDefault, "token-secret-1"),
  384. core.NewUpdateAction(schema.GroupVersionResource{Version: "v1", Resource: "secrets"}, metav1.NamespaceDefault, serviceAccountTokenSecret()),
  385. },
  386. },
  387. "added token secret without ca data": {
  388. ClientObjects: []runtime.Object{serviceAccountTokenSecretWithoutCAData()},
  389. ExistingServiceAccount: serviceAccount(tokenSecretReferences()),
  390. AddedSecret: serviceAccountTokenSecretWithoutCAData(),
  391. ExpectedActions: []core.Action{
  392. core.NewGetAction(schema.GroupVersionResource{Version: "v1", Resource: "secrets"}, metav1.NamespaceDefault, "token-secret-1"),
  393. core.NewUpdateAction(schema.GroupVersionResource{Version: "v1", Resource: "secrets"}, metav1.NamespaceDefault, serviceAccountTokenSecret()),
  394. },
  395. },
  396. "added token secret with mismatched ca data": {
  397. ClientObjects: []runtime.Object{serviceAccountTokenSecretWithCAData([]byte("mismatched"))},
  398. ExistingServiceAccount: serviceAccount(tokenSecretReferences()),
  399. AddedSecret: serviceAccountTokenSecretWithCAData([]byte("mismatched")),
  400. ExpectedActions: []core.Action{
  401. core.NewGetAction(schema.GroupVersionResource{Version: "v1", Resource: "secrets"}, metav1.NamespaceDefault, "token-secret-1"),
  402. core.NewUpdateAction(schema.GroupVersionResource{Version: "v1", Resource: "secrets"}, metav1.NamespaceDefault, serviceAccountTokenSecret()),
  403. },
  404. },
  405. "added token secret without namespace data": {
  406. ClientObjects: []runtime.Object{serviceAccountTokenSecretWithoutNamespaceData()},
  407. ExistingServiceAccount: serviceAccount(tokenSecretReferences()),
  408. AddedSecret: serviceAccountTokenSecretWithoutNamespaceData(),
  409. ExpectedActions: []core.Action{
  410. core.NewGetAction(schema.GroupVersionResource{Version: "v1", Resource: "secrets"}, metav1.NamespaceDefault, "token-secret-1"),
  411. core.NewUpdateAction(schema.GroupVersionResource{Version: "v1", Resource: "secrets"}, metav1.NamespaceDefault, serviceAccountTokenSecret()),
  412. },
  413. },
  414. "added token secret with custom namespace data": {
  415. ClientObjects: []runtime.Object{serviceAccountTokenSecretWithNamespaceData([]byte("custom"))},
  416. ExistingServiceAccount: serviceAccount(tokenSecretReferences()),
  417. AddedSecret: serviceAccountTokenSecretWithNamespaceData([]byte("custom")),
  418. ExpectedActions: []core.Action{
  419. // no update is performed... the custom namespace is preserved
  420. },
  421. },
  422. "updated secret without serviceaccount": {
  423. ClientObjects: []runtime.Object{serviceAccountTokenSecret()},
  424. UpdatedSecret: serviceAccountTokenSecret(),
  425. ExpectedActions: []core.Action{
  426. core.NewGetAction(schema.GroupVersionResource{Version: "v1", Resource: "serviceaccounts"}, metav1.NamespaceDefault, "default"),
  427. core.NewDeleteAction(schema.GroupVersionResource{Version: "v1", Resource: "secrets"}, metav1.NamespaceDefault, "token-secret-1"),
  428. },
  429. },
  430. "updated secret with serviceaccount": {
  431. ExistingServiceAccount: serviceAccount(tokenSecretReferences()),
  432. UpdatedSecret: serviceAccountTokenSecret(),
  433. ExpectedActions: []core.Action{},
  434. },
  435. "updated token secret without token data": {
  436. ClientObjects: []runtime.Object{serviceAccountTokenSecretWithoutTokenData()},
  437. ExistingServiceAccount: serviceAccount(tokenSecretReferences()),
  438. UpdatedSecret: serviceAccountTokenSecretWithoutTokenData(),
  439. ExpectedActions: []core.Action{
  440. core.NewGetAction(schema.GroupVersionResource{Version: "v1", Resource: "secrets"}, metav1.NamespaceDefault, "token-secret-1"),
  441. core.NewUpdateAction(schema.GroupVersionResource{Version: "v1", Resource: "secrets"}, metav1.NamespaceDefault, serviceAccountTokenSecret()),
  442. },
  443. },
  444. "updated token secret without ca data": {
  445. ClientObjects: []runtime.Object{serviceAccountTokenSecretWithoutCAData()},
  446. ExistingServiceAccount: serviceAccount(tokenSecretReferences()),
  447. UpdatedSecret: serviceAccountTokenSecretWithoutCAData(),
  448. ExpectedActions: []core.Action{
  449. core.NewGetAction(schema.GroupVersionResource{Version: "v1", Resource: "secrets"}, metav1.NamespaceDefault, "token-secret-1"),
  450. core.NewUpdateAction(schema.GroupVersionResource{Version: "v1", Resource: "secrets"}, metav1.NamespaceDefault, serviceAccountTokenSecret()),
  451. },
  452. },
  453. "updated token secret with mismatched ca data": {
  454. ClientObjects: []runtime.Object{serviceAccountTokenSecretWithCAData([]byte("mismatched"))},
  455. ExistingServiceAccount: serviceAccount(tokenSecretReferences()),
  456. UpdatedSecret: serviceAccountTokenSecretWithCAData([]byte("mismatched")),
  457. ExpectedActions: []core.Action{
  458. core.NewGetAction(schema.GroupVersionResource{Version: "v1", Resource: "secrets"}, metav1.NamespaceDefault, "token-secret-1"),
  459. core.NewUpdateAction(schema.GroupVersionResource{Version: "v1", Resource: "secrets"}, metav1.NamespaceDefault, serviceAccountTokenSecret()),
  460. },
  461. },
  462. "updated token secret without namespace data": {
  463. ClientObjects: []runtime.Object{serviceAccountTokenSecretWithoutNamespaceData()},
  464. ExistingServiceAccount: serviceAccount(tokenSecretReferences()),
  465. UpdatedSecret: serviceAccountTokenSecretWithoutNamespaceData(),
  466. ExpectedActions: []core.Action{
  467. core.NewGetAction(schema.GroupVersionResource{Version: "v1", Resource: "secrets"}, metav1.NamespaceDefault, "token-secret-1"),
  468. core.NewUpdateAction(schema.GroupVersionResource{Version: "v1", Resource: "secrets"}, metav1.NamespaceDefault, serviceAccountTokenSecret()),
  469. },
  470. },
  471. "updated token secret with custom namespace data": {
  472. ClientObjects: []runtime.Object{serviceAccountTokenSecretWithNamespaceData([]byte("custom"))},
  473. ExistingServiceAccount: serviceAccount(tokenSecretReferences()),
  474. UpdatedSecret: serviceAccountTokenSecretWithNamespaceData([]byte("custom")),
  475. ExpectedActions: []core.Action{
  476. // no update is performed... the custom namespace is preserved
  477. },
  478. },
  479. "deleted secret without serviceaccount": {
  480. DeletedSecret: serviceAccountTokenSecret(),
  481. ExpectedActions: []core.Action{},
  482. },
  483. "deleted secret with serviceaccount with reference": {
  484. ClientObjects: []runtime.Object{serviceAccount(tokenSecretReferences())},
  485. ExistingServiceAccount: serviceAccount(tokenSecretReferences()),
  486. DeletedSecret: serviceAccountTokenSecret(),
  487. ExpectedActions: []core.Action{
  488. core.NewGetAction(schema.GroupVersionResource{Version: "v1", Resource: "serviceaccounts"}, metav1.NamespaceDefault, "default"),
  489. core.NewUpdateAction(schema.GroupVersionResource{Version: "v1", Resource: "serviceaccounts"}, metav1.NamespaceDefault, serviceAccount(emptySecretReferences())),
  490. },
  491. },
  492. "deleted secret with serviceaccount without reference": {
  493. ExistingServiceAccount: serviceAccount(emptySecretReferences()),
  494. DeletedSecret: serviceAccountTokenSecret(),
  495. ExpectedActions: []core.Action{
  496. core.NewGetAction(schema.GroupVersionResource{Version: "v1", Resource: "serviceaccounts"}, metav1.NamespaceDefault, "default"),
  497. },
  498. },
  499. }
  500. for k, tc := range testcases {
  501. klog.Infof(k)
  502. // Re-seed to reset name generation
  503. utilrand.Seed(1)
  504. generator := &testGenerator{Token: "ABC"}
  505. client := fake.NewSimpleClientset(tc.ClientObjects...)
  506. for _, reactor := range tc.Reactors {
  507. client.Fake.PrependReactor(reactor.verb, reactor.resource, reactor.reactor(t))
  508. }
  509. informers := informers.NewSharedInformerFactory(client, controller.NoResyncPeriodFunc())
  510. secretInformer := informers.Core().V1().Secrets().Informer()
  511. secrets := secretInformer.GetStore()
  512. serviceAccounts := informers.Core().V1().ServiceAccounts().Informer().GetStore()
  513. controller, err := NewTokensController(informers.Core().V1().ServiceAccounts(), informers.Core().V1().Secrets(), client, TokensControllerOptions{TokenGenerator: generator, RootCA: []byte("CA Data"), MaxRetries: tc.MaxRetries})
  514. if err != nil {
  515. t.Fatalf("error creating Tokens controller: %v", err)
  516. }
  517. if tc.ExistingServiceAccount != nil {
  518. serviceAccounts.Add(tc.ExistingServiceAccount)
  519. }
  520. for _, s := range tc.ExistingSecrets {
  521. secrets.Add(s)
  522. }
  523. if tc.AddedServiceAccount != nil {
  524. serviceAccounts.Add(tc.AddedServiceAccount)
  525. controller.queueServiceAccountSync(tc.AddedServiceAccount)
  526. }
  527. if tc.UpdatedServiceAccount != nil {
  528. serviceAccounts.Add(tc.UpdatedServiceAccount)
  529. controller.queueServiceAccountUpdateSync(nil, tc.UpdatedServiceAccount)
  530. }
  531. if tc.DeletedServiceAccount != nil {
  532. serviceAccounts.Delete(tc.DeletedServiceAccount)
  533. controller.queueServiceAccountSync(tc.DeletedServiceAccount)
  534. }
  535. if tc.AddedSecret != nil {
  536. secrets.Add(tc.AddedSecret)
  537. controller.queueSecretSync(tc.AddedSecret)
  538. }
  539. if tc.AddedSecretLocal != nil {
  540. controller.updatedSecrets.Mutation(tc.AddedSecretLocal)
  541. }
  542. if tc.UpdatedSecret != nil {
  543. secrets.Add(tc.UpdatedSecret)
  544. controller.queueSecretUpdateSync(nil, tc.UpdatedSecret)
  545. }
  546. if tc.DeletedSecret != nil {
  547. secrets.Delete(tc.DeletedSecret)
  548. controller.queueSecretSync(tc.DeletedSecret)
  549. }
  550. // This is the longest we'll wait for async tests
  551. timeout := time.Now().Add(30 * time.Second)
  552. waitedForAdditionalActions := false
  553. for {
  554. if controller.syncServiceAccountQueue.Len() > 0 {
  555. controller.syncServiceAccount()
  556. }
  557. if controller.syncSecretQueue.Len() > 0 {
  558. controller.syncSecret()
  559. }
  560. // The queues still have things to work on
  561. if controller.syncServiceAccountQueue.Len() > 0 || controller.syncSecretQueue.Len() > 0 {
  562. continue
  563. }
  564. // If we expect this test to work asynchronously...
  565. if tc.IsAsync {
  566. // if we're still missing expected actions within our test timeout
  567. if len(client.Actions()) < len(tc.ExpectedActions) && time.Now().Before(timeout) {
  568. // wait for the expected actions (without hotlooping)
  569. time.Sleep(time.Millisecond)
  570. continue
  571. }
  572. // if we exactly match our expected actions, wait a bit to make sure no other additional actions show up
  573. if len(client.Actions()) == len(tc.ExpectedActions) && !waitedForAdditionalActions {
  574. time.Sleep(time.Second)
  575. waitedForAdditionalActions = true
  576. continue
  577. }
  578. }
  579. break
  580. }
  581. if controller.syncServiceAccountQueue.Len() > 0 {
  582. t.Errorf("%s: unexpected items in service account queue: %d", k, controller.syncServiceAccountQueue.Len())
  583. }
  584. if controller.syncSecretQueue.Len() > 0 {
  585. t.Errorf("%s: unexpected items in secret queue: %d", k, controller.syncSecretQueue.Len())
  586. }
  587. actions := client.Actions()
  588. for i, action := range actions {
  589. if len(tc.ExpectedActions) < i+1 {
  590. t.Errorf("%s: %d unexpected actions: %+v", k, len(actions)-len(tc.ExpectedActions), actions[i:])
  591. break
  592. }
  593. expectedAction := tc.ExpectedActions[i]
  594. if !reflect.DeepEqual(expectedAction, action) {
  595. t.Errorf("%s:\nExpected:\n%s\ngot:\n%s", k, spew.Sdump(expectedAction), spew.Sdump(action))
  596. continue
  597. }
  598. }
  599. if len(tc.ExpectedActions) > len(actions) {
  600. t.Errorf("%s: %d additional expected actions", k, len(tc.ExpectedActions)-len(actions))
  601. for _, a := range tc.ExpectedActions[len(actions):] {
  602. t.Logf(" %+v", a)
  603. }
  604. }
  605. }
  606. }