-
Notifications
You must be signed in to change notification settings - Fork 85
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
Replacing Paramiko with ssh connection to host #830
Replacing Paramiko with ssh connection to host #830
Conversation
This ssh connection to host is used to get directory paths, htxerr files and many other validations. Signed-off-by: Tasmiya Nalatwad <tasmiya@linux.vnet.ibm.com>
8f496af
to
91205e6
Compare
@abdhaleegit All the logs i have attached to jira covering all the scenarios. |
dev_base = self.execute_remote_command('basename $(realpath {})'.format(dev_path)) | ||
if device == dev_base: | ||
return dev_path | ||
return device |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@abdhaleegit I am removing these functions as i am directly running commands on host and getting the device paths, Tested this by providing 2 different paths as '/dev/mapper/mpatha' and '/dev/sda' it worked fine in both the cases.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@PraveenPenguin Removed paramiko as per your suggestion.. please ack |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This ssh connection to host is used to get directory paths, htxerr files and many other validations.