-
Notifications
You must be signed in to change notification settings - Fork 206
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
Add instructions for bootstrapping faasd on digitalocean #70
Conversation
Signed-off-by: Mehdi Yedes <mehdi.yedes@gmail.com>
Signed-off-by: Mehdi Yedes <mehdi.yedes@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you're OK with it, can we retain my gist as the non-TLS example, and add your example underneath?
could the folder dgitalocean-tls be renamed to digitalocean-terraform?
Signed-off-by: Mehdi Yedes <mehdi.yedes@gmail.com>
Done
|
@@ -56,7 +56,9 @@ You can run this tutorial on your Raspberry Pi, or adapt the steps for a regular | |||
|
|||
Automate everything within < 60 seconds and get a public URL and IP address back. Customise as required, or adapt to your preferred cloud such as AWS EC2. | |||
|
|||
* [Provision faasd 0.7.5 on DigitalOcean with Terraform 0.12.0](https://gist.github.com/alexellis/fd618bd2f957eb08c44d086ef2fc3906) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I meant not to delete this? Or did you just copy my gist into docs/bootstrap/README.md
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Correct, I have only updated the faasd version to 0.8.1
and copied it to docs/bootstrap/README.md
.
Is that ok, or would you prefer to keep it as a gist?
I thought we'd keep both options: TLS and non-TLS at the same place
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK thanks for explaining
|
||
1) [Sign up to DigitalOcean](https://www.digitalocean.com/?refcode=2962aa9e56a1&utm_campaign=Referral_Invite&utm_medium=Referral_Program&utm_source=CopyPaste) | ||
2) [Download Terraform](https://www.terraform.io) | ||
3) Clone this gist using the URL from the address bar |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just noticed I am still referring to the gist here. Should be changed to refer to the faasd repo instead.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please
Has anyone run through this to test for you? I think @utsavanand2 and then @Waterdrips both committed to doing that, but I haven't seen anything on the PR yet. |
Let's merge this, then it's easier for others to test. Could you also refer to the repo instead of the gist as per your comment above @mehyedes ? If you have other suggestions for changes please feel free to propose them too. |
Not yet unfortunately. |
Signed-off-by: Mehdi Yedes mehdi.yedes@gmail.com
Added instructions for provisioning a faasd instance on Digitalocean with or without TLS.
Description
This PR integrates the following gists:
into the faasd repository.
I have also updated the
README.md
file to include the guides.Motivation and Context
The issue is: Ability to bootstrap a production-ready faasd with TLS support #67
How Has This Been Tested?
Tested the provisioning with terraform on Digitalocean.
Types of changes
Checklist:
Commits:
git commit -s
for the Developer Certificate of Origin (DCO)Code:
Docs: