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

feature: blackbox exporter user/group configurable #172

Conversation

tvenieris
Copy link
Contributor

Also, binary and configuration owner is changed to root. User blackbox-exp should be prevented from modifying or deleting either the configuration or itself, according to the principle of least privilege.

Also, binary and configuration owner is changed to root.
User blackbox-exp should be prevented from modifying or deleting either
the configuration or itself, according to the principle of least
privilege.

Signed-off-by: Thomas Venieris <[email protected]>
Copy link
Contributor

@SuperQ SuperQ left a comment

Choose a reason for hiding this comment

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

Thanks

@SuperQ
Copy link
Contributor

SuperQ commented Jul 29, 2023

Strange that the label trigger didn't happen on the first pass.

@codecov-commenter
Copy link

codecov-commenter commented Jul 29, 2023

Codecov Report

❗ No coverage uploaded for pull request base (main@19fda62). Click here to learn what that means.
The diff coverage is n/a.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@           Coverage Diff            @@
##             main      #172   +/-   ##
========================================
  Coverage        ?   100.00%           
========================================
  Files           ?         2           
  Lines           ?        34           
  Branches        ?         2           
========================================
  Hits            ?        34           
  Misses          ?         0           
  Partials        ?         0           
Flag Coverage Δ
integration 100.00% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

@SuperQ SuperQ merged commit c0eb6bb into prometheus-community:main Jul 29, 2023
@SuperQ
Copy link
Contributor

SuperQ commented Jul 29, 2023

Oops, forgot to update the meta/argument_specs.yml.

@tvenieris tvenieris deleted the blackbox_exporter_user_configurable branch July 29, 2023 17:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants