-
Notifications
You must be signed in to change notification settings - Fork 31
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 security policy schema and resource with unit tests #53
add security policy schema and resource with unit tests #53
Conversation
f5cd7ba
to
84f2220
Compare
examples/resources/security_policy/resource_cluster_group_security_policy.tf
Outdated
Show resolved
Hide resolved
Can you break it into multiple logical commits |
Sure |
internal/resources/policy/type/security/resource_security_policy.go
Outdated
Show resolved
Hide resolved
9d53e98
to
e6c298b
Compare
2d1801e
to
3deb7fd
Compare
0e34b92
to
c61367f
Compare
examples/resources/security_policy/resource_cluster_baseline_security_policy.tf
Outdated
Show resolved
Hide resolved
examples/resources/security_policy/resource_cluster_custom_security_policy.tf
Outdated
Show resolved
Hide resolved
c61367f
to
f832cd1
Compare
f832cd1
to
30dff58
Compare
Done @shreyassreenivas |
PTAL again, @shreyassreenivas , @Vasundharashukla and @shobha2626 |
30dff58
to
0c5d9e9
Compare
Signed-off-by: Ishan Gupta <gishan@vmware.com>
Signed-off-by: Ishan Gupta <gishan@vmware.com>
Signed-off-by: Ishan Gupta <gishan@vmware.com>
Signed-off-by: Ishan Gupta <gishan@vmware.com>
Signed-off-by: Ishan Gupta <gishan@vmware.com>
Signed-off-by: Ishan Gupta <gishan@vmware.com>
3f4567e
to
2565db1
Compare
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.
LGTM
allow_privilege_escalation = true | ||
allow_host_namespace_sharing = true | ||
allow_host_network = true | ||
read_only_root_file_system = true |
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.
super nit : check the indentation of =
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.
Done @shreyassreenivas
If all the edge cases are tested and validated, feel free to merge the feature branch into master. |
Signed-off-by: Ishan Gupta <gishan@vmware.com>
Signed-off-by: Ishan Gupta <gishan@vmware.com>
2565db1
to
a14191f
Compare
yes. Merging this to the feature branch. After this, I will add the acceptance test. Also will add the |
|
Signed-off-by: Ishan Gupta gishan@vmware.com
What this PR does / why we need it: This PR adds security policy schema and resource with unit tests.
Additional information: Tested for all the scenarios.