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

Deprecate componenttest/obsreporttest check metrics functions #12185

Open
16 tasks done
sincejune opened this issue Jan 27, 2025 · 0 comments
Open
16 tasks done

Deprecate componenttest/obsreporttest check metrics functions #12185

sincejune opened this issue Jan 27, 2025 · 0 comments

Comments

@sincejune
Copy link
Contributor

sincejune commented Jan 27, 2025

componenttest/obsreporttest is an untested file and we're planning to replace it with metadata.AssertMetrics and metadatatest.AssertEqualMetric series functions.

github-merge-queue bot pushed a commit that referenced this issue Jan 27, 2025

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
Deprecate `CheckExporterLogs` functions, also verified
`CheckExporterLogs` function is not being used in the contrib repo as
well.

<!-- Issue number if applicable -->
#### Link to tracking issue
Part of #12185 

<!--Describe what testing was performed and which tests were added.-->
#### Testing
Updated

<!--Describe the documentation added.-->
#### Documentation
Added

<!--Please delete paragraphs that you did not use before submitting.-->
github-merge-queue bot pushed a commit that referenced this issue Jan 28, 2025

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
Deprecate `CheckReceiverLogs` functions, also verified
`CheckReceiverLogs` function is not being used in the contrib repo as
well.

<!-- Issue number if applicable -->
#### Link to tracking issue
Part of #12185 

<!--Describe what testing was performed and which tests were added.-->
#### Testing
Updated

<!--Describe the documentation added.-->
#### Documentation
Added

<!--Please delete paragraphs that you did not use before submitting.-->
github-merge-queue bot pushed a commit that referenced this issue Jan 28, 2025

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
…#12198)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
Deprecate `CheckExporterEnqueue*` functions, also verified
`CheckExporterEnqueue*` functions are not being used in the contrib repo
as well.


<!-- Issue number if applicable -->
#### Link to tracking issue
Part of #12185 

<!--Describe what testing was performed and which tests were added.-->
#### Testing
Updated

<!--Describe the documentation added.-->
#### Documentation
Added

<!--Please delete paragraphs that you did not use before submitting.-->
github-merge-queue bot pushed a commit that referenced this issue Jan 28, 2025

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
…12200)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
Deprecate `CheckExporterMetrics` functions, also verified
`CheckExporterMetrics` function is not being used in the contrib repo as
well.

<!-- Issue number if applicable -->
#### Link to tracking issue
Part of #12185 

<!--Describe what testing was performed and which tests were added.-->
#### Testing
Updated

<!--Describe the documentation added.-->
#### Documentation
Added

<!--Please delete paragraphs that you did not use before submitting.-->
github-merge-queue bot pushed a commit that referenced this issue Jan 28, 2025

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
…12199)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
Deprecate `CheckExporterTraces` functions, also verified
`CheckExporterTraces` function is not being used in the contrib repo as
well.

<!-- Issue number if applicable -->
#### Link to tracking issue
Part of #12185 

<!--Describe what testing was performed and which tests were added.-->
#### Testing
Updated

<!--Describe the documentation added.-->
#### Documentation
Added

<!--Please delete paragraphs that you did not use before submitting.-->
github-merge-queue bot pushed a commit that referenced this issue Jan 30, 2025

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
…ons (#12210)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
Deprecate `CheckReceiverMetricGauge` functions, also verified
`CheckReceiverMetricGauge` function is not being used in the contrib
repo as well.

<!-- Issue number if applicable -->
#### Link to tracking issue
Part of #12185 

<!--Describe what testing was performed and which tests were added.-->
#### Testing
Updated

<!--Describe the documentation added.-->
#### Documentation
Added

<!--Please delete paragraphs that you did not use before submitting.-->

---------

Co-authored-by: Bogdan Drutu <bogdandrutu@gmail.com>
sfc-gh-sili pushed a commit to sfc-gh-sili/opentelemetry-collector that referenced this issue Feb 3, 2025
…-telemetry#12186)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
Deprecate `CheckExporterLogs` functions, also verified
`CheckExporterLogs` function is not being used in the contrib repo as
well.

<!-- Issue number if applicable -->
#### Link to tracking issue
Part of open-telemetry#12185 

<!--Describe what testing was performed and which tests were added.-->
#### Testing
Updated

<!--Describe the documentation added.-->
#### Documentation
Added

<!--Please delete paragraphs that you did not use before submitting.-->
sfc-gh-sili pushed a commit to sfc-gh-sili/opentelemetry-collector that referenced this issue Feb 3, 2025
…-telemetry#12187)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
Deprecate `CheckReceiverLogs` functions, also verified
`CheckReceiverLogs` function is not being used in the contrib repo as
well.

<!-- Issue number if applicable -->
#### Link to tracking issue
Part of open-telemetry#12185 

<!--Describe what testing was performed and which tests were added.-->
#### Testing
Updated

<!--Describe the documentation added.-->
#### Documentation
Added

<!--Please delete paragraphs that you did not use before submitting.-->
sfc-gh-sili pushed a commit to sfc-gh-sili/opentelemetry-collector that referenced this issue Feb 3, 2025
…open-telemetry#12198)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
Deprecate `CheckExporterEnqueue*` functions, also verified
`CheckExporterEnqueue*` functions are not being used in the contrib repo
as well.


<!-- Issue number if applicable -->
#### Link to tracking issue
Part of open-telemetry#12185 

<!--Describe what testing was performed and which tests were added.-->
#### Testing
Updated

<!--Describe the documentation added.-->
#### Documentation
Added

<!--Please delete paragraphs that you did not use before submitting.-->
sfc-gh-sili pushed a commit to sfc-gh-sili/opentelemetry-collector that referenced this issue Feb 3, 2025
…pen-telemetry#12200)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
Deprecate `CheckExporterMetrics` functions, also verified
`CheckExporterMetrics` function is not being used in the contrib repo as
well.

<!-- Issue number if applicable -->
#### Link to tracking issue
Part of open-telemetry#12185 

<!--Describe what testing was performed and which tests were added.-->
#### Testing
Updated

<!--Describe the documentation added.-->
#### Documentation
Added

<!--Please delete paragraphs that you did not use before submitting.-->
sfc-gh-sili pushed a commit to sfc-gh-sili/opentelemetry-collector that referenced this issue Feb 3, 2025
…pen-telemetry#12199)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
Deprecate `CheckExporterTraces` functions, also verified
`CheckExporterTraces` function is not being used in the contrib repo as
well.

<!-- Issue number if applicable -->
#### Link to tracking issue
Part of open-telemetry#12185 

<!--Describe what testing was performed and which tests were added.-->
#### Testing
Updated

<!--Describe the documentation added.-->
#### Documentation
Added

<!--Please delete paragraphs that you did not use before submitting.-->
sfc-gh-sili pushed a commit to sfc-gh-sili/opentelemetry-collector that referenced this issue Feb 3, 2025
…ons (open-telemetry#12210)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
Deprecate `CheckReceiverMetricGauge` functions, also verified
`CheckReceiverMetricGauge` function is not being used in the contrib
repo as well.

<!-- Issue number if applicable -->
#### Link to tracking issue
Part of open-telemetry#12185 

<!--Describe what testing was performed and which tests were added.-->
#### Testing
Updated

<!--Describe the documentation added.-->
#### Documentation
Added

<!--Please delete paragraphs that you did not use before submitting.-->

---------

Co-authored-by: Bogdan Drutu <bogdandrutu@gmail.com>
github-merge-queue bot pushed a commit that referenced this issue Feb 5, 2025

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
…12227)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
Deprecate `CheckReceiverTraces` functions

<!-- Issue number if applicable -->
#### Link to tracking issue
Part of #12185 

<!--Describe what testing was performed and which tests were added.-->
#### Testing
Updated

<!--Describe the documentation added.-->
#### Documentation
Added

<!--Please delete paragraphs that you did not use before submitting.-->
mx-psi pushed a commit to open-telemetry/opentelemetry-collector-contrib that referenced this issue Feb 10, 2025

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
…#37623)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
Remove references of `componenttest.CheckReceiver*` functions

<!-- Issue number (e.g. #1234) or full URL to issue, if applicable. -->
#### Link to tracking issue
Part of
open-telemetry/opentelemetry-collector#12185

<!--Describe what testing was performed and which tests were added.-->
#### Testing
Updated

<!--Describe the documentation added.-->
#### Documentation
n/a

<!--Please delete paragraphs that you did not use before submitting.-->
github-merge-queue bot pushed a commit that referenced this issue Feb 18, 2025

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
#12420)

…s and CheckReceiverTraces functions

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
Removing the deprecated functions

<!-- Issue number if applicable -->
#### Link to tracking issue
Part of #12185 

<!--Describe what testing was performed and which tests were added.-->
#### Testing
n/a

<!--Describe the documentation added.-->
#### Documentation
n/a

<!--Please delete paragraphs that you did not use before submitting.-->
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

No branches or pull requests

1 participant