Boot environments and zfs canmount=noauto

[Ultima]

Is this actually required? Instead of the extra steps, how about just
moving/creating each dataset into each BE (or at least the ones desired)
and setting them to mountpoint=inherit. With zfs_enable="YES" this should
mount the dataset appropriately. Only the root BE should need
canmount=noauto because if the parent dataset is not mounted the child will
inherently not mount.

Is this not sufficient?