add reference(s) for quickstart guide; fix ansible playbook example#198
add reference(s) for quickstart guide; fix ansible playbook example#198pathcl wants to merge 6 commits intohostinger:mainfrom
Conversation
| firecracker: | ||
| binary_path: firecracker | ||
| kernel_image_path: /usr/local/share/firecracker/vmlinux.bin | ||
| # obtained from https://s3.amazonaws.com/spec.ccfc.min/firecracker-ci/v1.10/x86_64/vmlinux-5.10.223 |
There was a problem hiding this comment.
I think this will need separate documentation page, as we also create our own kernel images here. Need to define the differences, I'll handle this upcoming week, then You can update the reference here.
It should work with the one from Firecracker, even though some crucial options will be missing.
There was a problem hiding this comment.
Great! indeed I downloaded it from there and used it to test fireactions. Im curious what crucial options we would be missing?
There was a problem hiding this comment.
Hi, sorry for the wait, been busy. You can update it to this reference, change the PR title to match conventional commit guidelines (we have squash merge with title as commit message enabled) and it's good to go!
Regarding the differences between kernels - I really can't remember, should have documented when building:( But what I can remember is that it's built off the Firecracker kernel config. I think we had some issues with running Docker/Kubernetes workloads.
|
I'd like to give this PR a bump. The docs are still referencing the broken Playbook configuration. |
|
@konradasb Would you accept a rebased version of this from me? Or do you prefer the shell script that is in the repository root? If you prefer the script, I would open another docs PR. |
e16953d to
fb35e90
Compare
If https://fireactions.io/user-guide/installation/#step-2-create-an-ansible-playbook is used we'll have issues running it as an ansible playbook because of this line. Also is not explicit from where to obtain a kernel image && runner image.