If it gets longer than 63 characters, then replace the label value by a hash to do not abort the VM import. The origin value is still used as `VirtualMachineImageSpec.DisplayName`. The label is currently used to search for the `VirtualMachineImage` object during the reconciliation phase.
Related to: https://github.com/harvester/harvester/issues/6463
Signed-off-by: Volker Theile <vtheile@suse.com>
- Import OpenStack server by UUID
- Import OpenStack server with upper case characters in its name
The following improvements have been done:
- Sanitize the configured `VirtualMachineName` field, e.g. convert upper case to lower case to make it RFC 1123 compliant.
- Convert UUID to real name for OpenStack imports
- Reduce waiting time to recheck if created VM is running from 5min to 2min
- Rename variable `uuid` to `serverUUID` in the OpenStack client code to do not collide with the imported uuid module
- Improve error and log messages
- Fix typos
- Add comments
Related to: https://github.com/harvester/harvester/issues/6500
Related to: https://github.com/harvester/harvester/issues/6505
Signed-off-by: Volker Theile <vtheile@suse.com>
Add the fields `UploadImageRetryCount` and `UploadImageRetryDelay` to `OpenstackSourceSpec`.
Related to: https://github.com/harvester/harvester/issues/6675
Signed-off-by: Volker Theile <vtheile@suse.com>
- A new `apiGroups` is added to the `ClusterRole` to be able to list `network-attachment-definitions`.
- The `network-attachment-definitions` CRD has to be generated in the test env.
Related to: https://github.com/harvester/harvester/issues/6491
Signed-off-by: Volker Theile <vtheile@suse.com>
Add log messages to better track what the importer is doing and to better isolate errors if necessary.
Related to: https://github.com/harvester/harvester/issues/6512
Signed-off-by: Volker Theile <vtheile@suse.com>
added more debug info
close lease earlier
stage vmware migration improvements
improve openstack migration to check uefi/tpm/secureboot from associated image
improve firmware lookup for openstack, and add ability to specify custom storage classes
fix storage class caching issue
drop need for sync
fix imports
Log errors in the OpenStack and VMWare import controllers when the
client fails to verify.
The client for VMWare and OpenStack VM import are constructed and then
verified in an additional step. Instead of just indicating that there
was an error with the status conditions, this change adds a log message
to the controllers logs, which gives an indication why the verification
failed.
related-to: https://github.com/harvester/vm-import-controller/issues/26
Signed-off-by: Moritz Röhrich <moritz.rohrich@suse.com>
simplify disk condition checks
convert disk image status reconcile into a wrapper
move VM reconcile to a wrapper method
move preflight checks to a wrapper
more wrappers
bumped up harvester/harvester to v1.1.0-rc2
changed vmware client to check VM state before attempting poweroff, add closure for ovf client to fix Operation timed out errors
fine tuned integration tests
changed logging message
