Skip to content
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

fix: cis checks validate (api-server, controller-manager, scheduler and etcd) args #110

Merged
merged 1 commit into from
Apr 17, 2024

Conversation

chen-keinan
Copy link
Contributor

@chen-keinan chen-keinan commented Apr 14, 2024

Description

cis checks validate (api-server, controller-manager, scheduler and etcd) args
args might appear together with command or placed with the separate args field
this change check for args in both fields command and args

@chen-keinan chen-keinan requested a review from simar7 as a code owner April 14, 2024 11:27
@chen-keinan chen-keinan changed the title fix: cis checks validate api-server args fix: cis checks validate (api-server, controller-manager, scheduler and etcd) args Apr 14, 2024
@chen-keinan chen-keinan force-pushed the fix/cis-checks-validate-args branch from a504a24 to 16c0a9a Compare April 14, 2024 14:35
@chen-keinan chen-keinan force-pushed the fix/cis-checks-validate-args branch from 16c0a9a to ac7cf4b Compare April 14, 2024 14:35
@chen-keinan
Copy link
Contributor Author

@simar7 please review this PR

@simar7 simar7 added this pull request to the merge queue Apr 17, 2024
Merged via the queue into aquasecurity:main with commit 932169b Apr 17, 2024
5 checks passed
@chen-keinan
Copy link
Contributor Author

@simar7 could you please cut a new release so the updated checks could be consumed by trivy

@simar7
Copy link
Member

simar7 commented Apr 17, 2024

@simar7 could you please cut a new release so the updated checks could be consumed by trivy

Yes https://github.com/aquasecurity/trivy-checks/pkgs/container/trivy-policies/204711166?tag=0 was published.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants