Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove debug_files handling in TW. Do it in scheduler #8747

Merged
merged 2 commits into from
Oct 22, 2024

Conversation

belforte
Copy link
Member

this is point 4.5 of #6544
and completes handling S3 tarballs in the scheduler, w/o using TaskWorker and man in the middle and w/o shipping them using condor transfer_input_files

@cmsdmwmbot
Copy link

Jenkins results:

  • Python3 Pylint check: failed
    • 1 warnings and errors that must be fixed
    • 75 comments to review
  • Pycodestyle check: succeeded
    • 161 comments to review

Details at https://cmssdt.cern.ch/dmwm-jenkins/view/All/job/DMWM-CRABServer-PR-test/2214/artifact/artifacts/PullRequestReport.html

@cmsdmwmbot
Copy link

Jenkins results:

  • Python3 Pylint check: succeeded
    • 57 comments to review
  • Pycodestyle check: succeeded
    • 154 comments to review

Details at https://cmssdt.cern.ch/dmwm-jenkins/view/All/job/DMWM-CRABServer-PR-test/2215/artifact/artifacts/PullRequestReport.html

@cmsdmwmbot
Copy link

Jenkins results:

  • Python3 Pylint check: succeeded
    • 7 comments to review
  • Pycodestyle check: succeeded
    • 149 comments to review

Details at https://cmssdt.cern.ch/dmwm-jenkins/view/All/job/DMWM-CRABServer-PR-test/2216/artifact/artifacts/PullRequestReport.html

@belforte
Copy link
Member Author

remaining pylint comments are in parts of the code which I do not fully understand and better not touch !

@belforte belforte force-pushed the expand-debug_files-in-AdjustSites branch from 7c38055 to 21b3fc2 Compare October 21, 2024 22:11
@cmsdmwmbot
Copy link

Jenkins results:

  • Python3 Pylint check: succeeded
    • 7 comments to review
  • Pycodestyle check: succeeded
    • 149 comments to review

Details at https://cmssdt.cern.ch/dmwm-jenkins/view/All/job/DMWM-CRABServer-PR-test/2217/artifact/artifacts/PullRequestReport.html

@belforte
Copy link
Member Author

belforte commented Oct 21, 2024

@belforte belforte merged commit 33239f7 into dmwm:master Oct 22, 2024
2 checks passed
@belforte belforte deleted the expand-debug_files-in-AdjustSites branch October 22, 2024 07:17
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.

2 participants