Skip to content

cluster: removed array of regions due to they could be changed #158

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

Merged
merged 1 commit into from
Feb 27, 2020

Conversation

PacoDw
Copy link
Contributor

@PacoDw PacoDw commented Feb 26, 2020

closes #157

Copy link
Collaborator

@shum shum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. We can defer the error handling to the API endpoint.

@PacoDw
Copy link
Contributor Author

PacoDw commented Feb 27, 2020

Thank you so much @shum !

Copy link
Contributor

@marinsalinas marinsalinas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@PacoDw PacoDw merged commit 72bddd0 into master Feb 27, 2020
@PacoDw PacoDw deleted the fix-#157 branch February 27, 2020 16:32
@kostyazgara
Copy link

Can you say me, when changes will be applied in terraform provider and I can use them?

@PacoDw
Copy link
Contributor Author

PacoDw commented Feb 27, 2020

@kostyazgara I think the new release coming in the next week, but for now, you can use the master branch to use this new change, to use it you can follow these steps:

  • Download the repository, make the following command to create the binary: go build
  • Once you get the binary you can cut and paste in your project root. (I think you will need to remove the release with you are working)
  • Later make: terraform init and that's it.
    Note: When the new release is done, you need to remove the current binary and make 'terraform init' to download automatically the last plugin version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can't create cluster ME_SOUTH_1 region
5 participants