-
Notifications
You must be signed in to change notification settings - Fork 375
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
Provide commit sha for RelEnv #3713
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3713 +/- ##
=======================================
Coverage 97.91% 97.91%
=======================================
Files 1246 1246
Lines 74999 74999
Branches 3627 3627
=======================================
Hits 73436 73436
Misses 1563 1563 ☔ View full report in Codecov by Sentry. |
a4c9887
to
b6e83b0
Compare
BenchmarksBenchmark execution time: 2024-07-12 11:07:07 Comparing candidate commit 9cf93d3 in PR branch Found 1 performance improvements and 0 performance regressions! Performance is the same for 9 metrics, 2 unstable metrics. scenario:Propagation - Datadog
|
Uhh, I like this idea! Fingers crossed it can work nicely :) |
9d11c36
to
9d556a4
Compare
9d556a4
to
53beb04
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems reasonable, thank you 👍
@@ -232,9 +232,27 @@ onboarding_tests: | |||
variables: | |||
PRODUCT_NAME: auto_inject-ruby | |||
|
|||
save_versions: | |||
image: $DOCKER_REGISTRY/images/mirror/ruby:3.2.2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor: We may be able to use a simpler image without Ruby ;)
Motivation:
In order to release
1.x-stable
, we need to update commit sha for RelEnv deploymentWhat does this PR do?
This PR attempt to provide the compute the latest commit sha, so we don't need to manually update RelEnv.