-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Bump Azure Stack Terraform Provider for Go 1.18 #5865
Bump Azure Stack Terraform Provider for Go 1.18 #5865
Conversation
Let's make sure |
/retest |
The goal here is to ensure compatibility with Go 1.18, but not yet require it. |
Prior to this change, building the provider is failing similar to this:
|
dbbc837
to
c67e4a6
Compare
Sifted through the files and there was only the one go.mod file changed. Verified the version of sys that was updated. |
/lgtm |
c67e4a6
to
3e5d009
Compare
Needed to bump x/sys dependency due to golang/go#49219 to ensure Go 1.18 compatibility on darwin. This commit revendors our AzureStack fork to bring in: openshift/terraform-provider-azurestack#5 which updated x/sys with the following commands: go get -u golang.org/x/sys go mod tidy -compat=1.17 && go mod vendor
3e5d009
to
6badee4
Compare
Reran with |
/lgtm |
/test e2e-azurestack |
e2e-azurestack tests are successfully running terraform. failure is after bootstrap complete. /approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: patrickdillon 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 |
@patrickdillon: The following tests failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
Bump Azure Stack Terraform Provider for Go 1.18
Builds for Mac are failing with Go 1.18 due to an outdated dependency in the Azure Stack Terraform provider on x/sys: golang/go#49219
I tested this using
registry.ci.openshift.org/ocp/builder:rhel-8-golang-1.18-openshift-4.11
and it resolves the issue.