create login_startup_script

[AlexeySaff]

add login_startup_script similar to controller_startup_script.

the script to execute on login node creation.

[AlexeySaff]

I tried something like this, but it doesn't work

• id: runner-startup
  source: modules/scripts/startup-script
  settings:
  runners:
  - type: shell
  destination: "runner-startup.sh"
  content: |
  #!/bin/sh
  echo "Hello world"

• id: slurm_login
  source: community/modules/scheduler/schedmd-slurm-gcp-v6-login
  use:

  • network
  • runner-startup

[AlexeySaff]

@kvenkatachala333 possible to take a look?

[kvenkatachala333]

Hi @AlexeySaff Apologies for the delay.

looked into it and found that the schedmd-slurm-gcp-v6-login module was missing the startup_script input variable, which is why the blueprint generator was silently dropping the runner-startup attachment. Furthermore, the controller module needed to be updated to accept and inject these scripts for login nodes.

I've just opened a Draft PR that fixes this by properly plumbing the startup_script variable through the login module and into the controller's login node schema - #5800

Please let me know if these changes looks good to you?

[AlexeySaff]

That's great, but does it mean, it will recreate the existing controller?

[kvenkatachala333]

No, it will not recreate your existing controller.

The startup_script input variable is scoped explicitly to the login node configurations (login_nodes). Terraform will only update the instance template for the login nodes and apply those changes. The controller instance template and VM remain completely isolated from this change and will be untouched

[AlexeySaff]

@kvenkatachala333 any ideas what release it can be included?