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

ci: Notify slack of codegen failures #5273

Merged

Conversation

jonathan-innis
Copy link
Contributor

Fixes #N/A

Description

This PR adds handling to send slack messages over to the notification channel when the codegen for Karpenter fails

How was this change tested?

Does this change impact docs?

  • Yes, PR includes docs updates
  • Yes, issue opened: #
  • No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@jonathan-innis jonathan-innis requested a review from a team as a code owner December 7, 2023 21:37
Copy link

netlify bot commented Dec 7, 2023

Deploy Preview for karpenter-docs-prod canceled.

Name Link
🔨 Latest commit dceb3dc
🔍 Latest deploy log https://app.netlify.com/sites/karpenter-docs-prod/deploys/65723b5348434a000805d7c0

@coveralls
Copy link

coveralls commented Dec 7, 2023

Pull Request Test Coverage Report for Build 7134314948

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage decreased (-0.02%) to 82.478%

Files with Coverage Reduction New Missed Lines %
pkg/providers/amifamily/ami.go 1 90.32%
Totals Coverage Status
Change from base Build 7134237878: -0.02%
Covered Lines: 4919
Relevant Lines: 5964

💛 - Coveralls

@jonathan-innis jonathan-innis merged commit 2790e61 into aws:main Dec 7, 2023
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants