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.

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

Jimit/e2e tracing test1 Adding basic e2e Firestore Client trace test #1602

Open
wants to merge 48 commits into
base: main
Choose a base branch
from
Open

Jimit/e2e tracing test1 Adding basic e2e Firestore Client trace test #1602

wants to merge 48 commits into from

Conversation

jimit-j-shah
Copy link

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> ☕️

If you write sample code, please follow the samples format.

ehsannas and others added 30 commits December 14, 2023 16:49
@ehsannas
feat: Enable client-side tracing using OpenTelemetry.
56b6cc9
@ehsannas
Remove unused dependencies.
c32d018
@ehsannas
Merge remote-tracking branch 'origin/main' into client-side-tracing-u…
006114a 
…sing-otel-2
@ehsannas
format.
633a883
@ehsannas
fix merge error.
9d1ca65
@ehsannas
print debugging gha.
89ee6ef
@ehsannas
Set project/db for tests.
ce58e6b
@ehsannas
set project id for tests.
c12732c
@ehsannas
remove printf debugging.
9a55dbe
@ehsannas
check for null before adding settings attributes.
f544449
@ehsannas
Move integration tests to it/
ded785f
@ehsannas
remove unused deps. Add test deps.
ca0185e
@ehsannas
fix deps.
d1450f5
@ehsannas
Use EnvironmentVariablesRule to test env vars.
9e0adca
@ehsannas
remove mockito-inline.
4f14f6c 
/
@ehsannas
remove env var tests for now.
5d87b6f
@ehsannas
Move the unit test to com.google.cloud.firestore
4928864
@ehsannas
experiment with terminating the bulkwriter executor.
5c79770
@ehsannas
Merge remote-tracking branch 'origin/main' into client-side-tracing-u…
6911736 
…sing-otel-2
@ehsannas
don't add public APIs to firestore for getting the TraceUtil.
56de0f7
@ehsannas
Simpler way to pass TraceUtil to AggregateQueryResponseObserver.
9fcf1db
@ehsannas
Don't use .* imports.
178ca6c
@ehsannas
setTracingEnabled instead of setTelemetryCollectionEnabled.
968ad82
@ehsannas
Don't use .* imports.
075e12e
@ehsannas
better comment wording.
36a9b3c
@ehsannas
Better variable name.
2208f36
@ehsannas
Don't use .* import.
0222176
@ehsannas
gRPC channel configurator shouldn't use globalopentelemetry.
3e2d058
@ehsannas
Add more comments.
367669b
@ehsannas
Don't pass TraceUtil to AggregateQueryResponseObserver.
204412c
ehsannas and others added 11 commits January 8, 2024 13:51
@ehsannas
Add memory utlization as trace attrbiute.
3becd0a
@ehsannas
Remove the OpenTelemetryUtil class.
391f34a
@ehsannas
Merge remote-tracking branch 'origin/main' into client-side-tracing-u…
62fcc8e 
…sing-otel-2-new
@ehsannas
Add more trace tests.
449aa91
@gcf-owl-bot
🦉 Updates from OwlBot post-processor
7d30eb8 
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
@ehsannas
Add Trace tests for DocumentReference and Query.
e2cb9bc
@gcf-owl-bot
🦉 Updates from OwlBot post-processor
1d9fc85 
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
@ehsannas
Add transaction tests.
00e4835
@ehsannas
Add WriteBatch test.
98ef318
@gcf-owl-bot
🦉 Updates from OwlBot post-processor
54b9082 
See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
@jimit-j-shah
Adding Basic Trace Test w/ Custom Root Span
25fe5fe
@jimit-j-shah jimit-j-shah requested review from a team as code owners March 9, 2024 00:49
@product-auto-label product-auto-label bot added size: xl Pull request size is extra large. api: firestore Issues related to the googleapis/java-firestore API. labels Mar 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
api: firestore Issues related to the googleapis/java-firestore API. size: xl Pull request size is extra large.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@jimit-j-shah @ehsannas