-
Notifications
You must be signed in to change notification settings - Fork 321
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(knative&crossplane): add support to knative and crossplane #5634
Conversation
Bumps [github.com/BurntSushi/toml](https://github.com/BurntSushi/toml) from 1.1.0 to 1.2.0. - [Release notes](https://github.com/BurntSushi/toml/releases) - [Commits](BurntSushi/toml@v1.1.0...v1.2.0) --- updated-dependencies: - dependency-name: github.com/BurntSushi/toml dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…5628) Bumps [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go) from 1.44.58 to 1.44.59. - [Release notes](https://github.com/aws/aws-sdk-go/releases) - [Changelog](https://github.com/aws/aws-sdk-go/blob/main/CHANGELOG.md) - [Commits](aws/aws-sdk-go@v1.44.58...v1.44.59) --- updated-dependencies: - dependency-name: github.com/aws/aws-sdk-go dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: João Reigota <[email protected]> Co-authored-by: Rafaela Soares <[email protected]>
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.
A few additional considerations:
-
Maybe it will make sense to use the walk function to reduce the number of policies (at the moment, the query is looking kinda the same for both kinds)
-
Is it metadata.name required in Crossplane?
-
Please, update the branch with the release/1.6
assets/queries/crossplane/aws/cloudfront_logging_disabled/query.rego
Outdated
Show resolved
Hide resolved
assets/queries/crossplane/aws/cloudfront_logging_disabled/query.rego
Outdated
Show resolved
Hide resolved
assets/queries/crossplane/aws/cloudfront_logging_disabled/query.rego
Outdated
Show resolved
Hide resolved
assets/queries/crossplane/gcp/google_container_node_pool_auto_repair_disabled/query.rego
Outdated
Show resolved
Hide resolved
assets/queries/crossplane/gcp/google_container_node_pool_auto_repair_disabled/query.rego
Outdated
Show resolved
Hide resolved
assets/queries/crossplane/gcp/google_container_node_pool_auto_repair_disabled/query.rego
Outdated
Show resolved
Hide resolved
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.
Please, merge the branch with release/1.6
assets/queries/ansible/aws/elb_using_weak_ciphers/metadata.json
Outdated
Show resolved
Hide resolved
assets/queries/crossplane/aws/cloudwatch_without_retention_period_specified/metadata.json
Outdated
Show resolved
Hide resolved
assets/queries/crossplane/aws/cloudwatch_without_retention_period_specified/query.rego
Outdated
Show resolved
Hide resolved
assets/queries/crossplane/aws/cloudwatch_without_retention_period_specified/query.rego
Outdated
Show resolved
Hide resolved
assets/queries/crossplane/aws/db_instance_storage_not_encrypted/metadata.json
Outdated
Show resolved
Hide resolved
assets/queries/crossplane/aws/neptune_database_cluster_encryption_disabled/query.rego
Outdated
Show resolved
Hide resolved
assets/queries/crossplane/aws/cloudwatch_without_retention_period_specified/query.rego
Outdated
Show resolved
Hide resolved
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 🚀
Proposed Changes
I submit this contribution under the Apache-2.0 license.