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

chore(docs): Update docs to reflect existence of patch.tolerations #175

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

andrewjeffree
Copy link

Purpose

Observed today when deploying this that the helm docs nor the values files imply that this is a supported option but it is indeed there inside the patch job definitions.

Changes

  • added patch.tolerations to values.yaml and values.schema.json
  • updated Readme

Testing

Code Author Checklist

  • Bump the chart version (Chart.yaml -> version)
  • JSON Schema updated (values.schema.json)
  • Update README.md via helm-docs (or make prep)
  • Run pre-commit run --all-files via pre-commit (or make prep)
  • Update Artifacthub annotation (Chart.yaml -> artifacthub.io/changes, artifacthub.io/images)

Observed today when deploying this that the helm docs nor the values
files imply that this is a supported option but it is indeed there
inside the patch job definitions.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant