Skip to content

Conversation

@segunjkf
Copy link
Contributor

@segunjkf segunjkf commented Dec 21, 2024

Summary of changes

Changes introduced in this pull request:

  • Add URL validation for remote write and logs sink URLs
  • Make flag requirements conditional on enabled features (metrics/logs)
  • Improve error messages and logging
  • Add config file validation
  • Add docs for configuration options
  • enable cadvisor

Reference issue to close (if applicable)

Closes

Other information and links

Change checklist

  • I have performed a self-review of my own code,
  • I have made corresponding changes to the documentation,
  • I have added tests that prove my fix is effective or that my feature works (if possible),
  • I have made sure the [CHANGELOG][1] is up-to-date. All user-facing changes should be reflected in this document.

… when specificed

Signed-off-by: segunjkf <sege.timz12@gmail.com>
Signed-off-by: segunjkf <sege.timz12@gmail.com>
Signed-off-by: segunjkf <sege.timz12@gmail.com>
Signed-off-by: segunjkf <sege.timz12@gmail.com>
Signed-off-by: segunjkf <sege.timz12@gmail.com>
@segunjkf segunjkf changed the title fix: fixed the config file generation error and ignored --config flag when specificed config: improve validation and error handling Dec 21, 2024
Signed-off-by: segunjkf <sege.timz12@gmail.com>
Signed-off-by: segunjkf <sege.timz12@gmail.com>
Signed-off-by: segunjkf <sege.timz12@gmail.com>
Signed-off-by: segunjkf <sege.timz12@gmail.com>
@segunjkf segunjkf marked this pull request as ready for review January 28, 2025 08:03
@segunjkf segunjkf requested a review from a team as a code owner January 28, 2025 08:03
@segunjkf segunjkf changed the title config: improve validation and error handling feat: improve validation and error handling Jan 28, 2025
@segunjkf segunjkf merged commit 946e455 into main Jan 28, 2025
1 of 2 checks passed
</p>

<h1 align="center">Blocks</h1>
<h1 align="center">BlocksOp Network</h1>
Copy link
Contributor

Choose a reason for hiding this comment

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

Telescope

Enable both metrics and logs:

```bash
agent \
Copy link
Contributor

Choose a reason for hiding this comment

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

the binary name should be telescope

Create a YAML configuration file and run:

```bash
agent --config-file=telescope_config.yaml
Copy link
Contributor

Choose a reason for hiding this comment

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

please update the binary name here as well

"sort"
"strings"

networksConfig "github.com/grafana/agent/internal/static/config/networks"
Copy link
Contributor

Choose a reason for hiding this comment

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

we need this to point to our own repo not grafana's, can you make the update in the go.mod file

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.

3 participants