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

upgrade aws provider #8

Closed
wants to merge 6 commits into from

Conversation

AlyRagab
Copy link

@AlyRagab AlyRagab commented Dec 6, 2020

  • Upgrading the aws provider version to be "~> 3.0" because the current "2.0" does not work if we have another resources depend on version 3.0

  • Error :

Initializing provider plugins...

  • terraform.io/builtin/terraform is built in to Terraform
  • Finding hashicorp/null versions matching "~> 2.0"...
  • Finding hashicorp/aws versions matching "~> 3.0, ~> 2.0"...
  • Using hashicorp/null v2.1.2 from the shared cache directory

Error: Failed to query available provider packages

Could not retrieve the list of available versions for provider hashicorp/aws:
no available releases match the given constraints ~> 3.0, ~> 2.0

@AlyRagab AlyRagab requested a review from a team as a code owner December 6, 2020 00:48
@joe-niland
Copy link
Member

Hi @AlyRagab
We are using lower bound pinning going forward.

Could you please adjust this for Terraform and for the providers?

@joe-niland
Copy link
Member

Here's an example cloudposse/terraform-aws-codebuild#70

versions.tf Outdated
@@ -2,7 +2,7 @@ terraform {
required_version = ">= 0.12.0, < 0.14.0"
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
required_version = ">= 0.12.0, < 0.14.0"
required_version = ">= 0.12.0"

versions.tf Outdated
@@ -2,7 +2,7 @@ terraform {
required_version = ">= 0.12.0, < 0.14.0"

required_providers {
aws = "~> 2.0"
aws = "~> 3.0"
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
aws = "~> 3.0"
aws = ">= 2.0"

versions.tf Outdated
@@ -2,7 +2,7 @@ terraform {
required_version = ">= 0.12.0, < 0.14.0"

required_providers {
aws = "~> 2.0"
aws = "~> 3.0"
null = "~> 2.0"
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
null = "~> 2.0"
null = "=> 2.0"

Copy link
Member

@aknysh aknysh left a comment

Choose a reason for hiding this comment

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

thanks @AlyRagab , please see comments.
Please also run the following commands from the repo root:

make init
make github/init
make readme

@AlyRagab
Copy link
Author

AlyRagab commented Dec 6, 2020

Thanks @aknysh for your comments

@aknysh
Copy link
Member

aknysh commented Dec 6, 2020

/test all

Copy link
Member

@aknysh aknysh left a comment

Choose a reason for hiding this comment

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

@AlyRagab
Sorry, but we also need to update the test to use Go modules.
Can you please run these commands:

make init
make github/init
cd test/src/
go mod init github.com/cloudposse/$(cd ../../ && basename `pwd`)
go mod tidy
git rm Gopkg.*
git add go.*

Thanks a lot for your contribution.

@aknysh
Copy link
Member

aknysh commented Dec 6, 2020

/rebuild-readme

@aknysh
Copy link
Member

aknysh commented Dec 6, 2020

/test all

@joe-niland
Copy link
Member

/test all

@AlyRagab
Copy link
Author

AlyRagab commented Dec 8, 2020

/rebuild-readme

@AlyRagab
Copy link
Author

AlyRagab commented Dec 8, 2020

/test all

@AlyRagab
Copy link
Author

AlyRagab commented Dec 8, 2020

/test all

@Nuru Nuru mentioned this pull request Dec 21, 2020
@maximmi maximmi closed this in #9 Dec 22, 2020
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.

4 participants