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

Update agent config during init only in case of installer mode #718

Merged
merged 2 commits into from
Apr 14, 2022

Conversation

0sewa0
Copy link
Contributor

@0sewa0 0sewa0 commented Apr 11, 2022

If securityContext with user/group is used on a user's pod that we inject a csi volume into, then the standalone-init will fail as it doesn't have the permissions to update the ruxitagentproc.conf file.

This happens because the csi-driver creates/updates that file with a different user/group then what the user's pod's securityContext has which is used by the standalone-init.

How can this be tested?

Have cloud-native and inject into a sample app that has a securityContext with a user/group that is different from the one that the csi-driver is using. The sample app should just work.

For bonus-points: Have app-monitoring WITHOUT csi-driver, deploy the same sample app, check that its running, and check if the ruxitagentproc.conf was updated.

  • under opt/dynatrace/oneagent-paas/agent/conf/ in the injected container, if there is a _ruxitagentproc.conf if it was updated

Checklist

  • Unit tests have been updated/added
  • PR is labeled accordingly

@0sewa0 0sewa0 requested a review from a team as a code owner April 11, 2022 08:39
@0sewa0 0sewa0 added the bug Something isn't working label Apr 14, 2022
@0sewa0 0sewa0 enabled auto-merge (squash) April 14, 2022 12:13
@0sewa0 0sewa0 merged commit f0ea28a into master Apr 14, 2022
@0sewa0 0sewa0 deleted the fix/only-get-ruxit-in-installer-mode branch April 14, 2022 12:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants