-
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 content library CRUD operations #58716
Conversation
The test
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 |
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.
Rest LGTM
lib/ansible/modules/cloud/vmware/vmware_content_library_manager.py
Outdated
Show resolved
Hide resolved
lib/ansible/modules/cloud/vmware/vmware_content_library_manager.py
Outdated
Show resolved
Hide resolved
lib/ansible/modules/cloud/vmware/vmware_content_library_manager.py
Outdated
Show resolved
Hide resolved
lib/ansible/modules/cloud/vmware/vmware_content_library_manager.py
Outdated
Show resolved
Hide resolved
lib/ansible/modules/cloud/vmware/vmware_content_library_manager.py
Outdated
Show resolved
Hide resolved
lib/ansible/modules/cloud/vmware/vmware_content_library_manager.py
Outdated
Show resolved
Hide resolved
lib/ansible/modules/cloud/vmware/vmware_content_library_manager.py
Outdated
Show resolved
Hide resolved
Thanks @Akasurde for reviews. Committed the changes. Please review |
The test
|
The test
|
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.
Otherwise lgtm
lib/ansible/modules/cloud/vmware/vmware_content_library_manager.py
Outdated
Show resolved
Hide resolved
lib/ansible/modules/cloud/vmware/vmware_content_library_manager.py
Outdated
Show resolved
Hide resolved
lib/ansible/modules/cloud/vmware/vmware_content_library_manager.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.
Handled review comments.
test/integration/targets/vmware_content_library/content_lib_crud_ops.yml
Outdated
Show resolved
Hide resolved
test/integration/targets/vmware_content_library/content_lib_crud_ops.yml
Outdated
Show resolved
Hide resolved
test/integration/targets/vmware_content_library/content_lib_crud_ops.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.
Hi @goneri, Thanks for review comments. Please let me know your inputs on my queries
Thanks
test/integration/targets/vmware_content_library/content_lib_crud_ops.yml
Outdated
Show resolved
Hide resolved
test/integration/targets/vmware_content_library/content_lib_crud_ops.yml
Outdated
Show resolved
Hide resolved
test/integration/targets/vmware_content_library/content_lib_crud_ops.yml
Outdated
Show resolved
Hide resolved
You may find this new doc useful: https://github.com/ansible/ansible/blob/devel/docs/docsite/rst/dev_guide/platforms/vmware_guidelines.rst It explains how to run the test-suite locally. |
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.
Made changes as per review comments
test/integration/targets/vmware_content_library/content_lib_crud_ops.yml
Outdated
Show resolved
Hide resolved
The test
|
The test
|
b8b58a1
to
a816e4b
Compare
* Fixing file format issue. converted file from dos2unix * Review comments changes * Changes as per review comments. Moved integration test files to module name folder * Fixed precheck failures * Fixed few more precheck failures. Visual indentation fix and testname duplication fix in integration fix
SUMMARY
New module to create, Update & delete content library.
Content library created with datastore backing
ISSUE TYPE
COMPONENT NAME
VMWare content library