Migrate from opencensus library: extension/healthcheck #33469
Assignees
Labels
never stale
Issues marked with this label will be never staled and automatically removed
Comments
codeboten
changed the title
codeboten
added a commit
to codeboten/opentelemetry-collector-contrib
that referenced
this issue
Jun 10, 2024
This PR removes the dependency on opencensus by removing the code that sets up a view to trigger healthcheck status changes. This functionality does not work as documented in the README: https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/extension/healthcheckextension#health-check I'm proposing to remove the code that brings an unnecessary dependency for functionality that isn't working. Fixes open-telemetry#33469 Signed-off-by: Alex Boten <223565+codeboten@users.noreply.github.com>
codeboten
added
the
never stale
codeboten
added a commit
that referenced
this issue
Jun 11, 2024
This PR removes the dependency on opencensus by removing the code that sets up a view to trigger healthcheck status changes. This functionality does not work as documented in the README: https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/extension/healthcheckextension#health-check I'm proposing to remove the code that brings an unnecessary dependency for functionality that isn't working. Fixes #33469 As discussed previously #29782, an improved functionality will be implemented in the v2 healthcheck extension --------- Signed-off-by: Alex Boten <223565+codeboten@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
No description provided.
The text was updated successfully, but these errors were encountered: