-
Notifications
You must be signed in to change notification settings - Fork 98
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
feat: implement instance and table level IAM policy #584
feat: implement instance and table level IAM policy #584
Conversation
Codecov Report
@@ Coverage Diff @@
## master #584 +/- ##
==========================================
+ Coverage 86.82% 87.24% +0.41%
==========================================
Files 33 33
Lines 1981 2046 +65
Branches 349 373 +24
==========================================
+ Hits 1720 1785 +65
Misses 233 233
Partials 28 28
Continue to review full report at Codecov.
|
@crwilcox I added implementation for Instance level Iam policy and documents snippets. |
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.
@AVaksman mind adding a bit of a description in the issue comment about this feature, just a high level of outline of the behavior.
a3c83f5
to
7c6979c
Compare
* feat: add table level IAM policy controls * lint: lint * feat: add instance level IAM policy * docs: update description * docs: add document snippets * lint: lint
* Reads and filters WIP, working Reads tests and current format for Reads snippets * working tests for solo filters * Composing filters * add bulk region tags * Change testing instance * Linted * feat: implement instance and table level IAM policy (#584) * feat: add table level IAM policy controls * lint: lint * feat: add instance level IAM policy * docs: update description * docs: add document snippets * lint: lint * Linted Fixed headers Removed unnceccessary asyncs Arrow functions where possible * Arrow functions * destructure Bigtable require * Using snapshots for testing * Change functions to use a switch * Fix timezone issue for tests * Lint snapshot files * Clean up loops, add update snapshot method
The new features allow user to check and tune IAM policy for the resource (
instance
,table
) level:getIamPolicy
- allows a user obtain the current resource IAM policy.setIamPolicy
- allows a user to set resource level IAM policy.testIamPermissions
- allows a user to pass a list ofpermissions
and get back a sub-list of granted permissions.Ensure the tests and linter pass
Code coverage does not decrease (if any source code was changed)
Appropriate docs were updated (if necessary)
Fixes #581