search

uselagoon / lagoon-sync

Apache License 2.0
7 stars 5 forks source link

Fix defaults for postgres config so that local config override is not needed #101

Closed rocketeerbkw closed 1 year ago

rocketeerbkw commented 1 year ago

closes #99

rocketeerbkw commented 1 year ago

We use drupal as the default for mariadb, so I'd say keep them in sync either way https://github.com/uselagoon/lagoon-sync/blob/main/synchers/mariadb.go#L43

tobybellwood commented 1 year ago

We use drupal as the default for mariadb, so I'd say keep them in sync either way https://github.com/uselagoon/lagoon-sync/blob/main/synchers/mariadb.go#L43

Good point, that's an oversight. Agree to keep them in sync then, but should work out a better way for both.

bomoko commented 1 year ago

@rocketeerbkw and @tobybellwood - this seems good to me, let me know if you're happy for me to merge.