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

add emptyDir agent and CR volume for SA receiver collectd config #839

Merged
merged 2 commits into from
Jul 18, 2023

Conversation

rmfitzpatrick
Copy link
Contributor

In cases like openshift's default SecurityContextConstraints when the agent is set to have a read-only root filesystem, collectd monitor use isn't possible without having a declared volume mount for its run directory:

error	collectd/monitorcore.go:126	Could not render collectd plugin config	{"kind": "receiver", "name": "smartagent/collectd/mysql", "data_type": "metrics", "error": "failed to create collectd config dir at /usr/lib/splunk-otel-collector/agent-bundle/run/collectd/monitor-smartagentcollectdmysql/managed_config: mkdir /usr/lib/splunk-otel-collector/agent-bundle/run/collectd/monitor-smartagentcollectdmysql: read-only file system", "monitorID": "smartagentcollectdmysql", "monitorType": "collectd/mysql", "path": "/usr/lib/splunk-otel-collector/agent-bundle/run/collectd/monitor-smartagentcollectdmysql/managed_config/20-mysql.smartagentcollectdmysql.conf"}

@rmfitzpatrick rmfitzpatrick requested review from a team as code owners June 28, 2023 19:34
@rmfitzpatrick
Copy link
Contributor Author

rmfitzpatrick commented Jun 28, 2023

It'd probably make sense to add these to the CR deployment as well.

edit: updated.

@rmfitzpatrick rmfitzpatrick changed the title add emptyDir daemonset volume for SA receiver collectd config add emptyDir agent and CR volume for SA receiver collectd config Jun 28, 2023
@rmfitzpatrick rmfitzpatrick merged commit 9a28a6d into main Jul 18, 2023
@atoulme atoulme deleted the osagentbundle branch September 11, 2023 20:13
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