-
Notifications
You must be signed in to change notification settings - Fork 176
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] Drop the system instance level in the CS #7186
Conversation
d23e35d
to
ad15178
Compare
So, IIUC:
At a minimum we need some instructions, to be added to the v9 wiki. A script that does the CS update would be better. |
OK, I created #7808 in order to not forget. |
This PR allows to drop the System instance section level from the CS. The presence of the /DIRAC/NoSetup=True option in the CS suggests that the configuration is defined without Setups (to be introduced in 9.0 #7671). This forces the system section not to contain the instance level in the output of getSystemSection() and similar cases. This way it should be compatible with CS both with and without Setups defined.
BEGINRELEASENOTES
*Configuration
CHANGE: PathFinder.getSystemInstance() returns empty string if Setups are dropped from the CS
*WorkloadManagement
CHANGE: StalledJobAgent, Watchdog - use getSystemSection() instead of getSystemInstance()
ENDRELEASENOTES