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

machinectl become plugin does not specify it requires a tty #6932

Closed
1 task done
bcoca opened this issue Jul 14, 2023 · 2 comments · Fixed by #6935
Closed
1 task done

machinectl become plugin does not specify it requires a tty #6932

bcoca opened this issue Jul 14, 2023 · 2 comments · Fixed by #6935
Labels
become become plugin bug This issue/PR relates to a bug plugins plugin (any type)

Comments

@bcoca
Copy link
Contributor

bcoca commented Jul 14, 2023

Summary

see ansible/ansible#81254

if the plugin sets the class attribute:

 require_tty = True

It would automatically disable pipelining and avoid such errors

Issue Type

Bug Report

Component Name

become/machinectl

Ansible Version

$ ansible --version

all

Community.general Version

$ ansible-galaxy collection list community.general

all

Configuration

$ ansible-config dump --only-changed

N/A

OS / Environment

N/A

Steps to Reproduce

Use machinectl become plugin + pipelining

Expected Results

it works TM

Actual Results

   "msg": "MODULE FAILURE\nSee stdout/stderr for the exact error",

Code of Conduct

  • I agree to follow the Ansible Code of Conduct
@ansibullbot
Copy link
Collaborator

Files identified in the description:

If these files are incorrect, please update the component name section of the description or use the !component bot command.

click here for bot help

@ansibullbot
Copy link
Collaborator

cc @None
click here for bot help

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
become become plugin bug This issue/PR relates to a bug plugins plugin (any type)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants