Skip to content

Commit 63840ee

Browse files
committed
simplify lustre defaults
1 parent 9f47321 commit 63840ee

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

ansible/roles/lustre/README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,12 +8,12 @@ Install and configure a Lustre client. This builds RPM packages from source.
88
## Role Variables
99

1010
- `lustre_version`: Optional str. Version of lustre to build, default '2.15.64'. TODO: EXPLAIN. See https://wiki.whamcloud.com/display/PUB/Lustre+Support+Matrix
11-
- `lustre_mounts`: Optional list. Define Lustre filesystems and mountpoints as a list of dicts with possible keys:
11+
- `lustre_mounts`: Required list. Define Lustre filesystems and mountpoints as a list of dicts with possible keys:
1212
- `mgs_nid`: The NID for the MGS, e.g. `192.168.227.11@tcp1`
1313
- `fs_name`: The name of the filesystem to mount
1414
- `mount_point`: Path to mount filesystem at. Default is `/mnt/lustre/{{ lustre_fs_name}}`
1515
- `mount_state`: Mountpoint state, as for [ansible.posix.mount](https://docs.ansible.com/ansible/latest/collections/ansible/posix/mount_module.html#parameter-state). Default `mounted`.
16-
These parameters may alternatively be specified as role variables prefixed `lustre_`, which are lower-priority than entries in `lustre_mounts`.
16+
Any of these parameters may alternatively be specified as role variables prefixed `lustre_`. If both are given entries in `lustre_mounts` take priority.
1717

1818
The following variables control the package build and and install and should not generally be required:
1919
- `lustre_build_packages`: Optional list. Prerequisite packages required to build Lustre. See `defaults/main.yml`.

ansible/roles/lustre/defaults/main.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
lustre_version: '2.15.5-RC3' # TODO: FIXME to 2.15.5 on release see http://lists.lustre.org/pipermail/lustre-discuss-lustre.org/2024-June/019163.html
2-
lustre_mounts: [{}] # Ensures if all top-level vars are provided then this isn't required
2+
lustre_mounts: []
33
#lustre_mgs_nid:
44
#lustre_fs_name:
55
lustre_mount_point: "/mnt/lustre/{{ lustre_fs_name }}"

0 commit comments

Comments
 (0)