-
Notifications
You must be signed in to change notification settings - Fork 65
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
Added snmp user facts changes #649
Conversation
@@ -31,9 +31,9 @@ | |||
config='config.json' |
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.
change copyright year
@@ -68,8 +68,9 @@ def test_should_get_all_snmp_v3_users(self): | |||
) | |||
|
|||
def test_should_get_by_id_snmp_v3_users(self): |
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.
change test case name to test_should_get_by_name_snmp_v3_users
Also update this resource in CHANGELOG and endpoints file |
…sible into refactored_snmp_usersv3
@@ -44,7 +44,7 @@ | |||
hostname: 172.16.101.48 | |||
username: administrator | |||
password: my_password | |||
api_version: 800 |
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.
change hpeOneView version to 6.0.0
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
Description
[Added snmp user facts changes]
Issues Resolved
[List any issues this PR will resolve. e.g., Fixes #1]
Check List
$ ./build.sh
).