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

Write karmor probe data #822

Merged
merged 1 commit into from
Aug 23, 2022
Merged

Write karmor probe data #822

merged 1 commit into from
Aug 23, 2022

Conversation

Essietom
Copy link
Contributor

@Essietom Essietom commented Aug 18, 2022

Ref kubearmor/kubearmor-client#19

Signed-off-by: Esther Adenekan essietom@gmail.com

@daemon1024
Copy link
Member

Ref kubearmor/kubearmor-client#19

@Essietom Essietom changed the title read karmor probe data Read karmor probe data Ref kubearmor/kubearmor-client#19 Aug 18, 2022
@codecov-commenter
Copy link

codecov-commenter commented Aug 21, 2022

Codecov Report

Merging #822 (5a291ea) into main (a28c4ed) will increase coverage by 0.42%.
The diff coverage is 72.72%.

❗ Current head 5a291ea differs from pull request most recent head 7a7eba7. Consider uploading reports for the commit 7a7eba7 to get more accurate results

@@            Coverage Diff             @@
##             main     #822      +/-   ##
==========================================
+ Coverage   39.77%   40.20%   +0.42%     
==========================================
  Files          30       31       +1     
  Lines        9285     9328      +43     
==========================================
+ Hits         3693     3750      +57     
+ Misses       5101     5083      -18     
- Partials      491      495       +4     
Impacted Files Coverage Δ
KubeArmor/common/common.go 43.24% <40.00%> (-0.14%) ⬇️
KubeArmor/core/karmorprobedata.go 81.81% <81.81%> (ø)
KubeArmor/core/kubeArmor.go 50.66% <100.00%> (ø)
KubeArmor/monitor/logUpdate.go 68.19% <0.00%> (-3.28%) ⬇️
KubeArmor/monitor/processTree.go 94.62% <0.00%> (-1.08%) ⬇️
KubeArmor/monitor/syscallParser.go 62.27% <0.00%> (-1.02%) ⬇️
KubeArmor/core/containerdHandler.go 68.14% <0.00%> (+15.55%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@daemon1024 daemon1024 changed the title Read karmor probe data Ref kubearmor/kubearmor-client#19 Write karmor probe data Aug 22, 2022
Copy link
Member

@daemon1024 daemon1024 left a comment

Choose a reason for hiding this comment

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

Some refactoring suggestions, Lemme know WDYT bout em.

Copy link
Member

@daemon1024 daemon1024 left a comment

Choose a reason for hiding this comment

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

Let's squash the commits

Copy link
Member

@Ankurk99 Ankurk99 left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: Esther Adenekan <essietom@gmail.com>

get active lsm not supported lsm

Signed-off-by: Esther Adenekan <essietom@gmail.com>

get active lsm

Signed-off-by: Esther Adenekan <essietom@gmail.com>

refactor container and host security check code

Signed-off-by: Esther Adenekan <essietom@gmail.com>

get host and container default posture and refactor code

Signed-off-by: Esther Adenekan <essietom@gmail.com>

add comment to setkarmordata method

Signed-off-by: Esther Adenekan <essietom@gmail.com>

edit comment

Signed-off-by: Esther Adenekan <essietom@gmail.com>
@Ankurk99 Ankurk99 merged commit cfe270e into kubearmor:main Aug 23, 2022
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.

5 participants