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

postgresql_user priv is deprecated, needs to be migrated to postgresql_privs #254

Closed
C0rn3j opened this issue Dec 31, 2023 · 7 comments
Closed
Labels

Comments

@C0rn3j
Copy link
Contributor

C0rn3j commented Dec 31, 2023

See https://docs.ansible.com/ansible/latest/collections/community/postgresql/postgresql_user_module.html#parameter-priv

Copy link

github-actions bot commented May 3, 2024

This issue has been marked 'stale' due to lack of recent activity. If there is no further activity, the issue will be closed in another 30 days. Thank you for your contribution!

Please read this blog post to see the reasons why I mark issues as stale.

@github-actions github-actions bot added the stale label May 3, 2024
@C0rn3j
Copy link
Contributor Author

C0rn3j commented May 3, 2024

@geerlingguy please stop threatening to autoclose, just mark it stale and let it be.

@geerlingguy geerlingguy added bug and removed stale labels May 4, 2024
@geerlingguy
Copy link
Owner

Easier to mark as a bug (because it will be once the deprecation is deprecated), that way stale bot won't eat it up.

@zerwes
Copy link
Contributor

zerwes commented Jan 5, 2025

For now we are at community.postgresql collection version 3.9.1
The option has been deprecated and will be removed in community.postgresql 4.0.0.
4.x.y: to be released; not earlier than after Ansible 10 release (~May 2024) (https://github.com/ansible-collections/community.postgresql/tree/main?tab=readme-ov-file#releases-support-timeline)
Privileges are not cosmetic.
So I I would summarize it like this: this is not a bug, it is a ticking time bomb.

@geerlingguy
Copy link
Owner

geerlingguy commented Jan 5, 2025

Well I don't have a label for ticking time bombs, unfortunately :)

If someone wants to work on a PR for this I'd be happy to merge if it passes tests and someone else can check over it too.

@zerwes
Copy link
Contributor

zerwes commented Jan 6, 2025

Well I don't have a label for ticking time bombs, unfortunately :)

👍
Indeed, not just a label for this, a due date with reminder notifications / escalations is something I am missing @ github

If someone wants to work on a PR for this ...

OK, see #266 - hope it fits all needs

@geerlingguy
Copy link
Owner

Fixed, in 4.0.0 now!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants