-
Notifications
You must be signed in to change notification settings - Fork 106
Support external Slurm DBD #2690
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
Merged
hanwen-pcluste
merged 24 commits into
aws:develop
from
hanwen-pcluste:wip/feature/external_slurmdbd
Apr 16, 2024
Merged
Changes from 1 commit
Commits
Show all changes
24 commits
Select commit
Hold shift + click to select a range
72219ca
Add entrypoints for external slurmdbd.
hehe7318 4cd6cc3
Add cluster_user attribute, add template slurmdbd.service, add templa…
hehe7318 2e20575
Create new slurm_external_slurmdbd.conf. Add logic in slurmdbd.conf.e…
hehe7318 c5af3d2
Address comments.
hehe7318 7a1c7e9
Address issues in External Slurmdbd cookbook recipes (#2585)
hehe7318 6ad49b0
Create slurm_external_slurmdbd.conf for external slurmdbd
jdeamicis 44d27d3
Disable Slurm database bootstrap from external slurmdbd
jdeamicis 407a57d
Add support for external slurmdbd in Slurm configuration (#2595)
jdeamicis 33d8493
Increase debug level on external slurmdbd.conf
jdeamicis cb64b70
Use static IP to populate slurmdbd's DbdAddr parameter
jdeamicis be546b7
Configure CloudWatch agent for External Slurm Dbd
NSsirena 45084d5
Configure S3 mountpoint for external slurmdbd (#2633)
jdeamicis f76ed73
Fetch slurmdbd configuration from S3 if available
jdeamicis bbe65d8
Add recipe to disable not required services in ExternalSlurmdbd
NSsirena 84a97f1
Make DbdPort configurable in external slurmdbd recipes
jdeamicis 630023b
Extend remote_object resource with owner, group and mode
jdeamicis 3a7de07
Configure External Slurmdbd Host and Port in cookbook
jdeamicis 85785b0
Fix failing unit tests and cookstyle linter
jdeamicis 2d66917
Improve update_munge_key.sh
hanwen-cluster 578098e
Remove unnecessary code in external_slurmdbd_config
hanwen-cluster 887cda3
Simplify slurm_external_slurmdbd.conf.erb
hanwen-cluster 366fa2c
Remove duplicate attribute node['is_external_slurmdbd']
hanwen-cluster 48a031b
[Github CI] Add entrypoints spec test to Github checks
hanwen-cluster ba81080
Add spec tests for external slurmdbd config recipe
hanwen-cluster File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next
Next commit
Add entrypoints for external slurmdbd.
- Loading branch information
commit 72219ca05b7092d78d61979a7df0bd1b949d75da
There are no files selected for viewing
26 changes: 26 additions & 0 deletions
26
cookbooks/aws-parallelcluster-entrypoints/recipes/external_slurmdbd_config.rb
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,26 @@ | ||
# frozen_string_literal: true | ||
|
||
# | ||
# Copyright:: 2023 Amazon.com, Inc. or its affiliates. All Rights Reserved. | ||
# | ||
# Licensed under the Apache License, Version 2.0 (the "License"). You may not use this file except in compliance with the | ||
# License. A copy of the License is located at | ||
# | ||
# http://aws.amazon.com/apache2.0/ | ||
# | ||
# or in the "LICENSE.txt" file accompanying this file. This file is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES | ||
# OR CONDITIONS OF ANY KIND, express or implied. See the License for the specific language governing permissions and | ||
# limitations under the License. | ||
|
||
node.default['cluster']['slurm']['install_dir'] = '/opt/slurm' | ||
node.default['cluster']['slurm']['user'] = 'slurm' | ||
node.default['cluster']['slurm']['group'] = 'slurm' | ||
node.default['cluster']['scripts_dir'] = '/opt/parallelcluster/scripts' | ||
node.default['cluster']['region'] = node['region'] | ||
node.default['cluster']['slurmdbd_response_retries'] = 30 | ||
# TODO: attributes for stack name | ||
|
||
include_recipe "aws-parallelcluster-slurm::config_slurm_accounting" | ||
|
||
# TODO: modify logic in config_slurm_accounting or create a new recipe for external slurmdbd | ||
# to use parameter in dna.json instead of digging ARN from config |
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.