search

patte / fly-tailscale-exit

Run a VPN with global exit nodes with fly.io, tailscale and github!
https://news.ycombinator.com/item?id=36064305
1.57k stars 96 forks source link

Update to new Fly template naming scheme #5

Closed bslatyer closed 2 years ago

bslatyer commented 2 years ago

Hi @patte,

I've noticed that the new flyctl looks for Fly Templates now named fly.toml which causes the current fly-template.toml to not be found.

My suggestion would be to update all documentation and files to now refer to fly.toml instead of fly-template.toml

bslatyer commented 2 years ago

I can open a merge request if you want me to fix it?

patte commented 2 years ago

Sure you can. I’d love to review it.

bslatyer commented 2 years ago

No worries, I'll do it now.

bslatyer commented 2 years ago

Hey @patte

Still working on this.

bslatyer commented 2 years ago

Hey @patte,

It seems like Fly.io have deprecated the fly-template.toml naming and moved to fly.toml. My suggestions would be to inform users to change the app-name=tailwings to their own app created in Fly.io like tailscale-bs by using flyctl create. They would then use flyctl deploy to deploy their own app from the same folder once everything is ready.

This seems to work for me perfectly.

patte commented 2 years ago

done by: https://github.com/patte/fly-tailscale-exit/pull/17 and https://github.com/patte/fly-tailscale-exit/pull/19