XenAPI: improve the image handler configure

Registered by Jianghua Wang on 2017-08-24

We have a XenAPI config named as "image_upload_handler" which is used to set a class path which contains the function to upload images.

We need improve the image handler due to the following reasons:
1. It's easy to lost control by giving the ability to specify a class path and loading dynamically at runtime.
2. We want to use a new image handler which will avoid depending on a file system based SR
type in XenServer so that we can use some non-FS SR types: e.g. lvm, iscsi. But there is no way to specify the image download handler at the moment.

The proposed changes include:
1. deprecate the existing option of "image_upload_handler"; keep the existing class path loader for one more release.
2. add a new config named as "image_handler" to replace the deprecated option. It will contain both uploader handler and download handler.
  - Maintain a supported plugin list for image handler. Each image handler plugin should support the functionality to upload and download images. Give each plugin a short name.
  - The option of "image_handler" only accepts the short name from the above plugin list.
3. Add the new image handler which can support non-FS SR types.

Blueprint information

Status:
Complete
Approver:
melanie witt
Priority:
Low
Drafter:
Jianghua Wang
Direction:
Approved
Assignee:
Naichuan Sun
Definition:
Approved
Series goal:
Accepted for rocky
Implementation:
Implemented
Milestone target:
milestone icon rocky-3
Started by
Jianghua Wang on 2018-03-23
Completed by
Matt Riedemann on 2018-06-14

Related branches

Sprints

Whiteboard

Gerrit topic: https://review.openstack.org/#q,topic:bp/xenapi-image-handler-option-improvement,n,z

Addressed by: https://review.openstack.org/497201
    XenAPI: image handler option improvement

Gerrit topic: https://review.openstack.org/#q,topic:vdiStream,n,z

I'm deferring this to the Rocky release since it wasn't approved by the first milestone for Queens which was also our spec freeze. Please bring this up early in the Queens release when development opens so we can get the proper attention on this. -- mriedem 20171120

Addressed by: https://review.openstack.org/486475
    XenAPI: define a new image handler to use vdi streaming

This was discussed at the PTG and in today's nova meeting to approve for Rocky. -- melwitt 20180322

Addressed by: https://review.openstack.org/563077
    XenAPI: document change for supporting image handler

Addressed by: https://review.openstack.org/568318
    XenAPI: Pass expected return codes to resize2fs

Addressed by: https://review.openstack.org/568444
    XenAPI: update the document related to vdi streaming

Addressed by: https://review.openstack.org/574318
    DNM: default [xenserver]/image_handler to vdi_remote_stream

(?)

Work Items

This blueprint contains Public information 
Everyone can see this information.

Subscribers

No subscribers.