Fixes: HA provisioning with fab fails if config nodes in testbed.py

are not is sequential order ( The chosen master node is not the first node)

While comparing the openstack and cfgm node sets, we are sorting the
nodes which changes the order in which they are mentioned.

Changing the compare to use sets instead of using sort.

Change-Id: I312664cb67ad51244cdcc5ae0eecdb739644a8df
ClosesBug: 1455698

fabfile/tasks/helpers.py

@@ -1118,17 +1118,18 @@ def pre_check():
         print "\nERROR: \n\tkeystone_ip(VIP) needs to be set in testbed.py for HA, when more than one openstack node is defined."
         exit(1)
     if (len(env.roledefs['openstack']) > 1 and
-        env.roledefs['openstack'].sort() == env.roledefs['cfgm'].sort() and
+        set(env.roledefs['openstack']) == set(env.roledefs['cfgm']) and
         get_openstack_internal_vip() != get_openstack_internal_vip()):
         print "\nERROR: \n\tOpenstack and cfgm nodes are same, No need for contrail_internal_vip to be specified in testbed.py."
         exit(1)
     if (len(env.roledefs['openstack']) > 1 and
-        env.roledefs['openstack'].sort() != env.roledefs['cfgm'].sort() and
+        set(env.roledefs['openstack']) != set(env.roledefs['cfgm']) and
         get_openstack_internal_vip() == get_openstack_internal_vip()):
         print "\nERROR: \n\tOpenstack and cfgm nodes are different, Need to specify  contrail_internal_vip testbed.py."
         exit(1)
-        print "\nERROR: \n\tOpenstack and cfgm nodes are same, No need for contrail_internal_vip to be specified in testbed.py."
-        execute('add_openstack_reserverd_ports')
+    else:
+        print "\nINFO: \n\tOpenstack and cfgm nodes are same, No need for contrail_internal_vip to be specified in testbed.py."
+    execute('add_openstack_reserverd_ports')
 
 def role_to_ip_dict(role=None):
     role_to_ip_dict = {}