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

Fix ssh command and update documentation #205

Merged
merged 2 commits into from
Mar 4, 2024
Merged

Conversation

tescande
Copy link
Contributor

@tescande tescande commented Mar 4, 2024

No description provided.

@tescande tescande requested review from stormi and gduperrey March 4, 2024 13:24
@tescande tescande self-assigned this Mar 4, 2024
This change imports and uses lib.commands.ssh() function instead of
calling subprocess.run() as the ssh() function handles various corner
cases and ssh options.

Signed-off-by: Thierry Escande <[email protected]>
This change specifies that raw strings must be used for the host version
regular expression patterns in vm_data.py (i.e. r"PATTERN"). This
corrects warning about invalid escaped sequences.

Signed-off-by: Thierry Escande <[email protected]>
@tescande tescande merged commit 9bf7adb into master Mar 4, 2024
4 checks passed
@tescande tescande deleted the fix/ssh_command branch March 4, 2024 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants