Skip to content

[DOP-22729] Generate log_url on worker side#220

Merged
IlyasDevelopment merged 1 commit intodevelopfrom
feature/DOP-22729
Mar 18, 2025
Merged

[DOP-22729] Generate log_url on worker side#220
IlyasDevelopment merged 1 commit intodevelopfrom
feature/DOP-22729

Conversation

@IlyasDevelopment
Copy link
Copy Markdown
Contributor

@IlyasDevelopment IlyasDevelopment commented Mar 18, 2025

Change Summary

Ensured that log_url is always populated on the worker side, even when Transfer is started from the scheduler, instead of relying solely on the server.

Checklist

  • Commit message and PR title is comprehensive
  • Keep the change as small as possible
  • Unit and integration tests for the changes exist
  • Tests pass on CI and coverage does not decrease
  • Documentation reflects the changes where applicable
  • docs/changelog/next_release/<pull request or issue id>.<change type>.rst file added describing change
    (see CONTRIBUTING.rst for details.)
  • My PR is ready to review.

@IlyasDevelopment IlyasDevelopment added the ci:skip-changelog Add this label to skip changelog file check label Mar 18, 2025
@IlyasDevelopment IlyasDevelopment self-assigned this Mar 18, 2025
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 18, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.85%. Comparing base (3e3d663) to head (be4d322).
Report is 139 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #220      +/-   ##
===========================================
- Coverage    91.86%   91.85%   -0.01%     
===========================================
  Files          194      194              
  Lines         4669     4667       -2     
  Branches       350      350              
===========================================
- Hits          4289     4287       -2     
  Misses         297      297              
  Partials        83       83              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@IlyasDevelopment IlyasDevelopment merged commit d558a6b into develop Mar 18, 2025
40 of 42 checks passed
@IlyasDevelopment IlyasDevelopment deleted the feature/DOP-22729 branch March 18, 2025 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:skip-changelog Add this label to skip changelog file check

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants