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

Allow to set login_cmd_template for delegate driver #2989

Merged
merged 1 commit into from
Nov 22, 2020

Conversation

apatard
Copy link
Contributor

@apatard apatard commented Nov 20, 2020

…managed case

The current code only allows to configure the login command
if molecule is configured to use non managed instance but it may
be helpful to be able to set it even if the instance is managed.

For instance, if the instance (eg a chroot or a container) can be
created with the create.yml playbook, which means that this instance is
reachable with something else than ssh.

Signed-off-by: Arnaud Patard apatard@hupstream.com

Please include details of what it is, how to use it, how it's been tested

PR Type

  • Bugfix Pull Request
  • Docs Pull Request
  • Feature Pull Request

…managed case

The current code only allows to configure the login command
if molecule is configured to use non managed instance but it may
be helpful to be able to set it even if the instance is managed.

For instance, if the instance (eg a chroot or a container) can be
created with the create.yml playbook, which means that this instance is
reachable with something else than ssh.

Signed-off-by: Arnaud Patard <apatard@hupstream.com>
@ssbarnea ssbarnea changed the title src/molecule/driver/delegated.py: Allow to set login_cmd_template in … Allow to set login_cmd_template for delegate driver Nov 22, 2020
@ssbarnea ssbarnea added the bug label Nov 22, 2020
@ssbarnea ssbarnea merged commit bcad8cb into ansible:master Nov 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants