Skip to content

Commit a77ef88

Browse files
committed
extend timeout for slurmdbd startup to cope with major version upgrade on startup
1 parent 929dde2 commit a77ef88

File tree

2 files changed

+16
-5
lines changed

2 files changed

+16
-5
lines changed

environments/common/inventory/group_vars/all/openhpc.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -57,3 +57,8 @@ ohpc_openhpc_repos:
5757
ohpc_default_extra_repos:
5858
"9": []
5959
"8": []
60+
61+
# systemd.service.unit.TimeoutStartSec to wait for slurmdbd startup
62+
# Set long enought to avoid problems with a major version upgrade
63+
# Currently implemented in environments/common/inventory/group_vars/all/systemd.yml
64+
openhpc_slurmdbd_timeout_start_sec: '45 minutes'

environments/common/inventory/group_vars/all/systemd.yml

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,11 @@
11
_systemd_requiresmount_statedir: |
2+
{% if appliances_state_dir is defined %}
23
[Unit]
34
RequiresMountsFor={{ appliances_state_dir | default('') }}
5+
{% endif %}
46
5-
_systemd_dropins_statedir:
6-
# mysql not included as role handles state dir correctly
7+
systemd_dropins:
8+
# NB: mysql does not need _systemd_requiresmount_statedir as role handles state dir correctly
79
opensearch:
810
group: opensearch
911
content: "{{ _systemd_requiresmount_statedir }}"
@@ -12,12 +14,16 @@ _systemd_dropins_statedir:
1214
content: "{{ _systemd_requiresmount_statedir }}"
1315
slurmdbd:
1416
group: openhpc
15-
content: "{{ _systemd_requiresmount_statedir }}"
17+
content: |
18+
{{ _systemd_requiresmount_statedir }}
19+
20+
[Service]
21+
# Allow slurmdbd to complete major version upgrades
22+
TimeoutStartSec={{ openhpc_slurmdbd_timeout_start_sec }}
23+
1624
slurmctld:
1725
group: openhpc
1826
content: "{{ _systemd_requiresmount_statedir }}"
1927
prometheus:
2028
group: prometheus
2129
content: "{{ _systemd_requiresmount_statedir }}"
22-
23-
systemd_dropins: "{{ _systemd_dropins_statedir if appliances_state_dir is defined else {} }}"

0 commit comments

Comments
 (0)