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

ET-6505: Aurora defaults update #203

Merged
merged 2 commits into from
Aug 7, 2024
Merged

ET-6505: Aurora defaults update #203

merged 2 commits into from
Aug 7, 2024

Conversation

kzonov
Copy link
Contributor

@kzonov kzonov commented Aug 6, 2024

Two changes:

  1. The engine version. It is apparently already updated on the AWS side, but we didn't adjust our terraform earlier.
  2. The CA, I set it to the default recommended value. I don't think we'll ever need a custom value here, so I didn't put it into variables. Variables are for the things that we potentially may want to customize, this is meant to be fixed.

@kzonov kzonov requested a review from joecks August 6, 2024 11:15
@kzonov kzonov self-assigned this Aug 6, 2024
@joecks
Copy link
Collaborator

joecks commented Aug 7, 2024

LGTM

@kzonov kzonov merged commit 5077ecc into main Aug 7, 2024
3 checks passed
@kzonov kzonov deleted the ET-6505-ca-update-aurora branch August 7, 2024 12:24
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