Skip to content

Fix gradle change detection for config tasks#8181

Merged
jack-berg merged 1 commit intoopen-telemetry:mainfrom
jack-berg:fix-sdk-incubator-gradle
Mar 12, 2026
Merged

Fix gradle change detection for config tasks#8181
jack-berg merged 1 commit intoopen-telemetry:mainfrom
jack-berg:fix-sdk-incubator-gradle

Conversation

@jack-berg
Copy link
Copy Markdown
Member

With the existing setup, you get build flakes when switching between branches where the opentelemetry-configuration version has changes because the tasks aren't properly setup for gradle change detection.

  • Move the downloaded zip to tmp/ so it's outside unzipConfigurationSchema's output directory (configuration/)
  • Encode the version in the zip filename so that a configurationTag bump produces a new path, triggering a fresh download despite overwrite(false); overwrite(false) is retained to
    avoid redundant network requests on repeat builds (the de.undercouch plugin always reports itself as not up-to-date)
  • Switch unzipConfigurationSchema from Copy to Sync so stale files from old schema versions are removed

@jack-berg jack-berg requested a review from a team as a code owner March 11, 2026 22:14
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.29%. Comparing base (839f235) to head (c534422).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #8181   +/-   ##
=========================================
  Coverage     90.29%   90.29%           
  Complexity     7650     7650           
=========================================
  Files           843      843           
  Lines         23059    23059           
  Branches       2309     2309           
=========================================
  Hits          20822    20822           
  Misses         1519     1519           
  Partials        718      718           

☔ 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.

@jack-berg jack-berg merged commit c383dc5 into open-telemetry:main Mar 12, 2026
27 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.

2 participants