-
Notifications
You must be signed in to change notification settings - Fork 258
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
Add PostgreSQL configuration resource and test #1139
Add PostgreSQL configuration resource and test #1139
Conversation
Signed-off-by: Steffen Exler <steffen@exivity.com>
Signed-off-by: Steffen Exler <steffen@exivity.com>
Taking a look now- thank you so much! Also, running |
digitalocean/database/resource_database_postgres_config_test.go
Outdated
Show resolved
Hide resolved
Co-authored-by: danaelhe <42972711+danaelhe@users.noreply.github.com>
Signed-off-by: Steffen Exler <steffen@exivity.com>
Co-authored-by: danaelhe <42972711+danaelhe@users.noreply.github.com>
Thanks for reviewing. I applied your suggestions. |
…_config Signed-off-by: Steffen Exler <steffen@exivity.com>
Thanks! Still working through the review- looks great though!! |
I had some time today so I pushed up some changes as well. Mostly minor things, like adding Thank you so much for your contribution! We should be able to get this merged and a new release cut today! |
Thank you a lot @danaelhe 💃 |
This pull request adds a new PostgreSQL configuration resource and a corresponding test. The resource allows users to configure various PostgreSQL settings for a DigitalOcean database cluster, similar like to MySQL.
I tried to implement all options from godo. For the test I'm not sure if they are working correct, I guess here it will need some more work.