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: AREX delegation not found #7219

Merged
merged 1 commit into from
Oct 10, 2023

Conversation

aldbr
Copy link
Contributor

@aldbr aldbr commented Sep 26, 2023

Aims to solve #7217

For an unknown reason, a few ARC instances from different sites got some discrepancies at some point. Here is an example:

GET https://<ce>:<port>/arex/rest/1.0/delegations -> {'delegation': [{'id': '02cfaf01c2ab'}]}
POST https://<ce>:<port>/arex/rest/1.0/delegations/02bvdf19c7at?action=get -> Error 404: No delegation found

CE admins are not able to find any detail about those buggy delegations on their side.
I submitted a ticket to inform the ARC developers, but as we don't know how to reproduce the issue, it is going to be hard for them to provide a fix: https://bugzilla.nordugrid.org/show_bug.cgi?id=4133

To avoid that, in this PR, we relax some constraints: if a delegation is not found, then instead of returning an error, we just continue so that a new delegation is generated.
I will monitor a few ARC instances for a few days to make sure it is not a recurrent issue and that it does not generate many delegations.

BEGINRELEASENOTES
*WorkloadManagement
FIX: AREX does not return an error if delegation not found
ENDRELEASENOTES

@DIRACGridBot DIRACGridBot added the alsoTargeting:integration Cherry pick this PR to integration after merge label Sep 26, 2023
@aldbr aldbr force-pushed the v8.0_FIX_AREX-delegation-404 branch 3 times, most recently from 172e4a6 to dd8a46d Compare September 28, 2023 12:47
@aldbr aldbr force-pushed the v8.0_FIX_AREX-delegation-404 branch from dd8a46d to 0569143 Compare October 10, 2023 06:35
@aldbr aldbr marked this pull request as ready for review October 10, 2023 06:36
@aldbr aldbr requested review from fstagni and atsareg as code owners October 10, 2023 06:36
@fstagni fstagni merged commit 2bd536c into DIRACGrid:rel-v8r0 Oct 10, 2023
20 checks passed
@DIRACGridBot DIRACGridBot added the sweep:done All sweeping actions have been done for this PR label Oct 10, 2023
DIRACGridBot pushed a commit to DIRACGridBot/DIRAC that referenced this pull request Oct 10, 2023
@DIRACGridBot
Copy link

Sweep summary

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

Successful:

  • integration

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.

3 participants