Skip to content

Conversation

claudia-lola
Copy link
Contributor

@claudia-lola claudia-lola commented Oct 1, 2025

Closes #706.

@claudia-lola claudia-lola requested a review from a team as a code owner October 1, 2025 14:41
Copy link
Collaborator

@sjpb sjpb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Doesn't actually work, did you test this?

.stackhpc/ (venv) [rocky@steveb-dev tofu]$ pwd
/home/rocky/slurm-app-rl9/environments/.stackhpc/tofu
.stackhpc/ (venv) [rocky@steveb-dev tofu]$ git diff
diff --git i/environments/.stackhpc/tofu/main.tf w/environments/.stackhpc/tofu/main.tf
index 649f2f7c..ed737068 100644
--- i/environments/.stackhpc/tofu/main.tf
+++ w/environments/.stackhpc/tofu/main.tf
@@ -79,6 +79,7 @@ module "cluster" {
     head = {
       nodes  = ["login-0"]
       flavor = var.other_node_flavor
+      fip_addresses = ["45.135.58.69"]
     }
   }
   compute = {
│ Call to function "templatefile" failed: ../../site/tofu/inventory.tpl:29,49-64: Invalid index; The given key does not identify an element in this collection value..

@claudia-lola claudia-lola requested a review from sjpb October 3, 2025 09:37
@sjpb sjpb self-requested a review October 3, 2025 11:09
Copy link
Collaborator

@sjpb sjpb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sjpb sjpb merged commit b504f10 into main Oct 3, 2025
31 checks passed
@sjpb sjpb deleted the claudia-fip branch October 3, 2025 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Should expose FIPs via ansible
2 participants