Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add Printer pages counters #1353

Merged
merged 2 commits into from
Feb 6, 2025
Merged

Add Printer pages counters #1353

merged 2 commits into from
Feb 6, 2025

Conversation

eflumerf
Copy link
Contributor

@eflumerf eflumerf commented Feb 3, 2025

@bastischubert
Copy link
Member

Hey @eflumerf thanks for your contribution !
to accept this PR please sign the commit and also include the updated snmp.yml.

@eflumerf
Copy link
Contributor Author

eflumerf commented Feb 4, 2025

Done!

@bastischubert
Copy link
Member

thx for fixing - can you please sign off the commits (see https://github.com/prometheus/snmp_exporter/pull/1353/checks?check_run_id=36656171662 ) then we should be ready to merge

@SuperQ
Copy link
Member

SuperQ commented Feb 6, 2025

This needs a DCO sign-off. You can use git commit -s --amend to add it.

@eflumerf eflumerf force-pushed the main branch 2 times, most recently from e80bd7f to 394f1e5 Compare February 6, 2025 16:02
@eflumerf
Copy link
Contributor Author

eflumerf commented Feb 6, 2025

I signed off on the second commit, should I do both?
Edit: Both commits are now signed off, following the recommendation from the DCO check

Eric Flumerfelt and others added 2 commits February 6, 2025 11:48
Signed-off-by: Eric Flumerfelt <eflumerf@fnal.gov>
Signed-off-by: Eric Flumerfelt <eflumerf@fnal.gov>
Copy link
Member

@bastischubert bastischubert left a comment

Choose a reason for hiding this comment

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

Looks good :)

@bastischubert bastischubert merged commit be3dcf7 into prometheus:main Feb 6, 2025
6 checks passed
@SuperQ SuperQ mentioned this pull request Feb 6, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants