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

Vcluster secrets labelling is missing #2283

Open
irizzant opened this issue Nov 20, 2024 · 3 comments · May be fixed by #2330
Open

Vcluster secrets labelling is missing #2283

irizzant opened this issue Nov 20, 2024 · 3 comments · May be fixed by #2330
Labels

Comments

@irizzant
Copy link

What happened?

Vcluster secrets are created without any labels

What did you expect to happen?

Vcluster to add labels to identify secrets

How can we reproduce it (as minimally and precisely as possible)?

Just create a vcluster

Anything else we need to know?

No response

Host cluster Kubernetes version

$ kubectl version
# paste output here

Client Version: v1.31.2
Kustomize Version: v5.4.2
Server Version: v1.31.2-eks-7f9249a

vcluster version

$ vcluster --version
# paste output here

vcluster version 0.20.4

VCluster Config

# My vcluster.yaml / values.yaml here
@irizzant irizzant changed the title Vcluster secret labelling Vcluster secrets labelling is missing Nov 20, 2024
@maheshbaliga
Copy link
Contributor

Hi @irizzant. Could you share more details about the issue/enhancement? Info like expected vs. actual behaviour, errors, or steps to reproduce would help.

@irizzant
Copy link
Author

irizzant commented Dec 6, 2024

Hi @maheshbaliga , when you create vclusters in a namespace there are some secrets created.

E.g. for a vcluster called example you have a secret called vc-example-vcluster containing the admin kubeconfig.

That secret does not have any metadata.labels applied to it thus making it difficult to choose among other kubernetes secrets in the same namespace.

There should be a label added to those secrets in metadata.labels, something like
vcluster=name of the vcluster
vcluster-kubeconfig=true

to help identify and chose those secrets in a kubernetes style without using the secret name

@maheshbaliga
Copy link
Contributor

Thanks @irizzant for providing the additional details about the issue. I was able to replicate the scenario based on the information you shared. I’ll raise a PR shortly to propose a fix; let’s wait for the maintainers to then review it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants