Skip to content

Bump openhpc role #690

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 0 additions & 8 deletions ansible/validate.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,14 +26,6 @@
- import_role:
name: stackhpc.openhpc
tasks_from: validate.yml
- assert:
that: "'enable_configless' in openhpc_config.SlurmctldParameters | default([])"
fail_msg: |
'enable_configless' not found in openhpc_config.SlurmctldParameters - is variable openhpc_config overridden?
Additional slurm.conf parameters should be provided using variable openhpc_config_extra.
success_msg: Checked Slurm will be configured for configless operation
delegate_to: localhost
run_once: true

- name: Validate filebeat configuration
hosts: filebeat
Expand Down
4 changes: 1 addition & 3 deletions environments/common/inventory/group_vars/all/openhpc.yml
Original file line number Diff line number Diff line change
Expand Up @@ -42,11 +42,9 @@ openhpc_packages_default:
- podman-compose
openhpc_packages_extra: []
openhpc_packages: "{{ (openhpc_packages_default + openhpc_packages_extra) | select | list }}"
openhpc_munge_key: "{{ vault_openhpc_mungekey | b64decode }}"
openhpc_munge_key_b64: "{{ vault_openhpc_mungekey }}"
openhpc_login_only_nodes: login
openhpc_config_default:
SlurmctldParameters:
- enable_configless
TaskPlugin: task/cgroup,task/affinity
ReturnToService: 2 # is stackhpc.openhpc default, but templating bug means it is needed here too
openhpc_config_rebuild:
Expand Down
2 changes: 1 addition & 1 deletion requirements.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ roles:
version: v25.3.2
name: stackhpc.nfs
- src: https://github.com/stackhpc/ansible-role-openhpc.git
version: v1.0.0
version: feat/only-configless # TODO: bump to release
name: stackhpc.openhpc
- src: https://github.com/stackhpc/ansible-node-exporter.git
version: stackhpc
Expand Down