Skip to content

add security-agent config#706

Merged
albertvaka merged 5 commits intoDataDog:mainfrom
florusboth:add-runtime-security-config
Aug 18, 2021
Merged

add security-agent config#706
albertvaka merged 5 commits intoDataDog:mainfrom
florusboth:add-runtime-security-config

Conversation

@florusboth
Copy link
Copy Markdown
Contributor

What does this PR do?

Add the security-agent config file and the needed update to system-prob.yaml

Motivation

Needed to activate and start the security-agent

Additional Notes

Describe your test plan

@florusboth florusboth requested a review from a team as a code owner June 15, 2021 13:34
Copy link
Copy Markdown
Contributor

@albertvaka albertvaka left a comment

Choose a reason for hiding this comment

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

Hi! Thanks a lot for contributing this ❤️ security-agent configuration was something missing in our puppet module, so it's really appreciated 🙇

I left some comments with some issues I found. If you don't have the time to make those changes, just let me know and I can make them for you.

Comment thread manifests/ubuntu.pp Outdated
Comment thread manifests/system_probe.pp Outdated
Comment thread manifests/security_agent.pp Outdated
Comment thread manifests/security_agent.pp Outdated
Comment thread manifests/security_agent.pp Outdated
Comment thread manifests/security_agent.pp Outdated
florusboth and others added 2 commits August 18, 2021 13:39
Co-authored-by: Albert Vaca Cintora <albertvaka@gmail.com>
Co-authored-by: Albert Vaca Cintora <albertvaka@gmail.com>
@albertvaka
Copy link
Copy Markdown
Contributor

LGTM, thanks a lot for your time contributing this :)

@albertvaka albertvaka merged commit 23e70dd into DataDog:main Aug 18, 2021
cegeka-jenkins pushed a commit to cegeka/puppet-datadog_agent that referenced this pull request Feb 5, 2026
Add the security-agent. config file and the needed update to system-probe.yaml
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.

3 participants