-
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: add new module vmware_export_ovf #50589
Conversation
@Tomorrow9, just so you are aware we have a dedicated Working Group for vmware. |
@MikeKlebolt @Akasurde @bedecarroll @CastawayEGR @chrrrles @dav1x @garbled1 @imjoseangel @jjahns @kamsz @kryptsi @lrivallain @nafpliot-ibm @nerzhul @oboukili @rhoop @ritzk @rmin @stravassac @tchernomax @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 |
@pdellaert @dericcrago @Im0 @tchernomax @ckotte @jeking3 Could you please review this ? |
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 had only minor improvements that could be made. There is no integration test for this, and I'm not sure the simulator would be able to do it... without an integration test and without unit tests, it could rot easily.
d2a87be
to
30d500e
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.
LGTM overall from a code perspective, will test in my environment tomorrow
@pdellaert Hi, did you get a chance to test this? |
@Tomorrow9 There is one pending comment related to |
85772e2
to
292dd96
Compare
@Akasurde Hi, I've made the change according to that review comment. Thanks. |
@Tomorrow9 Thanks for the contribution. |
SUMMARY
Create a new module named "vmware_export_ovf" to export VM to ovf template.
Fixes #42397
ISSUE TYPE
New Module Pull Request
COMPONENT NAME
vmware_export_ovf
ADDITIONAL INFORMATION
hosts: localhost
tasks:
name: test export ovf
vmware_export_ovf:
validate_certs: no
hostname: test.eng.com
username: root
password: test
name: test-debian9.4
datacenter: 6.7GA
export_dir: /tmp/
export_with_images: true
register: export_ovf