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.

Sign up for GitHub

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

Jump to bottom

Add tests to verify OTel integration using in-memory exporter #306

Merged

psx95 merged 1 commit into GoogleCloudPlatform:main from psx95:trace-exporter-unit-test

Mar 15, 2024

Contributor

psx95 commented Mar 14, 2024 •

edited

Loading

Adding a unit test demonstrating the use of InMemorySpanExporter to verify spans generated by the application.

Testing

Ran the sample and verified the attributes presence in the generated span.
The newly added test passed.

Re: #277

psx95 self-assigned this

psx95 mentioned this pull request

Remove usage of deprecated TraceServiceStub methods #305

Closed


          Add tests verifying spans via in-memory exporter

84bd354

psx95 force-pushed the trace-exporter-unit-test branch from c1a8d8f to 84bd354 Compare

March 14, 2024 18:55

psx95 mentioned this pull request

Please do not remove the option to set a TraceExporterStub on TraceExporter via builder #277

Closed

psx95 requested review from punya and dashpole

March 14, 2024 19:08

psx95 marked this pull request as ready for review

March 14, 2024 19:08

psx95 requested a review from a team as a code owner

March 14, 2024 19:08

dashpole approved these changes

View reviewed changes

psx95 merged commit 398e552 into GoogleCloudPlatform:main

17 checks passed

psx95 deleted the trace-exporter-unit-test branch

March 15, 2024 16:56

lqiu96 pushed a commit to googleapis/sdk-platform-java that referenced this pull request


          deps: update dependency com.google.cloud.opentelemetry:detector-resou…

e4ed176

…rces-support to v0.28.0 (#2649)

[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
|
[com.google.cloud.opentelemetry:detector-resources-support](https://togithub.com/GoogleCloudPlatform/opentelemetry-operations-java)
| `0.27.0` -> `0.28.0` |
[![age](https://developer.mend.io/api/mc/badges/age/maven/com.google.cloud.opentelemetry:detector-resources-support/0.28.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/maven/com.google.cloud.opentelemetry:detector-resources-support/0.28.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/maven/com.google.cloud.opentelemetry:detector-resources-support/0.27.0/0.28.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/maven/com.google.cloud.opentelemetry:detector-resources-support/0.27.0/0.28.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

> [!WARNING]
> Some dependencies could not be looked up. Check the Dependency
Dashboard for more information.

---

### Release Notes

<details>
<summary>GoogleCloudPlatform/opentelemetry-operations-java
(com.google.cloud.opentelemetry:detector-resources-support)</summary>

###
[`v0.28.0`](https://togithub.com/GoogleCloudPlatform/opentelemetry-operations-java/releases/tag/v0.28.0)

[Compare
Source](https://togithub.com/GoogleCloudPlatform/opentelemetry-operations-java/compare/v0.27.0...v0.28.0)

#### Release Highlights

- Add support for `CreateServiceTimeseries` in metrics exporter in
[#&#8203;318](https://togithub.com/GoogleCloudPlatform/opentelemetry-operations-java/issues/318).
- Add resource attributes as metrics labels by providing a Predicate
based filter to control which resource attributes end up as metric
labels in
[#&#8203;314](https://togithub.com/GoogleCloudPlatform/opentelemetry-operations-java/issues/314),
[#&#8203;319](https://togithub.com/GoogleCloudPlatform/opentelemetry-operations-java/issues/319).

#### What's Changed

- Add Readme for detector-resources-support module by
[@&#8203;psx95](https://togithub.com/psx95) in
[GoogleCloudPlatform/opentelemetry-operations-java#282
- Copy instrumentation quickstart out of java-docs-samples by
[@&#8203;aabmass](https://togithub.com/aabmass) in
[GoogleCloudPlatform/opentelemetry-operations-java#283
- Enable snippet-bot by [@&#8203;aabmass](https://togithub.com/aabmass)
in
[GoogleCloudPlatform/opentelemetry-operations-java#284
- Quota project fix by [@&#8203;aabmass](https://togithub.com/aabmass)
in
[GoogleCloudPlatform/opentelemetry-operations-java#286
- Use logstash-logback-encoder version that works with spring boot 2 by
[@&#8203;aabmass](https://togithub.com/aabmass) in
[GoogleCloudPlatform/opentelemetry-operations-java#285
- Update instrumentation quickstart README links by
[@&#8203;aabmass](https://togithub.com/aabmass) in
[GoogleCloudPlatform/opentelemetry-operations-java#287
- Make examples/instrumentation-quickstart a standalone gradle build by
[@&#8203;aabmass](https://togithub.com/aabmass) in
[GoogleCloudPlatform/opentelemetry-operations-java#289
- Add CI for examples/instrumentation-quickstart by
[@&#8203;aabmass](https://togithub.com/aabmass) in
[GoogleCloudPlatform/opentelemetry-operations-java#290
- Test against java 11 by
[@&#8203;aabmass](https://togithub.com/aabmass) in
[GoogleCloudPlatform/opentelemetry-operations-java#293
- Update TraceTranslator.java by
[@&#8203;WadeGulbrandsen](https://togithub.com/WadeGulbrandsen) in
[GoogleCloudPlatform/opentelemetry-operations-java#296
- feat: simplify logging in instrumentation quickstart by
[@&#8203;dashpole](https://togithub.com/dashpole) in
[GoogleCloudPlatform/opentelemetry-operations-java#298
- Update Gradle to v8.6 by [@&#8203;psx95](https://togithub.com/psx95)
in
[GoogleCloudPlatform/opentelemetry-operations-java#300
- Replace resource detector with upstream detector by
[@&#8203;psx95](https://togithub.com/psx95) in
[GoogleCloudPlatform/opentelemetry-operations-java#301
- Update links to the upstream detector by
[@&#8203;psx95](https://togithub.com/psx95) in
[GoogleCloudPlatform/opentelemetry-operations-java#302
- Add OTLP trace with ADC example by
[@&#8203;damemi](https://togithub.com/damemi) in
[GoogleCloudPlatform/opentelemetry-operations-java#297
- Update auto-exporter readme by
[@&#8203;psx95](https://togithub.com/psx95) in
[GoogleCloudPlatform/opentelemetry-operations-java#304
- Add tests to verify OTel integration using in-memory exporter by
[@&#8203;psx95](https://togithub.com/psx95) in
[GoogleCloudPlatform/opentelemetry-operations-java#306
- Add GH action step to test shadowJar by
[@&#8203;psx95](https://togithub.com/psx95) in
[GoogleCloudPlatform/opentelemetry-operations-java#309
- Add missing GOOGLE_CLOUD_QUOTA_PROJECT environment variable to compose
file by [@&#8203;aabmass](https://togithub.com/aabmass) in
[GoogleCloudPlatform/opentelemetry-operations-java#311
- Update instrumentation quickstart to new region tag name by
[@&#8203;aabmass](https://togithub.com/aabmass) in
[GoogleCloudPlatform/opentelemetry-operations-java#312
- Update instrumentation-quickstart README by
[@&#8203;psx95](https://togithub.com/psx95) in
[GoogleCloudPlatform/opentelemetry-operations-java#315
- Add resource attributes as metric labels by
[@&#8203;psx95](https://togithub.com/psx95) in
[GoogleCloudPlatform/opentelemetry-operations-java#314
- Update resource detection sample by
[@&#8203;psx95](https://togithub.com/psx95) in
[GoogleCloudPlatform/opentelemetry-operations-java#317
- Make setResourceAttributesFilter public by
[@&#8203;psx95](https://togithub.com/psx95) in
[GoogleCloudPlatform/opentelemetry-operations-java#319
- Add support for createServiceTimeSeries by
[@&#8203;psx95](https://togithub.com/psx95) in
[GoogleCloudPlatform/opentelemetry-operations-java#318
- Disable release for gcp-resource-detector by
[@&#8203;psx95](https://togithub.com/psx95) in
[GoogleCloudPlatform/opentelemetry-operations-java#321

#### New Contributors

- [@&#8203;WadeGulbrandsen](https://togithub.com/WadeGulbrandsen) made
their first contribution in
[GoogleCloudPlatform/opentelemetry-operations-java#296
- [@&#8203;damemi](https://togithub.com/damemi) made their first
contribution in
[GoogleCloudPlatform/opentelemetry-operations-java#297

**Full Changelog**:
GoogleCloudPlatform/opentelemetry-operations-java@v0.27.0...v0.28.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/googleapis/sdk-platform-java).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4zMDEuNCIsInVwZGF0ZWRJblZlciI6IjM3LjMwMS40IiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6W119-->

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment