Disable postgres on bekkalokk #106

Closed
opened 2024-08-06 19:39:36 +02:00 by oysteikt · 3 comments
Owner

lmao

lmao
oysteikt added the
nixos
bug
good first issue
labels 2024-08-06 19:39:49 +02:00
oysteikt added this to the Kanban project 2024-08-06 19:42:37 +02:00
oysteikt changed title from Disable postgres on bekkalokk to Disable postgres on `bekkalokk` 2024-08-06 19:58:16 +02:00
Author
Owner

The correct way to fix this would be to open a PR that fixes this assumption: f008a5b261/nixos/modules/services/misc/gitea.nix (L472-L481)

Yes, we'd like the module to create the database if it doesn't exist, yes we'd like to use postgres, no we're not using it on the same machine.

The quick and dirty way would be services.postgresql.enable = lib.mkForce false;

The correct way to fix this would be to open a PR that fixes this assumption: https://github.com/NixOS/nixpkgs/blob/f008a5b2616de80a25e705e9019d9b7283faab8b/nixos/modules/services/misc/gitea.nix#L472-L481 Yes, we'd like the module to create the database if it doesn't exist, yes we'd like to use postgres, no we're not using it on the same machine. The quick and dirty way would be `services.postgresql.enable = lib.mkForce false;`
Owner

I think the nixos-module behavior is correct, we should be setting createDatabase = false ensureUsers and siblings obviously won't work with an external database.

An upstream PR could add an assertion or set a more reasonable default than true when database-settings are set though

EDIT: But we do set createDatabase = false wtf?

I think the nixos-module behavior is correct, we should be setting `createDatabase = false` `ensureUsers` and siblings obviously won't work with an external database. An upstream PR could add an assertion or set a more reasonable default than `true` when database-settings are set though EDIT: But we do set `createDatabase = false` wtf?
Owner

postgresql is enabled in roundcube lol

postgresql is enabled in roundcube lol
Sign in to join this conversation.
No description provided.