karmada icon indicating copy to clipboard operation
karmada copied to clipboard

standardize the naming of karmada secrets in operator method

Open chaosi-zju opened this issue 1 year ago • 2 comments

What type of PR is this?

/kind cleanup

What this PR does / why we need it:

In karmada, here are two important secrets, which is mount by most karmada components. One is karmada-cert, which contains a series of cert files like ca.crt, apiserver.crt and so on; another is karmada-kubeconfig, which contains a kubeconfig of karmada-apiserver.

However, in different installation methods, we used inconsistent secret naming or file path naming, which can potentially cause some unnecessary problems, detail refer to https://github.com/karmada-io/karmada/issues/5363.

This PR aims to standardize the naming of karmada secrets in operator installation method.

Which issue(s) this PR fixes:

Fixes part of #5363

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

karmada-operator: standardize the naming of karmada secrets in operator installation method

chaosi-zju avatar Sep 20 '24 08:09 chaosi-zju

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Once this PR has been reviewed and has the lgtm label, please assign lonelycz for approval. For more information see the Kubernetes Code Review Process.

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 Sep 20 '24 08:09 karmada-bot

:warning: Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 0% with 163 lines in your changes missing coverage. Please review.

Project coverage is 33.93%. Comparing base (b8edec1) to head (4d827c4).

Files with missing lines Patch % Lines
operator/pkg/controlplane/controlplane.go 0.00% 40 Missing :warning:
operator/pkg/tasks/init/upload.go 0.00% 29 Missing :warning:
operator/pkg/controlplane/apiserver/apiserver.go 0.00% 22 Missing :warning:
operator/pkg/controlplane/etcd/etcd.go 0.00% 14 Missing :warning:
operator/pkg/controlplane/search/search.go 0.00% 14 Missing :warning:
operator/pkg/certs/certs.go 0.00% 9 Missing :warning:
.../pkg/controlplane/metricsadapter/metricsadapter.go 0.00% 8 Missing :warning:
operator/pkg/controlplane/webhook/webhook.go 0.00% 8 Missing :warning:
operator/pkg/tasks/deinit/cert.go 0.00% 7 Missing :warning:
operator/pkg/tasks/init/cert.go 0.00% 7 Missing :warning:
... and 3 more

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #5573   +/-   ##
=======================================
  Coverage   33.93%   33.93%           
=======================================
  Files         645      645           
  Lines       44857    44849    -8     
=======================================
+ Hits        15220    15221    +1     
+ Misses      28479    28471    -8     
+ Partials     1158     1157    -1     
Flag Coverage Δ
unittests 33.93% <0.00%> (+<0.01%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov-commenter avatar Sep 20 '24 09:09 codecov-commenter