-
Notifications
You must be signed in to change notification settings - Fork 32
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
[Manual] Add manual test plan for vm-import with EFI mode and secure boot #1518
[Manual] Add manual test plan for vm-import with EFI mode and secure boot #1518
Conversation
1. Get the available access to the existing or prepared vSphere client (v7.1) | ||
|
||
## Harvester Prerequisite Setup | ||
1. Harvester can connect to the OpenStack dashbord and API endpoint |
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.
Typo, dashboard
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 the check. Update the typos to dashboard.
|
||
## Harvester Prerequisite Setup | ||
1. Harvester can connect to the OpenStack dashbord and API endpoint | ||
1. Harvester can connect to the vShpere client dashboard and API endpoint |
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.
typo, vSphere.
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 the check. Update the typos to vSphere.
``` | ||
1. Check can list the image list on OpenStack Server | ||
``` | ||
openstack image list --os-auth-url http://172.19.109.182/identity --os-identity-api-version 3 --os-project-name admin --os-project-domain-name default --os-username <username> --os-password <password> |
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.
Avoid IP exposing?
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 the check. Remove the IP exposed on the check image command.
b5a93bb
to
85afea3
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
Thanks for this addition @TachunLin ! 👍 |
What this PR does / why we need it:
This PR is used to add manual test plan according to the latest vm-import-controller enhancement issue
harvester/harvester#5846
Test Scenarios
For OpenStack:
For VMware:
Special notes for your reviewer:
Tested on the local hugo server, image can be loaded well.
Additional documentation or context
Related task ticket:
#1509