-
Notifications
You must be signed in to change notification settings - Fork 270
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
feat(xo-web/import/esxi): import VM from ESXi #6663
Conversation
Tests done :
|
I would prefer a dedicated page, as explained in the original issue (Import/From VMware), allowing multi VMs import, which is a special process we don't have in the "Import/VM" view. |
|
|
test done : rebase from master ( since there is fix on import, merge from branch #6662 )
|
Test donetest Protocol
import XVA
import Ova
import from url
import from esxi
|
65e6bdc
to
9918303
Compare
9918303
to
3f204b5
Compare
de4ffc0
to
8d13f9a
Compare
Testprotocolonly need to pull the branch xva / ova import : okimport esxi : still running , but no error on start |
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.
functionnal tests are ok for me
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 but I don't know how to test it. @Rajaa-BARHTAOUI or @fbeauchamp, can you do a last end-to-end test after the latest changes?
It has been tested after the latest changes, and everything is still running. |
Ok, then this PR is ready to merge after #6662. |
32ea87e
to
cf17c77
Compare
vm.importFromEsxi
Screenshots
Check list
Fixes #007
orSee xoa-support#42
)CHANGELOG.unreleased.md
:${name} v${new version}
)cron/parse.spec.js
)xo-server
API changes, the corresponding test has been added to/updated onxo-server-test
Process
WiP:
(Work in Progress) if not ready to be mergedFrom the Four Agreements: