Skip to content

CLOUD-809: chore(aurora clone lambda) update from python 3.10 to 3.14 - #328

Merged
ruthwells merged 1 commit into
mainfrom
CLOUD-809_EOL_aurora_clone_lambda_versions
Jul 2, 2026
Merged

CLOUD-809: chore(aurora clone lambda) update from python 3.10 to 3.14#328
ruthwells merged 1 commit into
mainfrom
CLOUD-809_EOL_aurora_clone_lambda_versions

Conversation

@ruthwells

@ruthwells ruthwells commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Describe your changes

Python 3.10 is deprecated - this PR upgrades the lambdas used by the aurora clone mechanism to Python 3.14.

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have updated the Readme / docs
  • If this is a breaking change, I have discussed the roll-out and roll-back plan with the team(s) and I have provided detailed instructions.

I've done a search of citizensadvice github repos, and the only repo that uses the AuroraCloneRefresh construct is casebook (and the now dead witnessbox-v2). This is only in staging/preprod, so we will be able to test before releasing changes to production. I will communicate/manage this with the adviser engineers.

EDIT: @mlaycock @Slange-Mhath @MrJoosh I think this PR (once the version of ca-cdk-constructs is updated in the casebook repo) will also fix the EOL lambdas in the cita-devops account for the qa-DevCasebookDataExtractor. But my AWS permissions don't allow me to see if there are also any aurora clones in the cab-data-prod account 445513191207.

  • If it is a core feature, I have added thorough tests

@ruthwells
ruthwells requested a review from a team as a code owner June 29, 2026 11:30

@simonwgill simonwgill left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I don't like that there's no automated tests for this code at all... and that the lambda code can be run under a different runtime than the one used for any static type checking.

But since that's been the case forever and it's not used for any production infrastructure, I guess it's a case of send it, and see if the unlikely case of a failure breaks something.

@Slange-Mhath

Copy link
Copy Markdown

lgtm and thanks for doing this Ruth!

@MrJoosh MrJoosh left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM - we don't have any clones in cab-data-prod so all good from our side! Thanks Ruth!

@ruthwells
ruthwells merged commit 5c68a3f into main Jul 2, 2026
10 checks passed
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