ben-grande / qusal

Salt Formulas for Qubes OS.
14 stars 6 forks source link

`dom0.port-forward` installed by default in `dom0` Salt scripts but used only by `sys-syncthing` #25

Closed peakunshift closed 4 months ago

peakunshift commented 4 months ago

Commitment

I confirm that I have read the following resources:

Current problem (if any)

dom0.port-forward script is installed by default in the init.sls's dom0 Salt installs, but seems to be currently used only by sys-syncthing, thus not necessary as a default install.

Proposed solution

Remove the port-forward line from init.sls. This is already present in syncthing's create script. I also propose to install it in the fetcher salt scripts, because it can be necessary to port-forward when torrenting.

The value to a user, and who that user might be

Install only what's necessary when necessary.

ben-grande commented 4 months ago

KDE is not necessary, but it is good to have it for usability reasons.

I can see that port forwarding only happens in some circumstances and I will remove it from the default installation.

I also propose to install it in the fetcher salt scripts, because it can be necessary to port-forward when torrenting.

It is true that port forwarding can be used in some formulas such as fetcher, sys-pihole, sys-cacher, but without documenting in the Usage section of the readme, I am not doing that currently because: