Merge pull request #1510 from mozilla/update_severity_geomodel

Change severity of geomodel alert to warning
mozilla · Oct 29, 2019 · 18c65ce · 18c65ce
2 parents d5c0d0d + 7caea4d
commit 18c65ce
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 8 deletions.
diff --git a/alerts/geomodel_location.py b/alerts/geomodel_location.py
@@ -154,7 +154,7 @@ def onAggregation(self, agg):
                     hop.destination.city, hop.destination.country)
 
             alert_dict = self.createAlertDict(
-                summary, 'geomodel', ['geomodel'], events, 'INFO')
+                summary, 'geomodel', ['geomodel'], events, 'WARNING')
 
             # TODO: When we update to Python 3.7+, change to asdict(alert_produced)
             alert_dict['details'] = {

diff --git a/tests/alerts/test_geomodel_location.py b/tests/alerts/test_geomodel_location.py
@@ -171,7 +171,7 @@ class TestAlertGeoModel(GeoModelTest):
                 }
             ],
         },
-        'severity': 'INFO',
+        'severity': 'WARNING',
         'tags': ['geomodel'],
     }
 
@@ -333,7 +333,7 @@ class TestOnePreviousLocality(GeoModelTest):
                 }
             ],
         },
-        'severity': 'INFO',
+        'severity': 'WARNING',
         'tags': ['geomodel'],
     }
 
@@ -411,7 +411,7 @@ class TestInitialLocalityPositiveAlert(GeoModelTest):
                 }
             ],
         },
-        'severity': 'INFO',
+        'severity': 'WARNING',
         'tags': ['geomodel'],
     }
 
@@ -493,7 +493,7 @@ class TestSameCitiesOutsideRange(GeoModelTest):
         'category': 'geomodel',
         'summary': 'tester1 seen in Sherbrooke,CA then Sherbrooke,CA',
         'details': {'username': 'tester1'},
-        'severity': 'INFO',
+        'severity': 'WARNING',
         'tags': ['geomodel'],
     }
 
@@ -569,7 +569,7 @@ class TestMultipleEventsInWindow(GeoModelTest):
                 }
             ],
         },
-        'severity': 'INFO',
+        'severity': 'WARNING',
         'tags': ['geomodel'],
     }
 
@@ -708,7 +708,7 @@ class TestSameCitiesFarAway(GeoModelTest):
                 }
             ],
         },
-        'severity': 'INFO',
+        'severity': 'WARNING',
         'tags': ['geomodel'],
     }
 
@@ -795,7 +795,7 @@ class TestMultipleImpossibleJourneys(GeoModelTest):
         'category': 'geomodel',
         'summary': _summary_from_events(events),
         'details': {'username': 'tester1', 'hops': _hops_from_events(events)},
-        'severity': 'INFO',
+        'severity': 'WARNING',
         'tags': ['geomodel'],
     }