aboutsummaryrefslogtreecommitdiffstats
path: root/src/onaptests
diff options
context:
space:
mode:
authorjardellos <thierry.hardy@orange.com>2020-08-28 11:27:47 +0200
committerThierry Hardy <thierry.hardy@orange.com>2020-08-31 13:49:40 +0200
commit7664059073dd0691428864deb37c3861521e2cd8 (patch)
tree8817d8115de3846dee9a5f3a591ecf536e8147d6 /src/onaptests
parentf9b0c349a5c83f9278f6b115d334598201d9d7e6 (diff)
Add logger in python-sdktests
Adding of logger to onaptests setting the stream and file handlers based on settings.py onapsdk should evolve to include the same code This would avoid to add it in run.py Simplification of the proposal based on remarks from Michal and Morgan Issue-ID: TEST-252 Signed-off-by: jardellos <thierry.hardy@orange.com> Change-Id: I5b7baf77580df916c4f5e62965a47d8462b44c43
Diffstat (limited to 'src/onaptests')
-rw-r--r--src/onaptests/configuration/settings.py32
-rw-r--r--src/onaptests/steps/base.py18
2 files changed, 49 insertions, 1 deletions
diff --git a/src/onaptests/configuration/settings.py b/src/onaptests/configuration/settings.py
index f6959de..0e69b4e 100644
--- a/src/onaptests/configuration/settings.py
+++ b/src/onaptests/configuration/settings.py
@@ -6,6 +6,38 @@
# #
######################
+
+# Variables to set logger information
+# Possible values for logging levels in onapsdk: INFO, DEBUG , WARNING, ERROR
+LOG_CONFIG = {
+ "version": 1,
+ "disable_existing_loggers": False,
+ "formatters": {
+ "default": {
+ "class": "logging.Formatter",
+ "format": "%(asctime)s %(levelname)s %(lineno)d:%(filename)s(%(process)d) - %(message)s"
+ }
+ },
+ "handlers": {
+ "console": {
+ "level": "DEBUG",
+ "class": "logging.StreamHandler",
+ "formatter": "default"
+ },
+ "file": {
+ "level": "DEBUG",
+ "class": "logging.FileHandler",
+ "formatter": "default",
+ "filename": "./pythonsdk.debug.log",
+ "mode": "w"
+ }
+ },
+ "root": {
+ "level": "DEBUG",
+ "handlers": ["console", "file"]
+ }
+}
+
VENDOR_NAME = "sdktests_vendor"
VSP_NAME = "sdktests_vsp"
VSP_FILE_PATH = "vfw.zip"
diff --git a/src/onaptests/steps/base.py b/src/onaptests/steps/base.py
index b32e6d3..5f66935 100644
--- a/src/onaptests/steps/base.py
+++ b/src/onaptests/steps/base.py
@@ -1,10 +1,24 @@
+import logging
+import logging.config
+
from abc import ABC, abstractmethod
from typing import List
-
+from onapsdk.configuration import settings
class BaseStep(ABC):
"""Base step class."""
+ _logger: logging.Logger = logging.getLogger("")
+
+ def __init_subclass__(cls):
+ """Subclass initialization.
+
+ Add _logger property for any BaseStep subclass
+ """
+ super().__init_subclass__()
+ cls._logger: logging.Logger = logging.getLogger("")
+ logging.config.dictConfig(settings.LOG_CONFIG)
+
def __init__(self, cleanup: bool = False) -> None:
"""Step initialization.
@@ -16,6 +30,8 @@ class BaseStep(ABC):
self._cleanup: bool = cleanup
self._parent: "BaseStep" = None
+
+
def add_step(self, step: "BaseStep") -> None:
"""Add substep.