-
Notifications
You must be signed in to change notification settings - Fork 11
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
Add k8s cluster resource #140
Conversation
Fix error message when invalid k8s release is selected. Always check if keys of ParametersSchema exists.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Gets a "looks good to me so far" 😄 Just a couple of things that shouldn't be in the interface, I think.
@nvthongswansea @itakouna What release do we target with this 1.9.0? And how soon should we release this?
@@ -77,6 +77,7 @@ func Provider() terraform.ResourceProvider { | |||
"gridscale_snapshotschedule": resourceGridscaleStorageSnapshotSchedule(), | |||
"gridscale_backupschedule": resourceGridscaleStorageBackupSchedule(), | |||
"gridscale_paas": resourceGridscalePaaS(), | |||
"gridscale_k8s": resourceGridscaleK8s(), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@itakouna please take a look. Thanks!
BTW, I really like the validation errors above. More of that please, Sir 💪 |
Ping @twiebe with a request for comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes LGTM
Add a new k8s resource to gridscale terraform provider #120.
What changes:
What it does:
k8s_release
(e.g. 1.19), instead of inputtingservice_template_uuid
. Optionally, the values ofk8s_release
can be retrieved via gscloud.