Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create secrets for serviceaccounts #2121

Merged
merged 1 commit into from
Jun 30, 2022

Conversation

skitt
Copy link
Member

@skitt skitt commented Jun 30, 2022

K8s 1.24+ doesn't autocreate secret for serviceaccount. This
breaks submariner deployments which expects secret to be created
and associated to the corresponding serviceaccount.

The fix ensures the Secret of type service-account-token exists,
is annotated with ServiceAccount.Name, and is added to Secrets list in
the ServiceAccount.

Signed-off-by: Vishal Thapar [email protected]

K8s 1.24+ doesn't autocreate secret for serviceaccount. This
breaks submariner deployments which expects secret to be created
and associated to the corresponding serviceaccount.

The fix ensures the Secret of type service-account-token exists,
is annotated with ServiceAccount.Name, and is added to Secrets list in
the ServiceAccount.

Fixes #102

Signed-off-by: Vishal Thapar <[email protected]>
@submariner-bot
Copy link
Contributor

🤖 Created branch: z_pr2121/skitt/backport-subctl-125
🚀 Full E2E won't run until the "ready-to-test" label is applied. I will add it automatically once the PR has 2 approvals, or you can add it manually.

@submariner-bot submariner-bot added the ready-to-test When a PR is ready for full E2E testing label Jun 30, 2022
@nyechiel
Copy link
Member

@tpantelis ^^

@tpantelis tpantelis merged commit 86c7124 into submariner-io:release-0.12 Jun 30, 2022
@submariner-bot
Copy link
Contributor

🤖 Closed branches: [z_pr2121/skitt/backport-subctl-125]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-test When a PR is ready for full E2E testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants