dashboard icon indicating copy to clipboard operation
dashboard copied to clipboard

Fix Issue #24: Manually Generate Secret for K8s v1.24+

Open ada2468 opened this issue 3 years ago • 4 comments

What type of PR is this? /kind bug

What this PR does / why we need it: The background story is that since K8s v1.24+, service account by default doesn't create secret automatically. As the result, if we follow the old instruction, we can't get a login token, because there is no secret present.

Potential Solutions:

  1. Use K8s v1.23.
  2. Manually create a secret. (This PR)
  3. Adapt to new API, TokenRequest API

More discussion can be found here: https://github.com/karmada-io/karmada/issues/1961 https://github.com/karmada-io/karmada/pull/1972 https://itnext.io/big-change-in-k8s-1-24-about-serviceaccounts-and-their-secrets-4b909a4af4e0

My Solution: The second one. This assumes you use k8s v1.24+.

Add instruction in README.me for modifying karmada-dashboard-role.yaml, to manually create secret for us to see the login token. Then we can login the dashboard.

Which issue(s) this PR fixes: Fixes #24

Signed-off-by: Jun Xiang [email protected]

ada2468 avatar Aug 31 '22 23:08 ada2468

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: To complete the pull request process, please assign rainbowmango after the PR has been reviewed. You can assign the PR to them by writing /assign @rainbowmango in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

karmada-bot avatar Aug 31 '22 23:08 karmada-bot

@RainbowMango I propose a fix for issue #24. Please take a look.

ada2468 avatar Aug 31 '22 23:08 ada2468

I use your yaml, but can not create secret

image

it is very unbelievable

haiker2011 avatar Nov 28 '23 09:11 haiker2011

I test find the secret dashboardsecret is deleted immediately after created

image

haiker2011 avatar Nov 29 '23 02:11 haiker2011

/close

As we are currently Refactoring the dashboard and the readme file has been significantly changed. Thank you @ada2468 all the same.

RainbowMango avatar Sep 30 '24 03:09 RainbowMango

@RainbowMango: Closed this PR.

In response to this:

/close

As we are currently Refactoring the dashboard and the readme file has been significantly changed. Thank you @ada2468 all the same.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

karmada-bot avatar Sep 30 '24 03:09 karmada-bot