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 the password issue on console/sshd #6351

Merged
merged 1 commit into from
Dec 5, 2018

Conversation

pdostal
Copy link
Member

@pdostal pdostal commented Dec 5, 2018

This is fix of an issue discovered on Live systems, where root password is empty.
I use custom password for the sshboy user and cp instead of scp.

This was discovered in #6295 by @DimStar77

@pdostal
Copy link
Member Author

pdostal commented Dec 5, 2018

I implemented install instead of cp; chmod; chown as suggested by @dzedro (thank you) and it's already green on SLE12SP4 and pending on Tumbleweed@GNOME-Live.

tests/console/sshd.pm Outdated Show resolved Hide resolved
@OleksandrOrlov OleksandrOrlov merged commit ad6613d into os-autoinst:master Dec 5, 2018
@OleksandrOrlov
Copy link
Contributor

Hi @pdostal, Could you please take a look, as one of the tests is failed on SLE15. Seems like opensusebasetest::select_serial_terminal(); is not implemented for 'svirt' backend.

@OleksandrOrlov
Copy link
Contributor

OleksandrOrlov commented Dec 5, 2018

@pdostal, It also failed on s390x on different step. Would you like to revert both #6295 and #6351, so that you'll be able to fix all the issues and verify them without hurry?

@pdostal
Copy link
Member Author

pdostal commented Dec 5, 2018

@OleksandrOrlov preferably no. I was reminded in poo#44696 as well. Thank you for merging - we fixed the password problem with Live distributions. Now I will continue with the opensusebasetest::select_serial_terminal() on the svirt backend. I hope one day will be enough.

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