Skip to content

feat(java): remove SNAPSHOT tracking from registry - #1340

Merged
jaydeluca merged 1 commit into
open-telemetry:mainfrom
SurbhiAgarwal1:feat/1152-remove-java-snapshot
Oct 27, 2026
Merged

jaydeluca merged 1 commit into
open-telemetry:mainfrom
SurbhiAgarwal1:feat/1152-remove-java-snapshot

Conversation

@SurbhiAgarwal1

Copy link
Copy Markdown
Contributor

What changed

Removed SNAPSHOT tracking from the Java instrumentation watcher and cleaned up existing snapshot data:

  • Removed update_snapshot() from InstrumentationSync and updated sync() to only backfill missing README indexes and process new releases.
  • Removed snapshot reporting from main.py sync summary.
  • Deleted ecosystem-registry/java/javaagent/v2.33.2-SNAPSHOT/ and pruned the 21 unreferenced orphaned READMEs from ecosystem-registry/java/javaagent/library_readmes/.
  • Updated unit tests in test_instrumentation_sync.py and documentation in java-instrumentation-watcher/README.md.

Why

Part of #1152.

As discussed in #1152, tracking unreleased SNAPSHOT versions from main in the registry adds no practical value since neither explorer-db-builder nor v1-registry-sync consumes snapshot data (the user-facing explorer only displays official releases). Removing snapshot tracking eliminates nightly automated PR noise when there are no new releases upstream. Per discussion with @jaydeluca, we are removing snapshots one language per PR, starting with Java.

How it was tested

  • Ran pytest ecosystem-automation/java-instrumentation-watcher/ (all 104 unit tests passed).
  • Ran ruff check ecosystem-automation/java-instrumentation-watcher (clean).
  • Ran ruff format --check ecosystem-automation/java-instrumentation-watcher (clean).
  • Verified prune_orphan_readmes() cleanly pruned only the 21 unreferenced READMEs associated with the snapshot.

Is this a breaking change?

No. Downstream consumers (explorer-db-builder, etc.) already filter out SNAPSHOT directories.

Special notes for your reviewer

This is the first PR in the series addressing #1152 (Java). Once all languages have had snapshot tracking removed, the shared methods in watcher-common will be cleaned up in a follow-up PR.

Checklist

  • Tests added or updated for new behavior
  • Screenshots attached for any UI changes
  • I wrote this PR description myself (AI tools must not check this box on behalf of the author)

Stop tracking unreleased SNAPSHOT versions from the main branch in ecosystem-registry/java/javaagent and remove existing snapshot directory and unreferenced README files. The java-instrumentation-watcher now tracks official releases only. Part of open-telemetry#1152.

Signed-off-by: Surbhi Agarwal <SurbhiAgarwal1@users.noreply.github.com>
@SurbhiAgarwal1
SurbhiAgarwal1 requested review from a team as code owners October 27, 2026 16:37
Copilot AI balanced review requested due to automatic review settings October 27, 2026 16:37

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

@netlify

netlify Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for otel-ecosystem-explorer ready!

Name Link
🔨 Latest commit ebeed40
🔍 Latest deploy log https://app.netlify.com/projects/otel-ecosystem-explorer/deploys/6ae0d3303b5ac5db35f1285e
😎 Deploy Preview https://deploy-preview-1340--otel-ecosystem-explorer.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@opentelemetry-pr-dashboard

opentelemetry-pr-dashboard Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

Pull request dashboard status

Merged · refreshed 2026-10-27 17:12 UTC

Status above doesn't look right?
  • Anything look wrong? Report it with what you expected; it helps us improve the dashboard.

@jaydeluca
jaydeluca added this pull request to the merge queue Oct 27, 2026
Merged via the queue into open-telemetry:main with commit ab41acc Oct 27, 2026
32 checks passed
@otelbot

otelbot Bot commented Oct 27, 2026

Copy link
Copy Markdown
Contributor

Thank you for your contribution @SurbhiAgarwal1! 🎉 We would like to hear from you about your experience contributing to OpenTelemetry by taking a few minutes to fill out this survey.

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.

3 participants