feat: add deployment solution to digital ocean app platform

asyncapi / server-api

Server API providing official AsyncAPI tools

Apache License 2.0

31 stars 33 forks source link

feat: add deployment solution to digital ocean app platform #430

Closed Shurtu-gal closed 10 months ago

Shurtu-gal commented 11 months ago

Description

Add workflow for deployment to digital ocean droplets
Remove k8s helm chart files
Add digital ocean terraform files

Proof that it works

The site is hosted at http://142.93.209.65/v1/help/validate
Successful workflow run:- https://github.com/ash17290/server-api/actions/runs/7476130464/job/20345827991

Things to be noted by the maintainer

You have to add a public ssh key in https://github.com/Shurtu-gal/server-api/blob/feat/droplet/deployments/droplets/server-api.yaml#L14
Install terraform
Run terraform init and terraform apply
Copy the ipv4 from output.
Add the following secrets:
- HOST: ipv4 address of container
- USERNAME: asyncapi
- PORT: 22
- KEY: Full Private Key for the public ssh key

Note: this is a single-time process.

Related issue(s) Fixes #317

Shurtu-gal commented 11 months ago

@derberg A small review of GitHub workflow would be appreciated.

smoya commented 11 months ago

Comment moved to https://github.com/asyncapi/server-api/issues/317#issuecomment-1886993658

Shurtu-gal commented 10 months ago

@smoya @derberg The PR has been updated.

Successful workflow run :- https://github.com/ash17290/server-api/actions/runs/7525186615/job/20481139953 Deployed Link :- https://server-api-ek54f.ondigitalocean.app/v1/help/validate

asyncapi-bot commented 10 months ago

Hello, @Shurtu-gal! 👋🏼

    I'm 🧞🧞🧞 Genie 🧞🧞🧞 from the magic lamp. Looks like somebody needs a hand!

    At the moment the following comments are supported in pull requests:

    - `/ready-to-merge` or `/rtm` - This comment will trigger automerge of PR in case all required checks are green, approvals in place and do-not-merge label is not added
    - `/do-not-merge` or `/dnm` - This comment will block automerging even if all conditions are met and ready-to-merge label is added
    - `/autoupdate` or `/au` - This comment will add `autoupdate` label to the PR and keeps your PR up-to-date to the target branch's future changes. Unless there is a merge conflict or it is a draft PR.

smoya commented 10 months ago

Digital Ocean Application has been depoyed successfully using the terraform file @Shurtu-gal created. @fmvilas added the proper CNAME record into api.asyncapi.com to server-api-84f2u.ondigitalocean.app, which is the internal Digital Ocean domain for server-api app.

All seems working properly, the transition from K8s to App Platform happened smoothly. BTW, we do not have K8s on Digital Ocean anymore 🎉 cc @derberg

Now I'm gonna proceed configuring the env vars for the GH action to work.

smoya commented 10 months ago

Now I'm gonna proceed configuring the env vars for the GH action to work.

Ok, this is embarrassing but I have no powers for setting secrets for the CI. Would you @derberg @fmvilas help me? Thanks!

smoya commented 10 months ago

@Shurtu-gal Do you believe you could update the docs of this repo mentioning how the deploy of the infra works, and how deploys of the app work as well? Can be done in a different PR, not affecting in this case to the Bounty's timeline unless you want to do it here.

Shurtu-gal commented 10 months ago

Sure, I can do it here itself.

Shurtu-gal commented 10 months ago

@smoya done the readme update.