Skip to content

Conversation

@anthonyeleven
Copy link
Contributor

@anthonyeleven anthonyeleven commented Aug 17, 2025

chore(ipmi_exporter): Improve ipmi_exporter docs

Address minor nits.

Fixes: #637

Signed-off-by: Anthony D'Atri [email protected]

@github-actions
Copy link
Contributor

Docs Build 📝

Thank you for contribution!✨

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

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/17016959693

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/ipmi_exporter_role.html b/home/runner/work/ansible/ansible/docsbuild/head/ipmi_exporter_role.html
index 2f9c2f4..af5b9c2 100644
--- a/home/runner/work/ansible/ansible/docsbuild/base/ipmi_exporter_role.html
+++ b/home/runner/work/ansible/ansible/docsbuild/head/ipmi_exporter_role.html
@@ -165,7 +165,7 @@ To check whether it is installed, run <code class="code docutils literal notrans
 <section id="synopsis">
 <h3><a class="toc-backref" href="#id2" role="doc-backlink">Synopsis</a><a class="headerlink" href="#synopsis" title="Link to this heading"></a></h3>
 <ul class="simple">
-<li><p>Deploy prometheus <a class="reference external" href="https://github.com/prometheus-community/ipmi_exporter">ipmi exporter</a> using ansible</p></li>
+<li><p>Deploy the Prometheus <a class="reference external" href="https://github.com/prometheus-community/ipmi_exporter">ipmi exporter</a> using Ansible</p></li>
 </ul>
 </section>
 <section id="parameters">
@@ -181,7 +181,7 @@ To check whether it is installed, run <code class="code docutils literal notrans
 <div class="ansibleOptionAnchor" id="parameter-main--ipmi_exporter_basic_auth_users"></div><p class="ansible-option-title" id="ansible-collections-prometheus-prometheus-ipmi-exporter-role-parameter-main-ipmi-exporter-basic-auth-users"><strong>ipmi_exporter_basic_auth_users</strong></p>
 <a class="ansibleOptionLink" href="#parameter-main--ipmi_exporter_basic_auth_users" title="Permalink to this option"></a><p class="ansible-option-type-line"><span class="ansible-option-type">dictionary</span></p>
 </div></td>
-<td><div class="ansible-option-cell"><p>Dictionary of users and password for basic authentication. Passwords are automatically hashed with bcrypt.</p>
+<td><div class="ansible-option-cell"><p>Dictionary of users and passwords for basic authentication. Passwords are automatically hashed with bcrypt.</p>
 </div></td>
 </tr>
 <tr class="row-odd"><td><div class="ansible-option-cell">
@@ -189,7 +189,7 @@ To check whether it is installed, run <code class="code docutils literal notrans
 <a class="ansibleOptionLink" href="#parameter-main--ipmi_exporter_binary_install_dir" title="Permalink to this option"></a><p class="ansible-option-type-line"><span class="ansible-option-type">string</span></p>
 </div></td>
 <td><div class="ansible-option-cell"><p><em>Advanced</em></p>
-<p>Directory to install ipmi_exporter binary</p>
+<p>Directory in which to install ipmi_exporter binary</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;/usr/local/bin&quot;</span></code></p>
 </div></td>
 </tr>
@@ -229,7 +229,7 @@ To check whether it is installed, run <code class="code docutils literal notrans
 <div class="ansibleOptionAnchor" id="parameter-main--ipmi_exporter_local_cache_path"></div><p class="ansible-option-title" id="ansible-collections-prometheus-prometheus-ipmi-exporter-role-parameter-main-ipmi-exporter-local-cache-path"><strong>ipmi_exporter_local_cache_path</strong></p>
 <a class="ansibleOptionLink" href="#parameter-main--ipmi_exporter_local_cache_path" title="Permalink to this option"></a><p class="ansible-option-type-line"><span class="ansible-option-type">string</span></p>
 </div></td>
-<td><div class="ansible-option-cell"><p>Local path to stash the archive and its extraction</p>
+<td><div class="ansible-option-cell"><p>Local path at which to download and extract the archive</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;/tmp/ipmi_exporter-{{</span> <span class="pre">ansible_facts['system']</span> <span class="pre">|</span> <span class="pre">lower</span> <span class="pre">}}-{{</span> <span class="pre">_ipmi_exporter_go_ansible_arch</span> <span class="pre">}}/{{</span> <span class="pre">ipmi_exporter_version</span> <span class="pre">}}&quot;</span></code></p>
 </div></td>
 </tr>
@@ -295,7 +295,7 @@ To check whether it is installed, run <code class="code docutils literal notrans
 <div class="ansibleOptionAnchor" id="parameter-main--ipmi_exporter_web_listen_address"></div><p class="ansible-option-title" id="ansible-collections-prometheus-prometheus-ipmi-exporter-role-parameter-main-ipmi-exporter-web-listen-address"><strong>ipmi_exporter_web_listen_address</strong></p>
 <a class="ansibleOptionLink" href="#parameter-main--ipmi_exporter_web_listen_address" title="Permalink to this option"></a><p class="ansible-option-type-line"><span class="ansible-option-type">string</span></p>
 </div></td>
-<td><div class="ansible-option-cell"><p>Address on which ipmi exporter will listen</p>
+<td><div class="ansible-option-cell"><p>Address on which the ipmi_exporter will listen</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;0.0.0.0:9290&quot;</span></code></p>
 </div></td>
 </tr>

@gardar gardar changed the title chore(ipmi_exporter): Improve ipmi_exporter docs docs(ipmi_exporter): Improve ipmi_exporter docs Aug 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Minor docs and text improvements for ipmi_exporter

2 participants