X-Git-Url: https://osm.etsi.org/gitweb/?a=blobdiff_plain;f=osm_policy_module%2Fcore%2Fagent.py;h=f7cd679c897faf9b9d20c1931ebf0bf5ca5c264e;hb=28d5e049e96472f4482b4c6ae59b818b889abe47;hp=8a1a31ca850d5b025a560fbfc28c6a18378214aa;hpb=023473295ef70215b9603e1fa0de8d71c0f6d0a2;p=osm%2FPOL.git diff --git a/osm_policy_module/core/agent.py b/osm_policy_module/core/agent.py index 8a1a31c..f7cd679 100644 --- a/osm_policy_module/core/agent.py +++ b/osm_policy_module/core/agent.py @@ -36,10 +36,12 @@ from osm_policy_module.common.mon_client import MonClient from osm_policy_module.core import database from osm_policy_module.core.config import Config from osm_policy_module.core.database import ScalingGroup, ScalingAlarm, ScalingPolicy, ScalingCriteria, DatabaseManager +from osm_policy_module.core.exceptions import VdurNotFound +from osm_policy_module.utils.vnfd import VnfdUtils log = logging.getLogger(__name__) -ALLOWED_KAFKA_KEYS = ['instantiated', 'scaled', 'notify_alarm'] +ALLOWED_KAFKA_KEYS = ['instantiated', 'scaled', 'terminated', 'notify_alarm'] class PolicyModuleAgent: @@ -71,6 +73,7 @@ class PolicyModuleAgent: await consumer.start() try: async for msg in consumer: + log.info("Message arrived: %s", msg) await self._process_msg(msg.topic, msg.key, msg.value) finally: await consumer.stop() @@ -87,6 +90,9 @@ class PolicyModuleAgent: if key == 'instantiated' or key == 'scaled': await self._handle_instantiated_or_scaled(content) + if key == 'terminated': + await self._handle_terminated(content) + if key == 'notify_alarm': await self._handle_alarm_notification(content) else: @@ -96,13 +102,13 @@ class PolicyModuleAgent: async def _handle_alarm_notification(self, content): log.debug("_handle_alarm_notification: %s", content) - alarm_id = content['notify_details']['alarm_uuid'] + alarm_uuid = content['notify_details']['alarm_uuid'] metric_name = content['notify_details']['metric_name'] operation = content['notify_details']['operation'] threshold = content['notify_details']['threshold_value'] vdu_name = content['notify_details']['vdu_name'] vnf_member_index = content['notify_details']['vnf_member_index'] - ns_id = content['notify_details']['ns_id'] + nsr_id = content['notify_details']['ns_id'] log.info( "Received alarm notification for alarm %s, \ metric %s, \ @@ -111,9 +117,9 @@ class PolicyModuleAgent: vdu_name %s, \ vnf_member_index %s, \ ns_id %s ", - alarm_id, metric_name, operation, threshold, vdu_name, vnf_member_index, ns_id) + alarm_uuid, metric_name, operation, threshold, vdu_name, vnf_member_index, nsr_id) try: - alarm = self.database_manager.get_alarm(alarm_id) + alarm = self.database_manager.get_alarm(alarm_uuid) delta = datetime.datetime.now() - alarm.scaling_criteria.scaling_policy.last_scale log.debug("last_scale: %s", alarm.scaling_criteria.scaling_policy.last_scale) log.debug("now: %s", datetime.datetime.now()) @@ -121,15 +127,15 @@ class PolicyModuleAgent: if delta.total_seconds() < alarm.scaling_criteria.scaling_policy.cooldown_time: log.info("Time between last scale and now is less than cooldown time. Skipping.") return - log.info("Sending scaling action message for ns: %s", alarm_id) - await self.lcm_client.scale(ns_id, + log.info("Sending scaling action message for ns: %s", nsr_id) + await self.lcm_client.scale(nsr_id, alarm.scaling_criteria.scaling_policy.scaling_group.name, alarm.vnf_member_index, alarm.action) alarm.scaling_criteria.scaling_policy.last_scale = datetime.datetime.now() alarm.scaling_criteria.scaling_policy.save() except ScalingAlarm.DoesNotExist: - log.info("There is no action configured for alarm %s.", alarm_id) + log.info("There is no action configured for alarm %s.", alarm_uuid) async def _handle_instantiated_or_scaled(self, content): log.debug("_handle_instantiated_or_scaled: %s", content) @@ -139,15 +145,28 @@ class PolicyModuleAgent: nsr_id = nslcmop['nsInstanceId'] log.info("Configuring scaling groups for network service with nsr_id: %s", nsr_id) await self._configure_scaling_groups(nsr_id) + log.info("Checking for orphaned alarms to be deleted for network service with nsr_id: %s", nsr_id) + await self._delete_orphaned_alarms(nsr_id) else: log.info( "Network service is not in COMPLETED or PARTIALLY_COMPLETED state. " "Current state is %s. Skipping...", nslcmop['operationState']) + async def _handle_terminated(self, content): + log.debug("_handle_deleted: %s", content) + nsr_id = content['nsr_id'] + if content['operationState'] == 'COMPLETED' or content['operationState'] == 'PARTIALLY_COMPLETED': + log.info("Deleting scaling groups and alarms for network service with nsr_id: %s", nsr_id) + await self._delete_scaling_groups(nsr_id) + else: + log.info( + "Network service is not in COMPLETED or PARTIALLY_COMPLETED state. " + "Current state is %s. Skipping...", + content['operationState']) + async def _configure_scaling_groups(self, nsr_id: str): log.debug("_configure_scaling_groups: %s", nsr_id) - # TODO: Add support for non-nfvi metrics alarms_created = [] with database.db.atomic() as tx: try: @@ -247,8 +266,13 @@ class PolicyModuleAgent: ) ) elif 'vnf-metric' in vnf_monitoring_param: - log.warning("vnf-metric is not currently supported.") - continue + vdu = VnfdUtils.get_mgmt_vdu(vnfd) + vdurs = list( + filter( + lambda vdur: vdur['vdu-id-ref'] == vdu['id'], + vnfr['vdur'] + ) + ) else: log.warning( "Scaling criteria is referring to a vnf-monitoring-param that does not " @@ -280,13 +304,14 @@ class PolicyModuleAgent: operation=scaling_criteria['scale-in-relational-operation'], statistic=vnf_monitoring_param['aggregation-type'] ) - ScalingAlarm.create( - alarm_id=alarm_uuid, + alarm = ScalingAlarm.create( + alarm_uuid=alarm_uuid, action='scale_in', vnf_member_index=int(vnfr['member-vnf-index-ref']), vdu_name=vdur['name'], scaling_criteria=scaling_criteria_record ) + alarms_created.append(alarm) alarm_uuid = await self.mon_client.create_alarm( metric_name=vnf_monitoring_param['id'], ns_id=nsr_id, @@ -296,13 +321,14 @@ class PolicyModuleAgent: operation=scaling_criteria['scale-out-relational-operation'], statistic=vnf_monitoring_param['aggregation-type'] ) - ScalingAlarm.create( - alarm_id=alarm_uuid, + alarm = ScalingAlarm.create( + alarm_uuid=alarm_uuid, action='scale_out', vnf_member_index=int(vnfr['member-vnf-index-ref']), vdu_name=vdur['name'], scaling_criteria=scaling_criteria_record ) + alarms_created.append(alarm) except Exception as e: log.exception("Error configuring scaling groups:") @@ -310,5 +336,59 @@ class PolicyModuleAgent: if len(alarms_created) > 0: log.info("Cleaning alarm resources in MON") for alarm in alarms_created: - await self.mon_client.delete_alarm(*alarm) + await self.mon_client.delete_alarm(alarm.scaling_criteria.scaling_policy.scaling_group.nsr_id, + alarm.vnf_member_index, + alarm.vdu_name, + alarm.alarm_uuid) + raise e + + async def _delete_scaling_groups(self, nsr_id: str): + with database.db.atomic() as tx: + try: + for scaling_group in ScalingGroup.select().where(ScalingGroup.nsr_id == nsr_id): + for scaling_policy in scaling_group.scaling_policies: + for scaling_criteria in scaling_policy.scaling_criterias: + for alarm in scaling_criteria.scaling_alarms: + try: + await self.mon_client.delete_alarm( + alarm.scaling_criteria.scaling_policy.scaling_group.nsr_id, + alarm.vnf_member_index, + alarm.vdu_name, + alarm.alarm_uuid) + except ValueError: + log.exception("Error deleting alarm in MON %s", alarm.alarm_uuid) + alarm.delete_instance() + scaling_criteria.delete_instance() + scaling_policy.delete_instance() + scaling_group.delete_instance() + + except Exception as e: + log.exception("Error deleting scaling groups and alarms:") + tx.rollback() + raise e + + async def _delete_orphaned_alarms(self, nsr_id): + with database.db.atomic() as tx: + try: + for scaling_group in ScalingGroup.select().where(ScalingGroup.nsr_id == nsr_id): + for scaling_policy in scaling_group.scaling_policies: + for scaling_criteria in scaling_policy.scaling_criterias: + for alarm in scaling_criteria.scaling_alarms: + try: + self.db_client.get_vdur(nsr_id, alarm.vnf_member_index, alarm.vdu_name) + except VdurNotFound: + log.info("Deleting orphaned alarm %s", alarm.alarm_uuid) + try: + await self.mon_client.delete_alarm( + alarm.scaling_criteria.scaling_policy.scaling_group.nsr_id, + alarm.vnf_member_index, + alarm.vdu_name, + alarm.alarm_uuid) + except ValueError: + log.exception("Error deleting alarm in MON %s", alarm.alarm_uuid) + alarm.delete_instance() + + except Exception as e: + log.exception("Error deleting orphaned alarms:") + tx.rollback() raise e