Include additional values

Working on including "mgmt_only" as an interface value that gets moved across when adding an interface.

Author: NetTech2001

netbox_interface_synchronization/templates/netbox_interface_synchronization/interface_comparison.html

@@ -37,9 +37,9 @@
 
 <p>
 {% if templates_count == interfaces_count %}
-    The Device Type and Device have the same Interfaces.
+    The Device Type and Device have the same number of Interfaces.
 {% else %}
-    The Device Type and Device have different Interfaces.<br>
+    The Device Type and Device have a different number of Interfaces.<br>
     Device Type: {{ templates_count }}<br>
     Device: {{ interfaces_count }}
 {% endif %}
@@ -154,7 +154,7 @@
         {% endfor %}
     </table>
     <div class="text-right">
-        <input type="submit" value="Apply" class="btn btn-primary">
+        <input type="submit" value="Apply Changes" class="btn btn-green">
     </div>
 </form>
 

netbox_interface_synchronization/utils.py

@@ -24,6 +24,7 @@ class UnifiedInterface:
     name: str
     type: str
     type_display: str
+    mgmt_only: bool
     is_template: bool = False
 
     def __eq__(self, other):

netbox_interface_synchronization/views.py

@@ -24,7 +24,7 @@ def get(self, request, device_id):
 
         unified_interfaces = [UnifiedInterface(i.id, i.name, i.type, i.get_type_display()) for i in interfaces]
         unified_interface_templates = [
-            UnifiedInterface(i.id, i.name, i.type, i.get_type_display(), is_template=True) for i in interface_templates]
+            UnifiedInterface(i.id, i.name, i.type, i.get_type_display(), i.mgmt_only, is_template=True) for i in interface_templates]
 
         # List of interfaces and interface templates presented in the unified format
         overall_interfaces = list(set(unified_interface_templates + unified_interfaces))
@@ -79,19 +79,19 @@ def post(self, request, device_id):
             # Add selected interfaces to the device and count them
             add_to_device_interfaces = InterfaceTemplate.objects.filter(id__in=add_to_device)
             interfaces_created = len(Interface.objects.bulk_create([
-                Interface(device=device, name=i.name, type=i.type) for i in add_to_device_interfaces
+                Interface(device=device, name=i.name, type=i.type, mgmt_only=i.mgmt_only) for i in add_to_device_interfaces
             ]))
 
             # Getting and validating a list of interfaces to rename
             fix_name_interfaces = filter(lambda i: str(i.id) in request.POST.getlist("fix_name"), interfaces)
             # Casting interface templates into UnifiedInterface objects for proper comparison with interfaces for renaming
             unified_interface_templates = [
-                UnifiedInterface(i.id, i.name, i.type, i.get_type_display()) for i in interface_templates]
+                UnifiedInterface(i.id, i.name, i.type,i.mgmt_only, i.get_type_display()) for i in interface_templates]
 
             # Rename selected interfaces
             interfaces_fixed = 0
             for interface in fix_name_interfaces:
-                unified_interface = UnifiedInterface(interface.id, interface.name, interface.type, interface.get_type_display())
+                unified_interface = UnifiedInterface(interface.id, interface.name, interface.type, interface.mgmt_only, interface.get_type_display())
                 try:
                     # Try to extract an interface template with the corresponding name
                     corresponding_template = unified_interface_templates[unified_interface_templates.index(unified_interface)]