-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
Return a clearer error when terraform is used on an unsupported provider #11953
Conversation
It would be good to extend |
@johngmyers I added completion for |
@rifelpet You use |
@@ -89,6 +89,8 @@ var ( | |||
OldestSupportedKubernetesVersion = "1.17.0" | |||
// OldestRecommendedKubernetesVersion is the oldest kubernetes version that is not deprecated in Kops | |||
OldestRecommendedKubernetesVersion = "1.19.0" | |||
// TerraformCloudProviders is the list of cloud providers with terraform target support | |||
TerraformCloudProviders = []kops.CloudProviderID{kops.CloudProviderAWS, kops.CloudProviderGCE, kops.CloudProviderALI} |
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.
Could use a sets.String()
/retest Review the full test history for this PR. Silence the bot with an |
2 similar comments
/retest Review the full test history for this PR. Silence the bot with an |
/retest Review the full test history for this PR. Silence the bot with an |
/hold |
/hold cancel |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: johngmyers The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
ref: #11951
How I determined the list of supported providers: