[draft] Add load balancer server type

austinkregel commented 3 months ago

This PR aims to add a new server type for load balancing with nginx. Inspired in part by this discussion thread, and my personal usage of load balancing servers 😄

Goal: Deploy a Load Balancing server from the UI

Acceptance Criteria:

[x] (done) Have "load balancing" be a new server type, that configures only an nginx server
[ ] (in progress) Have all sites under a "load balancing" server change their "application" page to reflect the load-balancer configuration.
[ ] (todo) Only allow load balancing to privately networked servers. (I personally use Zerotier, when I'm not using DigitalOcean)
[ ] (todo) Allow configuring of the load balancing method (round robin, ip_hash, hash, last_conn)
[ ] (todo) Allow configuring the host and port of the servers we're balancing to.
[ ] (todo) Allow setting server as "backup" or as "down"

Optional concepts to explore:

Having the load balancing server auto-provision the site on servers you connect to. This would require defining the project repo, deployment script, php version, project directory, and any extra fields based on the site type when creating the site on the load balancing server.

Old Laravel Forge load balancing config page to convey basic page layout. Exact style, format, or design will vary.

rtraselbd commented 3 months ago

Go ahead, waiting for it.

austinkregel commented 3 months ago

I've fallen into a huge rabbit hole learning about Bison and language parsing 😵 I found an article about using bison and PHP to parse the Nginx config into AST and have been able to extend the project and I might be able to fully support all nginx directives :eyes:

With one of the goal being more advanced nginx config parsing and eventually editing so projects like Vito can work with any existing config.

With that said, I've lost ~~about 2 weeks in this hole 🙈~~ a little over half a week but it feels like longer 😂 I need a bit of a break to re-calibrate, and reset to tackle this specifically and not nginx parsing as a whole.

saeedvaziry commented 3 months ago

@austinkregel first of all thanks for this contribution, I am sure it will be a big update.

second, I am sure there is easier workarounds and personally don't think parsing nginx files is really needed.

Feel free to reach me on Discord to have a chat about it.

saeedvaziry commented 2 days ago

@austinkregel shall we close this PR?

vitodeploy / vito

[draft] Add load balancer server type #244