Skip to content

Commit eba6697

Browse files
committed
improve ood app install logic
1 parent 724cdb4 commit eba6697

File tree

1 file changed

+9
-4
lines changed

1 file changed

+9
-4
lines changed

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

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -82,16 +82,21 @@ openondemand_clusters_grafana:
8282
jobid: "jobid"
8383

8484
# define installation of openondemand apps (see openondemand_apps_* below for configuration):
85-
openondemand_install_apps_default:
85+
openondemand_install_app_jupyter:
8686
jupyter:
8787
repo: https://github.com/OSC/bc_example_jupyter.git
8888
version: master # defaults (optional)
89-
openondemand_install_apps_extra: # mapping, values as for ansible.builtin.git: repo (required)/dest/version(default main)/umask
89+
openondemand_install_app_rstudio: # mapping, values as for ansible.builtin.git: repo (required)/dest/version(default main)/umask
9090
rstudio:
9191
repo: https://github.com/stackhpc/ood-stackhpc-rstudio.git
9292
version: main
9393
# osc:ood role var (NB only active when not in configure):
94-
ood_install_apps: "{{ openondemand_install_apps_default if openondemand_jupyter_partition | default(none) else {} | combine(openondemand_install_apps_extra) }}"
94+
ood_install_apps: >-
95+
{{
96+
{}
97+
| combine(openondemand_install_app_jupyter if openondemand_jupyter_partition else {})
98+
| combine(openondemand_install_app_rstudio if openondemand_rstudio_partition else {})
99+
}}
95100
96101
# https://github.com/OSC/ondemand/tree/master/apps/bc_desktop
97102
# also https://osc.github.io/ood-documentation/latest/enable-desktops/custom-job-submission.html#enable-desktops-custom-job-submission
@@ -225,7 +230,7 @@ openondemand_apps:
225230
ood_brand_bg_color: "#0e6ec8"
226231
ood_dashboard_title: "{{ openhpc_cluster_name }}"
227232

228-
ood_apps: "{{ openondemand_apps | combine(openondemand_apps_jupyter) | combine(openondemand_apps_desktop) }}"
233+
ood_apps: "{{ openondemand_apps | combine(openondemand_apps_jupyter) | combine(openondemand_apps_desktop) | combine(openondemand_apps_rstudio) }}"
229234

230235
# Prometheus config for Open Ondemand exporter
231236
# https://osc.github.io/ood-documentation/latest/monitoring/prometheus.html#prometheus-configuration

0 commit comments

Comments
 (0)