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

Recommend host hardening policies #662

Merged
merged 1 commit into from Feb 17, 2023

Conversation

vishnusomank
Copy link
Contributor

  • show KubearmorHostPolicies while recommending hardening policies based on flag value

Signed-off-by: Vishnu Soman vishnu@accuknox.com

src/config/configManager.go Outdated Show resolved Hide resolved
src/recommendpolicy/helperFunctions.go Outdated Show resolved Hide resolved
Copy link
Contributor

@seswarrajan seswarrajan left a comment

Choose a reason for hiding this comment

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

LGTM

@seswarrajan
Copy link
Contributor

@vishnusomank Please fix gosec issues

Copy link
Contributor

@seswarrajan seswarrajan left a comment

Choose a reason for hiding this comment

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

LGTM.
Is this validated with the latest change ?

Since it is host policy, is it tested with running as systemd in local machine ?

@vishnusomank
Copy link
Contributor Author

LGTM. Is this validated with the latest change ?

Since it is host policy, is it tested with running as systemd in local machine ?

yes this was tested locally on a k3s cluster @seswarrajan

Copy link
Contributor

@achrefbensaad achrefbensaad left a comment

Choose a reason for hiding this comment

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

@seswarrajan
Copy link
Contributor

Kindly resolve conflicts and update.

- show KubearmorHostPolicies while recommending hardening policies based on flag value

Signed-off-by: Vishnu Soman <vishnu@accuknox.com>
@vishnusomank
Copy link
Contributor Author

Kindly resolve conflicts and update.

@seswarrajan i've resolved the conflicts and updated. kindly review

Copy link
Contributor

@seswarrajan seswarrajan left a comment

Choose a reason for hiding this comment

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

LGTM

@seswarrajan seswarrajan merged commit f7e1a8a into accuknox:dev Feb 17, 2023
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

3 participants