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

[Flow] Add new discovery.file component #4404

Merged
merged 8 commits into from
Jul 13, 2023
Merged

Conversation

spartan0x117
Copy link
Contributor

@spartan0x117 spartan0x117 commented Jul 12, 2023

PR Description

Adds the new discovery.file component, which implements Prometheus file-based SD.

Note that there is a banner added to the new discovery.file reference docs. Let me know if the wording should be changed or it should be moved elsewhere.

Closes #3346

PR Checklist

  • CHANGELOG updated
  • Documentation added
  • Tests updated

@spartan0x117 spartan0x117 marked this pull request as ready for review July 12, 2023 16:35
@spartan0x117 spartan0x117 requested review from a team and clayton-cornell as code owners July 12, 2023 16:35
@spartan0x117 spartan0x117 changed the title [WIP] Add new discovery.file component [Flow] Add new discovery.file component Jul 12, 2023
# discovery.file

> **NOTE:** In `v0.35.0` of the Grafana Agent, the `discovery.file` component was renamed to [local.file_match][]. From `v0.35.0` onwards, the `discovery.file`
> component behaves as documented here.
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe specify here briefly what local.file_match does, and why they may want to go there instead.

Copy link
Contributor

@captncraig captncraig left a comment

Choose a reason for hiding this comment

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

Code looks good to me. I'd say wait for @clayton-cornell to weigh in on docs.

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.

LGTM

@rfratto rfratto merged commit 000b569 into main Jul 13, 2023
@rfratto rfratto deleted the spartan0x117/flow-discovery-file branch July 13, 2023 00:35
@github-actions github-actions bot added the frozen-due-to-age Locked due to a period of inactivity. Please open new issues or PRs if more discussion is needed. label Feb 22, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
frozen-due-to-age Locked due to a period of inactivity. Please open new issues or PRs if more discussion is needed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support for file_sd_configs like behavior in flow
4 participants