-
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
terraform: update aws, azure, google providers to latest #2396
Conversation
/test e2e-gcp /test e2e-azure |
Lots of:
Needs a Go bump? |
0.12.9 version brings new new dependency on alibaba-cloud-sdk, which requires overrides to work correctly with aws-sdk-go The other overrides are almost similar
2e75567
to
a4cecfd
Compare
Build FAILURE, see build http://10.8.144.11:8080/job/dev-tools/1165/ |
/test e2e-gcp /test e2e-azure |
/retest |
/lgtm baremetal IPI is working fine with this PR, got a good install locally |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: abhinavdahiya, stbenjam 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 |
/retest |
/hold /test e2e-azure |
/test e2e-gcp |
New changes are detected. LGTM label has been removed. |
/test e2e-azure /test e2e-gcp |
pushed 68631f5 to fix the deprecation notices and errors in https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/openshift_installer/2396/pull-ci-openshift-installer-master-e2e-azure/191 left the private zone one, as that needs more work for now. |
need few fixes in the azure provider to use the private_dns resources that were introduced in 1.34.0 hashicorp/terraform-provider-azurerm#4452 |
68631f5
to
b337462
Compare
/test e2e-azure /test e2e-gcp |
/retest |
/test e2e-azure |
/test e2e-azure looks like it failed with a boskos error |
….9 terraform aws: =2.29.0 https://github.com/terraform-providers/terraform-provider-aws/tree/v2.29.0 azure: =1.34.0 https://github.com/terraform-providers/terraform-provider-azurerm/tree/v1.34.0 google: =2.15.0 https://github.com/terraform-providers/terraform-provider-google/tree/v2.15.0 leaves the ignition as-is as it has the forked patch leaves openstack, ironic as-is as they are tighly coupled to each other due to dependency on gophercloud lib
27cd1fb
to
365dfda
Compare
/test e2e-gcp /test e2e-azure |
still need fixes from hashicorp/terraform-provider-azurerm#4501 and hashicorp/terraform-provider-azurerm#4453 for azure provider |
/refresh |
/test all |
/skip |
/test all |
/test e2e-azure |
/test e2e-gcp |
@abhinavdahiya: The following tests failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. 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. |
/close |
@abhinavdahiya: Closed this PR. In response to this:
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. |
aws:
=2.29.0
https://github.com/terraform-providers/terraform-provider-aws/tree/v2.29.0azure:
=1.34.0
https://github.com/terraform-providers/terraform-provider-azurerm/tree/v1.34.0google:
=2.15.0
https://github.com/terraform-providers/terraform-provider-google/tree/v2.15.0terraform:
=0.12.9
https://github.com/hashicorp/terraform/tree/v0.12.9/cc @openshift/openshift-team-installer @Fedosin @russellb