-
Notifications
You must be signed in to change notification settings - Fork 540
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
Feature/onstor collects diskshare informations #953
Feature/onstor collects diskshare informations #953
Conversation
kula1922
commented
Jul 7, 2014
- Onstor plugin collects diskshare informations
@@ -13,6 +13,7 @@ | |||
from ralph.scan.plugins import get_base_result_template | |||
from ralph.util import parse | |||
from ralph.util.network import connect_ssh, check_tcp_port | |||
from ralph.discovery.models import DiskShare |
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.
Append this import to line 11.
} | ||
) | ||
return disk_shares | ||
|
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.
What about some basic unit tests for those 3 functions above..?
Update CHANGELOG txt |
finally: | ||
ssh.close() | ||
return device_info | ||
|
||
|
||
def scan_address(ip_address, **kwargs): | ||
if kwargs.get('http_family') not in ('sscccc',): | ||
if kwargs.get('http_family') not in ('sscccc'): |
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.
I'd rather leave this comma here (previously it was a tuple, now it's just a string). Consider this:
'a' in ('aaa',)
False
'a' in ('aaa')
True
…re-informations Feature/onstor collects diskshare informations