Merge pull request #1733 from jmbredal/active_alerts_port_details

Add active alerts tab for port details page (Fixes #1509)

python/nav/web/ipdevinfo/views.py

@@ -37,6 +37,7 @@
 from nav.models.arnold import Identity
 from nav.models.service import Service
 from nav.models.profiles import Account
+from nav.models.event import AlertHistory
 from nav.ipdevpoll.config import get_job_descriptions
 from nav.util import is_valid_ip
 from nav.web.ipdevinfo.utils import create_combined_urls
@@ -597,11 +598,34 @@ def port_details(request, netbox_sysname, port_type=None, port_id=None,
             'graphite_error': graphite_error,
             'detention': detention,
             'sensor_metrics': sensor_metrics,
+            'alert_info': get_recent_alerts_interface(port)
         },
         context_instance=RequestContext(
             request, processors=[search_form_processor]))
 
 
+def get_recent_alerts_interface(interface, days_back=7):
+    """Returns the most recent linkState events for this interface"""
+    lowest_end_time = dt.datetime.now() - dt.timedelta(days=days_back)
+    alerts = AlertHistory.objects.filter(
+        event_type='linkState',
+        subid=interface.pk,
+        end_time__gt=lowest_end_time)
+    for alert in alerts:
+        try:
+            message = alert.messages.filter(type='sms')[0].message
+        except IndexError:
+            message = None
+        alert.message = message
+
+    return {
+        'alerts': alerts,
+        'count': alerts.count(),
+        'days_back': days_back,
+        'has_unresolved_alerts': any(a.is_open() for a in alerts),
+    }
+
+
 def port_counter_graph(request, interfaceid, kind='Octets'):
     """Creates an url to Graphite for rendering a graph as an image
 

templates/ipdevinfo/port-details-recent-alerts-frag.html

@@ -0,0 +1,66 @@
+<div id="alerts">
+  {% if port %}
+
+    {# Table for listing recent alerts #}
+    <h4>
+      Alerts last {{ alert_info.days_back }}
+      day{{ alert_info.days_back|pluralize }}
+    </h4>
+    <table class="listtable">
+      <thead>
+        <tr>
+          <th>Event</th>
+          <th>Message</th>
+          <th>Start</th>
+          <th>End</th>
+          <th>Downtime</th>
+        </tr>
+      </thead>
+
+      <tfoot>
+        <tr>
+          <th colspan="5">
+            {{ alert_info.alerts|length }}
+            alert{{ alert_info.alerts|length|pluralize }}
+            found
+          </th>
+        </tr>
+      </tfoot>
+
+      <tbody>
+        {% for a in alert_info.alerts %}
+          <tr>
+            <td>{{ a.event_type }}</td>
+            <td>
+	      <a href="{% url 'event-details' a.pk %}" title="See event details">
+                {{ a.message|default:"N/A" }}
+              </a>
+	    </td>
+            <td>{{ a.start_time|date|default:"N/A" }}
+              {{ a.start_time|time }}</td>
+            {% if a.is_open %}
+              <td class="status_down">Unresolved</td>
+            {% else %}
+              <td>{{ a.end_time|date|default:"N/A" }}
+                {{ a.end_time|time }}</td>
+            {% endif %}
+            <td>
+              {% if a.is_stateful %}
+                {% if a.is_open %}
+                  {{ a.start_time|timesince }}
+                {% else %}
+                  {{ a.start_time|timesince:a.end_time }}
+                {% endif %}
+              {% endif %}
+            </td>
+          </tr>
+        {% empty %}
+          <tr>
+            <td>No matching alerts found.</td>
+          </tr>
+        {% endfor %}
+      </tbody>
+    </table>
+
+  {% endif %}
+</div>

templates/ipdevinfo/port-details.html

@@ -12,6 +12,8 @@
         }
       });
       tabs.removeClass('notvisible').show();
+      // Mark error tab
+      $('li[data-mark-as-error="True"]', tabs).removeClass('ui-state-default').addClass('ui-state-error');
     });
   </script>
 {% endblock %}
@@ -61,6 +63,9 @@
       <li><a href="#port-details-activity-graphs">Activity graphs</a></li>
       <li><a href="#port-details-metrics-list">Metric list</a></li>
       <li><a href="#port-details-sensors-list">Port Sensors</a></li>
+      <li data-mark-as-error="{{ alert_info.has_unresolved_alerts }}">
+        <a href="#port-details-recent-alerts">Recent alerts</a>
+      </li>
     </ul>
 
 
@@ -118,6 +123,11 @@ <h3>Metrics</h3>
       {% include 'ipdevinfo/port-details-sensors-frag.html' %}
     </div>
 
+    {# Recent alerts for this interface #}
+    <div id="port-details-recent-alerts">
+      {% include "ipdevinfo/port-details-recent-alerts-frag.html" %}
+    </div>
+
 
   </div>