-
Notifications
You must be signed in to change notification settings - Fork 71
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
Add ability to test individual as well as multiple organisations #110
Add ability to test individual as well as multiple organisations #110
Conversation
dd0be11
to
eae3ad3
Compare
Signed-Off-By: Walter Dolce <[email protected]>
eae3ad3
to
b639110
Compare
Signed-Off-By: Walter Dolce <[email protected]>
The above is based on:
|
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.
Thanks for this @walterdolce, looks great! How about we add the controls for this resource but make them disabled by default with a flag? You could use the below control as an example of doing that:
https://github.com/inspec/inspec-gcp/blob/master/test/integration/verify/controls/google_kms_crypto_key.rb
Signed-Off-By: Walter Dolce <[email protected]>
b5bb1e8
to
c36114f
Compare
@skpaterson sounds good. Done! |
Brilliant, thanks again. For completeness do you mind adding the control for the singular |
Signed-Off-By: Walter Dolce <[email protected]>
@skpaterson certainly. Also done! |
Templates, inspec.yaml for bigquery table
I wasn't really sure how I should have gone about testing this as you need to have a G Suite account for this (and I did not want to use the company one), but I promise everything works. I have a screenshot for it! 😛