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

Set the securityContext for the HTTPD Apache instrumentation #2051

Merged
merged 11 commits into from
Oct 16, 2023

Conversation

iblancasa
Copy link
Contributor

Resolves #2050 .

@@ -145,7 +145,7 @@ func injectApacheHttpdagent(_ logr.Logger, apacheSpec v1alpha1.ApacheHttpd, pod
Command: []string{"/bin/sh", "-c"},
Args: []string{
// Copy agent binaries to shared volume
"cp -ar /opt/opentelemetry/* " + apacheAgentDirFull + " && " +
Copy link
Member

Choose a reason for hiding this comment

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

@chrlic could you please review?

@iblancasa
Copy link
Contributor Author

#2069 is a blocker for this PR.

@iblancasa iblancasa marked this pull request as ready for review October 13, 2023 12:07
@iblancasa iblancasa requested a review from a team October 13, 2023 12:07
@iblancasa
Copy link
Contributor Author

Blocked by #2225

@iblancasa iblancasa requested a review from pavolloffay October 13, 2023 14:37
@pavolloffay
Copy link
Member

CI failed

@pavolloffay pavolloffay merged commit 83a7b34 into open-telemetry:main Oct 16, 2023
30 checks passed
ItielOlenick pushed a commit to ItielOlenick/opentelemetry-operator that referenced this pull request May 1, 2024
…lemetry#2051)

* Set the securityContext for the HTTPD Apache instrumentation

Signed-off-by: Israel Blancas <[email protected]>

* Fix unit tests

Signed-off-by: Israel Blancas <[email protected]>

* Add changelog

Signed-off-by: Israel Blancas <[email protected]>

---------

Signed-off-by: Israel Blancas <[email protected]>
Co-authored-by: Jacob Aronoff <[email protected]>
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.

SecurityContext is not propagated to apache-httpd instrumentation init containers.
3 participants