Disable foreman proxy TFTP support by default

Currently the installer enabled TFTP support by default, but I think this doesn’t make a huge amount of sense. Just TFTP doesn’t enable any feature. AFAIK you need DHCP for bare metal boot.

Am I missing anything or can we disable it by default?

On a technical level it’d flip true to false here:

2 Likes

+1 for disabling, I guess it was enabled since it’s easy to install and usually does not interfere with other services on the same network

As long as the upgrade won’t disable TFTP for everyone, I am good.

This would be limited to new installations. Touching existing installations is something I avoid when possible.

1 Like

We rely on the default true as out install requires DHCP+TFTP. Default False is not a deal breaker for us as long as an option override is supported.

https://projects.theforeman.org/issues/30198

To be clear, this only affects defaults. That means only fresh installations when using the installer. Direct users of the Puppet module also need to modify their setup.

I’ll also go over the manual to see where we need to suggest --foreman-proxy-tftp true and write an upgrade warning.

1 Like