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

[PR #6118/9a97bc7c backport][stable-6] Add 'inventory_hostname' to the jail plugin documentation #6137

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Mar 3, 2023

This is a backport of PR #6118 as merged into main (9a97bc7).

SUMMARY

On FreeBSD, using the jail connection plugin will print the following warning when processing each task:

[WARNING]: The "jail" connection plugin has an improperly configured remote target value, forcing "inventory_hostname" templated value instead of the string

This was already fixed in other plugins by including "inventory_hostname" in the documentation string.

See for example 5e5af45 and 905f9ec.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME
  • jail.py

This fixes the following warning on FreeBSD:

  [WARNING]: The "jail" connection plugin has an improperly configured
  remote target value, forcing "inventory_hostname" templated value
  instead of the string

(cherry picked from commit 9a97bc7)
@ansibullbot
Copy link
Collaborator

cc @None
click here for bot help

@ansibullbot ansibullbot added backport bug This issue/PR relates to a bug connection connection plugin new_contributor Help guide this first time contributor plugins plugin (any type) labels Mar 3, 2023
@felixfontein felixfontein merged commit 8c1b7e3 into stable-6 Mar 3, 2023
@felixfontein felixfontein deleted the patchback/backports/stable-6/9a97bc7c1146f612e02268a8b6df92490ad10ec2/pr-6118 branch March 3, 2023 22:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue/PR relates to a bug connection connection plugin new_contributor Help guide this first time contributor plugins plugin (any type)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants