Device interface(s) tab for tracking (#30)

* (feature): First Phase plumbing for Inventory/Device Interface options and tracking/priorty interface selection.

* (feature): Implement new tabular UX design for device CRUD in particular the Interfaces tab. Ability to add interfaces in order to "track" for Statseeker alarms and future feature implementations.

* (feature): Updated dockerfile.

Author: blake-russell

CHANGELOG.md

@@ -1,5 +1,18 @@
 # Changelog
 
+## v0.3.0 - 10-10-2025
+- 🖥️ Inventory: Re-design into single modal/page for CRUD process.
+- 🆕 Inventory Tracked Interface: Added ability to add interfaces in order to "track" for particular scripts and cards such as Statseeker
+
+## v0.2.9 - 10-3-2025
+- 🆕 Inventory Discovery: Added SNMP/SSH device discovery functionality.
+- 🗃️ DiscoveryJob Tasks: Added capability for job queuing discovery tasks.
+
+## v0.2.8 - 09-26-2025
+- 🆕 CiscoAdvisory: Added user assignment and status workflow.
+- ✅ On-call: Fix parsehandler bugs for Cogent email processing.
+- ✅ ASAVPN: Fixed unreachable stats.
+
 ## v0.2.7 - 09-03-2025
 - 🆕 CiscoAdvisory: Added functionality to interact with field notices to maintain impact and remediation.
 - ⚙️ CiscoAdvisory: Field Notices set as "No Impact" will auto-archive at specified On-Call auto-archive timeframe.

Dockerfile.ci

@@ -20,7 +20,8 @@ ENV PYTHONDONTWRITEBYTECODE=1
 
 # Install Python dependencies
 COPY requirements.txt .
-RUN pip install --no-cache-dir -r requirements.txt
+RUN pip install --upgrade pip
+RUN pip install --no-cache-dir --timeout=60 -r requirements.txt
 
 # Copy App
 COPY . .

src/network_ops_dashboard/inventory/forms.py

@@ -10,7 +10,7 @@ class Meta:
         model = Inventory
         fields = ('name', 'name_lookup', 'site', 'platform', 'serial_number', \
             'ipaddress_mgmt', 'ipaddress_rest', 'ipaddress_gmni', 'port_rest', 'port_netc', \
-            'port_gnmi', 'device_tag', 'priority_interfaces', 'creds_ssh', 'creds_rest')
+            'port_gnmi', 'device_tag', 'creds_ssh', 'creds_rest')
     name = forms.CharField(label="Device Name:", help_text="<br>Enter name that is resolved from mgmt IP excluding<br>the domain/subdomain.", required=True)
     name_lookup = forms.CharField(label="Name Lookup:", help_text="<br>ie: devicename.companyname.com", required=False)
     site = forms.ModelChoiceField(label="Site:", queryset=Site.objects.all(), required=False)
@@ -23,7 +23,7 @@ class Meta:
     port_netc = forms.CharField(label="Port (NETCONF):", initial='830')
     port_gnmi = forms.CharField(label="Port (gNMI):", initial='9339')
     device_tag = forms.ModelMultipleChoiceField(label="Device Tags:", queryset=DeviceTag.objects.all(), widget=forms.SelectMultiple(attrs={'class': 'form-select'}), required=False)
-    priority_interfaces = forms.CharField(label="Priority Interfaces:", help_text="<br>ie: GigabitEthernet2/1/1, TenGigabitEthernet1/1/1, etc", required=False)
+    # priority_interfaces = forms.CharField(label="Priority Interfaces:", help_text="<br>ie: GigabitEthernet2/1/1, TenGigabitEthernet1/1/1, etc", required=False)
     creds_ssh = forms.ModelChoiceField(label="SSH Credential:", queryset=NetworkCredential.objects.all(), required=False)
     creds_rest = forms.ModelChoiceField(label="REST Credential:", queryset=NetworkCredential.objects.all(), required=False)
 

src/network_ops_dashboard/inventory/models.py

@@ -171,7 +171,8 @@ class Status(models.TextChoices):
     discovery_source = models.CharField(max_length=64, blank=True, default="")
     sw_version = models.CharField(max_length=128, blank=True, default="")
     # Priority/Tracked Interfaces
-    priority_interfaces = models.CharField(max_length=750, blank=True)
+    priority_interfaces = models.JSONField(default=list, blank=True, null=True)
+    # stores ["Ethernet1/1", "Ethernet2/43", "mgmt0"]
     def get_priority_interfaces(self):
         return [k.strip() for k in (self.priority_interfaces or "").split(",") if k.strip()]
     def set_priority_interfaces(self, items):
@@ -194,6 +195,31 @@ def __str__(self):
         return str(self.name)
 
 class InventoryInterface(models.Model):
-    device = models.ForeignKey(Inventory, related_name="interfaces", on_delete=models.CASCADE)
-    name = models.CharField(max_length=128)
-    is_priority = models.BooleanField(default=False)
+    device = models.ForeignKey(
+        Inventory,
+        related_name="interfaces",
+        on_delete=models.CASCADE
+    )
+    name = models.CharField(max_length=64)
+    prefix = models.CharField(max_length=32, null=True, blank=True)
+    rack = models.PositiveSmallIntegerField(null=True, blank=True)
+    slot = models.PositiveSmallIntegerField(null=True, blank=True)
+    subslot = models.PositiveSmallIntegerField(null=True, blank=True)
+    port = models.PositiveSmallIntegerField(null=True, blank=True)
+    is_priority = models.BooleanField(default=False)
+
+    def display_name(self):
+        if self.prefix:
+            nums = [n for n in [self.rack, self.slot, self.subslot, self.port] if n is not None]
+            suffix = "/".join(str(n) for n in nums)
+            return f"{self.prefix}{suffix}" if suffix else self.prefix
+        return self.name
+    
+    class Meta:
+        ordering = ["rack", "slot", "subslot", "port", "name"]
+        constraints = [
+            models.UniqueConstraint(fields=["device", "name"], name="uniq_device_interface")
+        ]
+
+    def __str__(self):
+        return f"{self.device.name} {self.name}"

src/network_ops_dashboard/inventory/views.py

@@ -3,10 +3,12 @@
 from django.views.decorators.cache import never_cache
 from django.http import JsonResponse, HttpResponse
 from django.db.models import Q
+from django.urls import reverse
+from django.contrib import messages
 import logging
 from network_ops_dashboard.decorators import *
 from network_ops_dashboard.models import *
-from network_ops_dashboard.inventory.models import Inventory, Site, Platform, DeviceTag
+from network_ops_dashboard.inventory.models import Inventory, InventoryInterface, Site, Platform, DeviceTag
 from network_ops_dashboard.inventory.discovery.forms import DiscoveryForm
 from network_ops_dashboard.inventory.forms import *
 
@@ -29,17 +31,96 @@ def inventory_home(request):
         "discovery_form": discovery_form,
     })
 
-@login_required(login_url='/accounts/login/')
+@login_required
 def inventory_edit_modal(request, pk):
     device = get_object_or_404(Inventory, pk=pk)
+
     if request.method == "POST":
         form = InventoryForm(request.POST, instance=device)
         if form.is_valid():
             form.save()
-            return HttpResponse('<script>window.location.reload()</script>')
+
+        # --- Update Priority Interfaces ---
+        selected_ids = request.POST.getlist("priority_interfaces")
+        device.interfaces.update(is_priority=False)
+        if selected_ids:
+            device.interfaces.filter(id__in=selected_ids).update(is_priority=True)
+
+        # --- Add New Interfaces ---
+        prefix = request.POST.get("prefix")
+        rack = request.POST.get("rack")
+        slot = request.POST.get("slot")
+        port_start = request.POST.get("port_start")
+        port_end = request.POST.get("port_end") or port_start
+
+        if prefix:
+            if port_start:  # Range / numbered interfaces
+                try:
+                    start = int(port_start)
+                    end = int(port_end)
+                except ValueError:
+                    start, end = None, None
+
+                if start is not None and end is not None:
+                    for port in range(start, end + 1):
+                        slot_bits = []
+                        if rack:
+                            slot_bits.append(str(rack))
+                        if slot:
+                            slot_bits.append(str(slot))
+                        slot_bits.append(str(port))
+                        slot_str = "/".join(slot_bits)
+
+                        iface_name = f"{prefix}{slot_str}"
+
+                        InventoryInterface.objects.get_or_create(
+                            device=device,
+                            name=iface_name,
+                            defaults={
+                                "rack": int(rack) if rack else None,
+                                "slot": int(slot) if slot else None,
+                                "port": port,
+                            },
+                        )
+            else:
+                # Singleton free-text interface (Mgmt0, Port-Channel1.2405, Vlan10…)
+                InventoryInterface.objects.get_or_create(
+                    device=device,
+                    name=prefix.strip(),
+                    defaults={
+                        "rack": int(rack) if rack else None,
+                        "slot": int(slot) if slot else None,
+                    },
+                )
+
+        if request.POST.get("action") == "add_interface":
+            # Stay in modal > re-render with updated interface list
+            interfaces = device.interfaces.all().order_by("rack", "slot", "port", "name")
+            return render(
+                request,
+                "network_ops_dashboard/inventory/_interfaces_list.html",
+                {"device": device, "interfaces": interfaces, "form": form},
+            )
+        else:
+            # Save clicked > close modal + refresh page
+            resp = HttpResponse()
+            resp["HX-Redirect"] = reverse("inventory_home")
+            return resp
+
     else:
         form = InventoryForm(instance=device)
