-
Notifications
You must be signed in to change notification settings - Fork 183
fix(blackbox_exporter): blackbox dependencies syntax #668
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
base: main
Are you sure you want to change the base?
Conversation
Docs Build 📝Thank you for contributing!✨ The docs for this PR have been published here: You can compare to the docs for the The docsite for this PR is also available for download as an artifact from this run: File changes: Click to see the diff comparison.NOTE: only file modifications are shown here. New and deleted files are excluded. diff --git a/home/runner/work/ansible/ansible/docsbuild/base/_common_role.html b/home/runner/work/ansible/ansible/docsbuild/head/_common_role.html
index 46c59b8..8d1b8a3 100644
--- a/home/runner/work/ansible/ansible/docsbuild/base/_common_role.html
+++ b/home/runner/work/ansible/ansible/docsbuild/head/_common_role.html
@@ -412,7 +412,7 @@ To check whether it is installed, run <code class="code docutils literal notrans
<a class="ansibleOptionLink" href="#parameter-preflight--_common_dependencies" 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>Package dependencies to install</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">"{%</span> <span class="pre">if</span> <span class="pre">(ansible_facts['pkg_mgr']</span> <span class="pre">==</span> <span class="pre">'apt')</span> <span class="pre">%}{{</span> <span class="pre">('python-apt'</span> <span class="pre">if</span> <span class="pre">ansible_facts['python_version']</span> <span class="pre">is</span> <span class="pre">version('3',</span> <span class="pre">'<')</span> <span class="pre">else</span> <span class="pre">'python3-apt')</span> <span class="pre">}}</span> <span class="pre">{%</span> <span class="pre">else</span> <span class="pre">%}{%</span> <span class="pre">endif</span> <span class="pre">%}"</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">"{{</span> <span class="pre">['python-apt']</span> <span class="pre">if</span> <span class="pre">ansible_facts['pkg_mgr']</span> <span class="pre">==</span> <span class="pre">'apt'</span> <span class="pre">and</span> <span class="pre">ansible_facts['python_version']</span> <span class="pre">is</span> <span class="pre">version('3',</span> <span class="pre">'<')\n</span>   <span class="pre">else</span> <span class="pre">['python3-apt']</span> <span class="pre">if</span> <span class="pre">ansible_facts['pkg_mgr']</span> <span class="pre">==</span> <span class="pre">'apt'\n</span>   <span class="pre">else</span> <span class="pre">[]</span> <span class="pre">}}"</span></code></p>
</div></td>
</tr>
<tr class="row-odd"><td><div class="ansible-option-cell">
|
Signed-off-by: Kuba Orlik <[email protected]>
Signed-off-by: Kuba Orlik <[email protected]>
|
Thanks, works for me! I directly patched this into the installed prometheus role: |
Fixes #667
The yaml syntax was such that the dependency list was a string (
"['python3-apt', 'libcap2-bin'] "— note the whitespace at the end!), not a list, and caused errors described in #667