Skip to content

Commit 8baf1b5

Browse files
committed
Change ClinVar Batch to overall health check so we always know about batches left as pending
1 parent faa9ab3 commit 8baf1b5

File tree

1 file changed

+9
-10
lines changed

1 file changed

+9
-10
lines changed

eventlog/signals/active_users_health_check.py

Lines changed: 9 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,8 @@
77
from classification.models import DiscordanceReportTriage, ClinVarExportBatch, ClinVarExportBatchStatus
88
from eventlog.models import Event, ViewEvent
99
from library.guardian_utils import bot_group
10-
from library.health_check import health_check_signal, HealthCheckRequest, HealthCheckRecentActivity
10+
from library.health_check import health_check_signal, HealthCheckRequest, HealthCheckRecentActivity, \
11+
health_check_overall_stats_signal, HealthCheckTotalAmount
1112
from snpdb.models import UserPreview
1213

1314

@@ -102,19 +103,17 @@ def discordance_triage_health_check(sender, health_request: HealthCheckRequest,
102103
)
103104

104105

105-
@receiver(signal=health_check_signal)
106+
@receiver(signal=health_check_overall_stats_signal)
106107
def clinvar_export_batch_healthcheck(sender, health_request: HealthCheckRequest, **kwargs):
107108

108-
accepted_statuses = [ClinVarExportBatchStatus.AWAITING_UPLOAD, ClinVarExportBatchStatus.UPLOADING]
109-
recent_batches = ClinVarExportBatch.objects.filter(status__in=accepted_statuses,
110-
created__gte=health_request.since,
111-
created__lt=health_request.now)
109+
accepted_statuses = (ClinVarExportBatchStatus.AWAITING_UPLOAD, ClinVarExportBatchStatus.UPLOADING)
110+
recent_batches = ClinVarExportBatch.objects.filter(status__in=accepted_statuses)
112111

113112
if count := recent_batches.count():
114-
return HealthCheckRecentActivity(
113+
return HealthCheckTotalAmount(
115114
emoji=":package:",
116-
name="ClinVar Export Batches",
115+
name="Pending ClinVar Export Batches",
117116
amount=count,
118-
extra=", ".join([f'{batch.clinvar_key.name} - *{batch.get_status_display()}*' for batch in recent_batches]),
119-
stand_alone=True,
117+
extra=", ".join([f'{batch.clinvar_key.name} *{batch.get_status_display()}*' for batch in recent_batches])
120118
)
119+

0 commit comments

Comments
 (0)