Skip to content
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

additional_userdata script needs '#!/bin/sh at top to work with AL2023 AMI #317

Open
1 task done
canpmh opened this issue Jan 13, 2025 · 0 comments · May be fixed by #320
Open
1 task done

additional_userdata script needs '#!/bin/sh at top to work with AL2023 AMI #317

canpmh opened this issue Jan 13, 2025 · 0 comments · May be fixed by #320
Labels
bug Something isn't working new Added to an issue when it's new ;)

Comments

@canpmh
Copy link
Collaborator

canpmh commented Jan 13, 2025

Terraform Version Details

No response

Terraform Variable File Details

No response

Steps to Reproduce

update to an AMI using AL2023 for the compute node pool

include "custom_data" = "files/custom-data/additional_userdata.sh" in compute node pool definition within terraform.tfvars

Expected Behavior

NVME drives should mount and be available at the /nvme/disk symlink.

Actual Behavior

cloud-init won't run the scripts-user module during config, it fails with this error

Exec format error. Missing #! in script

Googling a bit I came across this post,

which suggests we simply need to add #!/bin/sh to the top of our additional_userdata.sh script in order for cloud-init to execute it. This wasn’t necessary on AL2 images, but apparently is needed for AL2023.

Additional Context

No response

References

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@canpmh canpmh added bug Something isn't working new Added to an issue when it's new ;) labels Jan 13, 2025
joshcoburn added a commit to joshcoburn/viya4-iac-aws that referenced this issue Jan 31, 2025
@joshcoburn joshcoburn linked a pull request Jan 31, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working new Added to an issue when it's new ;)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant