LGTM, thanks
On Wed, Apr 10, 2013 at 8:24 PM, Guido Trotter <[email protected]> wrote: > The test tries to scp the instance disks from the primary node to the > new destination, but connects to the master node instead. Thus it works > only when the primary node is the master node. Fixing by correcting to > the right node. > > Signed-off-by: Guido Trotter <[email protected]> > --- > qa/qa_instance.py | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/qa/qa_instance.py b/qa/qa_instance.py > index 14bdf2c..37c12c5 100644 > --- a/qa/qa_instance.py > +++ b/qa/qa_instance.py > @@ -636,7 +636,7 @@ def TestInstanceModifyPrimaryAndBack(instance, > currentnode, othernode): > pathutils.SSH_KNOWN_HOSTS_FILE, > "-oCheckHostIp=no", "-oStrictHostKeyChecking=yes", > "-oHashKnownHosts=no", "-oHostKeyAlias=%s" % > cluster_name, > - "-r", disk, "%s:%s" % (other, filestorage)]) > + "-r", disk, "%s:%s" % (other, filestorage)], > node=current) > AssertCommand(["gnt-instance", "modify", "--new-primary=%s" % other, > name]) > AssertCommand(["gnt-instance", "startup", name]) > > -- > 1.7.10.4 > >
