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

RDS: Allow more db-instance operations in custom subnets #8440

Conversation

snordhausen
Copy link
Contributor

Follow up to #8374 that fixes a new deepcopy call from #8373 in modify_db_instance and another one in restore_db_instance_to_point_in_time. Previously, this failed with

TypeError: cannot pickle 'generator' object

Previously, this failed with
    TypeError: cannot pickle 'generator' object
when the code tried to deepcopy the `database.db_subnet_group`.

But a deepcopy isn't needed in this case, the subsequent
    database.master_user_secret_status = ...
will not affect the shallow copy. Also, the copy is only needed
to render the response template.
Previously, this failed with
    TypeError: cannot pickle 'generator' object
when the code tried to deepcopy the `backend`.

Merge it with the existing code that removed "db_subnet_group".
@snordhausen
Copy link
Contributor Author

Hmm, the test failures seem to be unrelated to my changes.

@bblommers bblommers added this to the 5.0.26 milestone Dec 27, 2024
@bblommers
Copy link
Collaborator

bblommers commented Dec 27, 2024

Hmm, the test failures seem to be unrelated to my changes.

Yeah, they are unrelated. That's due to a recent bug in botocore, so don't worry about them

Copy link
Collaborator

@bblommers bblommers left a comment

Choose a reason for hiding this comment

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

LGTM!

@bblommers bblommers merged commit cbc7a51 into getmoto:master Dec 27, 2024
44 of 51 checks passed
@snordhausen snordhausen deleted the allow-modify_db_instance-in-custom-subnet-2 branch December 30, 2024 07: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