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

[8.0] feat(HTCondorCE): add option to submit with SSL #7630

Merged
merged 1 commit into from
May 31, 2024

Conversation

andresailer
Copy link
Contributor

@andresailer andresailer commented May 30, 2024

This let's me submit pilots to DESY, and hopefully also maybe other sites that will do the same.

BEGINRELEASENOTES

*Resources
NEW: HTCondorCE: Added UseSSLSubmission option. Allows one to use a configured DN at given Sites for job submission, instead of proxies or tokens. Only at participating CEs and conditions apply.

ENDRELEASENOTES

PS: Currently pilots are at the site, but DESY still has to configure something to make them run. So maybe there will be more changes.

@DIRACGridBot DIRACGridBot added the alsoTargeting:integration Cherry pick this PR to integration after merge label May 30, 2024
@andresailer andresailer changed the title feat(HTCondorCE): add option to submit with SSL [8.0] feat(HTCondorCE): add option to submit with SSL May 30, 2024
@andresailer andresailer force-pushed the 8.0_condor_ssl branch 2 times, most recently from decf1af to 7590beb Compare May 30, 2024 13:30
@andresailer andresailer force-pushed the 8.0_condor_ssl branch 3 times, most recently from 07476b0 to 5f54c3b Compare May 30, 2024 14:16
set UseSSLSubmission = True to a computing element and provide the local userkey and usercert files in coordination with participating Sites
as they have to configure DN of the submitting user
@fstagni
Copy link
Contributor

fstagni commented May 31, 2024

"UseSSLSubmission" should be added in some documentation.

@andresailer
Copy link
Contributor Author

"UseSSLSubmission" should be added in some documentation.

It is documented in the module docstring.

cf. https://dirac--7630.org.readthedocs.build/en/7630/CodeDocumentation/Resources/Computing/HTCondorCEComputingElement.html#module-DIRAC.Resources.Computing.HTCondorCEComputingElement

@fstagni fstagni merged commit bf44c17 into DIRACGrid:rel-v8r0 May 31, 2024
26 checks passed
@DIRACGridBot DIRACGridBot added sweep:done All sweeping actions have been done for this PR sweep:failed Sweeping failed and needs manual intervention labels May 31, 2024
@DIRACGridBot
Copy link

Sweep summary

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

Failed:

  • integration
    cherry-pick bf44c17 into integration failed
    check merge conflicts on a local copy of this repository
    git fetch upstream
    git checkout upstream/integration -b cherry-pick-2-bf44c17f0-integration
    git cherry-pick -x -m 1 bf44c17f0
    # Fix the conflicts
    git cherry-pick --continue
    git commit --amend -m 'sweep: #7630 feat(HTCondorCE): add option to submit with SSL' --author='Andre Sailer <[email protected]>'
    git push -u origin cherry-pick-2-bf44c17f0-integration
    
    # If you have the GitHub CLI installed the PR can be made with
    gh pr create \
         --label 'sweep:from rel-v8r0' \
         --base integration \
         --repo DIRACGrid/DIRAC \
         --title '[sweep:integration] feat(HTCondorCE): add option to submit with SSL' \
         --body 'Sweep #7630 `feat(HTCondorCE): add option to submit with SSL` to `integration`.
    
    Adding original author @andresailer as watcher.
    
    BEGINRELEASENOTES
    
    *Resources
    NEW: HTCondorCE: Added UseSSLSubmission option. Allows one to use a configured DN at given Sites for job submission, instead of proxies or tokens. Only at participating CEs and conditions apply.
    
    ENDRELEASENOTES
    Closes #7635'

@andresailer andresailer deleted the 8.0_condor_ssl branch June 1, 2024 06:56
andresailer pushed a commit to andresailer/DIRAC that referenced this pull request Jun 4, 2024
[8.0] feat(HTCondorCE): add option to submit with SSL

(cherry picked from commit bf44c17)
andresailer added a commit to andresailer/DIRAC that referenced this pull request Jun 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
alsoTargeting:integration Cherry pick this PR to integration after merge sweep:done All sweeping actions have been done for this PR sweep:failed Sweeping failed and needs manual intervention
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants