Skip to content

Conversation

goruha
Copy link
Contributor

@goruha goruha commented Oct 8, 2025

what

  • Update engine version in serverless.yaml

why

  • 13.15 is EOL on AWS

references

Summary by CodeRabbit

  • Chores

    • Updated default Aurora PostgreSQL engine versions in configuration templates: provisioned to 15.13 and serverless to 13.21.
    • Aligns templates with supported releases for improved compatibility and security; no behavioral or logic changes.
  • Tests

    • Updated test fixtures to reflect the new engine versions.

@mergify mergify bot requested review from a team October 8, 2025 10:16
@mergify mergify bot added the triage Needs triage label Oct 8, 2025
Copy link

coderabbitai bot commented Oct 8, 2025

Walkthrough

Bumped Postgres engine_version values in two YAML test fixtures: test/fixtures/stacks/catalog/usecase/basic.yaml (15.3 → 15.13) and test/fixtures/stacks/catalog/usecase/serverless.yaml (13.15 → 13.21). No other changes.

Changes

Cohort / File(s) Summary of Changes
Test fixtures: engine_version bumps
test/fixtures/stacks/catalog/usecase/basic.yaml, test/fixtures/stacks/catalog/usecase/serverless.yaml
Updated Postgres engine_version strings in YAML fixtures: basic.yaml 15.315.13; serverless.yaml 13.1513.21. No other edits.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

I nudged the dials, soft hops in tune,
Numbers climbed under a YAML moon.
Fifteen point thirteen, thirteen point twenty-one,
Tiny hops done—now off to run. 🐇✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title correctly refers to the bump in serverless.yaml but omits the related engine version update in basic.yaml, so it only partially represents the full set of changes.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch update-rds-postgresql-13.22

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 840c705 and b92417c.

📒 Files selected for processing (1)
  • test/fixtures/stacks/catalog/usecase/basic.yaml (1 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
  • test/fixtures/stacks/catalog/usecase/basic.yaml
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: Summary

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@goruha
Copy link
Contributor Author

goruha commented Oct 8, 2025

/terratest

@goruha
Copy link
Contributor Author

goruha commented Oct 8, 2025

/terratest

RoseSecurity
RoseSecurity previously approved these changes Oct 8, 2025
@mergify mergify bot removed the triage Needs triage label Oct 8, 2025
@mergify mergify bot requested a review from a team October 8, 2025 11:35
@goruha
Copy link
Contributor Author

goruha commented Oct 8, 2025

/terratest

@mergify mergify bot added the triage Needs triage label Oct 8, 2025
@RoseSecurity RoseSecurity added this pull request to the merge queue Oct 8, 2025
@mergify mergify bot removed the triage Needs triage label Oct 8, 2025
@mergify mergify bot requested a review from a team October 8, 2025 16:23
Merged via the queue into main with commit 3540534 Oct 8, 2025
19 checks passed
@RoseSecurity RoseSecurity deleted the update-rds-postgresql-13.22 branch October 8, 2025 17:14
Copy link

github-actions bot commented Oct 9, 2025

These changes were released in v1.540.1.

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