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

fix: disabledVOs not available when doing dirac-configure #7284

Merged
merged 1 commit into from
Nov 9, 2023

Conversation

aldbr
Copy link
Contributor

@aldbr aldbr commented Nov 9, 2023

"chicken and egg" situation here.
When doing dirac-configure, the CS is not yet available to the user, so we cannot get the VO and the DisabledVOs value.
Therefore, DisabledVOs is equal to [] and the condition is always True, which triggers calls to diracx, which is not available.

I proposed the following "hack" to perform the DIRAC hackathon.
The problem with that approach is that you would need to perform dirac-proxy-init again, after dirac-configure, to get a token along with a proxy.

BEGINRELEASENOTES
*Framework
FIX: disabledVOs not available when doing dirac-configure
ENDRELEASENOTES

@aldbr aldbr requested review from chaen and chrisburr November 9, 2023 10:13
@aldbr aldbr changed the title fix: disabledVOs not availabe when doing dirac-configure fix: disabledVOs not available when doing dirac-configure Nov 9, 2023
@chrisburr
Copy link
Member

We already do it a second time so I think this makes sense as there is the same problem with adding VOMS extensions and uploadijng the proxy:

["dirac-proxy-init", "--pwstdin"],

@chrisburr chrisburr merged commit 4f14f50 into DIRACGrid:integration Nov 9, 2023
22 checks passed
@DIRACGridBot DIRACGridBot added the sweep:ignore Prevent sweeping from being ran for this PR label Nov 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sweep:ignore Prevent sweeping from being ran for this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants