[feat] Support multi-cluster operation in Slurm backends#3639
[feat] Support multi-cluster operation in Slurm backends#3639vkarak wants to merge 1 commit intoreframe-hpc:developfrom
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #3639 +/- ##
===========================================
+ Coverage 91.64% 91.72% +0.08%
===========================================
Files 62 62
Lines 13530 13537 +7
===========================================
+ Hits 12399 12417 +18
+ Misses 1131 1120 -11 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
@vkarak I have pulled from your fork and can confirm it is polling the the correct cluster. Something that I think could improve the user experience would be to include it against the Here is a snippet of my partitions for the test I ran, you can see that I currently need to set it in {
'name': 'cluster1',
'scheduler': 'slurm',
'launcher': 'local',
'environs': ['slurm_multi_cluster_mode'],
'access': ['-M tst1'],
'sched_options': {
'slurm_multi_cluster_mode': ['cluster1']
}
},
{
'name': 'cluster2',
'scheduler': 'slurm',
'launcher': 'local',
'environs': ['slurm_multi_cluster_mode'],
'access': ['-M tst2'],
'sched_options': {
'slurm_multi_cluster_mode': ['cluster2']
}
} |
Yes, that make sense! I'll update the PR, so that the |
This PR introduces a new configuration option for Slurm backends named
slurm_multi_cluster_modethat supports Slurm's Multi-Cluster Operation. If not specified, nothing changes. If it is, then the clusters listed are being passed to Slurm's-Moption. If set to["all"], this is equivalent to-M alland all clusters are queried.Closes #3559.
@JimPaine Would you mind trying this PR with your setup?