Skip to content

Run grpc tests with ipv4#11087

Merged
laurit merged 3 commits into
open-telemetry:mainfrom
laurit:grpc-ipv4
Apr 11, 2024
Merged

Run grpc tests with ipv4#11087
laurit merged 3 commits into
open-telemetry:mainfrom
laurit:grpc-ipv4

Conversation

@laurit laurit requested a review from a team April 10, 2024 11:29
jvmArgs("-Dotel.instrumentation.grpc.capture-metadata.client.request=some-client-key")
jvmArgs("-Dotel.instrumentation.grpc.capture-metadata.server.request=some-server-key")
jvmArgs("-Dotel.instrumentation.common.experimental.controller-telemetry.enabled=true")
jvmArgs("-Djava.net.preferIPv4Stack=true")

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you add comment why?

is it just so we can verify network.type=ipv4?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is network.type and network.peer.address. Seems to happen only in latest dep tests and not always.

@laurit laurit merged commit 79d50d1 into open-telemetry:main Apr 11, 2024
@laurit laurit deleted the grpc-ipv4 branch April 11, 2024 06:16
laurit added a commit to laurit/opentelemetry-java-instrumentation that referenced this pull request Apr 13, 2024
Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
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

Successfully merging this pull request may close these issues.

3 participants