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

Allow user to override node bootstrap script #522

Merged
merged 2 commits into from
Feb 8, 2019
Merged

Conversation

errordeveloper
Copy link
Contributor

@errordeveloper errordeveloper commented Feb 7, 2019

Description

This should close #69. It introduces a way for user to override the entier bootstrap.{al2,ubuntu}.sh with their own version.

Checklist

  • Code compiles correctly (i.e make build)
  • Added tests that cover your change (if possible)
  • All unit tests passing (i.e. make test)
  • All integration tests passing (i.e. make integration-test)

@errordeveloper errordeveloper changed the title WIP: custom bootstrap script Allow user to override node bootstrap script Feb 7, 2019
jrryjcksn
jrryjcksn previously approved these changes Feb 7, 2019
Copy link
Contributor

@jrryjcksn jrryjcksn left a comment

Choose a reason for hiding this comment

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

👍

This should allow those who _know what their are doing_ to pass
a custom script to run inside of cloud-init. Anyone looking to
use this will need to defined a shell script that would resemble
`bootstrap.{al2,ubuntu}.sh`. This should cater for any of advanced
use-cases, where additional files must be created on EC2 instances
prior to boot or anything of that nature. Alternative approach is
to use a custom AMI, or propose an eksctl feature, but this is
meant as a stop-gap for testing things out.
@errordeveloper
Copy link
Contributor Author

@jrryjcksn would you mind to re-approve this please?

Copy link
Contributor

@jrryjcksn jrryjcksn left a comment

Choose a reason for hiding this comment

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

Looks much better

@mickeyhovel
Copy link

Not sure I have understood how should I provide to the eksctl command the user data I want to be used in the worker node

torredil pushed a commit to torredil/eksctl that referenced this pull request May 20, 2022
…verwrite

Allow volume attach limit overwrite via command line parameter
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.

Allow additional user-data on EKS nodes
3 participants