-
Notifications
You must be signed in to change notification settings - Fork 487
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
Update discovery.kubernetes.md #4522
Conversation
Added selectors section to the Limited Searched Namespaces section and changed the description of the section to add the specific labels aspect.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some rewording of the text to make it more active and simplify the sentence a little
@grafana/grafana-agent-maintainers Can someone from the team double check the River example and approve. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few small corrections, but LGTM once those are added.
@dhalperin Thanks for your input here. Can you review and commit the suggestions. I could do it, but it's nice if you can see what has been suggestions before the PR disappears into the merge history :-) I'll merge once you've had a chance to see/review things here. |
Co-authored-by: Clayton Cornell <[email protected]>
Co-authored-by: Mischa Thompson <[email protected]>
Co-authored-by: Mischa Thompson <[email protected]>
Edits looked good so committed the changes. Feel free to merge when ready. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good
@dhalperin I'll get this merged and published :-)
* Update discovery.kubernetes.md Added selectors section to the Limited Searched Namespaces section and changed the description of the section to add the specific labels aspect. * Update docs/sources/flow/reference/components/discovery.kubernetes.md Co-authored-by: Clayton Cornell <[email protected]> * Update docs/sources/flow/reference/components/discovery.kubernetes.md Co-authored-by: Mischa Thompson <[email protected]> * Update docs/sources/flow/reference/components/discovery.kubernetes.md Co-authored-by: Mischa Thompson <[email protected]> * Minor fix to remove full stop --------- Co-authored-by: Clayton Cornell <[email protected]> Co-authored-by: Mischa Thompson <[email protected]> (cherry picked from commit 3e2c52f)
* Update discovery.kubernetes.md Added selectors section to the Limited Searched Namespaces section and changed the description of the section to add the specific labels aspect. * Update docs/sources/flow/reference/components/discovery.kubernetes.md Co-authored-by: Clayton Cornell <[email protected]> * Update docs/sources/flow/reference/components/discovery.kubernetes.md Co-authored-by: Mischa Thompson <[email protected]> * Update docs/sources/flow/reference/components/discovery.kubernetes.md Co-authored-by: Mischa Thompson <[email protected]> * Minor fix to remove full stop --------- Co-authored-by: Clayton Cornell <[email protected]> Co-authored-by: Mischa Thompson <[email protected]> (cherry picked from commit 3e2c52f)
* Update discovery.kubernetes.md Added selectors section to the Limited Searched Namespaces section and changed the description of the section to add the specific labels aspect. * Update docs/sources/flow/reference/components/discovery.kubernetes.md Co-authored-by: Clayton Cornell <[email protected]> * Update docs/sources/flow/reference/components/discovery.kubernetes.md Co-authored-by: Mischa Thompson <[email protected]> * Update docs/sources/flow/reference/components/discovery.kubernetes.md Co-authored-by: Mischa Thompson <[email protected]> * Minor fix to remove full stop --------- Co-authored-by: Clayton Cornell <[email protected]> Co-authored-by: Mischa Thompson <[email protected]> (cherry picked from commit 3e2c52f)
* Update discovery.kubernetes.md Added selectors section to the Limited Searched Namespaces section and changed the description of the section to add the specific labels aspect. * Update docs/sources/flow/reference/components/discovery.kubernetes.md Co-authored-by: Clayton Cornell <[email protected]> * Update docs/sources/flow/reference/components/discovery.kubernetes.md Co-authored-by: Mischa Thompson <[email protected]> * Update docs/sources/flow/reference/components/discovery.kubernetes.md Co-authored-by: Mischa Thompson <[email protected]> * Minor fix to remove full stop --------- Co-authored-by: Clayton Cornell <[email protected]> Co-authored-by: Mischa Thompson <[email protected]> (cherry picked from commit 3e2c52f) Co-authored-by: Devin H <[email protected]>
* Update discovery.kubernetes.md Added selectors section to the Limited Searched Namespaces section and changed the description of the section to add the specific labels aspect. * Update docs/sources/flow/reference/components/discovery.kubernetes.md Co-authored-by: Clayton Cornell <[email protected]> * Update docs/sources/flow/reference/components/discovery.kubernetes.md Co-authored-by: Mischa Thompson <[email protected]> * Update docs/sources/flow/reference/components/discovery.kubernetes.md Co-authored-by: Mischa Thompson <[email protected]> * Minor fix to remove full stop --------- Co-authored-by: Clayton Cornell <[email protected]> Co-authored-by: Mischa Thompson <[email protected]> (cherry picked from commit 3e2c52f) Co-authored-by: Devin H <[email protected]>
* Update discovery.kubernetes.md Added selectors section to the Limited Searched Namespaces section and changed the description of the section to add the specific labels aspect. * Update docs/sources/flow/reference/components/discovery.kubernetes.md Co-authored-by: Clayton Cornell <[email protected]> * Update docs/sources/flow/reference/components/discovery.kubernetes.md Co-authored-by: Mischa Thompson <[email protected]> * Update docs/sources/flow/reference/components/discovery.kubernetes.md Co-authored-by: Mischa Thompson <[email protected]> * Minor fix to remove full stop --------- Co-authored-by: Clayton Cornell <[email protected]> Co-authored-by: Mischa Thompson <[email protected]> (cherry picked from commit 3e2c52f) Co-authored-by: Devin H <[email protected]>
* Update discovery.kubernetes.md Added selectors section to the Limited Searched Namespaces section and changed the description of the section to add the specific labels aspect. * Update docs/sources/flow/reference/components/discovery.kubernetes.md Co-authored-by: Clayton Cornell <[email protected]> * Update docs/sources/flow/reference/components/discovery.kubernetes.md Co-authored-by: Mischa Thompson <[email protected]> * Update docs/sources/flow/reference/components/discovery.kubernetes.md Co-authored-by: Mischa Thompson <[email protected]> * Minor fix to remove full stop --------- Co-authored-by: Clayton Cornell <[email protected]> Co-authored-by: Mischa Thompson <[email protected]>
* Update discovery.kubernetes.md Added selectors section to the Limited Searched Namespaces section and changed the description of the section to add the specific labels aspect. * Update docs/sources/flow/reference/components/discovery.kubernetes.md Co-authored-by: Clayton Cornell <[email protected]> * Update docs/sources/flow/reference/components/discovery.kubernetes.md Co-authored-by: Mischa Thompson <[email protected]> * Update docs/sources/flow/reference/components/discovery.kubernetes.md Co-authored-by: Mischa Thompson <[email protected]> * Minor fix to remove full stop --------- Co-authored-by: Clayton Cornell <[email protected]> Co-authored-by: Mischa Thompson <[email protected]>
* Update discovery.kubernetes.md Added selectors section to the Limited Searched Namespaces section and changed the description of the section to add the specific labels aspect. * Update docs/sources/flow/reference/components/discovery.kubernetes.md Co-authored-by: Clayton Cornell <[email protected]> * Update docs/sources/flow/reference/components/discovery.kubernetes.md Co-authored-by: Mischa Thompson <[email protected]> * Update docs/sources/flow/reference/components/discovery.kubernetes.md Co-authored-by: Mischa Thompson <[email protected]> * Minor fix to remove full stop --------- Co-authored-by: Clayton Cornell <[email protected]> Co-authored-by: Mischa Thompson <[email protected]>
Added selectors section to the Limited Searched Namespaces section and changed the description of the section to add the specific labels aspect.
PR Description
Which issue(s) this PR fixes
Notes to the Reviewer
PR Checklist