-    return render(request, "network_ops_dashboard/inventory/_inventory_form.html", {"form": form, "device": device})
+
+    interfaces = device.interfaces.all().order_by("rack", "slot", "port", "name")
+    return render(
+        request,
+        "network_ops_dashboard/inventory/_inventory_form.html",
+        {
+            "device": device,
+            "form": form,
+            "interfaces": interfaces,
+        },
+    )
+
 
 @login_required(login_url='/accounts/login/')
 def inventory_add_modal(request):
@@ -104,6 +185,29 @@ def inventory_data(request):
     response["Pragma"] = "no-cache"
     return response
 
+@login_required
+def inventory_add_interface(request, pk):
+    device = get_object_or_404(Inventory, pk=pk)
+
+    if request.method == "POST":
+        prefix = request.POST.get("prefix")
+        slot = request.POST.get("slot") or None
+        port_start = int(request.POST.get("port_start"))
+        port_end = request.POST.get("port_end")
+        port_end = int(port_end) if port_end else port_start
+
+        for port in range(port_start, port_end + 1):
+            InventoryInterface.objects.get_or_create(
+                device=device,
+                prefix=prefix,
+                slot=slot,
+                port=port,
+                defaults={"name": f"{prefix}{slot}/{port}" if slot else f"{prefix}{port}"}
+            )
+
+        messages.success(request, f"Added interfaces {prefix}{slot}/{port_start}-{port_end}")
+        return redirect("inventory_edit_modal", pk=device.id)
+
 @login_required(login_url='/accounts/login/')
 def platform_home(request):
     platform_all = Platform.objects.all().order_by('name')

src/network_ops_dashboard/templates/network_ops_dashboard/inventory/_interfaces_list.html

@@ -0,0 +1,51 @@
+<div id="interfaces-list">
+    <div class="accordion" id="interfacesAccordion">
+      {% regroup interfaces by slot as slot_list %}
+      {% for slot in slot_list %}
+        <div class="accordion-item">
+          <h2 class="accordion-header" id="heading{{ forloop.counter }}">
+            <button class="accordion-button collapsed" type="button" data-bs-toggle="collapse" data-bs-target="#collapse{{ forloop.counter }}">
+              Slot {{ slot.grouper|default:"N/A" }}
+            </button>
+          </h2>
+          <div id="collapse{{ forloop.counter }}" class="accordion-collapse collapse">
+            <div class="accordion-body">
+              {% for iface in slot.list %}
+                <div class="form-check">
+                  <input class="form-check-input" type="checkbox" name="priority_interfaces" value="{{ iface.id }}" id="iface{{ iface.id }}" {% if iface.is_priority %}checked{% endif %}>
+                  <label class="form-check-label" for="iface{{ iface.id }}">{{ iface.display_name }}</label>
+                </div>
+              {% endfor %}
+            </div>
+          </div>
+        </div>
+      {% endfor %}
+    </div>
+  
+    <!-- Inline Add Form -->
+    <div class="mt-3">
+      <div class="row g-2">
+        <div class="col"><input type="text" name="prefix" class="form-control" placeholder="Prefix/Name"></div>
+        <div class="col"><input type="text" name="rack" class="form-control" placeholder="Rack#"></div>
+        <div class="col"><input type="text" name="slot" class="form-control" placeholder="Slot#"></div>
+        <div class="col"><input type="text" name="port_start" class="form-control" placeholder="Port Start"></div>
+        <div class="col"><input type="text" name="port_end" class="form-control" placeholder="Port End"></div>
+        <div class="col-auto">
+          <button type="submit"
+                  name="action"
+                  value="add_interface"
+                  class="btn btn-sm btn-primary"
+                  hx-post="{% url 'inventory_edit_modal' device.pk %}"
+                  hx-target="#interfaces-list"
+                  hx-swap="outerHTML">
+            Add
+          </button>
+        </div>
+        <div class="form-text mt-1">
+            <p class="mb-0 text-muted small">* To configure a range enter Prefix name ("Ethernet", "GigabitEthernet", etc) plus rack/slot and start/end ports.</p>
+            <p class="mb-0 text-muted small">* For singletons just enter full name into Interface Name (eg "Port-Channel220", "Vlan220", "Mgmt0").</p>
+            <p class="mb-0 text-muted small">* Rack# is mainly for IOS-XR chassis like CRS/ASR9K.</p>
+          </div>
+      </div>
+    </div>
+  </div>

src/network_ops_dashboard/templates/network_ops_dashboard/inventory/_inventory_form.html

@@ -1,30 +1,59 @@
-{% block content %}
 <div class="modal-header">
-    <h5 class="modal-title">
-      {% if device %}Edit {{ device.name }}{% else %}Add Network Device{% endif %}
-    </h5>
-    <button type="button" class="btn-close" data-bs-dismiss="modal"></button>
-  </div>
-  <form method="post"
-        hx-post="{% if device %}{% url 'inventory_edit_modal' device.pk %}{% else %}{% url 'inventory_add_modal' %}{% endif %}"
-        hx-target="#inventoryModal .modal-content"
-        hx-swap="outerHTML">
-    {% csrf_token %}
-    <div class="modal-body">
+  <h5 class="modal-title">
+    {% if device %}Edit {{ device.name }}{% else %}Add Network Device{% endif %}
+  </h5>
+  <button type="button" class="btn-close" data-bs-dismiss="modal"></button>
+</div>
+
+<form method="post"
+      hx-post="{% if device %}{% url 'inventory_edit_modal' device.pk %}{% else %}{% url 'inventory_add_modal' %}{% endif %}"
+      hx-target="#inventoryModal .modal-content"
+      hx-swap="outerHTML">
+  {% csrf_token %}
+
+  <ul class="nav nav-tabs" id="invTab" role="tablist">
+    <li class="nav-item">
+      <a class="nav-link active" data-bs-toggle="tab" href="#deviceTab">Device</a>
+    </li>
+    <li class="nav-item">
+      <a class="nav-link" data-bs-toggle="tab" href="#interfacesTab">Interfaces</a>
+    </li>
+    <li class="nav-item">
+      <a class="nav-link" data-bs-toggle="tab" href="#configTab">Config</a>
+    </li>
+  </ul>
+
+  <div class="tab-content">
+    <!-- Device tab -->
+    <div class="tab-pane fade show active p-3" id="deviceTab">
       {{ form.as_p }}
     </div>
-    <div class="modal-footer">
-      <button type="submit" class="btn btn-primary">Save</button>
-      {% if device %}
-        <button type="button"
-          class="btn btn-danger"
-          hx-post="{% url 'inventory_delete_modal' device.pk %}"
-          hx-vals='{"_method":"DELETE"}'
-          hx-confirm="Really delete {{ device.name }}?">
-            Delete
-        </button>
-      {% endif %}
-      <button type="button" class="btn btn-secondary" data-bs-dismiss="modal">Cancel</button>
+
+    <!-- Interfaces tab -->
+      <div class="tab-pane fade p-3" id="interfacesTab" role="tabpanel">
+        <div class="d-flex justify-content-between align-items-center mb-2">
+          <p class="mb-0 text-muted small">✅ Check an interface to track</p>
+        </div>
+        {% include "network_ops_dashboard/inventory/_interfaces_list.html" %}
+      </div>
+
+    <!-- Config tab -->
+    <div class="tab-pane fade p-3" id="configTab">
+      <em>Coming soon…</em>
     </div>
-  </form>
-{% endblock %}
+  </div>
+
+  <div class="modal-footer">
+    <button type="submit" class="btn btn-primary">Save</button>
+    {% if device %}
+      <button type="button"
+              class="btn btn-danger"
+              hx-post="{% url 'inventory_delete_modal' device.pk %}"
+              hx-vals='{"_method":"DELETE"}'
+              hx-confirm="Really delete {{ device.name }}?">
+        Delete
+      </button>
+    {% endif %}
+    <button type="button" class="btn btn-secondary" data-bs-dismiss="modal">Cancel</button>
+  </div>
+</form>

src/network_ops_dashboard/templates/network_ops_dashboard/inventory/home.html

@@ -1,5 +1,6 @@
 {% extends 'network_ops_dashboard/base.html' %}
 {% block content %}
+
 <h3>Network Inventory</h3>
 
 <form id="filters" class="row g-2 align-items-end mb-3">