Skip to content

Comments

docs: document safer procedures for updates#435

Open
psafont wants to merge 1 commit intoxcp-ng:masterfrom
psafont:dev-pau-update
Open

docs: document safer procedures for updates#435
psafont wants to merge 1 commit intoxcp-ng:masterfrom
psafont:dev-pau-update

Conversation

@psafont
Copy link

@psafont psafont commented Jan 19, 2026

Previous procedure recommended updating the packages in all hosts, and after that was complete, restart all hosts. This is dangerous sometimes because the processes running in a host and the filesystem are out of sync. Instead recommend updating and restarting one host at a time.

The previous procedure did not mention how to avoid running tasks in a host before updating, now this is done using host-disable.

Now the choice of rebooting or restarting xapi is put before they are explained, because the former needs to shut down the VMs, or evacuate them onto other hosts, while the latter doesn't, so they must choose before starting the procedure to be consistent.

Before submitting the pull request, you must agree with the following statements by checking both boxes with a 'x'.

  • "I accept that my contribution is placed under the CC BY-SA 2.0 license [1]."
  • "My contribution complies with the Developer Certificate of Origin [2]."

[1] https://creativecommons.org/licenses/by-sa/2.0/
[2] https://docs.xcp-ng.org/project/contributing/#developer-certificate-of-origin-dco

@olivierlambert
Copy link
Member

We probably need to double check with the XO team about our RPU algorithm, if it's taking this into account (I think it does, but it won't hurt to make sure it does).

Copy link
Member

@stormi stormi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the improvement. I was sure that we already mentioned host-disable and host-evacuate, yet apparently we didn't (I'm puzzled. I did compare the update process in our docs with that applied by Xen Orchestra a few months or years ago...).

@stormi
Copy link
Member

stormi commented Jan 22, 2026

We probably need to double check with the XO team about our RPU algorithm, if it's taking this into account (I think it does, but it won't hurt to make sure it does).

I also think it does, however comparing to make sure we follow the same steps in a manual update vs an XO RPU would be good.

@stormi
Copy link
Member

stormi commented Jan 22, 2026

Adding @Wescoeur to ensure that the updated steps are fully compatible with an linstor upgrade (I think they are, but better double check).

@stormi stormi requested a review from Wescoeur January 22, 2026 17:47
Previous procedure recommended updating the packages in all hosts, and
after that was complete, restart all hosts. This is dangerous sometimes
because the processes running in a host and the filesystem are out of
sync. Instead recommend updating and restarting one host at a time.

The previous procedure did not mention how to avoid running tasks in a
host before updating, now this is done using host-disable.

Now the choice of rebooting or restarting xapi is put before they are
explained, because the former needs to shut down the VMs, or
evacuate them onto other hosts, while the latter doesn't, so they must
choose before starting the procedure to be consistent.

Signed-off-by: Pau Ruiz Safont <pau.safont@vates.tech>
Copy link
Member

@Wescoeur Wescoeur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Regarding LINSTOR, I think that's ok.

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.

5 participants