Skip to content

Commit

Permalink
Merge pull request #1224 from DSD-DBS/fix-push-loki-logs
Browse files Browse the repository at this point in the history
fix: Ensure that loki labels are strings
  • Loading branch information
MoritzWeber0 authored Dec 11, 2023
2 parents b98fd66 + 159f89c commit 5e6c6de
Showing 1 changed file with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -144,8 +144,8 @@ def _fetch_events_of_job_run(run: models.DatabasePipelineRun):
]

labels = {
"job_name": run.reference_id,
"pipeline_run_id": run.id,
"job_name": run.reference_id or "",
"pipeline_run_id": str(run.id),
"log_type": "events",
}

Expand Down Expand Up @@ -178,8 +178,8 @@ def _fetch_logs_of_job_runs(run: models.DatabasePipelineRun):
]

labels = {
"job_name": run.reference_id,
"pipeline_run_id": run.id,
"job_name": run.reference_id or "",
"pipeline_run_id": str(run.id),
"log_type": "logs",
}

Expand Down

0 comments on commit 5e6c6de

Please sign in to comment.