Remove dependency on onaplogging in vnfmgr

Change-Id: I36f25fb8bd6d93d971009787ca8c3bd633b3c225
Issue-ID: VFC-1810
Signed-off-by: dyh <dengyuanhong@chinamobile.com>

diff --git a/mgr/mgr/log.yml b/mgr/mgr/log.yml
index 4af8faa..154a328 100644 (file)
--- a/mgr/mgr/log.yml
+++ b/mgr/mgr/log.yml
@@ -26,7 +26,7 @@ handlers:
             "logging.handlers.RotatingFileHandler"
         filename: "/var/log/onap/vfc/gvnfm-vnfmgr/runtime_mgr.log"
         formatter:
-            "mdcFormat"
+            "standard"
         maxBytes: 52428800
         backupCount: 10
     django_handler:
@@ -42,9 +42,3 @@ formatters:
     standard:
         format:
             "%(asctime)s:[%(name)s]:[%(filename)s]-[%(lineno)d] [%(levelname)s]:%(message)s"
-    mdcFormat:
-        format:
-            "%(asctime)s|||||%(name)s||%(thread)s||%(funcName)s||%(levelname)s||%(message)s||||%(mdc)s \t"
-        mdcfmt: "{requestID} {invocationID} {serviceName} {serviceIP}"
-        datefmt: "%Y-%m-%d %H:%M:%S"
-        (): onaplogging.mdcformatter.MDCFormatter

diff --git a/mgr/mgr/middleware.py b/mgr/mgr/middleware.py
deleted file mode 100644 (file)
index dfb330c..0000000
--- a/mgr/mgr/middleware.py
+++ /dev/null
@@ -1,69 +0,0 @@
-# Copyright (c) 2017-2018 ZTE, Inc.
-#
-# Licensed under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at:
-#
-#       http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-
-import uuid
-from onaplogging.mdcContext import MDC
-
-from mgr.pub.config.config import FORWARDED_FOR_FIELDS, SERVICE_NAME
-
-
-class LogContextMiddleware(object):
-    #  the last IP behind multiple proxies,  if no exist proxies
-    #  get local host ip.
-    def __init__(self, get_response):
-        self.get_response = get_response
-
-    def _getLastIp(self, request):
-
-        ip = ""
-        try:
-            for field in FORWARDED_FOR_FIELDS:
-                if field in request.META:
-                    if ',' in request.META[field]:
-                        parts = request.META[field].split(',')
-                        ip = parts[-1].strip().split(":")[0]
-                    else:
-                        ip = request.META[field].split(":")[0]
-
-            if ip == "":
-                ip = request.META.get("HTTP_HOST").split(":")[0]
-
-        except Exception:
-            pass
-
-        return ip
-
-    def process_request(self, request):
-        # Fetch TRANSACTIONID Id and pass to plugin server
-        ReqeustID = request.META.get("HTTP_X_ONAP-RequestID", None)
-        if ReqeustID is None:
-            ReqeustID = uuid.uuid3(uuid.NAMESPACE_URL, SERVICE_NAME)
-            request.META["HTTP_X_ONAP-RequestID"] = ReqeustID
-        MDC.put("requestID", ReqeustID)
-        # generate the unique  id
-        InovocationID = uuid.uuid3(uuid.NAMESPACE_DNS, SERVICE_NAME)
-        MDC.put("invocationID", InovocationID)
-        MDC.put("serviceName", SERVICE_NAME)
-        # access ip
-        MDC.put("serviceIP", self._getLastIp(request))
-
-        return None
-
-    def process_response(self, request, response):
-        MDC.clear()
-        return response
-
-    def __call__(self, request):
-        self.process_request(request)
-        response = self.get_response(request)
-        self.process_response(request, response)
-        return response

diff --git a/mgr/mgr/settings.py b/mgr/mgr/settings.py
index 3de34bf..c64a219 100644 (file)
--- a/mgr/mgr/settings.py
+++ b/mgr/mgr/settings.py
@@ -21,8 +21,8 @@ import mgr.pub.redisco
 from mgr.pub.config.config import REDIS_HOST, REDIS_PORT, REDIS_PASSWD
 from mgr.pub.config.config import DB_NAME, DB_IP, DB_USER, DB_PASSWD, DB_PORT
 from logging import config
-from onaplogging import monkey
-monkey.patch_all()
+
+import yaml
 
 # Build paths inside the project like this: os.path.join(BASE_DIR, ...)
 BASE_DIR = os.path.dirname(os.path.dirname(os.path.abspath(__file__)))
@@ -61,7 +61,6 @@ MIDDLEWARE = [
     'django.contrib.auth.middleware.AuthenticationMiddleware',
     'django.contrib.messages.middleware.MessageMiddleware',
     'django.middleware.clickjacking.XFrameOptionsMiddleware',
-    'mgr.middleware.LogContextMiddleware'
 ]
 
 ROOT_URLCONF = 'mgr.urls'
@@ -156,12 +155,19 @@ if platform.system() == 'Windows' or 'test' in sys.argv:
     }
 else:
     LOGGING_CONFIG = None
-    LOGGING_FILE = os.path.join(BASE_DIR, 'mgr/log.yml')
-    config.yamlConfig(filepath=LOGGING_FILE, watchDog=True)
 
+    log_path = '/var/log/onap/vfc/gvnfm-vnfmgr'
+    if not os.path.exists(log_path):
+        os.makedirs(log_path)
+
+    LOGGING_FILE = os.path.join(BASE_DIR, 'mgr/log.yml')
+    with open(file=LOGGING_FILE, mode='r', encoding="utf-8")as file:
+        logging_yaml = yaml.load(stream=file, Loader=yaml.FullLoader)
+    config.dictConfig(config=logging_yaml)
 
 if 'test' in sys.argv:
     from mgr.pub.config import config
+
     config.REG_TO_MSB_WHEN_START = False
     DATABASES = {}
     DATABASES['default'] = {

diff --git a/mgr/requirements.txt b/mgr/requirements.txt
index d5a4429..78380b6 100644 (file)
--- a/mgr/requirements.txt
+++ b/mgr/requirements.txt
@@ -29,4 +29,6 @@ swagger-spec-validator>=2.1.0
 onappylog>=1.0.9
 
 # uwsgi for parallel processing
-uwsgi
\ No newline at end of file
+uwsgi
+
+PyYAML==5.3.1
\ No newline at end of file