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

[8.0] fix: support file:/path/to/pilot_files as a location for the pilot files #7565

Merged
merged 1 commit into from
Apr 16, 2024
Merged

[8.0] fix: support file:/path/to/pilot_files as a location for the pilot files #7565

merged 1 commit into from
Apr 16, 2024

Conversation

aldbr
Copy link
Contributor

@aldbr aldbr commented Apr 12, 2024

Aims at fixing this issue:

Trying https://file:/cvmfs/<repo>/dirac/pilot
2024-04-09 13:24:18 UTC ERROR    https://file:/cvmfs/<repo>/dirac/pilot unreacheable (this is normal!)
2024-04-09 13:24:18 UTC ERROR    <urlopen error [Errno -2] Name or service not known>

BEGINRELEASENOTES
*WorkloadManagement
FIX: support file:/... as a location for the pilot files
ENDRELEASENOTES

@aldbr aldbr requested a review from fstagni April 12, 2024 14:06
@DIRACGridBot DIRACGridBot added the alsoTargeting:integration Cherry pick this PR to integration after merge label Apr 12, 2024
@aldbr aldbr force-pushed the v8.0_FIX_pilot-wrapper-https branch from 8320ca5 to a67db21 Compare April 15, 2024 09:22
@fstagni fstagni merged commit abb41b5 into DIRACGrid:rel-v8r0 Apr 16, 2024
25 checks passed
@DIRACGridBot DIRACGridBot added the sweep:done All sweeping actions have been done for this PR label Apr 16, 2024
DIRACGridBot pushed a commit to DIRACGridBot/DIRAC that referenced this pull request Apr 16, 2024
@aldbr @web-flow
sweep: DIRACGrid#7565 fix: support file:/path/to/pilot_files as a l…
c0b5fac 
…ocation for the pilot files
@DIRACGridBot DIRACGridBot mentioned this pull request Apr 16, 2024
Merged
@DIRACGridBot
Copy link

Sweep summary

Sweep ran in https://github.com/DIRACGrid/DIRAC/actions/runs/8702276077

Successful:

  • integration

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

@andresailer andresailer andresailer left review comments

@fstagni fstagni fstagni approved these changes

Assignees
No one assigned
Labels
alsoTargeting:integration Cherry pick this PR to integration after merge sweep:done All sweeping actions have been done for this PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@aldbr @DIRACGridBot @fstagni @andresailer