Skip to content

Commit

Permalink
fix by code review
Browse files Browse the repository at this point in the history
  • Loading branch information
ilan7empest committed Dec 9, 2024
1 parent 5627772 commit 0482f32
Showing 1 changed file with 16 additions and 18 deletions.
34 changes: 16 additions & 18 deletions src/components/ProjectsPage/projects.util.js
Original file line number Diff line number Diff line change
Expand Up @@ -292,38 +292,36 @@ export const generateMonitoringCounters = (data, dispatch) => {
}

data.forEach(project => {
monitoringCounters.jobs.total +=
project.runs_completed_recent_count ||
0 + project.runs_failed_recent_count ||
0 + project.runs_running_count ||
0
monitoringCounters.jobs.completed += project.runs_completed_recent_count || 0
monitoringCounters.jobs.failed += project.runs_failed_recent_count || 0
monitoringCounters.jobs.running += project.runs_running_count || 0
monitoringCounters.workflows.total +=
project.pipelines_completed_recent_count ||
0 + project.pipelines_failed_recent_count ||
0 + project.pipelines_running_count ||
0
monitoringCounters.jobs.total =
monitoringCounters.jobs.completed +
monitoringCounters.jobs.failed +
monitoringCounters.jobs.running

monitoringCounters.workflows.completed += project.pipelines_completed_recent_count || 0
monitoringCounters.workflows.failed += project.pipelines_failed_recent_count || 0
monitoringCounters.workflows.running += project.pipelines_running_count || 0
monitoringCounters.scheduled.total +=
project.distinct_scheduled_jobs_pending_count ||
0 + project.distinct_scheduled_pipelines_pending_count ||
0
monitoringCounters.workflows.total =
monitoringCounters.workflows.completed +
monitoringCounters.workflows.failed +
monitoringCounters.workflows.running

monitoringCounters.scheduled.jobs += project.distinct_scheduled_jobs_pending_count || 0
monitoringCounters.scheduled.workflows +=
project.distinct_scheduled_pipelines_pending_count || 0
monitoringCounters.scheduled.total =
monitoringCounters.scheduled.jobs + monitoringCounters.scheduled.workflows

monitoringCounters.alerts.endpoint += project.endpoint_alerts_count || 0
monitoringCounters.alerts.jobs += project.job_alerts_count || 0
monitoringCounters.alerts.application += project.other_alerts_count || 0
monitoringCounters.alerts.total =
monitoringCounters.alerts.endpoint +
monitoringCounters.alerts.jobs +
monitoringCounters.alerts.application
})
monitoringCounters.alerts.total =
monitoringCounters.alerts.endpoint +
monitoringCounters.alerts.jobs +
monitoringCounters.alerts.application

dispatch(projectsAction.setJobsMonitoringData(monitoringCounters))
}
Expand Down

0 comments on commit 0482f32

Please sign in to comment.