Skip to content

Conversation

@rStelmach
Copy link
Contributor

@rStelmach rStelmach commented Aug 12, 2025

related to : #220609

Summary 📚

When testing the pipeline for serverless nightly workflow, Kubernetes OTel test was failing :
https://github.com/elastic/ensemble/actions/runs/16877811257/job/47821772894

This PR fixes Kubernetes OTEL e2e test failure in serverless by implementing environment-aware detection.
The test was hardcoded to look for serviceLink_opentelemetry/java/elastic which exists in stateful environments, but serverless uses agent-based patterns like serviceLink_java.

Solution ✅

  • Added environment detection using serverless flag

  • Implemented conditional service link selection:

    • Serverless: serviceLink_java (agent-based pattern)
    • Stateful: serviceLink_opentelemetry/java/elastic (existing pattern)
  • Fixed transaction assertion to handle multiple transaction types using .first()

Passed test : https://github.com/elastic/ensemble/actions/runs/16910418109/job/47910319822

@rStelmach rStelmach added release_note:fix backport:skip This PR does not require backporting labels Aug 12, 2025
@rStelmach rStelmach requested a review from a team as a code owner August 12, 2025 14:38
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-logs-team (Team:obs-ux-logs)

Copy link
Contributor

@gbamparop gbamparop left a comment

Choose a reason for hiding this comment

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

@gbamparop
Copy link
Contributor

@jackshirazi just to double check, for the OTel k8s flow in Stateful the agent name is java and in Serverless it's opentelemetry/java/elastic. Is this expected?

@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

✅ unchanged

History

@rStelmach rStelmach merged commit 24deb3a into elastic:main Aug 14, 2025
12 checks passed
@rStelmach rStelmach deleted the 220609-oblt-onboarding-kubernetes-test branch August 14, 2025 11:58
NicholasPeretti pushed a commit to NicholasPeretti/kibana that referenced this pull request Aug 18, 2025
…workflow (elastic#231462)

related to : elastic#220609
## Summary 📚 
When testing the pipeline for serverless nightly workflow, Kubernetes
OTel test was failing :

https://github.com/elastic/ensemble/actions/runs/16877811257/job/47821772894

This PR fixes Kubernetes OTEL e2e test failure in serverless by
implementing environment-aware detection.
The test was hardcoded to look for
`serviceLink_opentelemetry/java/elastic` which exists in stateful
environments, but serverless uses agent-based patterns like
`serviceLink_java`.

 ## Solution ✅

- Added environment detection using serverless flag

-  Implemented conditional service link selection:
    - Serverless: serviceLink_java (agent-based pattern)
- Stateful: serviceLink_opentelemetry/java/elastic (existing pattern)

- Fixed transaction assertion to handle multiple transaction types using
`.first()`

Passed test :
https://github.com/elastic/ensemble/actions/runs/16910418109/job/47910319822

---------

Co-authored-by: kibanamachine <[email protected]>
qn895 pushed a commit to qn895/kibana that referenced this pull request Aug 26, 2025
…workflow (elastic#231462)

related to : elastic#220609
## Summary 📚 
When testing the pipeline for serverless nightly workflow, Kubernetes
OTel test was failing :

https://github.com/elastic/ensemble/actions/runs/16877811257/job/47821772894

This PR fixes Kubernetes OTEL e2e test failure in serverless by
implementing environment-aware detection.
The test was hardcoded to look for
`serviceLink_opentelemetry/java/elastic` which exists in stateful
environments, but serverless uses agent-based patterns like
`serviceLink_java`.

 ## Solution ✅

- Added environment detection using serverless flag

-  Implemented conditional service link selection:
    - Serverless: serviceLink_java (agent-based pattern)
- Stateful: serviceLink_opentelemetry/java/elastic (existing pattern)

- Fixed transaction assertion to handle multiple transaction types using
`.first()`

Passed test :
https://github.com/elastic/ensemble/actions/runs/16910418109/job/47910319822

---------

Co-authored-by: kibanamachine <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants