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

job.yaml: Adding /var/lib/cni mount for proper CIS 1.1.9 and 1.1.10 checking #1547

Merged
merged 2 commits into from
Feb 11, 2024

Conversation

apolovov
Copy link
Contributor

Adding /var/lib/cni mount in job manifests for proper CIS 1.1.9 and 1.1.0 checking.
It was impossible to pass the checks without the mount. The result is WARN regardless of the actual file permissions on the host.

…hecking

Signed-off-by: Andrey Polovov <andrey.polovov@flant.com>
Signed-off-by: Andrey Pavlov <andrey.pavlov@flant.com>
Co-authored-by: Andrey Pavlov <andrey.pavlov@flant.com>
@CLAassistant
Copy link

CLAassistant commented Jan 10, 2024

CLA assistant check
All committers have signed the CLA.

@apolovov apolovov changed the title job.yaml: Adding /var/lib/cni mount for proper CIS 1.1.9 and 1.1.0 checking job.yaml: Adding /var/lib/cni mount for proper CIS 1.1.9 and 1.1.10 checking Jan 12, 2024
@apolovov
Copy link
Contributor Author

apolovov commented Feb 5, 2024

@chen-keinan could you review pls?

@mozillazg
Copy link
Collaborator

@apolovov Could you please sign the CLA? Thanks.

@apolovov
Copy link
Contributor Author

apolovov commented Feb 8, 2024

@mozillazg Signed, sorry for the delay.

Copy link
Collaborator

@mozillazg mozillazg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, Thanks for your contribution!

@chen-keinan chen-keinan merged commit faeceb5 into aquasecurity:main Feb 11, 2024
5 checks passed
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.

None yet

4 participants