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.

Sign up for GitHub

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

Jump to bottom

expand user tar files in /srv . Correct fix for #7661 #7708

Merged
merged 2 commits into from
Jun 30, 2023
Merged

expand user tar files in /srv . Correct fix for #7661 #7708

merged 2 commits into from
Jun 30, 2023

Conversation

belforte
Copy link
Member

@belforte belforte commented Jun 30, 2023 •
edited
Loading

No description provided.

@cmsdmwmbot

This comment was marked as outdated.

Sign in to view
@belforte belforte mentioned this pull request Jun 30, 2023
Merged
@belforte
Copy link
Member Author

belforte commented Jun 30, 2023 •
edited
Loading

test in https://cmsweb-test2.cern.ch/crabserver/ui/task/230630_211255%3Abelforte_crab_20230630_231250 FAILED

@cmsdmwmbot

This comment was marked as outdated.

Sign in to view
@belforte
Copy link
Member Author

belforte commented Jun 30, 2023 •
edited
Loading

test again after fixing above problem: https://cmsweb-test2.cern.ch/crabserver/ui/task/230630_220112%3Abelforte_crab_20230701_000107
fails again

@cmsdmwmbot

This comment was marked as outdated.

Sign in to view
@belforte
Copy link
Member Author

new test: https://cmsweb-test2.cern.ch/crabserver/ui/task/230630_223358%3Abelforte_crab_20230701_003353

@belforte
Copy link
Member Author

OK. Now user tar files are correclty expanded https://cmsweb.cern.ch:8443/scheddmon/0195/belforte/230630_223358:belforte_crab_20230701_003353/job_out.1.0.txt

@cmsdmwmbot
Copy link

Jenkins results:

  • Python3 Pylint check: failed
    • 32 warnings and errors that must be fixed
    • 14 warnings
    • 128 comments to review
  • Pycodestyle check: succeeded
    • 76 comments to review

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

@belforte
Copy link
Member Author

now job stdout has


==== Sandbox untarring in job's top dir STARTING at Fri Jun 30 22:53:29 2023 ====
expanding sandbox.tar.gz in /srv

expanding kh.tar in /srv

expanding kh2.tgz in /srv

expanding kh3.tar.gz in /srv

==== Sandbox untarring FINISHED at Fri Jun 30 22:53:31 2023 ====

@belforte belforte merged commit fc50785 into dmwm:master Jun 30, 2023
@belforte belforte deleted the fix-again-7661 branch June 30, 2023 22:57
@belforte belforte removed the request for review from mapellidario June 30, 2023 23:10
@belforte belforte mentioned this pull request Jul 1, 2023
Closed
belforte added a commit to belforte/CRABServer that referenced this pull request Jul 7, 2023
@belforte
expand user tar files in /srv . Correct fix for dmwm#7661 (dmwm#7708)
0419b24 
* expand user tar files in /srv . Correct fix for dmwm#7661

* final cleanup
novicecpp pushed a commit to novicecpp/CRABServer that referenced this pull request Jul 26, 2023
Revert "expand user tar files in /srv . Correct fix for dmwm#7661 (dm…
6161506 
…wm#7708)"

This reverts commit fc50785.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@belforte @cmsdmwmbot