Skip to content
This repository was archived by the owner on Jan 5, 2023. It is now read-only.

Add RDP login support for windows hosts#36

Merged
ssbarnea merged 2 commits intoansible-community:masterfrom
troyready:add_rdp_login
Feb 22, 2021
Merged

Add RDP login support for windows hosts#36
ssbarnea merged 2 commits intoansible-community:masterfrom
troyready:add_rdp_login

Conversation

@troyready
Copy link
Contributor

Current login command is hardcoded to SSH for all hosts. This enhancement adds a check for WinRM use, and for those hosts uses xfreerdp to open an RDP session instead.

@tbugfinder
Copy link
Collaborator

Hi @troyready
could you run the pre-commit hooks and also fix the tests?

@troyready
Copy link
Contributor Author

troyready commented Feb 18, 2021

Sure thing @tbugfinder . I ran pre-commit to fix the linting, and the test issues look to be unrelated (e.g. a bunch of ModuleNotFoundError: No module named 'sh' errors)

@tbugfinder
Copy link
Collaborator

Those sh errors should get fixed within #34
. That's my understanding :-)

@tbugfinder
Copy link
Collaborator

Hi @troyready
#34 was merged. Would you resync with master?

Current login command is hardcoded to SSH for all hosts. This
enhancement adds a check for WinRM use, and for those hosts uses
xfreerdp to open an RDP session instead.
@ssbarnea ssbarnea self-requested a review February 21, 2021 17:29
@ssbarnea ssbarnea added the enhancement New feature or request label Feb 21, 2021
@ssbarnea ssbarnea changed the title add RDP login support for windows hosts Add RDP login support for windows hosts Feb 21, 2021
@ssbarnea ssbarnea merged commit e69e5c1 into ansible-community:master Feb 22, 2021
@troyready troyready deleted the add_rdp_login branch February 22, 2021 17:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants