Skip to content

Fix incorrect -Sauto given to Ansible - #5183

Open
happz wants to merge 1 commit into
mainfrom
guest-ansible-sauto
Open

happz wants to merge 1 commit into
mainfrom
guest-ansible-sauto

Conversation

@happz

@happz happz commented Sep 21, 2026

Copy link
Copy Markdown
Contributor

-Sauto does not say "use multiplexing if you can, or don't, your call". Instead, it says "use multiplexing, and use socket ./auto". Which means that all ansible-playbook calls used the same socket no matter which guest they are running against.

Fixes #5182.

Pull Request Checklist

  • implement the feature
  • write the documentation

@happz happz added bug Something isn't working step | provision Stuff related to the provision step labels Sep 21, 2026
@happz happz added this to planning Sep 21, 2026
@github-project-automation github-project-automation Bot moved this to backlog in planning Sep 21, 2026
@happz happz added the ci | full test Pull request is ready for the full test execution label Sep 21, 2026
@happz happz moved this from backlog to implement in planning Sep 21, 2026
@thrix
thrix self-requested a review September 21, 2026 22:11

@thrix thrix left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, resolves the issue for both users reported the problem. Hotpatched in TF production.

@happz happz moved this from implement to review in planning Sep 22, 2026
@LecrisUT LecrisUT moved this from review to merge in planning Sep 23, 2026
`-Sauto` does not say "use multiplexing if you can, or don't, your
call". Instead, it says "use multiplexing, and use socket `./auto`".
Which means that all `ansible-playbook` calls used the same socket no
matter which guest they are running against.

Fixes #5182.
@happz
happz force-pushed the guest-ansible-sauto branch from f66d504 to cae2a06 Compare September 24, 2026 06:59

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working ci | full test Pull request is ready for the full test execution step | provision Stuff related to the provision step

Projects

Status: merge

Development

Successfully merging this pull request may close these issues.

Ansible runs against the wrong guest in multihost plans: -Sauto is a shared literal ControlPath

3 participants