Skip to content
This repository has been archived by the owner on Dec 11, 2023. It is now read-only.

Rename OneAgent app-only directory #251

Merged
merged 7 commits into from
May 13, 2020
Merged

Rename OneAgent app-only directory #251

merged 7 commits into from
May 13, 2020

Conversation

lrgar
Copy link
Contributor

@lrgar lrgar commented May 11, 2020

If we use /opt/dynatrace/oneagent to store the PaaS agent, and the full-stack agent is installed as well, then the init container will try to overwrite the files from the full-stack agent with the PaaS files, which will fail because of the lack of permissions (or not if the init container is running as root.)

While we don't define an expected behavior for this scenario, better to use different directories to avoid breaking stuff.

Using /opt/dynatrace/oneagent-paas for now.

@lrgar lrgar requested a review from DTMad May 11, 2020 15:39
@esukram
Copy link
Member

esukram commented May 11, 2020

Luis, why not having the paas directory configurable via CR and defaulting to your suggestion?

Markus

@lrgar
Copy link
Contributor Author

lrgar commented May 12, 2020

Luis, why not having the paas directory configurable via CR and defaulting to your suggestion?

May I ask about the use case? For example, if it would be a workaround for some conflicts with certain containers, then that sounds more like a Pod-specific thing and would better be configurable as a Pod annotation than to apply to all Pods.

@lrgar
Copy link
Contributor Author

lrgar commented May 12, 2020

Followed-up with Marco and decided to support an annotation on the Pods to allow configuration to a different path, e.g., oneagent.dynatrace.com/install-path: "/custom/dir" so will modify this PR accordingly.

@lrgar
Copy link
Contributor Author

lrgar commented May 12, 2020

PR updated, the customer containers will now have the agent on the /opt/dynatrace/oneagent-paas directory by default, but can be configured with the oneagent.dynatrace.com/install-path annotation on the Pods.

The init container however has been modified to only use /mnt/oneagent to extract the files to, since the directory shouldn't matter much for this container.

@lrgar lrgar requested a review from DTMad May 12, 2020 14:20
@lrgar lrgar merged commit 1f3ec4b into master May 13, 2020
@lrgar lrgar deleted the bugfix/rename-paas-dir branch May 13, 2020 08:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants