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

helm "podAnnotations" cannot applied #477

Closed
githubeto opened this issue Oct 8, 2024 · 4 comments
Closed

helm "podAnnotations" cannot applied #477

githubeto opened this issue Oct 8, 2024 · 4 comments
Labels
bug Something isn't working

Comments

@githubeto
Copy link

githubeto commented Oct 8, 2024

Brief summary

helm chart values

podAnnotations:
  "karpenter.sh/do-not-disrupt": "true"

install commands:

helm upgrade k6-operator -f k6-operator-values.yaml grafana/k6-operator

error log:

Error: UPGRADE FAILED: values don't meet the specifications of the schema(s) in the following chart(s):
k6-operator:
- podAnnotations: Additional property karpenter.sh/do-not-disrupt is not allowed

k6-operator version or image

0.0.17

Helm chart version (if applicable)

k6-operator-3.9.0

TestRun / PrivateLoadZone YAML

not.

Other environment details (if applicable)

No response

Steps to reproduce the problem

Expected behaviour

helm install.

Actual behaviour

I also tried these.

podAnnotations:
  karpenter.sh/do-not-disrupt: "true"
podAnnotations:
  karpenter.sh/do-not-disrupt: true
@githubeto githubeto added the bug Something isn't working label Oct 8, 2024
@flowjo-shaunr
Copy link

I am also experiencing this issue. It seems that even the sample yaml file (customAnnotationsAndLabels.yaml) cannot be used with the values schema.

% helm template -f test.yaml grafana/k6-operator
coalesce.go:220: warning: cannot overwrite table with non table for k6-operator.manager.env (map[])
Error: values don't meet the specifications of the schema(s) in the following chart(s):
k6-operator:
- (root): Additional property certManager is not allowed
- customLabels: Additional property customized-labels is not allowed
- manager.readinessProbe: Additional property successThreshold is not allowed
- manager.readinessProbe: Additional property timeoutSeconds is not allowed
- manager.readinessProbe: Additional property failureThreshold is not allowed
- manager.readinessProbe: Additional property httpGet is not allowed
- manager.readinessProbe: Additional property initialDelaySeconds is not allowed
- manager.readinessProbe: Additional property periodSeconds is not allowed
- manager.env: Invalid type. Expected: object, given: array
- manager.image: Additional property name is not allowed
- manager.livenessProbe: Additional property failureThreshold is not allowed
- manager.livenessProbe: Additional property httpGet is not allowed
- manager.livenessProbe: Additional property initialDelaySeconds is not allowed
- manager.livenessProbe: Additional property periodSeconds is not allowed
- manager.livenessProbe: Additional property successThreshold is not allowed
- manager.livenessProbe: Additional property timeoutSeconds is not allowed
- podLabels: Additional property environment is not allowed
- podLabels: Additional property owner is not allowed
- podAnnotations: Additional property customized-annotation is not allowed
- affinity: Additional property nodeAffinity is not allowed
- authProxy.resources: Additional property limits is not allowed
- authProxy.resources: Additional property requests is not allowed
- customAnnotations: Additional property customized-annotation is not allowed
- nodeSelector: Additional property disktype is not allowed
- tolerations: Invalid type. Expected: object, given: array

Why is "additionalProperties": false for all of these objects (customAnnotations, customLabels, etc) without any properties defined?

@stawr93
Copy link

stawr93 commented Oct 22, 2024

Got same issue. Rolled back to helm chart version 3.8.0 - works fine

@yorugac
Copy link
Collaborator

yorugac commented Oct 25, 2024

👋 @githubeto, sorry for the delay; but I believe this is a duplicate of #471 ?

@yorugac
Copy link
Collaborator

yorugac commented Oct 31, 2024

@flowjo-shaunr,

  • (root): Additional property certManager is not allowed

This part was fixed in #480

The rest of errors is expected to be fixed the same as in #471 (comment)

So closing this in favor of #471. Please watch that one: I'll update there once a new release is up.

@yorugac yorugac closed this as completed Oct 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants