Skip to content

Commit 258f1d5

Browse files
Merge pull request #1909 from sanchezl/no-token-secrets-01
OCPBUILD-13: UPSTREAM: <carry>: fix [sig-auth] ServiceAccounts no secret-based service account token should be auto-generated
2 parents 5fcea52 + 7921d3d commit 258f1d5

File tree

1 file changed

+11
-4
lines changed

1 file changed

+11
-4
lines changed

test/e2e/auth/service_accounts.go

+11-4
Original file line numberDiff line numberDiff line change
@@ -65,10 +65,17 @@ var _ = SIGDescribe("ServiceAccounts", func() {
6565
sa, err := f.ClientSet.CoreV1().ServiceAccounts(f.Namespace.Name).Get(ctx, "default", metav1.GetOptions{})
6666
framework.ExpectNoError(err)
6767

68-
// TODO: OpenShift creates a Secret with a prefix "default-dockercfg", is the
69-
// following assertion valid?
70-
gomega.Expect(sa.Secrets).To(gomega.HaveLen(1))
71-
gomega.Expect(sa.Secrets[0].Name).To(gomega.HavePrefix("default-dockercfg"))
68+
// TODO: Ignore the image pull secret that OpenShift sometimes creates and adds to the list of Secrets.
69+
// TODO: This patch can be removed once OpenShift stops adding the pull secret to the list of secrets in 4.16.
70+
secrets := sa.DeepCopy().Secrets
71+
sa.Secrets = nil
72+
for _, s := range secrets {
73+
if strings.HasPrefix(s.Name, "default-dockercfg") {
74+
continue
75+
}
76+
sa.Secrets = append(sa.Secrets, s)
77+
}
78+
gomega.Expect(sa.Secrets).To(gomega.BeEmpty())
7279
}
7380
})
7481

0 commit comments

Comments
 (0)