-
-
Notifications
You must be signed in to change notification settings - Fork 154
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
fix cloud-init schema validation error when {{ growpart_str }} is not empty #432
base: main
Are you sure you want to change the base?
Conversation
…factoring Refactor Cluster Autoscaler installation
…st as a YAML structure instead of manipulating it as text
…ml-patching Fix patching autoscaler container
Co-authored-by: darthmaim <[email protected]>
…nto more-refactoring
fix installation with adding v before versions
Upgrade `crest` shard
Nginx default ingress class
Quality Gate passedIssues Measures |
Thanks for the PR! It seems that I moved that section to the wrong place in one recent update. Are you using microOS for your cluster? |
Yes, I’m using microOS for the cluster nodes. I had to hardcode an extra environment variable into ‘master_install_script.sh’ and generate a new binary to make the installation work, though. Maybe I’ll open a new issue with the details. |
That would be nice, thanks! |
Hi @EnDjeee, sorry for the long delay. Work has been keeping me busy, so I haven’t had much time to focus on the project until this week. Now that I’m back, I’ve been working on a lot of things, but among other things I had to do a force push to remove a file that accidentally got published. Unfortunately, this has caused some issues with existing PRs. Instead of trying to fix the conflicts, it might be simpler to close this PR and open a new one with the same changes. If you do that, I promise I’ll merge it soon :) |
{{ growpart_str }}, when is not empty, breaks cloud-init schema validation.
This is easily fixed by moving {{ growpart_str }} up in the template file.