-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
VMware: New Module for VMWare content Library info #58703
Conversation
The test
The test
The test
|
@GBrawl @MikeKlebolt @Tomorrow9 @alongchamps @bedecarroll @CastawayEGR @chrrrles @dav1x @davidmhewitt @garbled1 @GyorgyPeter @imjoseangel @jjahns @kamsz @karstenjakobsen @kryptsi @lrivallain @michaeldeaton @nafpliot-ibm @nerzhul @oboukili @ritzk @rmin @stravassac @sumkincpp @vmwjoseph @warthog9 @woshihaoren As a maintainer of a module in the same namespace this new module has been submitted to, your vote counts for shipits. Please review this module and add |
The test
|
@pgbidkar this PR contains more than one new module. Please submit only one new module per pull request. For a detailed explanation, please read the grouped modules documentation |
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.
Minor changes, rest LGTM
Thanks @Akasurde for the review comments. committed new changes. Please review |
lib/ansible/modules/cloud/vmware/vmware_content_library_info.py
Outdated
Show resolved
Hide resolved
test/integration/targets/vmware_content_library/content_lib_details.yml
Outdated
Show resolved
Hide resolved
test/integration/targets/vmware_content_library/content_lib_details.yml
Outdated
Show resolved
Hide resolved
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.
Module lgtm when test comments are resolved.
test/integration/targets/vmware_content_library/content_lib_details.yml
Outdated
Show resolved
Hide resolved
test/integration/targets/vmware_content_library/tasks/content_lib_details.yml
Outdated
Show resolved
Hide resolved
test/integration/targets/vmware_content_library/tasks/content_lib_details.yml
Outdated
Show resolved
Hide resolved
lib/ansible/modules/cloud/vmware/vmware_content_library_info.py
Outdated
Show resolved
Hide resolved
lib/ansible/modules/cloud/vmware/vmware_content_library_info.py
Outdated
Show resolved
Hide resolved
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.
Hi @goneri, Thanks for review comments. Please let me know your inputs for my queries.
Thanks
lib/ansible/modules/cloud/vmware/vmware_content_library_info.py
Outdated
Show resolved
Hide resolved
lib/ansible/modules/cloud/vmware/vmware_content_library_info.py
Outdated
Show resolved
Hide resolved
test/integration/targets/vmware_content_library/tasks/content_lib_details.yml
Outdated
Show resolved
Hide resolved
test/integration/targets/vmware_content_library/tasks/content_lib_details.yml
Outdated
Show resolved
Hide resolved
74c2ca1
to
d1f662a
Compare
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.
Thanks @goneri , Did the changes. please review
lib/ansible/modules/cloud/vmware/vmware_content_library_info.py
Outdated
Show resolved
Hide resolved
test/integration/targets/vmware_content_library/tasks/content_lib_details.yml
Outdated
Show resolved
Hide resolved
The test
The test
|
d1f662a
to
8dfd648
Compare
Thanks @pgbidkar for the reactivity. It looks good to me. |
Changes as per review comments and renamed file to vmware_content_library_info Removing password field from publish info dictionary Removed current password field from publish info Modfied vmware_content_library_info module with review comments. Adding new module for content library manager Adding integration tests for content library Fixing format issue. Converted dos2unix Fixed precheck test failures Fixing some more precheck Errors Fixed few format issues in vmware_content_library_info. Deleting vmware_content_library_manager module as 2 modules are not allowed in same commit Fixed documentation format issue Fixing format issue. Converted dos2unix
… branch Changes as per review comments Review comments. Moved test .yml files under tasks folder Moved test files under module name folder. Handled library_get call under try catch to throw proper error message Fixed precheck failure bare except. Catching Exception
8dfd648
to
6457adf
Compare
SUMMARY
New module to get list of Existing Content Libraries
ISSUE TYPE
COMPONENT NAME
VMWare Content Library
ADDITIONAL INFORMATION
New module to get list of Existing Content Libraries. Get details of existing content library