Skip to content

C
configuration
Commit 1f136b42 by Feanil Patel
Options

Merge pull request #1229 from edx/feanil/optional_datadog 

Feanil/optional datadog
parents 25ccffd0 02d0b37f
Showing with 5 additions and 3 deletions
playbooks/edx-east/jenkins_master.yml → playbooks/edx-east/jenkins_testeng_master.yml
# Configure a Jenkins master instance # Configure a Jenkins master instance for testeng
# This has the Jenkins Java app, but none of the requirements # This has the Jenkins Java app, but none of the requirements
# to run the tests. # to run the tests.
...@@ -8,6 +8,9 @@ ...@@ -8,6 +8,9 @@
gather_facts: True gather_facts: True
vars: vars:
COMMON_DATA_DIR: "/mnt" COMMON_DATA_DIR: "/mnt"
COMMON_ENABLE_DATADOG: True
roles: roles:
- common - common
- role: datadog
when: COMMON_ENABLE_DATADOG
- jenkins_master - jenkins_master
playbooks/roles/jenkins_master/meta/main.yml
--- ---
dependencies: dependencies:
- common - common
- role: datadog
COMMON_ENABLE_DATADOG: True
playbooks/roles/jenkins_master/tasks/main.yml
...@@ -128,3 +128,4 @@ ...@@ -128,3 +128,4 @@
notify: start nginx notify: start nginx
- include: datadog.yml tags=datadog - include: datadog.yml tags=datadog
when: COMMON_ENABLE_DATADOG
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment