Skip to content

Conversation

@nasenov
Copy link
Contributor

@nasenov nasenov commented Apr 26, 2025

Fixes #298

@github-actions
Copy link
Contributor

github-actions bot commented Apr 26, 2025

Docs Build 📝

Thank you for contributing!✨

The docs for this PR have been published here:
https://prometheus-community.github.io/ansible/pr/586

You can compare to the docs for the main branch here:
https://prometheus-community.github.io/ansible/branch/main

The docsite for this PR is also available for download as an artifact from this run:
https://github.com/prometheus-community/ansible/actions/runs/19113981068

File changes:

Click to see the diff comparison.

NOTE: only file modifications are shown here. New and deleted files are excluded.
See the file list and check the published docs to see those files.

diff --git a/home/runner/work/ansible/ansible/docsbuild/base/smartctl_exporter_role.html b/home/runner/work/ansible/ansible/docsbuild/head/smartctl_exporter_role.html
index d266fed..8943f8e 100644
--- a/home/runner/work/ansible/ansible/docsbuild/base/smartctl_exporter_role.html
+++ b/home/runner/work/ansible/ansible/docsbuild/head/smartctl_exporter_role.html
@@ -305,7 +305,7 @@ To check whether it is installed, run <code class="code docutils literal notrans
 </div></td>
 <td><div class="ansible-option-cell"><p><em>Advanced</em></p>
 <p>System group for Smartctl exporter</p>
-<p class="ansible-option-line"><strong class="ansible-option-default-bold">Default:</strong> <code class="ansible-option-default docutils literal notranslate"><span class="pre">&quot;smartctl-exp&quot;</span></code></p>
+<p class="ansible-option-line"><strong class="ansible-option-default-bold">Default:</strong> <code class="ansible-option-default docutils literal notranslate"><span class="pre">&quot;root&quot;</span></code></p>
 </div></td>
 </tr>
 <tr class="row-even"><td><div class="ansible-option-cell">
@@ -314,7 +314,7 @@ To check whether it is installed, run <code class="code docutils literal notrans
 </div></td>
 <td><div class="ansible-option-cell"><p><em>Advanced</em></p>
 <p>Smartctl exporter user</p>
-<p class="ansible-option-line"><strong class="ansible-option-default-bold">Default:</strong> <code class="ansible-option-default docutils literal notranslate"><span class="pre">&quot;smartctl-exp&quot;</span></code></p>
+<p class="ansible-option-line"><strong class="ansible-option-default-bold">Default:</strong> <code class="ansible-option-default docutils literal notranslate"><span class="pre">&quot;root&quot;</span></code></p>
 </div></td>
 </tr>
 <tr class="row-odd"><td><div class="ansible-option-cell">

@nasenov nasenov changed the title Set root as default user and group for smartctl exporter fix(smartctl-exporter): set root as default user and group Apr 26, 2025
@github-actions github-actions bot added bugfix and removed bugfix labels Jul 23, 2025
@github-actions github-actions bot added bugfix and removed bugfix labels Nov 5, 2025
@nasenov
Copy link
Contributor Author

nasenov commented Nov 5, 2025

@gardar @SuperQ could you review the PR?

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.

smartctl exporter role: the default user and group should be root

1 participant