-
Notifications
You must be signed in to change notification settings - Fork 137
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
Give known configuration for tinkerbell on a Packet host #70
Comments
Provided sample vars works fine for default Ubuntu 18.04 packet installation. The issue is in the script itself: @gauravgahlot already updated the script to use ifconfig and I just tested it so I can confirm that supplied example vars works fine. Feel free to test yourself and let me know if it works for you too |
Thanks for helping with this. Can you elaborate on what you mean by "it works fine"? When I ran through the steps and it did not? Do you mean it will work fine when #62 is merged (or now that it's been updated)? |
By "it works fine" I mean you no longer hit "unknown interface" error and script will fail on a later step (logging to docker registry) due to different error. And I mean when #62 gets merged of course. But I'm seeing few more issues with networking which I guess makes docker login fail, so while the example configuration is fine - it won't work until all the issues in the script get resolved. |
Thank you for clarifying, so I am not going crazy 😄 |
@alexellis I tested the script for CentOS 7 and Ubuntu 18.04 on:
Executed the script with: $ curl https://raw.githubusercontent.com/infracloudio/tink/deploy_stack/setup.sh | bash Added the same to PR description and docs as well: |
I'm trying it now. |
I can confirm that the script completed, but there is't a clear next action.
I see that the tink CLI's source code is downloaded under tink/cli/tink, but it's not built? Do you plan on downloading a release binary or building as part of the script so that users can make use of it after bootstrap? A help message with what to do next would also be a good addition at the end of the script. |
This error is also confusing / unactionable. What is the value when used with compose? It's not documented here -> https://github.com/tinkerbell/tink/blob/master/docs/first-good-workflow.md or here -> https://github.com/tinkerbell/tink/blob/master/docs/setup.md
|
I think you still need to use tink from |
As soon as the PR #62 is merged, I will be sending another one with changes to first-good-workflow. And accordingly will also add a "next" section at the end of setup script. |
@alexellis Could you comment on the remaining delta between this request and what you now see at https://tinkerbell.org/setup please? We believe @kdeng3849 and others may have closed the gap but want to be sure we haven't missed anything out. |
@gauravgahlot #62 is merged now, did you have time to raise the PR you mentioned for the "next action" section at the end of the script? |
@alexellis I had updated the @mrmrcoleman WRT https://tinkerbell.org and the issue, I have done work. Since we could not take that up in the call today, I will be sending a few details on the channel. |
Thanks @gauravgahlot I would check @jacobsmith928 's comment in Slack about that issue, as his team are already working on updating the website. |
Additional feedback given on tinkerbell/tinkerbell.org#6 |
@alexellis Since we have other issues that describe current challenges with the setup, can we close this one now? |
This is covered now. At the moment setup.sh and the terraform both work and cover this. |
I would like to see a known configuration for the installer script in #62 which provides everything needed without any guess-work or troubleshooting.
I.e. "run these commands" and it results in Tinkerbell being up.
cc @gauravgahlot
A comment on this issue should be sufficient to get started, but the above should be in the docs eventually.
The text was updated successfully, but these errors were encountered: