Skip to content

Update remote service for Java HTTP calls#605

Merged
ezhang6811 merged 4 commits intomainfrom
update-java-remote-service
Mar 27, 2026
Merged

Update remote service for Java HTTP calls#605
ezhang6811 merged 4 commits intomainfrom
update-java-remote-service

Conversation

@ezhang6811
Copy link
Copy Markdown
Contributor

Issue description:

Description of changes:
ADOT Java is upgrading its OpenTelemetry Java Instrumentation dependency to 2.26.1, which includes a change that adds the server.port attribute to all HTTP client spans (upstream PR).

the ADOT logic appends server.port to RemoteService if it exists, so this PR updates the java E2E tests to match the expected behavior.

Rollback procedure:

<Can we safely revert this commit if needed? If not, detail what must be done to safely revert and why it is needed.>

Ensure you've run the following tests on your changes and include the link below:

To do so, create a test.yml file with name: Test and workflow description to test your changes, then remove the file for your PR. Link your test run in your PR description. This process is a short term solution while we work on creating a staging environment for testing.

NOTE: TESTS RUNNING ON A SINGLE EKS CLUSTER CANNOT BE RUN IN PARALLEL. See the needs keyword to run tests in succession.

  • Run Java EKS on e2e-playground in us-east-1 and eu-central-2
  • Run Python EKS on e2e-playground in us-east-1 and eu-central-2
  • Run metric limiter on EKS cluster e2e-playground in us-east-1 and eu-central-2
  • Run EC2 tests in all regions
  • Run K8s on a separate K8s cluster (check IAD test account for master node endpoints; these will change as we create and destroy clusters for OS patching)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@ezhang6811 ezhang6811 changed the title Update remote service for HTTP calls Update remote service for Java HTTP calls Mar 27, 2026
@ezhang6811 ezhang6811 merged commit a4de9f9 into main Mar 27, 2026
55 checks passed
@ezhang6811 ezhang6811 deleted the update-java-remote-service branch March 27, 2026 19:14
ezhang6811 added a commit that referenced this pull request Mar 27, 2026
*Issue description:*

*Description of changes:*
In
#605,
we updated the Java validations to include port in RemoteService.
However, the eks-otlp-ocb test does not use the latest version of
opentelemetry-java-instrumentation, so this PR reverts the validations
for those tests.

*Rollback procedure:*

<Can we safely revert this commit if needed? If not, detail what must be
done to safely revert and why it is needed.>

*Ensure you've run the following tests on your changes and include the
link below:*

To do so, create a `test.yml` file with `name: Test` and workflow
description to test your changes, then remove the file for your PR. Link
your test run in your PR description. This process is a short term
solution while we work on creating a staging environment for testing.

NOTE: TESTS RUNNING ON A SINGLE EKS CLUSTER CANNOT BE RUN IN PARALLEL.
See the
[needs](https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#jobsjob_idneeds)
keyword to run tests in succession.
- Run Java EKS on `e2e-playground` in us-east-1 and eu-central-2
- Run Python EKS on `e2e-playground` in us-east-1 and eu-central-2
- Run metric limiter on EKS cluster `e2e-playground` in us-east-1 and
eu-central-2
- Run EC2 tests in all regions
- Run K8s on a separate K8s cluster (check IAD test account for master
node endpoints; these will change as we create and destroy clusters for
OS patching)

By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 license.

Co-authored-by: ADOT Patch workflow <[email protected]>
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.

2 participants