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
Add support for push to VMWare #1169
Conversation
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.
Looks awesome! :-)
Do you plan to add an integration test for this feature?
Also, you need to run tools/prepare-source.sh
and force-push the changes, the source check is currently failing.
e156979
to
90ad259
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.
This is very, very nice! I had some minor comments about he API, but over all this is looking like a great start.
bf4fd73
to
2ea66ce
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.
Really great! Only thing I think we need before merging is an integration test. Something similarly to test/cases/aws.sh
.
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.
Looks good! Two super-minor comments inline.
233bc33
to
8104110
Compare
7debb4a
to
a1f88e9
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 for adding the tests @jmikovic! This looks great!
a1f88e9
to
f87a110
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.
Looks good to me, awesome work Jozef! :)
29f0115
to
33ec3d6
Compare
New upload target for VMWare, similar to the ones for AWS and Azure, allowing users to set credentials for their vSphere instance. Commit also includes function that performs the actual upload.
Extend Weldr API to allow user to specify option to upload built image to VMWare, makes use of previously define upload target.
…d name VMDK image has default name 'disk.vmdk' and there is no option to change the name when uploading to vSphere, so I'm using symlink so that uploaded image has the name user specified instead of the default one.
33ec3d6
to
947d250
Compare
@teg @ondrejbudai please re-review test is failing on openstack test - not related to this PR |
Thanks @jmikovic! Will do :) |
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.
Amazing to see this land! Thanks @jmikovic!
Error in image tests for f32 must be unrelated, as the image definitions did not change. Merging. |
Add upload target for VMWare images to weldr API to allow users to push them directly to their vSphere instance without the need to download and push the image manually.
This pull request includes: