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

Warnings when doing -f ./helm-values.yaml mirrord-operator metalbear/mirrord-operator #59

Closed
meowjesty opened this issue Apr 9, 2024 · 3 comments

Comments

@meowjesty
Copy link
Member

You should get these warnings:

W0409 17:39:21.686201   98566 warnings.go:70] unknown field "app.kubernetes.io/instance"
W0409 17:39:21.686237   98566 warnings.go:70] unknown field "app.kubernetes.io/managed-by"
W0409 17:39:21.686251   98566 warnings.go:70] unknown field "app.kubernetes.io/version"
W0409 17:39:21.686264   98566 warnings.go:70] unknown field "helm.sh/chart"
W0409 17:39:21.686276   98566 warnings.go:70] unknown field "metadata.labels:app.kubernetes.io/name"
NAME: mirrord-operator
LAST DEPLOYED: Tue Apr  9 17:39:21 2024
NAMESPACE: default
STATUS: deployed
REVISION: 1
TEST SUITE: None

When running helm install where the values.yaml file contains only:

license:
  key: "{whatever-you-want-to-put-here-as-key}"

I also saw these same warnings in a "fuller" values.yaml, but this is enough to trigger them.

@aviramha
Copy link
Member

Can you test if it's fixed with #61?

@meowjesty
Copy link
Member Author

helm install -f ./helm-values.yaml mirrord-operator metalbear/mirrord-operator                                                                                                
W0415 11:39:05.079690   40830 warnings.go:70] unknown field "app.kubernetes.io/instance"
W0415 11:39:05.079727   40830 warnings.go:70] unknown field "app.kubernetes.io/managed-by"
W0415 11:39:05.079739   40830 warnings.go:70] unknown field "app.kubernetes.io/version"
W0415 11:39:05.079747   40830 warnings.go:70] unknown field "helm.sh/chart"
W0415 11:39:05.079756   40830 warnings.go:70] unknown field "metadata.labels:app.kubernetes.io/name"
NAME: mirrord-operator
LAST DEPLOYED: Mon Apr 15 11:39:04 2024
NAMESPACE: default
STATUS: deployed
REVISION: 1
TEST SUITE: None

On latest main.

@aviramha
Copy link
Member

Fixed in #65

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

No branches or pull requests

2 participants