Skip to content

re-enable dependabot with 14-day cooldown#299

Closed
moezein0 wants to merge 1 commit intomainfrom
re-enable-dependabot-cooldown
Closed

re-enable dependabot with 14-day cooldown#299
moezein0 wants to merge 1 commit intomainfrom
re-enable-dependabot-cooldown

Conversation

@moezein0
Copy link
Copy Markdown
Contributor

Note

Merge only if this is still needed and your repo is not managed by ADMS.
If your repository is already managed by ADMS, feel free to close or ignore this PR.


We are adding a mandatory 14-day cooldown on dependencies to reduce the risk of zero-day vulnerabilities.

This PR re-enables your Dependabot configuration and introduces the cooldown setting. If you notice any other Dependabot configurations in your repo that are missing the cooldown, please ensure it is added.

If your repository is already managed by ADMS and no longer requires these configurations, feel free to close or ignore the PR.

Rename .disabled configs back and add cooldown: default-days: 14

Signed-off-by: Moe Zein <moe.zein@datadoghq.com>
@pr-commenter
Copy link
Copy Markdown

pr-commenter bot commented Mar 25, 2026

Benchmarks

Benchmark execution time: 2026-03-25 22:46:01

Comparing candidate commit a9f6b35 in PR branch re-enable-dependabot-cooldown with baseline commit 910e3d5 in branch main.

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

Explanation

This is an A/B test comparing a candidate commit's performance against that of a baseline commit. Performance changes are noted in the tables below as:

  • 🟩 = significantly better candidate vs. baseline
  • 🟥 = significantly worse candidate vs. baseline

We compute a confidence interval (CI) over the relative difference of means between metrics from the candidate and baseline commits, considering the baseline as the reference.

If the CI is entirely outside the configured SIGNIFICANT_IMPACT_THRESHOLD (or the deprecated UNCONFIDENCE_THRESHOLD), the change is considered significant.

Feel free to reach out to #apm-benchmarking-platform on Slack if you have any questions.

More details about the CI and significant changes

You can imagine this CI as a range of values that is likely to contain the true difference of means between the candidate and baseline commits.

CIs of the difference of means are often centered around 0%, because often changes are not that big:

---------------------------------(------|---^--------)-------------------------------->
                              -0.6%    0%  0.3%     +1.2%
                                 |          |        |
         lower bound of the CI --'          |        |
sample mean (center of the CI) -------------'        |
         upper bound of the CI ----------------------'

As described above, a change is considered significant if the CI is entirely outside the configured SIGNIFICANT_IMPACT_THRESHOLD (or the deprecated UNCONFIDENCE_THRESHOLD).

For instance, for an execution time metric, this confidence interval indicates a significantly worse performance:

----------------------------------------|---------|---(---------^---------)---------->
                                       0%        1%  1.3%      2.2%      3.1%
                                                  |   |         |         |
       significant impact threshold --------------'   |         |         |
                      lower bound of CI --------------'         |         |
       sample mean (center of the CI) --------------------------'         |
                      upper bound of CI ----------------------------------'

@moezein0
Copy link
Copy Markdown
Contributor Author

Closing this PR due to a script bug that caused YAML reformatting issues. Specifically, yaml.dump() was used to serialize the config after adding the cooldown block — this stripped quotes from string values, meaning time values like "09:00" became unquoted and can be coerced to a number (e.g., 32400) by Dependabot's YAML parser, potentially breaking the schedule. A corrected PR will follow shortly that only appends the cooldown block and preserves all original formatting exactly.

@moezein0 moezein0 closed this Mar 26, 2026
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.

1 participant