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

first draft of alloy overview rewrite #2031

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Conversation

LisaHJung
Copy link
Contributor

PR Description

Editing overview page of Alloy

Which issue(s) this PR fixes

Notes to the Reviewer

PR Checklist

  • CHANGELOG.md updated
  • Documentation added
  • Tests updated
  • Config converters updated

@LisaHJung LisaHJung added the type/docs Docs Squad label across all Grafana Labs repos label Nov 4, 2024
@LisaHJung LisaHJung self-assigned this Nov 4, 2024
docs/sources/_index.md Outdated Show resolved Hide resolved
docs/sources/_index.md Outdated Show resolved Hide resolved
docs/sources/_index.md Outdated Show resolved Hide resolved
docs/sources/_index.md Outdated Show resolved Hide resolved
docs/sources/_index.md Outdated Show resolved Hide resolved
docs/sources/_index.md Outdated Show resolved Hide resolved
docs/sources/_index.md Outdated Show resolved Hide resolved
Co-authored-by: Clayton Cornell <131809008+clayton-cornell@users.noreply.github.com>
docs/sources/_index.md Outdated Show resolved Hide resolved
docs/sources/_index.md Outdated Show resolved Hide resolved
docs/sources/_index.md Outdated Show resolved Hide resolved
LisaHJung and others added 3 commits November 4, 2024 14:04
Co-authored-by: Clayton Cornell <131809008+clayton-cornell@users.noreply.github.com>
@LisaHJung LisaHJung marked this pull request as draft November 4, 2024 21:20
@LisaHJung LisaHJung marked this pull request as ready for review November 4, 2024 21:23
Copy link
Contributor

@clayton-cornell clayton-cornell left a comment

Choose a reason for hiding this comment

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

Some initial thoughts and comments.

docs/sources/_index.md Outdated Show resolved Hide resolved
{{< param "PRODUCT_NAME" >}} is fully compatible with the OTel Collector, Prometheus Agent, and [Promtail][].
You can use {{< param "PRODUCT_NAME" >}} as an alternative to either of these solutions or combine it into a hybrid system of multiple collectors and agents.
You can deploy {{< param "PRODUCT_NAME" >}} anywhere within your IT infrastructure and pair it with your Grafana LGTM stack, a telemetry backend from Grafana Cloud, or any other compatible backend from any other vendor.
Getting the relevant telemetry data (i.e. metrics, logs, and traces) for analysis is an indispensable part of understanding the health of your system.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Getting the relevant telemetry data (i.e. metrics, logs, and traces) for analysis is an indispensable part of understanding the health of your system.
Collecting the relevant metrics, logs, and traces for analysis is an indispensable part of understanding the health of your system.

We shouldn't use i.e. in the docs.

Do we need to use telemetry data here? Is there other telemetry data collected other than metrics, logs and traces?

I feel like we are missing some of the details for things that Alloy can do beyond collecting metrics, logs and traces. The original text mentions things like alerts, hybrid configurations, being deployed anywhere in the LGTM stack, Grafana Cloud, or other compatible backends/vendors.

You can deploy {{< param "PRODUCT_NAME" >}} anywhere within your IT infrastructure and pair it with your Grafana LGTM stack, a telemetry backend from Grafana Cloud, or any other compatible backend from any other vendor.
Getting the relevant telemetry data (i.e. metrics, logs, and traces) for analysis is an indispensable part of understanding the health of your system.

Think of {{< param "PRODUCT_NAME" >}} as a Swiss army knife for collecting, processing, and forwarding telemetry data to the data source of your choosing.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Think of {{< param "PRODUCT_NAME" >}} as a Swiss army knife for collecting, processing, and forwarding telemetry data to the data source of your choosing.
{{< param "PRODUCT_NAME" >}} is a multifunction tool you can use to collect, process, and forward telemetry data to the data receiver of your choosing.

Where do we forward data? I don't think it's a data source. It's some kind of receiver., right? Either external or to the next step in the pipeline (eg the output of one component can be input for the next component),

docs/sources/_index.md Outdated Show resolved Hide resolved
docs/sources/_index.md Outdated Show resolved Hide resolved
Getting started with Alloy consists of 3 major steps:
1. Install {{< param "PRODUCT_NAME" >}}
1. Configure {{< param "PRODUCT_NAME" >}}
1. Collect and forward telemetry data to the data source of choice
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
1. Collect and forward telemetry data to the data source of choice
1. Collect and forward telemetry data to the data receiver of choice

I'm not sure receiver is the best word here...

docs/sources/_index.md Outdated Show resolved Hide resolved
docs/sources/_index.md Outdated Show resolved Hide resolved
Some of these features include custom components, GitOps compatibility, clustering support, security, and debugging utilities. Refer to the {{< param "PRODUCT_NAME" >}} [Introduction] for more information on these and other key features.
For a quick overview of this process, check out the following tutorials.
* [Use Grafana Alloy to send logs to Loki](https://grafana.com/docs/alloy/latest/tutorials/send-logs-to-loki/)
* [Use Grafana Alloy to send metrics to Prometheus](https://grafana.com/docs/alloy/latest/tutorials/send-metrics-to-prometheus/)
Copy link
Contributor

Choose a reason for hiding this comment

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

There are more tutorials available... do we want to include them?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hey @clayton-cornell ! I intentionally put those two tutorials there as these go over the steps mentioned above from start to finish.

What do you think about adding a third bullet point and linking the tutorial page?

  • For more tutorials, check out the tutorial page (or something like that?)

LisaHJung and others added 5 commits November 5, 2024 12:18
Co-authored-by: Clayton Cornell <131809008+clayton-cornell@users.noreply.github.com>
Co-authored-by: Clayton Cornell <131809008+clayton-cornell@users.noreply.github.com>
Co-authored-by: Clayton Cornell <131809008+clayton-cornell@users.noreply.github.com>
Co-authored-by: Clayton Cornell <131809008+clayton-cornell@users.noreply.github.com>
Co-authored-by: Clayton Cornell <131809008+clayton-cornell@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/docs Docs Squad label across all Grafana Labs repos
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants