Skip to content

Update Sample otelcol config#4397

Merged
bmorelli25 merged 4 commits into
mainfrom
otelcol-config
Oct 18, 2024
Merged

Update Sample otelcol config#4397
bmorelli25 merged 4 commits into
mainfrom
otelcol-config

Conversation

@graphaelli

@graphaelli graphaelli commented Oct 18, 2024

Copy link
Copy Markdown
Contributor

Description

Simplify otel collector sample config and make it valid.

Documentation sets edited in this PR

Check all that apply.

  • Stateful (docs/en/observability/*)
  • Serverless (docs/en/serverless/*)
  • Integrations Developer Guide (docs/en/integrations/*)
  • None of the above

Related issue

closes elastic/apm-server#13550
closes elastic/apm-server#13551

Checklist

  • Product/Engineering Review
  • Writer Review

Follow-up tasks

Select one.

  • This PR does not need to be ported to another doc set because:
    • The concepts in this PR only apply to one doc set (serverless or stateful)
    • The PR contains edits to both doc sets (serverless and stateful)
  • This PR needs to be ported to another doc set:
    • Port to stateful docs: <link to PR or tracking issue>
    • Port to serverless docs: <link to PR or tracking issue>

Gil Raphaelli added 2 commits October 17, 2024 19:56
With recent versions of OpenTelemetry Collector it is necessary to specify protocols
@graphaelli graphaelli added the backport-8.x Automated backport to the 8.x branch with mergify label Oct 18, 2024
@graphaelli graphaelli requested a review from a team as a code owner October 18, 2024 00:20
@github-actions

Copy link
Copy Markdown
Contributor

A documentation preview will be available soon.

Request a new doc build by commenting
  • Rebuild this PR: run docs-build
  • Rebuild this PR and all Elastic docs: run docs-build rebuild

run docs-build is much faster than run docs-build rebuild. A rebuild should only be needed in rare situations.

If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status here.

@graphaelli

graphaelli commented Oct 18, 2024

Copy link
Copy Markdown
Contributor Author

…try/otel-direct.asciidoc

Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com>
@graphaelli graphaelli requested a review from alaudazzi October 18, 2024 17:54
@bmorelli25 bmorelli25 added the backport-8.16 Automated backport with mergify label Oct 18, 2024
@bmorelli25 bmorelli25 merged commit 2ba3a80 into main Oct 18, 2024
@bmorelli25 bmorelli25 deleted the otelcol-config branch October 18, 2024 19:19
mergify Bot pushed a commit that referenced this pull request Oct 18, 2024
* update to valid otlp receiver config

With recent versions of OpenTelemetry Collector it is necessary to specify protocols

* simplify exporter configuration and note advanced option

* fix link syntax

* Update docs/en/observability/apm/collect-application-data/open-telemetry/otel-direct.asciidoc

Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com>

---------

Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com>
(cherry picked from commit 2ba3a80)
bmorelli25 pushed a commit that referenced this pull request Oct 18, 2024
* update to valid otlp receiver config

With recent versions of OpenTelemetry Collector it is necessary to specify protocols

* simplify exporter configuration and note advanced option

* fix link syntax

* Update docs/en/observability/apm/collect-application-data/open-telemetry/otel-direct.asciidoc

Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com>

---------

Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com>
(cherry picked from commit 2ba3a80)

Co-authored-by: Gil Raphaelli <gil@elastic.co>
@bmorelli25 bmorelli25 added backport-8.16 Automated backport with mergify and removed backport-8.16 Automated backport with mergify labels Oct 18, 2024
mergify Bot pushed a commit that referenced this pull request Oct 18, 2024
* update to valid otlp receiver config

With recent versions of OpenTelemetry Collector it is necessary to specify protocols

* simplify exporter configuration and note advanced option

* fix link syntax

* Update docs/en/observability/apm/collect-application-data/open-telemetry/otel-direct.asciidoc

Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com>

---------

Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com>
(cherry picked from commit 2ba3a80)
bmorelli25 pushed a commit that referenced this pull request Oct 18, 2024
* update to valid otlp receiver config

With recent versions of OpenTelemetry Collector it is necessary to specify protocols

* simplify exporter configuration and note advanced option

* fix link syntax

* Update docs/en/observability/apm/collect-application-data/open-telemetry/otel-direct.asciidoc

Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com>

---------

Co-authored-by: Arianna Laudazzi <46651782+alaudazzi@users.noreply.github.com>
(cherry picked from commit 2ba3a80)

Co-authored-by: Gil Raphaelli <gil@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-8.x Automated backport to the 8.x branch with mergify backport-8.16 Automated backport with mergify needs-writer-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs: OpenTelemetry Collector example exporter config is misleading docs: OpenTelemetry Collector example receiver config is invalid

3 participants