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 DD_VERSION #3091

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add DD_VERSION #3091

wants to merge 1 commit into from

Conversation

n0ts
Copy link
Contributor

@n0ts n0ts commented Dec 21, 2016

Hi, I add DD_VERSION environment variable in datadog agent scripts.
Could you review it?

Copy link
Contributor

@masci masci left a comment

Choose a reason for hiding this comment

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

A couple of nitpicks.
Out of curiosity, why do you need the DD_VERSION variable?

@@ -133,7 +139,12 @@ if [ $OS = "RedHat" ]; then
$sudo_cmd yum -y remove datadog-agent-base
fi
fi
$sudo_cmd yum -y --disablerepo='*' --enablerepo='datadog' install datadog-agent || $sudo_cmd yum -y install datadog-agent
if [ -n "$dd_version" ]; then
Copy link
Contributor

Choose a reason for hiding this comment

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

What about assigning dd_agent variable in the if/else block at line 58?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

dd_agent like 5.8.5

@@ -159,7 +170,12 @@ determine the cause.
If the cause is unclear, please contact Datadog support.
*****
"
$sudo_cmd apt-get install -y --force-yes datadog-agent
if [ -n "$dd_version" ]; then
Copy link
Contributor

Choose a reason for hiding this comment

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

Same as above

@@ -174,8 +190,13 @@ elif [ $OS = "SUSE" ]; then
echo -e "\033[34m\n* Refreshing repositories\n\033[0m"
$sudo_cmd zypper --non-interactive refresh

if [ -n "$dd_version" ]; then
Copy link
Contributor

Choose a reason for hiding this comment

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

Same

@masci masci added this to the Triage milestone Jan 24, 2017
Copy link
Contributor Author

@n0ts n0ts left a comment

Choose a reason for hiding this comment

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

I want to all same datadog agent version in my production servers.

@@ -133,7 +139,12 @@ if [ $OS = "RedHat" ]; then
$sudo_cmd yum -y remove datadog-agent-base
fi
fi
$sudo_cmd yum -y --disablerepo='*' --enablerepo='datadog' install datadog-agent || $sudo_cmd yum -y install datadog-agent
if [ -n "$dd_version" ]; then
Copy link
Contributor Author

Choose a reason for hiding this comment

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

dd_agent like 5.8.5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants