Skip to content
This repository has been archived by the owner on Jul 19, 2023. It is now read-only.

Add grafana agent docs #720

Merged
merged 5 commits into from Jun 7, 2023
Merged

Add grafana agent docs #720

merged 5 commits into from Jun 7, 2023

Conversation

Rperry2174
Copy link
Contributor

@Rperry2174 Rperry2174 commented May 29, 2023

Adding grafana agent docs related grafana/pyroscope#1958

Copy link
Collaborator

@cyriltovena cyriltovena left a comment

Choose a reason for hiding this comment

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

LGTM


## Running the agent
- `examples/your-application-example/agent/config/config.river`
Copy link
Contributor Author

Choose a reason for hiding this comment

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

add the actual code here

Copy link
Collaborator

Choose a reason for hiding this comment

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

How is the Grafana Agent scraping the available containers? Or is it a hardcoded list?

Copy link
Collaborator

@simonswine simonswine left a comment

Choose a reason for hiding this comment

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

LGTM

scheme: http
profiling_config:
path_prefix: '/app'
Create the following directory structure for the example files:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Wonder if it is better to describe a git clone instead.

We already have a bit of a demo here: https://github.com/grafana/phlare/tree/main/tools/docker-compose

It is "just" without Grafana Agent


## Running the agent
- `examples/your-application-example/agent/config/config.river`
Copy link
Collaborator

Choose a reason for hiding this comment

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

How is the Grafana Agent scraping the available containers? Or is it a hardcoded list?

@Rperry2174 Rperry2174 merged commit fbbcc95 into main Jun 7, 2023
17 checks passed
@Rperry2174 Rperry2174 deleted the add-grafana-agent-docs branch June 7, 2023 12:21
@Rperry2174
Copy link
Contributor Author

went ahead and merged -- will do another pass on docs here after talking with @rfratto next week and include suggestions @cyriltovena @simonswine

simonswine pushed a commit to simonswine/pyroscope that referenced this pull request Jun 30, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants