[Pull Request] Merged Refactoring codes to Master

.github/workflows/dispatch_daily_build.yaml

@@ -63,7 +63,7 @@ jobs:
 
   scan:
     needs: docker
-    runs-on: ubuntu-20.04
+    runs-on: ubuntu-latest
     steps:
       - name: Run Trivy vulnerability scanner
         id: trivy-scan
@@ -89,7 +89,7 @@ jobs:
           echo "$count"
 
       - name: slack
-        if:  ${{ steps.vulnerabilities.outputs.result_count != 0 }}
+        if: ${{ steps.vulnerabilities.outputs.result_count != 0 }}
         uses: 8398a7/action-slack@v3
         with:
           status: custom

deploy/helm/Chart.yaml

@@ -4,6 +4,6 @@ description: SpaceONE Cost Analysis Helm chart for Kubernetes
 
 type: application
 
-version: 1.4.1
+version: 1.4.2
 appVersion: 1.x.y
 

deploy/helm/values.yaml

@@ -78,6 +78,18 @@ application_scheduler:
       queue: cost_analysis_q
       interval: 1
       minute: ':00'
+    cost_report_run_scheduler:
+      backend: spaceone.cost_analysis.interface.task.v1.cost_report_run_scheduler.CostReportRunScheduler
+      queue: cost_analysis_q
+      interval: 1
+      minute: ':00'
+    budget_update_scheduler:
+      backend: spaceone.cost_analysis.interface.task.v1.budget_update_scheduler.BudgetUpdateScheduler
+      queue: cost_analysis_q
+      interval: 1
+      minute: ':00'
+
+
 
 # Overwrite worker config
 application_worker:

src/spaceone/cost_analysis/conf/global_conf.py

@@ -13,6 +13,10 @@
     }
 }
 
+# Budget Settings
+BUDGET_UPDATE_DAY = 1  # Every 1st day of month
+BUDGET_UPDATE_HOUR = 0
+
 # Cost Report Config Settings
 COST_REPORT_CONFIG_DEFAULT_ISSUE_DAY = 10
 COST_REPORT_DEFAULT_CURRENCY = "KRW"  # KRW | USD | JPY
@@ -98,12 +102,6 @@
 UNIFIED_COST_RUN_HOUR = 0  # Hour (UTC)
 UNIFIED_COST_AGGREGATION_DAY = 15  # Day
 
-DEFAULT_EXCHANGE_RATE = {
-    # 'KRW': 1178.7,
-    # 'JPY': 114,
-    # 'CNY': 6.3
-}
-
 INSTALLED_DATA_SOURCE_PLUGINS = [
     # {
     #     'name': '',

src/spaceone/cost_analysis/connector/datasource_plugin_connector.py

@@ -2,6 +2,7 @@
 
 from spaceone.core.connector import BaseConnector
 from spaceone.core.auth.jwt.jwt_util import JWTUtil
+from spaceone.core.connector.space_connector import SpaceConnector
 
 __all__ = ["DataSourcePluginConnector"]
 
@@ -24,9 +25,7 @@ def initialize(self, endpoint):
         if static_endpoint:
             endpoint = static_endpoint
 
-        self.client = self.locator.get_connector(
-            "SpaceConnector", endpoint=endpoint, token="NO_TOKEN"
-        )
+        self.client = SpaceConnector(endpoint=endpoint, token="NO_TOKEN")
 
         self.secret_data = self.config.get("secret_data")
         self.options = self.config.get("options")

src/spaceone/cost_analysis/error/budget.py

@@ -30,32 +30,38 @@ class ERROR_DATE_IS_WRONG(ERROR_INVALID_ARGUMENT):
 
 
 class ERROR_UNIT_IS_REQUIRED(ERROR_INVALID_ARGUMENT):
-    _message = (
-        "Unit is required for notifications (key = notifications, value = {value})"
-    )
+    _message = "Unit is required for notification (key = notification, value = {value})"
 
 
 class ERROR_NOTIFICATION_TYPE_IS_REQUIRED(ERROR_INVALID_ARGUMENT):
-    _message = "Notification type is required for notifications (key = notifications, value = {value})"
+    _message = "Notification type is required for notification (key = notification, value = {value})"
 
 
 class ERROR_THRESHOLD_IS_WRONG(ERROR_INVALID_ARGUMENT):
     _message = (
-        "Threshold must be greater than zero. (key = notifications, value = {value})"
+        "Threshold must be greater than zero. (key = notification, value = {value})"
     )
 
 
 class ERROR_THRESHOLD_IS_WRONG_IN_PERCENT_TYPE(ERROR_INVALID_ARGUMENT):
-    _message = "In percentage type, the threshold must be less than 100. (key = notifications, value = {value})"
+    _message = "In percentage type, the threshold must be less than 100. (key = notification, value = {value})"
 
 
 class ERROR_PROVIDER_FILTER_IS_EMPTY(ERROR_INVALID_ARGUMENT):
     _message = "Provider filter is empty. (key = provider_filter.providers, value = [])"
 
 
 class ERROR_BUDGET_ALREADY_EXIST(ERROR_INVALID_ARGUMENT):
-    _message = "Budget already exist. (service_account_id = {service_account_id}, workspace_id= {workspace_id}, target = {target})"
+    _message = "Budget already exist. (start = {start} end = {end}, target = {target}, workspace_id = {workspace_id})"
 
 
 class ERROR_NOTIFICATION_IS_NOT_SUPPORTED_IN_PROJECT(ERROR_INVALID_ARGUMENT):
     _message = "Notification is not supported in project. (target = {target})"
+
+
+class ERROR_BUDGET_MANAGER_IS_NOT_VERIFIED(ERROR_INVALID_ARGUMENT):
+    _message = "To assign as a budget manager to a user, email verification is required. (user_id = {user_id})"
+
+
+class ERROR_DUPLICATED_THRESHOLD(ERROR_INVALID_ARGUMENT):
+    _message = "Duplicate threshold in notification. (threshold = {threshold})"