"Block migraton" option on the VM´s Advanced tab

[manilx]

@abudef You can't migrate VM's between hosts and Disks between storages.

[abudef]

@manilx Thanks, how simple! I confused myself by thinking of "Block" as a noun, not a verb imperative, being influenced by the NBD term

[olivierlambert]

Ah yes I see, I wonder if we could make it more clear.

[olivierlambert]

Pinging @thomas-dkmt

[Andrew]

@abudef @olivierlambert @thomas-dkmt I agree. I read block the same way... how about disable or prevent. (French? empêcher)

[olivierlambert]

Disable or prevent are indeed a lot better.

[DustinB]

If I had my choice, Prevent Migration is more understandable.

Disable Migration, while it means the same thing, doesn't naturally come out of the English language.

[olivierlambert]

Food for thoughts @thomas-dkmt

[thomas-dkmt]

@olivierlambert @DustinB Thanks! It's always helpful to have a native speaker's perspective.

[DustinB]

@thomas-dkmt said in "Block migraton" option on the VM´s Advanced tab:

@olivierlambert @DustinB Thanks! It's always helpful to have a native speaker's perspective.

Happy to help, worth noting that if a setting like this is enabled on a given VM, there should also be some sort of visual indication on the home screen of XO to annotate that these VMs won't move unless this is disabled.

[abudef]

Done
https://github.com/vatesfr/xen-orchestra/commit/dba08f0fac866c76700a3785a178743770159883

0 MathieuRA committed to vatesfr/xen-orchestra

feat(xo-web/vm): rename 'block migration' to 'disable migration' (#8492)

Introduced by 11a801c
Related to #8222

[DustinB]

@abudef The English translation is meant to be "Prevent Migration".

I think some lines were crossed ha.

[john.c]

@DustinB said in "Block migraton" option on the VM´s Advanced tab:

@abudef The English translation is meant to be "Prevent Migration".

I think some lines were crossed ha.

I agree it really needs to actually read "Prevent Migration", that will then make it pretty clear.

[DustinB]

Looks to be correct on the current release of XO from Source and reads as "Prevent Migration"