Skip to content

Conversation

@vitorvasc
Copy link
Member

  • Removes an invalid entry for the Sentry Software vendor, which is returning an HTTP 404
  • Contributes to Refresh refcache #8494

Since this entry has no contact, its URL doesn't exist anymore, and there's no accurate replacement for it, we've discussed the following in #8494:

This entry is a 404,

- name: Sentry Software
nativeOTLP: true
url: https://www.sentrysoftware.com/products/hardware-sentry-opentelemetry-collector.html
contact:
oss: false

@svrnm @vitorvasc - I'd say, if an entry has no contact, and it doesn't exist anymore, that we drop it. WDYT? In case we don't want that in this case ... The only two entries I've found on their website are:

Originally posted by @chalin in #8494 (comment)

I agree. Looking at web.archive.org, there's an entry from earlier this year showing that this page also had a redirect to an URL that no longer exists, see: https://web.archive.org/web/20250415130819/https://www.sentrysoftware.com/products/hardware-sentry-opentelemetry-collector.html.

Originally posted by @vitorvasc in #8494 (comment)

@svrnm
Copy link
Member

svrnm commented Nov 26, 2025

@bertysentry can you take a look here

vitorvasc added a commit that referenced this pull request Nov 26, 2025
@bertysentry
Copy link
Contributor

@iguitton We need to fix this link asap! (in OpenTelemetry's Web site) Thank you!

@iguitton
Copy link
Contributor

iguitton commented Dec 1, 2025

@vitorvasc , Hardware Sentry OpenTelemetry has been replaced by MetricsHub.

can you please update the Sentry Software section with the one below?

- name: MetricsHub
  nativeOTLP: true
  url: https://www.metricshub.com
  contact:
  oss: true
  commercial: true

Thanks

@vitorvasc
Copy link
Member Author

Thanks @bertysentry @iguitton for the information!

Do you have any contact, could be an email or a GitHub handle, that we can reach out to in the future if needed?

@vitorvasc vitorvasc force-pushed the vendors_remove-entry-invalid-url branch from 2248e59 to 9c6dac2 Compare December 1, 2025 16:39
@vitorvasc
Copy link
Member Author

Oh, I see we already have it - it was added via #5378. The contact information is @iguitton's GitHub handle:

- name: MetricsHub
  nativeOTLP: true
  url: https://www.metricshub.com/docs/latest/
  contact: https://github.com/iguitton
  oss: true
  commercial: true

If there's nothing to update in this entry, I think we're good to move forward with the merge.

@iguitton
Copy link
Contributor

iguitton commented Dec 1, 2025

@vitorvasc , you can reach out to me (@iguitton) or @bertysentry in the future if needed.

@vitorvasc
Copy link
Member Author

Thank you so much @iguitton!

Mentioning @open-telemetry/docs-approvers for a rubber stamp here 😄

@vitorvasc vitorvasc enabled auto-merge December 1, 2025 16:49
@svrnm svrnm removed the blocked label Dec 4, 2025
@vitorvasc vitorvasc added this pull request to the merge queue Dec 4, 2025
Merged via the queue into open-telemetry:main with commit 8b25b18 Dec 4, 2025
22 checks passed
@vitorvasc vitorvasc deleted the vendors_remove-entry-invalid-url branch December 4, 2025 10:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants