search

tiny-pilot / ansible-role-tinypilot

DEPRECATED: Merged into core tiny-pilot/tinypilot repo
https://tinypilotkvm.com
MIT License
117 stars 37 forks source link

Remove legacy nginx vhosts cleanup #253

Closed jotaen4tinypilot closed 1 year ago

jotaen4tinypilot commented 1 year ago

Originally per 4e5ffeed7260c1dc18e246cc0ec0f20b2359cfdd (see https://github.com/tiny-pilot/ansible-role-tinypilot/pull/244), but then reverted per https://github.com/tiny-pilot/ansible-role-tinypilot/pull/245. Now, with https://github.com/tiny-pilot/gatekeeper/issues/83 resolved, we can go ahead with this.

Original issue description

Resolves https://github.com/tiny-pilot/ansible-role-tinypilot-pro/issues/132

This PR depends on first resolving https://github.com/tiny-pilot/gatekeeper/issues/83

Now that TinyPilot Community/Pro users are forced to upgrade to 1.8.0/2.5.0 (before upgrading any further), we can remove legacy cleanup code.

The ansible-role-tinypilot-pro version of this PR (i.e., merging community changes) would be to remove this ansible task.

Review on CodeApprove

jotaen4tinypilot commented 1 year ago

(Sorry @jdeanwallace for “stealing” the git credits… 🙈 )