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

Add a snmp_context argument to SNMP exporter #6923

Merged
merged 1 commit into from
May 24, 2024
Merged

Conversation

ptodev
Copy link
Contributor

@ptodev ptodev commented May 22, 2024

PR Description

Adds a new config argument which was added to upstream recently

Notes to the Reviewer

I still haven't tested this. I hope to test it tomorrow with some help from @bastischubert

PR Checklist

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

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.

Doc changes look OK to me. @erikbaranowski are good to go for a merge?

@ptodev ptodev merged commit 315599b into main May 24, 2024
10 checks passed
@ptodev ptodev deleted the ptodev/add-snmp_context branch May 24, 2024 13:12
erikbaranowski added a commit that referenced this pull request May 31, 2024
* Update pyroscope/ebpf to v0.4.6 (#678) (#6917)

* Update pyroscope/ebpf to v0.4.4

Note: Pyroscope updated its prometheus/common dependecy and hence this
required further updates.

* Also update windows exporter

* Allow pid maps to be configured

This also modifieds the way we do defaulting and valides those two new
fields.

* Update pyroscope/ebpf v0.4.5

This updates the pyroscope/ebpf annotations to both have alloy and
river ones.

* Add some testing for argument validation

* Update to v0.4.6 to fix debug data

This also includes a test to make sure debug data is marshalled
correctly.

Co-authored-by: Clayton Cornell <[email protected]>

* Add a snmp_context argument (#6923)

* Update `make docs` procedure (#6930)

Co-authored-by: grafanabot <[email protected]>

---------

Co-authored-by: Christian Simon <[email protected]>
Co-authored-by: Clayton Cornell <[email protected]>
Co-authored-by: Paulin Todev <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: grafanabot <[email protected]>
@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 Jun 24, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 24, 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.

4 participants