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

Add notification channel for build pipeline #4412

Merged
merged 1 commit into from
Feb 19, 2025

Conversation

randomanderson
Copy link
Contributor

What does this PR do?
Adds a notification channel for the build pipeline

Motivation:
One-pipeline added notification messages for failed releases. More notifications may be added in the future

Change log entry

@randomanderson randomanderson added dev/ci Involves CircleCI, GitHub Actions, or GitLab dev/internal Other internal work that does not need to be included in the changelog labels Feb 19, 2025
@randomanderson randomanderson requested a review from a team as a code owner February 19, 2025 21:18
Copy link

👋 Hey @DataDog/ruby-guild, please fill "Change log entry" section in the pull request description.

If changes need to be present in CHANGELOG.md you can state it this way

**Change log entry**

Yes. A brief summary to be placed into the CHANGELOG.md

(possible answers Yes/Yep/Yeah)

Or you can opt out like that

**Change log entry**

None.

(possible answers No/Nope/None)

Visited at: 2025-02-19 21:18:57 UTC

@datadog-datadog-prod-us1
Copy link
Contributor

Datadog Report

Branch report: landerson/notification
Commit report: b5d0766
Test service: dd-trace-rb

✅ 0 Failed, 20607 Passed, 1372 Skipped, 3m 7.2s Total Time

@pr-commenter
Copy link

pr-commenter bot commented Feb 19, 2025

Benchmarks

Benchmark execution time: 2025-02-19 21:40:20

Comparing candidate commit b5d0766 in PR branch landerson/notification with baseline commit f710dac in branch master.

Found 0 performance improvements and 1 performance regressions! Performance is the same for 30 metrics, 2 unstable metrics.

scenario:tracing - Propagation - Datadog

  • 🟥 throughput [-3500.114op/s; -3417.257op/s] or [-10.562%; -10.312%]

@randomanderson randomanderson merged commit 2df231f into master Feb 19, 2025
497 checks passed
@randomanderson randomanderson deleted the landerson/notification branch February 19, 2025 21:48
@github-actions github-actions bot added this to the 2.11.0 milestone Feb 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev/ci Involves CircleCI, GitHub Actions, or GitLab dev/internal Other internal work that does not need to be included in the changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants