Skip to content

Comments

Refresh remote session during long migrations (26.1-lcm)#6916

Open
gthvn1 wants to merge 1 commit intoxapi-project:26.1-lcmfrom
xcp-ng:gtn-refresh-session-lcm
Open

Refresh remote session during long migrations (26.1-lcm)#6916
gthvn1 wants to merge 1 commit intoxapi-project:26.1-lcmfrom
xcp-ng:gtn-refresh-session-lcm

Conversation

@gthvn1
Copy link
Contributor

@gthvn1 gthvn1 commented Feb 19, 2026

Long VM migrations can exceed the default SM session timeout (24h), causing the destination host to fail when using an expired session. This is especially problematic for large VDI copies.

This patch ensures migrations do not fail due to session expiration by refreshing the remote session periodically while the data copy is in progress.

@psafont
Copy link
Member

psafont commented Feb 19, 2026

Please, add the hash of the backported commit into the commit message: f7afc85e320adcc3384048e661fe891c24d6e5e8

@gthvn1
Copy link
Contributor Author

gthvn1 commented Feb 19, 2026

Please, add the hash of the backported commit into the commit message: f7afc85e320adcc3384048e661fe891c24d6e5e8

you mean the sha1 I used to cherry-pick that is e76c039103fc37fa5d063c0266091836ca744529 right?
Edit: if it is that then done

Long VM migrations can exceed the default SM session timeout (24h), causing the
destination host to fail when using an expired session. This is especially
problematic for large VDI copies.

This patch ensures migrations do not fail due to session expiration by
refreshing the remote session periodically while the data copy is in progress.

(cherry picked from commit e76c039)

Signed-off-by: Guillaume <guillaume.thouvenin@vates.tech>
@gthvn1 gthvn1 force-pushed the gtn-refresh-session-lcm branch from f7afc85 to 8bbfa01 Compare February 19, 2026 17:02
@gthvn1
Copy link
Contributor Author

gthvn1 commented Feb 19, 2026

And I will add an alias so cherry-pick will use -x ;)

@gthvn1
Copy link
Contributor Author

gthvn1 commented Feb 20, 2026

I looked to the OCaml test that failed but I didn't understand the problem. I ran it locally and it looks good to me.

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