-
Notifications
You must be signed in to change notification settings - Fork 217
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_grant_role resource #4
Conversation
Co-authored-by: Danny Hermes <[email protected]>
Co-authored-by: Danny Hermes <[email protected]>
Co-authored-by: Danny Hermes <[email protected]>
Co-authored-by: Danny Hermes <[email protected]>
@dvdliao Thanks for recreating it here, Hashicorp should have migrated the PRs & issues of the old repository but as it's still not the case we'll use this one. I'll take a look this week-end. Could you just rebase on master? I fixed the Travis build so it should pass now. |
@cyrilgdn done. thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot for your work on this 👍 , and really sorry for the response delay.
Could you just remove the errwrap
dependency (cf comment) and it's good for me.
(Other comments are not mandatory)
|
||
~> **Note:** This resource needs PostgreSQL version 9 or above. | ||
|
||
~> **Note:** `postgresql_grant_role` **cannot** be used in conjunction with `postgresql_role` or they will fight over what your role grants should be. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice 👍 I encountered this conflict during my test and I was about to say that we need that we need to add this warning but you already added it 😊
@cyrilgdn done and thanks! |
@cyrilgdn done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot 👍
hey @cyrilgdn any idea when a release will be cut? i see the milestone as completed https://github.com/cyrilgdn/terraform-provider-postgresql/milestone/2?closed=1 |
Update from upstream
Copying from hashicorp/terraform-provider-postgresql#189
Fix #6