Skip to content

Commit

Permalink
Use static IP to populate slurmdbd's DbdAddr parameter
Browse files Browse the repository at this point in the history
Export the IP into the instance dna.json.

Add minor comments to the code.

Signed-off-by: Jacopo De Amicis <jdamicis@amazon.it>
  • Loading branch information
jdeamicis committed Jan 29, 2024
1 parent 4e4456f commit 4fc4db6
Showing 3 changed files with 4 additions and 2 deletions.
Original file line number Diff line number Diff line change
@@ -49,8 +49,6 @@

include_recipe 'aws-parallelcluster-slurm::config_head_node_directories'

# TODO: configuration of munge systemd service;

# TODO: move this template to a separate recipe
# TODO: add a logic in update_munge_key.sh.erb to skip sharing munge key to shared dir
template "#{node['cluster']['scripts_dir']}/slurm/update_munge_key.sh" do
Original file line number Diff line number Diff line change
@@ -32,6 +32,7 @@
action :create_if_missing
variables(
dbd_host: "localhost",
dbd_addr: node['slurmdbd_ip'],
storage_host: node['dbms_uri'],
# TODO: expose additional CFN Parameter in template
storage_port: 3306,
@@ -74,6 +75,8 @@
# After starting slurmdbd the database may not be fully responsive yet and
# its bootstrapping may fail. We need to wait for sacctmgr to successfully
# query the database before proceeding.
# In case of an external slurmdbd the Slurm commands do not work, so this
# check cannot be executed.
execute "wait for slurm database" do
command "#{node['cluster']['slurm']['install_dir']}/bin/sacctmgr show clusters -Pn"
retries node['cluster']['slurmdbd_response_retries']
Original file line number Diff line number Diff line change
@@ -2,6 +2,7 @@
# Do not modify.
# Please add user-specific slurmdbd configuration options in slurmdbd.conf
DbdHost=<%= @dbd_host %>
DbdAddr=<%= @dbd_addr %>
StorageHost=<%= @storage_host %>
StoragePort=<%= @storage_port %>
StorageLoc=<%= @storage_loc %>

0 comments on commit 4fc4db6

Please sign in to comment.