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 #4912/905f9ec3 backport][stable-5] fix lxd connection plugin inventory_hostname #4935

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Jul 7, 2022

This is a backport of PR #4912 as merged into main (905f9ec).

SUMMARY

Fixes #4886

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

lxd connection plugin

ADDITIONAL INFORMATION

remote_addr value was being set to literal string 'inventory_hostname' instead of value of inventory_hostname variable. Fix taken from ansible/ansible#77894

* fixes lxd connection plugin issue #4886

remote_addr value was set to literal string 'inventory_hostname' instead
of the value for inventory_hostname variable. solution found in PR
ansible/ansible#77894

* changelog fragment - bugfix - lxd connection plugin

* correct changelog fragment

* Update changelogs/fragments/4886-fix-lxd-inventory-hostname.yml

Co-authored-by: Felix Fontein <felix@fontein.de>

* replace _host instance variable with calls to get 'remote_addr' option

suggested by felixfontein

Co-authored-by: Felix Fontein <felix@fontein.de>
(cherry picked from commit 905f9ec)
@ansibullbot
Copy link
Collaborator

@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 Jul 7, 2022
@felixfontein felixfontein merged commit ba18d62 into stable-5 Jul 7, 2022
@felixfontein felixfontein deleted the patchback/backports/stable-5/905f9ec399a6cac86c3f1cbc3a36cb97139741b1/pr-4912 branch July 7, 2022 20:40
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