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

Send an event to the notifications-service when there are no failed r… #4043

Merged
merged 2 commits into from
Jul 7, 2020

Conversation

satellite15
Copy link

…esources in a failed client run

Signed-off-by: Martin Scott mscott@chef.io

See #4039

🔩 Description: What code changed, and why?

Send an event to the notifications-service for a failed client run that does not have a failed resource.

⛓️ Related Resources

👍 Definition of Done

👟 How to Build and Test the Change

Build and deploy the automate-gateway

Set up a notification for slack, webhook and servicenow (can be same destination as webhook).
Perform a client run for a cookbook that does not compile. This is fairly easy to do, just type some random words into a cookbook and it will not compile.

Verify notifications are received.

✅ Checklist

📷 Screenshots, if applicable

Martin Scott added 2 commits July 7, 2020 11:44
…esources in a failed client run

Signed-off-by: Martin Scott <mscott@chef.io>
Signed-off-by: Martin Scott <mscott@chef.io>
@satellite15 satellite15 added customer-reported issues reported by customers notifications This issue or pull request pertains to notifications in Automate labels Jul 7, 2020
@satellite15 satellite15 self-assigned this Jul 7, 2020
Copy link
Contributor

@lancewf lancewf left a comment

Choose a reason for hiding this comment

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

Works for me. Thanks for fixing it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
customer-reported issues reported by customers notifications This issue or pull request pertains to notifications in Automate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Include client run failures without resources in notifications
2 participants