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

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"