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

[WIP] resource/aws_db_instance: Apply parameter group update when launching an RDS snapshot #1461

Closed
wants to merge 1 commit into from
Closed

[WIP] resource/aws_db_instance: Apply parameter group update when launching an RDS snapshot #1461

wants to merge 1 commit into from

Conversation

jameshaak
Copy link

My first contribution, so go easy on me. 😉

Fixes #218.

Still need to:

  • Reboot RDS instance to fully apply parameter group
  • Rebase onto current master
  • Update resource tests?

@radeksimko radeksimko added the bug Addresses a defect in current functionality. label Sep 27, 2017
@radeksimko
Copy link
Member

radeksimko commented Nov 9, 2017

Hi @jameshaak
thanks for your contribution.
Sorry for the delay in reviewing this.

I don't think that we need to reboot the instance in order to apply this change. Just one thing to ask - do you mind adding an acceptance test which changes the parameter group name?

See an example of a test which is updating subnet group here:

https://github.com/terraform-providers/terraform-provider-aws/blob/07ff8a7e3059e5a09f0a6ccd845294abe97d8e1c/aws/resource_aws_db_instance_test.go#L124-L151

and feel free to apply the "copy-paste-modify" approach 😃

@radeksimko radeksimko added waiting-response Maintainers are waiting on response from community or contributor. size/XS Managed by automation to categorize the size of a PR. labels Nov 9, 2017
@radeksimko radeksimko added the service/rds Issues and PRs that pertain to the rds service. label Jan 16, 2018
@radeksimko radeksimko changed the title [WIP] Apply parameter group update when launching an RDS snapshot [WIP] resource/aws_db_instance: Apply parameter group update when launching an RDS snapshot Jan 16, 2018
@jameshaak
Copy link
Author

Closing this PR since I haven't had time to work on it, and I've implemented a non-Terraform work-around in my project.

@jameshaak jameshaak closed this Jan 26, 2018
@ghost
Copy link

ghost commented Apr 8, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks!

@ghost ghost locked and limited conversation to collaborators Apr 8, 2020
@breathingdust breathingdust removed the waiting-response Maintainers are waiting on response from community or contributor. label Sep 17, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Addresses a defect in current functionality. service/rds Issues and PRs that pertain to the rds service. size/XS Managed by automation to categorize the size of a PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

RDS: aws_db_instance not respecting parameter_group_name on first run
3 participants