Skip to content

[Oblt Logs Essentials] Remove APM dependency from Onboarding#226799

Merged
mykolaharmash merged 3 commits intoelastic:mainfrom
mykolaharmash:226435-decouple-onboarding-from-apm
Jul 14, 2025
Merged

[Oblt Logs Essentials] Remove APM dependency from Onboarding#226799
mykolaharmash merged 3 commits intoelastic:mainfrom
mykolaharmash:226435-decouple-onboarding-from-apm

Conversation

@mykolaharmash
Copy link
Copy Markdown
Contributor

@mykolaharmash mykolaharmash commented Jul 7, 2025

Closes #226435

  • Adds managedOtlpServiceUrl to the Observability plugin contract
  • Switched from generating OTLP service URL (by transforming APM managed service URL) to reading the URL from the Observability plugin contract.
  • Removes APM dependency from onboarding as it was needed only for the managed service URL thing

How to test

  1. Add a dummy URL to the config/serverless.oblt.dev.yml file
xpack.observability.managedOtlpServiceUrl: "https://test-test-test.ingest.us-east-1.aws.elastic.cloud"
  1. Run Kibana in serverless mode
  2. Go to the Host → OTel onboarding flow and make sure the code snippet uses the URL you've set in the config
  3. Go to the Kubernetes → OTel onboarding flow and make sure the code snippet uses the URL you've set in the config

@mykolaharmash mykolaharmash marked this pull request as ready for review July 7, 2025 12:32
@mykolaharmash mykolaharmash requested review from a team as code owners July 7, 2025 12:32
@mykolaharmash mykolaharmash added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting labels Jul 7, 2025
Copy link
Copy Markdown
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.

LGTM, thanks!

@botelastic botelastic bot added the Team:actionable-obs Formerly "obs-ux-management", responsible for SLO, o11y alerting, significant events, & synthetics. label Jul 8, 2025
@elasticmachine
Copy link
Copy Markdown
Contributor

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

Copy link
Copy Markdown
Member

@gsoldevila gsoldevila left a comment

Choose a reason for hiding this comment

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

Core changes LGTM (config review only)

@mykolaharmash mykolaharmash enabled auto-merge (squash) July 14, 2025 11:31
@mykolaharmash mykolaharmash merged commit 5b3ec33 into elastic:main Jul 14, 2025
11 of 12 checks passed
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

Bluefinger pushed a commit to Bluefinger/kibana that referenced this pull request Jul 22, 2025
…#226799)

Closes elastic#226435

* Adds `managedOtlpServiceUrl` to the Observability plugin contract
* Switched from generating OTLP service URL (by transforming APM managed
service URL) to reading the URL from the Observability plugin contract.
* Removes APM dependency from onboarding as it was needed only for the
managed service URL thing

## How to test

1. Add a dummy URL to the `config/serverless.oblt.dev.yml` file
```yml
xpack.observability.managedOtlpServiceUrl: "https://test-test-test.ingest.us-east-1.aws.elastic.cloud"
```
2. Run Kibana in serverless mode
3. Go to the Host → OTel onboarding flow and make sure the code snippet
uses the URL you've set in the config
4. Go to the Kubernetes → OTel onboarding flow and make sure the code
snippet uses the URL you've set in the config
kertal pushed a commit to kertal/kibana that referenced this pull request Jul 25, 2025
…#226799)

Closes elastic#226435

* Adds `managedOtlpServiceUrl` to the Observability plugin contract
* Switched from generating OTLP service URL (by transforming APM managed
service URL) to reading the URL from the Observability plugin contract.
* Removes APM dependency from onboarding as it was needed only for the
managed service URL thing

## How to test

1. Add a dummy URL to the `config/serverless.oblt.dev.yml` file
```yml
xpack.observability.managedOtlpServiceUrl: "https://test-test-test.ingest.us-east-1.aws.elastic.cloud"
```
2. Run Kibana in serverless mode
3. Go to the Host → OTel onboarding flow and make sure the code snippet
uses the URL you've set in the config
4. Go to the Kubernetes → OTel onboarding flow and make sure the code
snippet uses the URL you've set in the config
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:actionable-obs Formerly "obs-ux-management", responsible for SLO, o11y alerting, significant events, & synthetics. v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Obs Logs Essentials] Decouple Onboarding plugin from APM

6 participants