uselagoon / lagoon

Lagoon, the developer-focused application delivery platform
https://docs.lagoon.sh/
Apache License 2.0
560 stars 149 forks source link

FIX: fixes migration deletion logic for duplicate environment_services #3705

Closed bomoko closed 6 months ago

bomoko commented 6 months ago

This PR changes the logic of the existing migration (described in #3704 ) This fix is done in the actual migration because

  1. If the current migration has already been run, there is nothing for a brand new migration to do
  2. If the current migration hasn't been run, running it will cause data loss. We could no-op the existing migration, but that would just mean a second migration with complex logic to determine which version of the current migration (broken or no-op) had been run.

This closes #3704