Skip to content

Add post-release workflow step to create a PR with version bump #4765

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

Merged
merged 12 commits into from
Jul 15, 2025

Conversation

y9v
Copy link
Member

@y9v y9v commented Jun 27, 2025

What does this PR do?
This PR adds a Github Actions workflow called Bump datadog gem version that generates a PR with the next prerelease version. This workflow is triggered manually.

This is done using existing rake version:bump task that increases the minor version and sets the tiny version to zero.

Motivation:
We want to have the next prerelease version in our version.rb file to simplify system-tests setup.

Change log entry
None. This change is internal.

Additional Notes:
This should not be merged yet, since we will need to do some changes to fast_castle to not break our release process.

How to test the change?
Here is the generated PR: #4797 (commit message has been fixed)

@y9v y9v self-assigned this Jun 27, 2025
@y9v y9v requested a review from a team as a code owner June 27, 2025 15:46
@github-actions github-actions bot added the dev/github Github repository maintenance and automation label Jun 27, 2025
@y9v y9v added the do-not-merge/WIP Not ready for merge label Jun 27, 2025
@codecov-commenter
Copy link

codecov-commenter commented Jun 27, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.54%. Comparing base (2a76c95) to head (8b032b3).
Report is 7 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4765      +/-   ##
==========================================
- Coverage   97.55%   97.54%   -0.01%     
==========================================
  Files        1484     1484              
  Lines       88722    88729       +7     
  Branches     4606     4606              
==========================================
  Hits        86553    86553              
- Misses       2169     2176       +7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@datadog-datadog-prod-us1
Copy link
Contributor

Datadog Report

Branch report: generate-pr-to-update-version-in-publish-workflow
Commit report: 9f9191a
Test service: dd-trace-rb

✅ 0 Failed, 21794 Passed, 1296 Skipped, 4m 54.14s Total Time

@pr-commenter
Copy link

pr-commenter bot commented Jun 27, 2025

Benchmarks

Benchmark execution time: 2025-07-15 10:16:28

Comparing candidate commit 8b032b3 in PR branch generate-pr-to-update-version-in-publish-workflow with baseline commit 2a76c95 in branch master.

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

scenario:error - error tracking with http - all

  • 🟥 throughput [-151.182op/s; -147.650op/s] or [-96.244%; -93.995%]

scenario:error - error tracking with http - third_party only

  • 🟥 throughput [-149.445op/s; -147.074op/s] or [-95.911%; -94.389%]

scenario:error - error tracking with http - user code only

  • 🟥 throughput [-151.008op/s; -148.121op/s] or [-96.133%; -94.295%]

scenario:error - without error tracking with http

  • 🟥 throughput [-149.853op/s; -146.188op/s] or [-96.218%; -93.865%]

Copy link
Member

@Strech Strech left a comment

Choose a reason for hiding this comment

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

The only question is fast_castle, will be there a clash on release?

@y9v
Copy link
Member Author

y9v commented Jun 30, 2025

@Strech yes, there will be, and there is one more problem - we need to know the previous released version, and with this change we won't have it. So there will be another PR for fast_castle this week

@y9v y9v force-pushed the generate-pr-to-update-version-in-publish-workflow branch 2 times, most recently from c5c475a to 79f23df Compare July 11, 2025 09:07
@y9v y9v force-pushed the generate-pr-to-update-version-in-publish-workflow branch from 79f23df to 8732484 Compare July 11, 2025 09:52
@y9v y9v force-pushed the generate-pr-to-update-version-in-publish-workflow branch 3 times, most recently from 7dfedb5 to e504446 Compare July 11, 2025 15:37
@y9v y9v force-pushed the generate-pr-to-update-version-in-publish-workflow branch from e504446 to fe463e2 Compare July 11, 2025 15:43
@y9v y9v requested a review from TonyCTHsu July 11, 2025 16:14
@y9v y9v force-pushed the generate-pr-to-update-version-in-publish-workflow branch from 321b9d5 to 9b2817f Compare July 14, 2025 14:32
@y9v y9v requested a review from TonyCTHsu July 14, 2025 15:07
@y9v y9v force-pushed the generate-pr-to-update-version-in-publish-workflow branch from 7270091 to 32032e4 Compare July 15, 2025 09:20
@y9v y9v force-pushed the generate-pr-to-update-version-in-publish-workflow branch from 32032e4 to e146741 Compare July 15, 2025 09:35
@y9v y9v merged commit 3492127 into master Jul 15, 2025
444 checks passed
@y9v y9v deleted the generate-pr-to-update-version-in-publish-workflow branch July 15, 2025 10:21
@github-actions github-actions bot added this to the 2.19.0 milestone Jul 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev/github Github repository maintenance and automation do-not-merge/WIP Not ready for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants