Hi,
On 23 Nov 2020, at 15:13, Daniel Weismüller daniel.weismueller@ipfire.org wrote:
Hi Leo,
that pleases me to hear and I gladly accept your offer. ;-)
I quickly made a draft and attached it. As I said it is only a draft so there is still plenty of room for improvement.
Great drawing skills.
I would propose to add a second row for the priority because you might not fit it all into one cell. The “priority” label in that box probably isn’t a good idea.
The checkbox switches the variable named ${ZONE}_STP to 0 or 1. The input field fills the variable named ${ZONE}_STP_PRIORITY. Here must a number between 1 and 65535 inserted.
I would like the default to be enabled on new systems only.
Can we make that happen?
-Michael
Daniel
Am 21.11.20 um 17:39 schrieb Leo Hofmann:
Hi Daniel,
a few days ago I finally submitted my patches for zoneconf.cgi and I would now have time to work on this as well.
(By the way, I almost forgot, thanks @Michael for reviewing my patches!)
If you want me to take this on, it would be very helpful if you could summarize how this should work. For example, which config parameters need to be modified. Perhaps you could even paint a simple GUI mock-up like you did last time?
Regards, Leo
Am 20.11.2020 um 19:31 schrieb Daniel Weismüller:
OK. ;-)
The first step will be the introduction of the possibility to enable STP.
The next step will be the implementation in the webif.
I hope I find someone who can do that.
Daniel
Am 20.11.2020 um 16:18 schrieb Kienker, Fred:
I'm with Michael on this one. If it deserves to be in IPFire, it deserves to be on the web interface. Don't created exceptions which are only available from a command line.
Best regards, Fred
-----Original Message----- From: Michael Tremer michael.tremer@ipfire.org Sent: Friday, November 20, 2020 5:55 AM To: Daniel Weismüller daniel.weismueller@ipfire.org Cc: development@lists.ipfire.org Subject: Re: [PATCH] Core 152: the script "network-hotplug-bridges" now reads the variable ${ZONE}_STP from /var/ipfire/ethernet/settings so that STP can be turned on and off for each bridge
Hi,
On 20 Nov 2020, at 06:58, Daniel Weismüller
daniel.weismueller@ipfire.org wrote:
Hello,
In my opinion it is sufficient to be able to set these parameters via
command line.
Why is that? IPFire is a distribution that is supposed to be managed entirely by the web user interface.
It should only be made sure that the settings are persitend and not
overwritten by a reboot or the webif.
They wont be as they are in /var/ipfire/ethernet/settings.
Best, -Michael
Daniel
Am 19.11.2020 um 15:56 schrieb Michael Tremer:
Hello Daniel,
This patch looks good to me.
I had assumed that we automatically enabled STP on all bridges, but
apparently we do not.
How do we process with this?
I suppose it is not the most user-friendly way to ask the user to
edit the configuration file. This either must be documented somewhere or the zoneconfig.cgi script needs to be extended to allow enabling STP.
Does anyone want to be able to change any STP parameters like
priority or cost of the ports?
Best, -Michael
> On 19 Nov 2020, at 13:18, Daniel Weismüller
daniel.weismueller@ipfire.org wrote:
> Signed-off-by: Daniel Weismüller daniel.weismueller@ipfire.org > --- > config/udev/network-hotplug-bridges | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/config/udev/network-hotplug-bridges
b/config/udev/network-hotplug-bridges
> index 33d6d65ba..7431377bb 100644 > --- a/config/udev/network-hotplug-bridges > +++ b/config/udev/network-hotplug-bridges > @@ -81,6 +81,7 @@ MODE="$(get_value "${ZONE}_MODE")" > > # The name of the virtual bridge > BRIDGE="$(get_value "${ZONE}_DEV")" > +STP="$(get_value "${ZONE}_STP")" > > case "${MODE}" in > bridge) > @@ -89,7 +90,8 @@ case "${MODE}" in > > # We need to create the bridge if it doesn't exist, yet > if [ ! -d "/sys/class/net/${BRIDGE}" ]; then > - ip link add "${BRIDGE}" address "${ADDRESS}" type
bridge
> + ip link add "${BRIDGE}" address "${ADDRESS}" type
bridge \
> + $([ "${STP}" = "on" ] && echo "stp_state 1") > #ip link set "${BRIDGE}" up > fi > > -- > 2.28.0 >
<STP.png>