Skip to content

Conversation

@benie-joy-possi
Copy link
Contributor

@benie-joy-possi benie-joy-possi commented Oct 16, 2025

description here: #26

Copy link

@marcjazz marcjazz left a comment

Choose a reason for hiding this comment

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

Group your script files under same directory.

Copy link
Contributor

@Franck-Sorel Franck-Sorel left a comment

Choose a reason for hiding this comment

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

Looks Okay !

@benie-joy-possi
Copy link
Contributor Author

@Franck-Sorel I don't quite get your question. Kindly ellaborate please!

- Add --auto-approve for non-interactive execution
- Moved all shell scripts to a new  directory
- Updated path to k_server_customization_script.sh in ami_builder.tf
- Added comprehensive README.md in scripts directory
Copy link
Contributor

@stephane-segning stephane-segning left a comment

Choose a reason for hiding this comment

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

Wow! Did you tested it already on any aws account?

@benie-joy-possi
Copy link
Contributor Author

Not yet @stephane-segning just did some local testing on my k8s cluster!

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.

5 participants