feat: Add variable to allow userdata to be passed as string to aws launch template

[rashidnhm]

Fix for the issue mentioned in #1960. This PR enables the passing of userdata script as a string to the module.

If both variables userdata_string and userdata_template are passed, then userdata_string would take precedence.

[npalm]

Note: check conflicts with #1956

[rashidnhm]

Note: check conflicts with #1956

Sounds good, I'll rebase my changes on top of that PR

[ScottGuymer]

As this is a quite advanced usecase I think this needs an example to show a potential user how to do this and call out some of the things they need to consider along the way.

[rashidnhm]

I am also open to adding a userdata example if you'd like. Let me know.

[ScottGuymer]

Yes, I think an example of how to use this is warranted. With some notes in the readme to guide the user when they might want to use it and what issues there could be.

[npalm]

@rashidnhm please can you rebase the PR. We have updated the module last week to AWS terraform provide 4.x

[ScottGuymer]

I think we are just missing the example now.

[rashidnhm]

I think we are just missing the example now.

Sounds good, currently working on getting that ready

[rashidnhm]

Hello, @ScottGuymer @npalm wondering if you got a chance to comb through this PR yet?

[npalm]

Will put it on my list, sorry the PR got out of sight. Please can you have a look on the failing CI, some formatting is most likely the issue.

[rashidnhm]

@npalm Sorry about the late reply, I've fixed the formatting issue. Please take a look and let me know.

[rashidnhm]

@npalm Just checking in to see if you got a chance to comb through this?

[npalm]

not yet, hope to check all the open PR's this week

[github-actions]

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed if no further activity occurs. Thank you for your contributions.

[github-actions]

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed if no further activity occurs. Thank you for your contributions.

[rashidnhm]

@npalm I will try to update this PR soon and pull in all latest changes, will you have time to review anytime soon?

[rashidnhm]

@npalm the reason I had this PR is to add functionality myself for having multiple runners. However, now that #2472 is in progress, this is no longer needed on my end.

[npalm]

I think in that case we should close, thanks fro reaching out