ghatanothoa: Neues Jitsi #17

Merged
nerf merged 3 commits from Gonne/nixConfig:ghatanothoa into main 2023-10-23 18:14:57 +00:00
Owner
No description provided.
Gonne added 2 commits 2023-10-20 21:13:00 +00:00
nerf requested changes 2023-10-21 09:21:11 +00:00
nerf left a comment
Owner

We probably want additional persistence of /var/lib/jisti-meet and /var/lib/prosody

Also is jitsi tied to hard to the machine? Can't we make a role out of it. Or a module with a little amount of options?
If that is possible I would prefer that.

We probably want additional persistence of `/var/lib/jisti-meet` and `/var/lib/prosody` Also is jitsi tied to hard to the machine? Can't we make a role out of it. Or a module with a little amount of options? If that is possible I would prefer that.
@ -0,0 +3,4 @@
imports = [
./hardware-configuration.nix
(import ./jitsi.nix flake-inputs)
Owner

jitsi.nix does not use flake-inputs ever, why not scratch that parameter from the module and don't apply it here

jitsi.nix does not use flake-inputs ever, why not scratch that parameter from the module and don't apply it here
nerf marked this conversation as resolved
@ -0,0 +1,23 @@
flake-inputs:
Owner

We never use this input, so why not scratch it

We never use this input, so why not scratch it
nerf marked this conversation as resolved
@ -0,0 +19,4 @@
services.nginx.virtualHosts."meet.mathebau.de".enableACME = false;
services.nginx.virtualHosts."meet.mathebau.de".forceSSL = false;
networking.firewall.allowedTCPPorts = [ 80 443 ];
networking.firewall.allowedUDPPorts = [ 10000 ];
Owner

isn't this handled by services.jitsi-videobridge.openFirewall = true; If it is, I would not want to have that again explicitly. (or at least a comment why we want to set it explicitly again). (Or maybe I'm wrong)

isn't this handled by `services.jitsi-videobridge.openFirewall = true;` If it is, I would not want to have that again explicitly. (or at least a comment why we want to set it explicitly again). (Or maybe I'm wrong)
nerf marked this conversation as resolved
Gonne was assigned by nerf 2023-10-21 09:24:21 +00:00
Gonne added 1 commit 2023-10-23 15:43:37 +00:00
Gonne force-pushed ghatanothoa from 41c4af643d to 4b0c3f7948 2023-10-23 15:59:55 +00:00 Compare
nerf requested changes 2023-10-23 16:20:50 +00:00
@ -0,0 +15,4 @@
enable = mkEnableOption "mathebau jitsi service";
hostName = mkOption {
type = str;
default = null;
Owner

I think setting default = null; is not the same as setting no default.
I guess you want no default so just delete the line.
(See lengthy reference)

I think setting `default = null;` is not the same as setting no default. I guess you want no default so just delete the line. (See [lengthy reference](https://nixos.org/nixos/manual/#sec-writing-modules))
Gonne marked this conversation as resolved
Gonne force-pushed ghatanothoa from 4b0c3f7948 to bb93d3aed5 2023-10-23 17:06:09 +00:00 Compare
nerf approved these changes 2023-10-23 18:14:26 +00:00
nerf merged commit bb93d3aed5 into main 2023-10-23 18:14:57 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: Fachschaft/nixConfig#17
No description provided.