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-23439] Removing PostgreSQL Minor Version in AWS Release tests #345

Merged
merged 1 commit into from
Jan 21, 2025

Conversation

harshit410
Copy link
Contributor

CHANGELOG:

no-impact

Background

This PR removes the dependency on the minor version of PostgreSQL and only specifies the major version for the release tests. This change allows AWS RDS to automatically utilize the latest available minor version for the specified major version at the time of creation.

Summary

Currently, some of the release tests associated with the AWS Environment have the minor version of the PostgreSQL Engine hardcoded. Some AWS Release tests have failed due to Amazon RDS discontinuing support for the specific minor version of PostgreSQL. One of the incidents encountered in the following run : https://github.com/hashicorp/terraform-enterprise/actions/runs/12774604392/job/35657266311 where the error was:
Error: creating RDS DB Instance (up-coral-tfe20250113001829240000000001): operation error RDS: CreateDBInstance, https response error StatusCode: 400, RequestID: a10524b7-3a93-40bd-8813-95bbedd620b5, api error InvalidParameterCombination: Cannot find version 16.2 for postgres

Does this PR make a substantial change to TFE's architecture?

NO, it's a tests only PR.

How Has This Been Tested

  • Made the docker_active_active_feature module of the terraform-enterprise/release/internal/spec/terraform/modules/fdo/aws/docker_active_active_feature/ repository to use this branch (as docker_active_active_feature module is using this repository directly )
    make ci-aws-fdo-docker-active-active-feature and GHA run
    Github Run = https://github.com/hashicorp/terraform-enterprise/actions/runs/12884720233

@harshit410 harshit410 requested a review from a team as a code owner January 21, 2025 14:13
Copy link
Contributor

@aaron-lane aaron-lane left a comment

Choose a reason for hiding this comment

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

LGTM 🐘

@harshit410 harshit410 merged commit 27d407d into main Jan 21, 2025
4 checks passed
@harshit410 harshit410 deleted the harshit410/amazonRDS_minorversion branch January 21, 2025 16:53
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