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] fix(Resources): AREX should renew the delegation before submitting a payload #7936

Merged

Conversation

aldbr
Copy link
Contributor

@aldbr aldbr commented Dec 5, 2024

As discussed with @marianne013 a few weeks ago

Otherwise,

  • if there is no pilot submission, there is no pilot monitoring, and
  • if there is no pilot monitoring, there is no delegation renewal, and
  • if there is no delegation renewal, then the delegation can expire (which happens in the certification instance)

I am not even sure it solves #7931 though

BEGINRELEASENOTES
*WorkloadManagement
FIX: renew delegation prior to submitting pilots
ENDRELEASENOTES

@DIRACGridBot DIRACGridBot added the alsoTargeting:integration Cherry pick this PR to integration after merge label Dec 5, 2024
Copy link
Contributor

@fstagni fstagni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do not see anything wrong

@aldbr aldbr changed the title fix(Resources): AREX should renew the delegation before submitting a payload [8.0] fix(Resources): AREX should renew the delegation before submitting a payload Dec 9, 2024
@chrisburr chrisburr merged commit 3edd724 into DIRACGrid:rel-v8r0 Dec 13, 2024
26 checks passed
@DIRACGridBot DIRACGridBot added the sweep:done All sweeping actions have been done for this PR label Dec 13, 2024
DIRACGridBot pushed a commit to DIRACGridBot/DIRAC that referenced this pull request Dec 13, 2024
@DIRACGridBot
Copy link

Sweep summary

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

Successful:

  • integration

@fstagni fstagni mentioned this pull request Dec 19, 2024
2 tasks
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants