Allow modifying nodes fully-qualified name #651

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.

Sign up for GitHub

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

Jump to bottom

Draft

sjpb wants to merge 1 commit into main from feat/cluster-nodename-template

Collaborator

sjpb commented Apr 15, 2025 •

edited

Loading

Adds an opentofu variable cluster_nodename_template and overrides nodename_template for compute and login node groups, to allow defining the fully-qualified hostname for all nodes.
Note the appliance (and roles it uses) assume that the short hostname is resolvable (normally ensured by etc_hosts role), and that this matches the Ansible inventory_hostname and the slurm node name.

TODO: test this properly
TODO: maybe add the environment name as a templatable var using $TF_VAR_environment_root/??


          add cluster_nodename_template and compute/login.nodename_template

3694bb6

sjpb force-pushed the feat/cluster-nodename-template branch from f71dcdb to 3694bb6 Compare

April 15, 2025 09:14

sjpb mentioned this pull request

Allow use of Ironic node name as instance name #630

Open

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet