Use separate timeouts for connection and reading. 29/39429/2
authorBenoit HERARD <benoit.herard@orange.com>
Wed, 16 Aug 2017 12:12:47 +0000 (14:12 +0200)
committerBenoit HERARD <benoit.herard@orange.com>
Wed, 16 Aug 2017 13:32:12 +0000 (15:32 +0200)
Use a short timeout for connection timeout to avoid to slow down testcase
when remote recording API is not available and use a longer one to read data
from API on successfull connections.

Change-Id: I743c02f890059bf55b47cb4240d52fe0741f7cae
Signed-off-by: Benoit HERARD <benoit.herard@orange.com>
functest/energy/energy.py

index 580bc6b..c410e84 100644 (file)
@@ -16,6 +16,7 @@ import urllib
 
 from functools import wraps
 import requests
+import urllib3
 
 import functest.utils.functest_utils as ft_utils
 
@@ -76,7 +77,7 @@ class EnergyRecorder(object):
     INITIAL_STEP = "running"
 
     # Default connection timeout
-    CONNECTION_TIMOUT = 1
+    CONNECTION_TIMOUT = urllib3.Timeout(connect=1, read=3)
 
     @staticmethod
     def load_config():
@@ -104,13 +105,13 @@ class EnergyRecorder(object):
                 "API recorder at: " + energy_recorder_uri + uri_comp)
 
             # Creds
-            user = ft_utils.get_functest_config(
+            creds_usr = ft_utils.get_functest_config(
                 "energy_recorder.api_user")
-            password = ft_utils.get_functest_config(
+            creds_pass = ft_utils.get_functest_config(
                 "energy_recorder.api_password")
 
-            if user != "" and password != "":
-                energy_recorder_api_auth = (user, password)
+            if creds_usr != "" and creds_pass != "":
+                energy_recorder_api_auth = (creds_usr, creds_pass)
             else:
                 energy_recorder_api_auth = None