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

support aurora instance's parameter group and modifyinstance #6865

Merged
merged 2 commits into from
Jun 3, 2016

Conversation

arcadiatea
Copy link
Contributor

This change for #6841.

@catsby
Copy link
Member

catsby commented Jun 2, 2016

Hey @arcadiatea – am I correct that we can simply use the existing aws_db_parameter_group, correct?

@catsby catsby mentioned this pull request Jun 2, 2016
Computed: true,
},

"status": &schema.Schema{
Copy link
Member

Choose a reason for hiding this comment

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

This attribute doesn't seem to be set anywhere, what is it meant to record? The DBInstanceStatus, or status from StatusInfos.DBInstanceStatusInfo.N ?

@catsby
Copy link
Member

catsby commented Jun 2, 2016

I have a question about the status attribute but I did test out this branch and it worked as expected 👍

Note to self:

fixes #5023 too

@arcadiatea
Copy link
Contributor Author

Hey, @catsby
This change is to allow Aurora instance be able to use customized parameter group set by aws_db_parameter_group.
The current issues is Aurora instance always created with default db parameter group and not be able to detect the modify requirement.

@catsby
Copy link
Member

catsby commented Jun 2, 2016

@arcadiatea yep! I tried it out and it works as expected. I do have an question about the added status parameter; it's not being set, and it's not clear to me what value this should be. Do you recall?

@arcadiatea
Copy link
Contributor Author

arcadiatea commented Jun 2, 2016

@catsby that status attribute is not be used for this modification , you are right.
I am working on another enhancement feature on aws_rds_cluster_instance (support security_group changes) and I can include the usage of status there.

status attribute be used in aws_db_instance for support debug information on resourceAwsDbInstanceStateRefreshFunc, and I plan to mimic the function in aws_rds_cluster_instance

@catsby
Copy link
Member

catsby commented Jun 2, 2016

Ok, thanks for clarifying! If you remove that attribute from this commit, I'll go ahead and merge this

@arcadiatea
Copy link
Contributor Author

@catsby status attributes is removed. This change probably also fix #5710, #5436

@catsby
Copy link
Member

catsby commented Jun 3, 2016

Thanks! I'm going to merge this with an addition to the test file as well.

@ghost
Copy link

ghost commented Apr 24, 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 have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators Apr 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants