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] Add option to include proxy on AREX token submission #7913

Merged
merged 1 commit into from
Nov 29, 2024

Conversation

sfayer
Copy link
Member

@sfayer sfayer commented Nov 29, 2024

Hi,

Here is my proposed fix to #7908 as discussed in the BiLD meeting. It simply adds an option to re-enable sending the proxies even if tokens are used; we should hopefully only need this until ARC7 is widely deployed (and someone has worked out how to do accounting without VOMS ownership data).

Regards,
Simon

BEGINRELEASENOTES
*Resources
CHANGE: Add option to include proxy on AREX token submission
ENDRELEASENOTES

@DIRACGridBot DIRACGridBot added the alsoTargeting:integration Cherry pick this PR to integration after merge label Nov 29, 2024
@fstagni fstagni requested a review from aldbr November 29, 2024 10:35
@fstagni
Copy link
Contributor

fstagni commented Nov 29, 2024

Looks OK to me

@fstagni fstagni merged commit c75abcc into DIRACGrid:rel-v8r0 Nov 29, 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 Nov 29, 2024
@DIRACGridBot
Copy link

Sweep summary

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

Failed:

  • integration
    cherry-pick c75abcc into integration failed
    check merge conflicts on a local copy of this repository
    git fetch upstream
    git checkout upstream/integration -b cherry-pick-2-c75abccab-integration
    git cherry-pick -x -m 1 c75abccab
    # Fix the conflicts
    git cherry-pick --continue
    git commit --amend -m 'sweep: #7913 Add option to include proxy on AREX token submission' --author='Simon Fayer <[email protected]>'
    git push -u origin cherry-pick-2-c75abccab-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] Add option to include proxy on AREX token submission' \
         --body 'Sweep #7913 `Add option to include proxy on AREX token submission` to `integration`.
    
    Adding original author @sfayer as watcher.
    
    BEGINRELEASENOTES
    *Resources
    CHANGE: Add option to include proxy on AREX token submission
    ENDRELEASENOTES
    Closes #7916'

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