Skip to content
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

HCPE-921: Upgrade to Go 1.16 #104

Merged
merged 6 commits into from
Apr 16, 2021
Merged

HCPE-921: Upgrade to Go 1.16 #104

merged 6 commits into from
Apr 16, 2021

Conversation

bcmdarroch
Copy link
Contributor

🛠️ Description

Go 1.16 provides support for the darwin-arm64 platform, so users on M1 MacBooks can now use the HCP provider. 🎉

This PR also sets the min Go version to 1.15 in compliance with terraform-provider-sdk-v2.4.4.

🏗️ Acceptance tests

Output from acceptance testing:

$ make testacc

PASS
ok  	github.com/hashicorp/terraform-provider-hcp/internal/provider	1852.706s

@bcmdarroch bcmdarroch requested a review from a team April 16, 2021 00:12
## 0.4.1 (April 09, 2021)

FIXES:
- resource/consul_cluster: Set "computed=true" option for the vm size (#100)

## 0.4.0 (April 07, 2021)

⚠️ Note: There is an issue with this version of the HCP Provider in which existing Consul clusters that do not specify size will be recommended by Terraform to be recreated on the next terraform apply, resulting in potential data loss. Please upgrade to the patch v0.4.1 or beyond to avoid this issue. ⚠️
Copy link
Contributor

Choose a reason for hiding this comment

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

👍 this is great!

README.md Outdated
@@ -11,6 +11,7 @@ The HashiCorp Cloud Platform (HCP) Terraform Provider is a plugin for Terraform
## Requirements

- [Terraform](https://www.terraform.io/downloads.html) >= 0.12.x
- [Go](https://golang.org/doc/install) >= 1.15
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we move this to the contributing/README.md instead, since users will not need to have Go installed, only developers

@bcmdarroch bcmdarroch merged commit ba2e18a into main Apr 16, 2021
@bcmdarroch bcmdarroch deleted the bump-go-1.16 branch April 16, 2021 21:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants