Page MenuHomeElementl

[dagit] Add Permissions query
ClosedPublic

Authored by dish on Aug 2 2021, 8:34 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Sep 24, 2:33 PM
Unknown Object (File)
Fri, Sep 16, 1:07 PM
Unknown Object (File)
Thu, Sep 15, 9:36 AM
Unknown Object (File)
Fri, Sep 2, 9:54 AM
Unknown Object (File)
Aug 24 2022, 3:43 PM
Unknown Object (File)
Aug 19 2022, 7:46 AM
Unknown Object (File)
Aug 18 2022, 7:37 PM
Unknown Object (File)
Aug 18 2022, 2:45 PM
Subscribers
None

Details

Summary

Create a PermissionsProvider that queries for permissions via GraphQL, as arranged in https://dagster.phacility.com/D9122.

Test Plan

Load Dagit, verify that permissions are queried and used properly.

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Aug 2 2021, 8:56 PM
Harbormaster failed remote builds in B34558: Diff 42736!

this looks good, one big thing that was missing from the diff this is patched on is python tests though - I may be able to tackle that this morning, given that permissions are fairly sensitive i think its important to have some coverage there

Does this require any changes in the internal repo in order to avoid any breaking changes there?

tests in https://dagster.phacility.com/D9193 - lets make sure we have a change ready for internal though if this is breaking there

This revision is now accepted and ready to land.Aug 3 2021, 1:19 PM

Compatibility, expose raw permission array.

Does this require any changes in the internal repo in order to avoid any breaking changes there?

Verdict is: yes. Will have a followup.

This revision was automatically updated to reflect the committed changes.