Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add tutorial for running Beyla in Alloy's Helm #776

Merged
merged 11 commits into from
May 15, 2024
Merged

Conversation

marctc
Copy link
Contributor

@marctc marctc commented Apr 23, 2024

This tutorial explains how to run Beyla using Alloy's Helm
in a Kubernetes cluster.

docs/sources/tutorial/helm-alloy.md Outdated Show resolved Hide resolved
docs/sources/tutorial/helm-alloy.md Outdated Show resolved Hide resolved
docs/sources/tutorial/helm-alloy.md Outdated Show resolved Hide resolved
docs/sources/tutorial/helm-alloy.md Show resolved Hide resolved
docs/sources/tutorial/helm-alloy.md Outdated Show resolved Hide resolved
docs/sources/tutorial/helm-alloy.md Outdated Show resolved Hide resolved
docs/sources/tutorial/helm-alloy.md Outdated Show resolved Hide resolved
- tutorial
---

# Run Beyla in Kubernetes using Grafana Alloy Helm's chart.
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [vale] reported by reviewdog 🐶
[Grafana.GoogleHeadingPunctuation] Don't put a period at the end of a heading.

For more information, refer to https://developers.google.com/style/capitalization#capitalization-in-titles-and-headings.

If the rule is incorrect or needs improving, report an issue.

If you have reason to diverge from the style guidance, to skip a rule, refer to Skip rules.

docs/sources/tutorial/helm-alloy.md Outdated Show resolved Hide resolved
docs/sources/tutorial/helm-alloy.md Outdated Show resolved Hide resolved
marctc and others added 2 commits April 23, 2024 15:29
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
docs/sources/tutorial/helm-alloy.md Outdated Show resolved Hide resolved

The `output` section configures that Beyla component sends traces to `otelcol.exporter.otlp` component. The `otelcol.exporter.otlp` section configures the [OTLP exporter]({{< relref "docs/alloy/latest/reference/components/otelcol.exporter.otlp/" >}}) to send the traces to Grafana Cloud Tempo.

For further details on the configuration options, refer to the documentation of the [Grafana Alloy Beyla component]({{< relref "docs/alloy/latest/reference/components/beyla.ebpf" >}})
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [doc-validator] reported by reviewdog 🐶
The relref '{{< relref "docs/alloy/latest/reference/components/beyla.ebpf" >}}' can resolve ambiguously because it is not a relative or absolute path.
If a relref has ambiguous resolution, the link won't work and will instead point to the current page.

A relative path begins with either ./ or ../.
An absolute path begins with a /.


The `prometheus.scrape` section configures the Prometheus scrape configuration to collect the metrics from Beyla. The `prometheus.remote_write` section configures the remote write to send the metrics to Grafana Cloud Prometheus.

The `output` section configures that Beyla component sends traces to `otelcol.exporter.otlp` component. The `otelcol.exporter.otlp` section configures the [OTLP exporter]({{< relref "/docs/alloy/latest/reference/components/otelcol.exporter.otlp" >}}) to send the traces to Grafana Cloud Tempo.
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [doc-validator] reported by reviewdog 🐶
The link '{{< relref "/docs/alloy/latest/reference/components/otelcol.exporter.otlp" >}}' references a page in the project that does not exist.
Check if the page has moved within the repository.

Verify the link by building the docs locally using the make docs webserver.


The `output` section configures that Beyla component sends traces to `otelcol.exporter.otlp` component. The `otelcol.exporter.otlp` section configures the [OTLP exporter]({{< relref "/docs/alloy/latest/reference/components/otelcol.exporter.otlp" >}}) to send the traces to Grafana Cloud Tempo.

For further details on the configuration options, refer to the documentation of the [Grafana Alloy Beyla component]({{< relref "/docs/alloy/latest/reference/components/beyla.ebpf" >}})
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [doc-validator] reported by reviewdog 🐶
The link '{{< relref "/docs/alloy/latest/reference/components/beyla.ebpf" >}}' references a page in the project that does not exist.
Check if the page has moved within the repository.

Verify the link by building the docs locally using the make docs webserver.

@codecov-commenter
Copy link

codecov-commenter commented Apr 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 39.34%. Comparing base (6fb3594) to head (4680402).
Report is 8 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #776       +/-   ##
===========================================
- Coverage   76.39%   39.34%   -37.06%     
===========================================
  Files         112      107        -5     
  Lines        9362     9280       -82     
===========================================
- Hits         7152     3651     -3501     
- Misses       1831     5422     +3591     
+ Partials      379      207      -172     
Flag Coverage Δ
integration-test ?
unittests 39.34% <ø> (+0.52%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@grcevski grcevski left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Contributor

@mariomac mariomac left a comment

Choose a reason for hiding this comment

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

👌

@mariomac
Copy link
Contributor

My only concern is that we should also add soon a Beyla Helm document (as soon as this PR gets merged grafana/helm-charts#3093 ).

We should later discuss how to avoid generating confusion to the users between different helm charts.

@marctc marctc added the do-not-merge WIP or something that musn't be merged label Apr 24, 2024
@marctc
Copy link
Contributor Author

marctc commented Apr 24, 2024

My only concern is that we should also add soon a Beyla Helm document (as soon as this PR gets merged grafana/helm-charts#3093 ).

We should later discuss how to avoid generating confusion to the users between different helm charts.

Agreed! We should document in a page the different deployment modes and their benefits.

@marctc marctc removed the do-not-merge WIP or something that musn't be merged label May 15, 2024
@marctc marctc merged commit 529f5b9 into main May 15, 2024
7 checks passed
@marctc marctc deleted the add-alloy-helm-tutorial branch July 2, 2024 12:31
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.

5 participants