Skip to content

Draft: Allowing granular permissions for warehouses

Rahul requested to merge rahulj51/permifrost:warehouse-granular-permissions into master

Merge Request Checklist

  • Link to an issue
  • Include the proposed fix or feature
  • Include and update tests for the modified code
  • Include a documentation change
  • Add an entry in the Unreleased section of the CHANGELOG

Allows users to specify granular privileges for warehouses instead of always giving all 3 of operate, usage and monitor permissions by default. The warehouse attribute is now a dictionary

  - sysadmin:
      warehouses:
        loading: ['usage', 'monitor', 'operate']
        transforming: ['usage', 'monitor', 'operate']
        reporting: ['usage', 'monitor', 'operate']

Note: This is a breaking change because the warehouses have to be defined as a hashmap now . Made it backward compatible to support either a list like before or a dict with the specific privileges.

  • closes #123 (closed)
  • Also upgrades black version to 22.3.0 to avoid it from failing.

I have not yet changed the documentation and CHANGELOG. Will do so if the change

Edited by Rahul

Merge request reports