[kavlang5k.rb] allow kavlan pattern to be given at the port level

Une modification pour prendre en compte le pattern Kavlan quand il est spécifié au niveau du port.

Sans cela, la génération puppet utilise le pattern de la linecard, et dans le cas du switch gw-new, ca n'ira pas pour les noeuds qui seront branchés en 25Gb/s (comme les troll, qui sont actuellement en 10Gb/s sur gw).

Le fichier de description du switch gw-new a été modifié pour:

• n'utiliser plus qu'une seule linecard, en position 0.
• changer la préparation (sous forme de commentaires) des troll pour tenir compte de cette MR

Pour tester, il faut:

• vérifier que rake gen:puppet:kavlang5k ne change rien dans le cdpl malgré le passage en une seule linecard
• vérifier l'impact quand on utilise la possibilité apportée par la MR
  • dé-commenter la section troll-1 dans gw-new.yaml
  • supprimer troll-1 de gw.yaml
  • faire un rake reference-api
  • faire un rake valid:network et constater qu'il y a une erreur de vitesse concernant troll-1 (normal car on n'a pas modifié sa vitesse dans troll-1.yaml).
  • faire un rake gen:puppet:kavlang5k et vérifier dans le cdpl que troll-1 a changé de switch et que son interface ethernet correspond à celle donnée dans le fichier gw-new.yaml

NB: par rapport à la MR originale, j'ai retiré les préparations/génération yaml/json concernant gw-new de manière à pouvoir merger indépendamment de la modification plus générale qui sera effectuée dans le cadre de la migration.

lib/refrepo/gen/puppet/kavlang5k.rb

@@ -4,9 +4,11 @@ def kavlan_switch_port_lookup(switch, node_uid, interface='')
   switch["linecards"].each_with_index do |lc, lc_uid|
     next if not lc["ports"]
     lc["ports"].each_with_index do |port, port_uid|
+      pattern_source = lc
       if port.is_a?(Hash)
         switch_remote_port = port["port"] || lc["port"] || ""
         switch_remote_uid = port["uid"]
+        pattern_source = port if port.has_key?('snmp_pattern') or port.has_key?('kavlan_pattern')
       else
         switch_remote_port = lc["port"] || ""
         switch_remote_uid = port
@@ -15,7 +17,7 @@ def kavlan_switch_port_lookup(switch, node_uid, interface='')
       if switch_remote_uid == node_uid and switch_remote_port == interface
         # Build port name from snmp_naming_pattern
         # Example: '3 2 GigabitEthernet%LINECARD%/%PORT%' -> 'GigabitEthernet3/2'
-        pattern = lc.has_key?("kavlan_pattern") ? lc["kavlan_pattern"] : lc["snmp_pattern"]
+        pattern = pattern_source.has_key?("kavlan_pattern") ? pattern_source["kavlan_pattern"] : pattern_source["snmp_pattern"]
         port_name = pattern.sub("%LINECARD%",lc_uid.to_s).sub("%PORT%",port_uid.to_s)
         return port_name
       end