Skip to content

Allow specifying an agent_flavor to install#686

Merged
albertvaka merged 8 commits intomasterfrom
albertvaka/add-agent-flavor
Feb 25, 2021
Merged

Allow specifying an agent_flavor to install#686
albertvaka merged 8 commits intomasterfrom
albertvaka/add-agent-flavor

Conversation

@albertvaka
Copy link
Copy Markdown
Contributor

What does this PR do?

Allow specifying an agent_flavor to install.

Motivation

Make it possible to install the IoT flavor of the Agent.

Additional notes

Caveats: If the regular Agent is installed, it will fail to install the IoT Agent until the regular Agent is uninstalled (since the two packages install the same files).

@albertvaka albertvaka requested a review from a team as a code owner February 24, 2021 12:53
Copy link
Copy Markdown
Member

@mx-psi mx-psi left a comment

Choose a reason for hiding this comment

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

LGTM!

I don't know if it's worth it but maybe we could check the agent_major_version to have a more descriptive error if someone tries to install IoT Agent version 6.x/5.x? Right now I am guessing they would just get a "pacakge not found" error which might be a bit confusing

Comment thread manifests/init.pp Outdated
@albertvaka
Copy link
Copy Markdown
Contributor Author

albertvaka commented Feb 24, 2021

maybe we could check the agent_major_version to have a more descriptive error

There are several settings that are not compatible with the iot-agent: configuring a python integration, configuring the system-probe, etc. I though about adding checks for those cases, but I think there are too many 😅

albertvaka and others added 2 commits February 24, 2021 18:11
Co-authored-by: Pablo Baeyens <pablo.baeyens@datadoghq.com>
@albertvaka albertvaka merged commit 32df664 into master Feb 25, 2021
@albertvaka albertvaka deleted the albertvaka/add-agent-flavor branch February 25, 2021 11:25
cegeka-jenkins pushed a commit to cegeka/puppet-datadog_agent that referenced this pull request Feb 5, 2026
Make it possible to install the IoT flavor of the Agent.

Caveats:
 - If the regular Agent is installed, it will fail to install the IoT
   Agent until the regular Agent is uninstalled (since the two
   packages install the same files).
- Some settings aren't compatible with the IoT flavor of the
   Agent (major version < 7, python integrations, APM...)
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.

2 participants