Fix INF-371 and INF-372 notification post data does not comply to spec

[pti/o2.git] / o2ims / service / command / notify_alarm_handler.py

diff --git a/o2ims/service/command/notify_alarm_handler.py b/o2ims/service/command/notify_alarm_handler.py
index 8ee4dd8..ac5bb39 100644 (file)
--- a/o2ims/service/command/notify_alarm_handler.py
+++ b/o2ims/service/command/notify_alarm_handler.py
@@ -18,7 +18,7 @@ import ssl
 import json
 from urllib.parse import urlparse
 
-# from o2common.config import config
+from o2common.config import conf
 from o2common.domain.filter import gen_orm_filter
 from o2common.service.unit_of_work import AbstractUnitOfWork
 from o2common.service.command.handler import get_https_conn_default
@@ -41,49 +41,67 @@ def notify_alarm_to_smo(
     logger.debug('In notify_alarm_to_smo')
     data = cmd.data
     with uow:
+        alarm = uow.alarm_event_records.get(data.id)
+        if alarm is None:
+            logger.debug('Alarm Event {} does not exists.'.format(data.id))
+            return
+
         subs = uow.alarm_subscriptions.list()
         for sub in subs:
             sub_data = sub.serialize()
             logger.debug('Alarm Subscription: {}'.format(
                 sub_data['alarmSubscriptionId']))
 
-            alarm = uow.alarm_event_records.get(data.id)
-            if alarm is None:
-                logger.debug('Alarm Event {} does not exists.'.format(data.id))
+            if not sub_data.get('filter', None):
+                callback_smo(sub, data, alarm)
                 continue
-            if sub_data.get('filter', None):
-                try:
-                    args = gen_orm_filter(AlarmEventRecord, sub_data['filter'])
-                except KeyError:
-                    logger.warning(
-                        'Alarm Subscription {} filter {} has wrong attribute '
-                        'name or value. Ignore the filter'.format(
-                            sub_data['alarmSubscriptionId'],
-                            sub_data['filter']))
-                    callback_smo(sub, data)
-                    continue
-                args.append(AlarmEventRecord.alarmEventRecordId == data.id)
-                ret = uow.alarm_event_records.list_with_count(*args)
-                if ret[0] != 0:
-                    logger.debug(
-                        'Alarm Event {} skip for subscription {} because of '
-                        'the filter.'
-                        .format(data.id, sub_data['alarmSubscriptionId']))
-                    continue
-
-            callback_smo(sub, data)
+            try:
+                args = gen_orm_filter(AlarmEventRecord, sub_data['filter'])
+            except KeyError:
+                logger.warning(
+                    'Alarm Subscription {} filter {} has wrong attribute '
+                    'name or value. Ignore the filter'.format(
+                        sub_data['alarmSubscriptionId'],
+                        sub_data['filter']))
+                callback_smo(sub, data, alarm)
+                continue
+            args.append(AlarmEventRecord.alarmEventRecordId == data.id)
+            ret = uow.alarm_event_records.list_with_count(*args)
+            if ret[0] != 0:
+                logger.debug(
+                    'Alarm Event {} skip for subscription {} because of '
+                    'the filter.'
+                    .format(data.id, sub_data['alarmSubscriptionId']))
+                continue
+            callback_smo(sub, data, alarm)
 
 
-def callback_smo(sub: AlarmSubscription, msg: AlarmEvent2SMO):
+def callback_smo(sub: AlarmSubscription, msg: AlarmEvent2SMO,
+                 alarm: AlarmEventRecord):
     sub_data = sub.serialize()
-    callback_data = json.dumps({
+    alarm_data = alarm.serialize()
+    callback = {
+        'globalCloudID': conf.DEFAULT.ocloud_global_id,
         'consumerSubscriptionId': sub_data['consumerSubscriptionId'],
         'notificationEventType': msg.notificationEventType,
         'objectRef': msg.objectRef,
-        'updateTime': msg.updatetime
-    })
+        'alarmEventRecordId': alarm_data['alarmEventRecordId'],
+        'resourceTypeID': alarm_data['resourceTypeId'],
+        'resourceID': alarm_data['resourceId'],
+        'alarmDefinitionID': alarm_data['alarmDefinitionId'],
+        'probableCauseID': alarm_data['probableCauseId'],
+        'alarmRaisedTime': alarm_data['alarmRaisedTime'],
+        'alarmChangedTime': alarm_data['alarmChangedTime'],
+        'alarmAcknowledgeTime': alarm_data['alarmAcknowledgeTime'],
+        'alarmAcknowledged': alarm_data['alarmAcknowledged'],
+        'perceivedSeverity': alarm_data['perceivedSeverity'],
+        'extensions': ''
+    }
+    # logger.warning(callback)
+    callback_data = json.dumps(callback)
     logger.info('URL: {}, data: {}'.format(
         sub_data['callback'], callback_data))
+
     o = urlparse(sub_data['callback'])
     if o.scheme == 'https':
         conn = get_https_conn_default(o.netloc)