Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test: add obsreport test support for components #741

Closed
pjanotti opened this issue Mar 29, 2020 · 1 comment
Closed

Test: add obsreport test support for components #741

pjanotti opened this issue Mar 29, 2020 · 1 comment
Labels
help wanted Good issue for contributors to OpenTelemetry Service to pick up

Comments

@pjanotti
Copy link
Contributor

pjanotti commented Mar 29, 2020

This should be supplied so components can check if they are generating the proper observability data. This will avoid trivial mistakes like using the wrong methods for a given component.

The mistake risk can also be reduced by changing the obsreport interfaces. Either by having strong types on the interfaces per component and data type or by splitting it in packages per component and data type. Either way test helpers are valuable in both cases.

@tigrannajaryan tigrannajaryan added the help wanted Good issue for contributors to OpenTelemetry Service to pick up label Apr 22, 2020
@bogdandrutu
Copy link
Member

Fixed in #1299

hughesjj pushed a commit to hughesjj/opentelemetry-collector that referenced this issue Apr 27, 2023
Bumps golang from 1.17.0-stretch to 1.17.1-stretch.

---
updated-dependencies:
- dependency-name: golang
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Troels51 pushed a commit to Troels51/opentelemetry-collector that referenced this issue Jul 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Good issue for contributors to OpenTelemetry Service to pick up
Projects
None yet
Development

No branches or pull requests

3 participants