Skip to content

Conversation

@patchback
Copy link

@patchback patchback bot commented Oct 18, 2022

This is a backport of PR #335 as merged into main (eed4e1f).

SUMMARY

Fixes #332
Fixes #333

Test task 332 executes the following queries:

ALTER DEFAULT PRIVILEGES REVOKE ALL ON SCHEMAS FROM \"ansible_db_user3\";
ALTER DEFAULT PRIVILEGES GRANT USAGE ON SCHEMAS TO \"ansible_db_user3\"

Test task 333 executes the following:

ALTER DEFAULT PRIVILEGES REVOKE ALL ON TABLES FROM \"ansible_db_user3\"
ALTER DEFAULT PRIVILEGES GRANT SELECT ON TABLES TO \"ansible_db_user3\"

Is it OK?

not-specified is introduced to preserve backwards compatibility, we can change it as a default in a later major release if needed.

ISSUE TYPE
  • Bugfix Pull Request

…e on schemas (#335)

* postgresql_privs: grant usage on schemas

* issue 332

* add changelog fragment

* issue 333

* update changelog and documentation

* Fix

(cherry picked from commit eed4e1f)
@codecov
Copy link

codecov bot commented Oct 18, 2022

Codecov Report

❗ No coverage uploaded for pull request base (stable-1@15423c6). Click here to learn what that means.
The diff coverage is n/a.

❗ Current head eed4e1f differs from pull request most recent head 941c37d. Consider uploading reports for the commit 941c37d to get more accurate results

@@             Coverage Diff             @@
##             stable-1     #355   +/-   ##
===========================================
  Coverage            ?   76.45%           
===========================================
  Files               ?       37           
  Lines               ?     4990           
  Branches            ?     1141           
===========================================
  Hits                ?     3815           
  Misses              ?      884           
  Partials            ?      291           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@hunleyd
Copy link
Collaborator

hunleyd commented Oct 21, 2022

isn't this a feature @Andersson007 ? as such, no backport.

@Andersson007
Copy link
Collaborator

@hunleyd good catch, yep, i think so, let's not backport

@Andersson007 Andersson007 deleted the patchback/backports/stable-1/eed4e1f4641acc6181647e7d18bd550261a282d5/pr-335 branch May 16, 2025 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants