Skip to content

hate#500

Merged
anandkumarpatel merged 1 commit intomasterfrom
IGNORE_STUPID_DD_NOTIFICATION_ERRORS
Jul 13, 2016
Merged

hate#500
anandkumarpatel merged 1 commit intomasterfrom
IGNORE_STUPID_DD_NOTIFICATION_ERRORS

Conversation

@und1sk0
Copy link
Copy Markdown
Contributor

@und1sk0 und1sk0 commented Jun 24, 2016

  • if the dd notification fails or gives a non-200, we don't care.

Reviewers

Tests

Test any modifications on one of our environments.


- name: send datadog message for deploy
when: datadog_api_key is defined and datadog_tags is defined and git_branch is defined
ignore_errors: true
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

http://docs.ansible.com/ansible/playbooks_error_handling.html

  failed_when: "'OK' not in command_result.msg"
  register: command_result

@anandkumarpatel anandkumarpatel merged commit d713c01 into master Jul 13, 2016
@anandkumarpatel anandkumarpatel deleted the IGNORE_STUPID_DD_NOTIFICATION_ERRORS branch July 13, 2016 23:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants