Cleanup of class scoped variables

After bug described in OS#3456 and fixed in last commit, let's
categorize and place variables in its correct plac to avoid similar
issus. We leave under the class keyword (class scoped variables) the
attributes which are to be used as static class attributes. All other
ones are initialized during __init__(). This way w avoid scenarios in
which while using an object from an instance attribute we end up reading
a class scoped variable which is shared among all instances.

Change-Id: I5ad4cac34a9f49eaf42966c01c9c5a4d3f3e9dc8
diff --git a/src/osmo_gsm_tester/suite.py b/src/osmo_gsm_tester/suite.py
index db4a8dc..c712a75 100644
--- a/src/osmo_gsm_tester/suite.py
+++ b/src/osmo_gsm_tester/suite.py
@@ -63,21 +63,17 @@
     PASS = 'PASS'
     FAIL = 'FAIL'
 
-    trial = None
-    status = None
-    start_timestamp = None
-    duration = None
-    resources_pool = None
-    reserved_resources = None
-    objects_to_clean_up = None
-    test_import_modules_to_clean_up = []
-    _resource_requirements = None
-    _config = None
-    _processes = None
-    _run_dir = None
-
     def __init__(self, trial, suite_scenario_str, suite_definition, scenarios=[]):
         super().__init__(log.C_TST, suite_scenario_str)
+        self.start_timestamp = None
+        self.duration = None
+        self.reserved_resources = None
+        self.objects_to_clean_up = None
+        self.test_import_modules_to_clean_up = []
+        self._resource_requirements = None
+        self._config = None
+        self._processes = None
+        self._run_dir = None
         self.trial = trial
         self.definition = suite_definition
         self.scenarios = scenarios