Skip to content

feat: add persisted logs - #1383

Draft
leafty wants to merge 52 commits into
mainfrom
leafty/feat-peristed-logs
Draft

feat: add persisted logs#1383
leafty wants to merge 52 commits into
mainfrom
leafty/feat-peristed-logs

Conversation

@leafty

@leafty leafty commented Jul 13, 2026

Copy link
Copy Markdown
Member

wip

/deploy #notest amalthea-sessions=leafty/feat-run-id-jobs renku-ui=leafty/feat-peristed-logs extra-values=dataService.imageBuilders.enabled=true,dataService.imageBuilders.strategyName=renku-buildpacks-v3,dataService.imageBuilders.outputImagePrefix=harbor.dev.renku.ch/renku-build/,dataService.imageBuilders.nodeSelector.renku.io/node-purpose=user,dataService.imageBuilders.tolerations[0].effect=NoSchedule,dataService.imageBuilders.tolerations[0].key=renku.io/dedicated,dataService.imageBuilders.tolerations[0].operator=Equal,dataService.imageBuilders.tolerations[0].value=user

@RenkuBot

Copy link
Copy Markdown
Contributor

You can access the deployment of this PR at https://renku-ci-ds-1383.dev.renku.ch

@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 29478825432

Warning

Build has drifted: This PR's base is out of sync with its target branch, so coverage data may include unrelated changes.
Quick fix: rebase this PR. Learn more →

Coverage at 86.069% (no base build to compare)

Details

  • Coverage remained the same as the base build.
  • Patch coverage: 69 uncovered changes across 4 files (210 of 279 lines covered, 75.27%).
  • No coverage regressions found.

Uncovered Changes

File Changed Covered %
components/renku_data_services/persisted_logs/collector.py 79 37 46.84%
components/renku_data_services/persisted_logs/db.py 30 10 33.33%
bases/renku_data_services/data_tasks/task_defs.py 7 1 14.29%
components/renku_data_services/persisted_logs/loki_api.py 30 29 96.67%
Total (14 files) 279 210 75.27%

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 31871
Covered Lines: 27431
Line Coverage: 86.07%
Coverage Strength: 1.49 hits per line

💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants