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

[9.0] more setup removals #7917

Merged
merged 1 commit into from
Dec 2, 2024

Conversation

fstagni
Copy link
Contributor

@fstagni fstagni commented Nov 29, 2024

No description provided.

@aldbr
Copy link
Contributor

aldbr commented Nov 29, 2024

Can you also remove this block please?

# -----------------------------------------------------------------------------------------------
# Temporary hack: in v9.0, the DIRAC/Setup is meant to be removed from the configuration
# Until then, we need to set it manually
cfg = CFG().loadFromFile(cfgFilename)
cfg.setOption("DIRAC/Setup", gConfig.getOption("DIRAC/Setup", "")["Value"])
cfg.writeToFile(cfgFilename)
# -----------------------------------------------------------------------------------------------

Thanks

@fstagni fstagni force-pushed the 90_more_setup_removals branch from 0835dd8 to 36c1b32 Compare November 29, 2024 14:29
@fstagni
Copy link
Contributor Author

fstagni commented Dec 2, 2024

I tested this in https://github.com/fstagni/Pilot/actions/runs/12118277471

I think it is fine to merge...

@fstagni fstagni merged commit 6997c75 into DIRACGrid:integration Dec 2, 2024
24 checks passed
@DIRACGridBot DIRACGridBot added the sweep:ignore Prevent sweeping from being ran for this PR label Dec 2, 2024
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