conn_graph_facts:
  conn_graph_facts.py returns device_vlan_map_list. This used to be a dictionary
  with key being hostname and value being a list of vlanids for the fanout ports.
  We have modified this where the value instead of being a list of vlanids, it is a dictionary
  with key being the port_index and the value being the vlan id. This port_index is what gets
  put in the topology file. We get the port
  by looking at the host_port_vlans defined in the conn_graph for that device. This
  host_port_vlans has key being the Ethernet port (like Ethernet10) and value being
  a dictionary with key 'vlanlist' being a the list of fanout vlans. We check against all the ports
  'vlanlist' to get the port on the DUT that connects to this fanout vlan, and then split on
  'Ethernet' to get the port index.

  For example - lets say on dut with hostname "dut1", we have port Ethernet10 connected to fanout w/ vlan 120, and
  Ethernet11 connected to fanout w/ vlan 121, then we would have:

  "host_port_vlans":
    {
      "Ethernet10": {
         "mode": "Access",
         "vlanids": "120",
         "vlanlist": [
            120
         ]
      },
      "Ethernet11": {
         "mode": "Access",
         "vlanids": "121",
         "vlanlist": [
            121
         ]
      }
   }

  "VlanList": [
     120,
     121
  ]

  For vlan 120 in VlanList, we would iterate through host_port_vlans to find the port
  that has vlan 120 - in our case "Ethernet10". The port_index would then be "10". Similarly,
  for vlan 121, the port_index would be "Ethernet11".

  So, returned device_vlan_map_list would be:
    "dut1" : {
       "10" : 120,
       "11" : 121
    }

- vlan_port/kvm_port/mellanox_simx_port:
   - Updated to return (dut_fp_ports) a dictionary with key being the port index (same as in the topo file)
     and vlan being the port - instead of the just the list of ports.

bind/unbind vm_topology:
   - vlan_index is now a string in the dictionary of dut_fp_ports
   - updated regexp for checking valid vlan for multi-dut to be of the format '<dutIndex>.<portIndex>@<ptfIndex>'

remove_dut_port.yml:
  - set cmd to "remove" instead of "create" in vlan_port module call.
                                                                                                                                                                                                                                                        }

- The dut_fp_ports is a dictionary with key being a string and not an integer

- We were using the port name like 'Ethernet10' and splitting in out 'Ethernet' to
  get the port_index in the device_vlan_map_list. This port_index is to correspond to
  what would be in the topology file. This would not work
  when ports are not consecutively names - like 'Ethernet0', 'Ethernet4', 'Ethernet8' .....
  In this scenario, the topology file would have host_interfaces/vlans still as 0,1,2,.....

  Fix is to get the port names based on 'hwksu' (like it is done for minigraph) and
  then use that list to get the port_index to be  put into device_vlan_map_list

…and conn_graph_facts to module_utils/port_utils

The port_alias_to_name_map is generated based on hwsku and is common
code between minigraph_facts and conn_graph_facts ansible module.

Now that module_utils are supported, added module_utils/port_utils.py to
have this common code. Modified the ansible modules to use this new
common code.

…tils.py

Since dut_fp_ports is a dictionary with key being the dut name, and the
value being a dictionary with key being the interface index as a string,
need to typecast the host_interfaces interface index as a string.