Merge pull request #72 from nyctef/sleekxmpp-to-slixmpp

Migrate from sleekxmpp to slixmpp

Pipfile

@@ -13,13 +13,11 @@ pytz = "==2016.4"
 redis = "==2.10.1"
 psycopg2-binary = "*"
 # connect to jabber
-sleekxmpp = "==1.3.1"
+slixmpp = "==1.7.0"
 # azure logging
 azure = "==0.8.1"
-# note these lines are disabled for now since they bring in pyasn1, which breaks sleekxmpp:
-# https://github.com/fritzy/SleekXMPP/issues/477#issuecomment-349638055
-# opencensus-ext-azure = "==1.0.4"
-# opencensus-ext-logging = "==0.1.0"
+opencensus-ext-azure = "==1.0.4"
+opencensus-ext-logging = "==0.1.0"
 # misc
 laboratory = "*"
 

README.md

@@ -1,6 +1,7 @@
 
 ```powershell
 pipenv --python 3.8
+pipenv install --dev
 pipenv shell
 ./run-tests.ps1
 

modules/Experiments.py

@@ -7,7 +7,19 @@
 class LoggingExperiment(laboratory.Experiment):
     def publish(self, result):
         if not result.match:
-            logging.error("Result mismatch!: " + pformat(result))
+            try:
+                logging.error(f"Experiment mismatch: output from {result.candidates[0].name} did not match {result.control.name}",
+                    extra={
+                        'custom_dimensions': {
+                            'expected_value': pformat(result.control.value),
+                            'actual_value': pformat(result.candidates[0].value),
+                            'actual_exception': pformat(result.candidates[0].exception),
+                        }
+                    })
+            except Exception as e:
+                # if the detailed reporting fails for some reason, make sure we fall back to something
+                # simpler that won't throw and interfere with the rest of the app
+                logging.error("Experiment mismatch!: " + pformat(result), extra={'custom_dimensions': {'exception': pformat(e)}})
 
 
 def method_missing(method_name, type_name):