Skip to content

Commit

Permalink
Merge pull request #229 from ImMin5/master
Browse files Browse the repository at this point in the history
Fix filter duplicate connected workspaces for DataSource related with DataSourceAccount
  • Loading branch information
ImMin5 authored May 14, 2024
2 parents f56a73d + fba7d82 commit 02cb4c1
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions src/spaceone/cost_analysis/manager/data_source_manager.py
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,7 @@ def _rollback(old_data):
def update_data_source_account_and_connected_workspace_count_by_vo(
self, data_source_vo: DataSource
) -> DataSource:
connected_workspaces = []
conditions = {
"data_source_id": data_source_vo.data_source_id,
"domain_id": data_source_vo.domain_id,
Expand All @@ -50,9 +51,9 @@ def update_data_source_account_and_connected_workspace_count_by_vo(

ds_account_vos = self.data_source_account_model.filter(**conditions)

connected_workspaces = [
data_source_vo.workspace_id for data_source_vo in ds_account_vos
]
for ds_account_vo in ds_account_vos:
if ds_account_vo.workspace_id:
connected_workspaces.append(ds_account_vo.workspace_id)

data_source_vo = self.update_data_source_by_vo(
{
Expand Down

0 comments on commit 02cb4c1

Please sign in to comment.