-
Notifications
You must be signed in to change notification settings - Fork 87
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
Automate remaining manual steps #19
Comments
I don't see sandbox as the magic way to install Tinkerbell without having to learn about it. I think hiding too much from the installation process does not help with learning about Tinkerbell. As a tool, I am sure it tends to be backed by strong and opinioned ops people that I presume will have their own way to deliver Tinkerbell. |
Hmm, good point, but I think it depends on the objective. If it is to learn how to use Tinkerbell, then I think those steps could be automated. If the objective is to learn an example how to set up Tinkerbell, then I think existing documentation is correct. README.md currently says:
I'd argue, that automating remaining steps would make it quicker 😄 Feel free to close it though, if you think those steps should remain manual. But maybe it's worth having a better look of what value those steps brings to the Tinkerbell users then. |
That's just my point of view 😄 I am not sure it right or not, let's see if other people say. I am sure we can make it better for our goal and as you suggested we should find a good one 💯 |
@jacobweinstock - does #90 address this? |
Yes, I believe it does. Once #90 lands we can probably close this. |
Going to close this as #90 has landed. If there is still some concern here please do reopen. |
The following commands could be automated with Terraform, so after running
terraform apply
, user gets fully functional Tinkerbell deployment:This is what currently Lokomotive is doing: kinvolk/lokomotive@d61e9d1#diff-99208a178620f1e4104324f48efe5fb0701755df8a81ab7893b32e27a86fae21R240-R248
The text was updated successfully, but these errors were encountered: