-
Notifications
You must be signed in to change notification settings - Fork 60
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
change default to unset storageclass #235
Conversation
Signed-off-by: Arlan Lloyd <arlanlloyd@gmail.com>
@meilisearch sync-manifest |
I was not aware of the And I think the change makes sense as well |
correct. ~ is shorthand for null. link to yaml source http://yaml.org/spec/1.2/spec.html#id2805071 |
I see the tests failed. Looks like it expects a version to be bumped. I am down to increment it, but figured that would be best to a release process. |
Hi @acelinkio can you update the chart version? So I can merge this PR? |
Updated version to 0.9.1 |
@meilisearch sync-manifest |
bors merge |
Build succeeded: |
Pull Request
Related issue
N/A
What does this PR do?
PR checklist
Please check if your PR fulfills the following requirements:
Thank you so much for contributing to Meilisearch!