Skip to content
This repository has been archived by the owner on Feb 29, 2024. It is now read-only.

ansible-runner: Drop flag files for failing environments #363

Merged
merged 1 commit into from
May 2, 2017

Conversation

gandelman-a
Copy link
Contributor

This updates ansible-runner to drop empty files in a known
directory for environments that are failing to complete their
playbooks. The files are named after the failing environment.
Upon successful ansible run, these files are cleaned up if
they exist.

This'll allow us to set a datadog check that fails when any
files exist here. Our current datadog check is broken and
relies on log scraping and doesn't really work with multi-env
bastions.

Related-Issue: BonnyCI/projman#224

Signed-off-by: Adam Gandelman [email protected]

This updates ansible-runner to drop empty files in a known
directory for environments that are failing to complete their
playbooks. The files are named after the failing environment.
Upon successful ansible run, these files are cleaned up if
they exist.

This'll allow us to set a datadog check that fails when any
files exist here.  Our current datadog check is broken and
relies on log scraping and doesn't really work with multi-env
bastions.

Related-Issue: BonnyCI/projman#224

Signed-off-by: Adam Gandelman <[email protected]>
gandelman-a added a commit to gandelman-a/hoist that referenced this pull request Apr 25, 2017
This adds a simple datadog check that fails if it finds any flag files for
failing ansible-runner environments.  This depends on PR BonnyCI#363
but should pass OK if it lands before that merges.

Related-Issue: BonnyCI/projman#224

Signed-off-by: Adam Gandelman <[email protected]>
gandelman-a added a commit to gandelman-a/hoist that referenced this pull request Apr 25, 2017
This adds a simple datadog check that fails if it finds any flag files for
failing ansible-runner environments.  This depends on PR BonnyCI#363
but should pass OK if it lands before that merges.

Related-Issue: BonnyCI/projman#224

Signed-off-by: Adam Gandelman <[email protected]>
Copy link
Contributor

@ConsoleCatzirl ConsoleCatzirl left a comment

Choose a reason for hiding this comment

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

Yay, more tests!

@bonnyci bonnyci bot merged commit fd66437 into BonnyCI:master May 2, 2017
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.

2 participants