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

Get sandbox in schedd #8645

Closed
wants to merge 42 commits into from
Closed

Get sandbox in schedd #8645

wants to merge 42 commits into from

Conversation

novicecpp
Copy link
Contributor

No description provided.

@novicecpp
test14
90570d7
@novicecpp
test14 (client)
03fa08c
@novicecpp
WIP: purge dry run
1044f49
@novicecpp
sandbox at schedd instead
f52f2f4
@novicecpp
Revert "WIP: purge dry run"
3f3c9a4 
This reverts commit dd9e76a.
@novicecpp
fix logger
ea75f6b
@novicecpp
move logger function inside getSandbox
3a2490e
@novicecpp
fix getSandBox function
419366e
@novicecpp
make downloadFromS3 to be sure that file is not broken
7df7f6e
@novicecpp
merge into single rest call for task info
039bb0d
@novicecpp
more print and beautify
c830e31
@novicecpp
unbuffer?
3d7039d
@novicecpp
some comment
4d5ebd0
@novicecpp
revert downloadFromS3 back to master (/dev/null => /dev/null_tmp)
ea48412
@novicecpp
make donwload sandbox robust at adjustsites instead
d86d45b
@novicecpp
disable submit --dryrun
7bc3d0f
@novicecpp
fix wording
1d5a356
@novicecpp
fix wording2
00d950f
@novicecpp
heck if sandbox exist
82756cc
@novicecpp
debug
99c7496
@novicecpp
S3HeadObeject
8ca7082
@novicecpp
fix sandboxName
47eb7b4
@novicecpp
test header range with wget
93aaf61
@novicecpp
rename
b9ccf4f
@novicecpp
break
1dd5824
@novicecpp
check s3 object
1f40a30
@novicecpp
try to run inside bash
963a904
@novicecpp
Revert "break"
02fb98c 
This reverts commit d8f170f.
@novicecpp
update comment
325d8b0
@novicecpp
comment a bit
34894d3
@cmsdmwmbot
Copy link

Jenkins results:

  • Python3 Pylint check: succeeded
  • Pycodestyle check: succeeded

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

@novicecpp novicecpp mentioned this pull request Sep 4, 2024
Closed
@belforte
Copy link
Member

belforte commented Nov 7, 2024

implemented in steps via #8767 #8725 #8745 #8743 #8740

@belforte belforte closed this Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@belforte belforte belforte left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

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