Skip to content

Remove image format from deprecated receive start#7151

Queued
gthvn1 wants to merge 1 commit into
xapi-project:26.1-lcmfrom
xcp-ng:gtn-fix-image-format-26.1-lcm
Queued

Remove image format from deprecated receive start#7151
gthvn1 wants to merge 1 commit into
xapi-project:26.1-lcmfrom
xcp-ng:gtn-fix-image-format-26.1-lcm

Conversation

@gthvn1

@gthvn1 gthvn1 commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

The field image_format added to receive_start and receive_start2 breaks migration from old XAPI to new XAPI.

Both are sending RPC to the destination, which runs them locally. If the destination has the field but not the source it breaks compatibility. On the other hand, receive_start3 runs on the source. It is the VDI.create that is sent remotely to the destination.

This patch removes the new field. That means that you will not be able to choose the format of the destination when migrating from old version of XAPI to new version of XAPI. As receive_start and receive_start2 are deprecated the situation will become more and more rare.

(cherry picked from commit 71a2576)

The field image_format added to receive_start and receive_start2 breaks
migration from old XAPI to new XAPI.

Both are sending RPC to the destination, which runs them locally. If the
destination has the field but not the source it breaks compatibility. On the
other hand, receive_start3 runs on the source. It is the VDI.create that is
sent remotely to the destination.

This patch removes the new field. That means that you will not be able to
choose the format of the destination when migrating from old version of XAPI to
new version of XAPI. As receive_start and receive_start2 are deprecated the
situation will become more and more rare.

Signed-off-by: Guillaume Thouvenin <guillaume.thouvenin@vates.tech>
(cherry picked from commit 71a2576)
@minglumlu minglumlu added this pull request to the merge queue Jun 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants