From mboxrd@z Thu Jan  1 00:00:00 1970
From: Daniel =?utf-8?q?Weism=C3=BCller?= <daniel.weismueller@ipfire.org>
To: 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
Date: Fri, 20 Nov 2020 12:36:35 +0100
Message-ID: <5ef263d0-d1b5-f19b-e839-ce3dfc01933a@ipfire.org>
In-Reply-To: <B877A7BF-9A5E-4D47-8944-6950CE7C9DDC@ipfire.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============8460319900936322024=="
List-Id: <development.lists.ipfire.org>

--===============8460319900936322024==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

Am 20.11.2020 um 11:55 schrieb Michael Tremer:
> Hi,
>
>> On 20 Nov 2020, at 06:58, Daniel Weism=C3=BCller <daniel.weismueller(a)ipf=
ire.org> wrote:
>>
>> Hello,
>>
>> In my opinion it is sufficient to be able to set these parameters via comm=
and line.
> Why is that? IPFire is a distribution that is supposed to be managed entire=
ly by the web user interface.

Ok I'll revise the script.


>> It should only be made sure that the settings are persitend and not overwr=
itten by a reboot or the webif.
> They won=E2=80=99t 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 appar=
ently 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 th=
e configuration file. This either must be documented somewhere or the zonecon=
fig.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=C3=BCller <daniel.weismueller(a)i=
pfire.org> wrote:
>>>>
>>>> Signed-off-by: Daniel Weism=C3=BCller <daniel.weismueller(a)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-h=
otplug-bridges
>>>> index 33d6d65ba..7431377bb 100644
>>>> --- a/config/udev/network-hotplug-bridges
>>>> +++ b/config/udev/network-hotplug-bridges
>>>> @@ -81,6 +81,7 @@ MODE=3D"$(get_value "${ZONE}_MODE")"
>>>>
>>>> # The name of the virtual bridge
>>>> BRIDGE=3D"$(get_value "${ZONE}_DEV")"
>>>> +STP=3D"$(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}" =3D "on" ] && echo "stp_state 1")
>>>> 			#ip link set "${BRIDGE}" up
>>>> 		fi
>>>>
>>>> --=20
>>>> 2.28.0
>>>>

--===============8460319900936322024==--