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

[TF-10414] Upgrade modules to AWS V5 provider #316

Merged
merged 3 commits into from
Nov 14, 2023

Conversation

miguelhrocha
Copy link

Background

Some of the AWS APIs supported in V4 are going to get deprecated by EOY. Not to mention that some flaky situations have been spotted during the execution of modules relying on V4, which have been addressed on V5.

This PR upgrades the version constraints of the modules to rely on the AWS V5 provider.

Relates OR Closes #0000

How Has This Been Tested

I ran terraform init in each submodule to verify this is not introducing breaking syntax.

@miguelhrocha miguelhrocha marked this pull request as ready for review November 14, 2023 12:51
@miguelhrocha miguelhrocha requested a review from a team as a code owner November 14, 2023 12:51
@miguelhrocha
Copy link
Author

miguelhrocha commented Nov 14, 2023

/test all destroy=true

❌ Terraform AWS Public Active/Active (Legacy) - Test Report

🔗 Action Summary Page

  • ✅ Terraform Init

  • ✅ Terraform Validate

  • ❌ Terraform Apply

  • ❌ Run k6 Smoke Test

  • ✅ Terraform Destroy

✅ Terraform AWS Private Active/Active (FDO) - Test Report

🔗 Action Summary Page

  • ✅ Terraform Init

  • ✅ Terraform Validate

  • ✅ Terraform Apply

  • ✅ Run k6 Smoke Test

  • ✅ Terraform Destroy

✅ Terraform AWS Active/Active RHEL7 Proxy (FDO) - Test Report

🔗 Action Summary Page

  • ✅ Terraform Init

  • ✅ Terraform Validate

  • ✅ Terraform Apply

  • ✅ Run k6 Smoke Test

  • ✅ Terraform Destroy

❌ Terraform AWS Private TCP Active/Active (FDO) - Test Report

🔗 Action Summary Page

  • ✅ Terraform Init

  • ✅ Terraform Validate

  • ✅ Terraform Apply

  • ❌ Run k6 Smoke Test

  • ✅ Terraform Destroy

✅ Terraform AWS Public Active/Active (FDO) - Test Report

🔗 Action Summary Page

  • ✅ Terraform Init

  • ✅ Terraform Validate

  • ✅ Terraform Apply

  • ✅ Run k6 Smoke Test

  • ✅ Terraform Destroy

✅ Terraform AWS Private Active/Active (Legacy) - Test Report

🔗 Action Summary Page

  • ✅ Terraform Init

  • ✅ Terraform Validate

  • ✅ Terraform Apply

  • ✅ Run k6 Smoke Test

  • ✅ Terraform Destroy

❌ Terraform AWS Private TCP Active/Active (Legacy) - Test Report

🔗 Action Summary Page

  • ✅ Terraform Init

  • ✅ Terraform Validate

  • ✅ Terraform Apply

  • ❌ Run k6 Smoke Test

  • ✅ Terraform Destroy

✅ Terraform AWS Standalone Vault (FDO) - Test Report

🔗 Action Summary Page

  • ✅ Terraform Init

  • ✅ Terraform Validate

  • ✅ Terraform Apply

  • ✅ Run k6 Smoke Test

  • ✅ Terraform Destroy

✅ Terraform AWS Standalone Vault (Legacy) - Test Report

🔗 Action Summary Page

  • ✅ Terraform Init

  • ✅ Terraform Validate

  • ✅ Terraform Apply

  • ✅ Run k6 Smoke Test

  • ✅ Terraform Destroy

❌ Terraform AWS Active/Active RHEL7 Proxy (Legacy) - Test Report

🔗 Action Summary Page

  • ✅ Terraform Init

  • ✅ Terraform Validate

  • ✅ Terraform Apply

  • ❌ Run k6 Smoke Test

  • ✅ Terraform Destroy

@anniehedgpeth
Copy link
Contributor

anniehedgpeth commented Nov 14, 2023

/test public-active-active-replicated destroy=false

✅ Terraform AWS Public Active/Active (Legacy) - Test Report

🔗 Action Summary Page

  • ✅ Terraform Init

  • ✅ Terraform Validate

  • ✅ Terraform Apply

  • ✅ Run k6 Smoke Test

@anniehedgpeth
Copy link
Contributor

anniehedgpeth commented Nov 14, 2023

/test private-tcp-active-active destroy=false

❌ Terraform AWS Private TCP Active/Active (FDO) - Test Report

🔗 Action Summary Page

  • ✅ Terraform Init

  • ✅ Terraform Validate

  • ✅ Terraform Apply

  • ❌ Run k6 Smoke Test

❌ Terraform AWS Private TCP Active/Active (FDO) - Test Report

🔗 Action Summary Page

  • ✅ Terraform Init

  • ✅ Terraform Validate

  • ✅ Terraform Apply

  • ❌ Run k6 Smoke Test

@anniehedgpeth
Copy link
Contributor

anniehedgpeth commented Nov 14, 2023

/test private-tcp-active-active-replicated destroy=false

❌ Terraform AWS Private TCP Active/Active (Legacy) - Test Report

🔗 Action Summary Page

  • ✅ Terraform Init

  • ✅ Terraform Validate

  • ✅ Terraform Apply

  • ❌ Run k6 Smoke Test

@anniehedgpeth
Copy link
Contributor

anniehedgpeth commented Nov 14, 2023

/test active-active-rhel7-proxy-replicated destroy=false

❌ Terraform AWS Active/Active RHEL7 Proxy (Legacy) - Test Report

🔗 Action Summary Page

  • ✅ Terraform Init

  • ✅ Terraform Validate

  • ✅ Terraform Apply

  • ❌ Run k6 Smoke Test

Copy link
Contributor

@anniehedgpeth anniehedgpeth left a comment

Choose a reason for hiding this comment

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

The failing tests appear to be the result of cloud resource errors and replicated.com needing to be added to the no_proxy list, not due to any changes in this PR. A follow up PR will be made to address the failures.

@anniehedgpeth
Copy link
Contributor

anniehedgpeth commented Nov 14, 2023

/destroy active-active-rhel7-proxy-replicated

✅ Terraform AWS Active/Active RHEL7 Proxy (Replicated) - Destruction Report

🔗 Action Summary Page

  • ✅ Terraform Init

  • ✅ Terraform Destroy

@anniehedgpeth
Copy link
Contributor

anniehedgpeth commented Nov 14, 2023

/destroy private-tcp-active-active-replicated

✅ Terraform AWS Private TCP Active/Active (Replicated) - Destruction Report

🔗 Action Summary Page

  • ✅ Terraform Init

  • ✅ Terraform Destroy

@anniehedgpeth
Copy link
Contributor

anniehedgpeth commented Nov 14, 2023

/destroy private-tcp-active-active

✅ Terraform AWS Private TCP Active/Active (FDO) - Destruction Report

🔗 Action Summary Page

  • ✅ Terraform Init

  • ✅ Terraform Destroy

@anniehedgpeth
Copy link
Contributor

anniehedgpeth commented Nov 14, 2023

/destroy public-active-active-replicated

✅ Terraform AWS Public Active/Active (Replicated) - Destruction Report

🔗 Action Summary Page

  • ✅ Terraform Init

  • ✅ Terraform Destroy

@miguelhrocha miguelhrocha merged commit 2749b9c into main Nov 14, 2023
3 checks passed
@miguelhrocha miguelhrocha deleted the TF-10414-upgrade-to-aws-v5-provider branch November 14, 2023 16:05
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