Instance failover: fix bug for INT_MIRROR cases
Instance failover: fix bug for INT_MIRROR cases

Patches db366d9a and aac4511a added support for EXT_MIRROR instances,
but inadvertently introduced a bug: for INT_MIRROR cases, we don't
need (actually we can't support) neither an iallocator nor a target
node.

To fix this, we move the iallocator/node checks in CheckPrereq (or
respectively in the tasklet CheckPrereq), where we have access to the
instance configuration, and additionally we check for and prevent
passing either of these two for INT_MIRROR instances.

Signed-off-by: Iustin Pop <acf83a5274f31a43af6b914614365999469c1a31@google.com>
Reviewed-by: René Nussbaumer <bc755ac45a0e799d1d490cba76febc56d80cc5fc@google.com>
Reviewed-by: Michael Hanselmann <68b67885ff74281219b4a4be33cc20fd45078d7c@google.com>

ganeti-github-testing/ganeti-test-1,ganeti/ganeti,andir/ganeti,ganeti-github-testing/ganeti-test-1,bitemyapp/ganeti,andir/ganeti,ganeti/ganeti,apyrgio/snf-ganeti,dimara/ganeti,apyrgio/ganeti,yiannist/ganeti,ganeti/ganeti,dimara/ganeti,leshchevds/ganeti,mbakke/ganeti,yiannist/ganeti,bitemyapp/ganeti,andir/ganeti,apyrgio/snf-ganeti,mbakke/ganeti,mbakke/ganeti,onponomarev/ganeti,yiannist/ganeti,leshchevds/ganeti,apyrgio/ganeti,grnet/snf-ganeti,grnet/snf-ganeti,leshchevds/ganeti,onponomarev/ganeti