From mboxrd@z Thu Jan 1 00:00:00 1970 From: Adolf Belka To: development@lists.ipfire.org Subject: Testing of rapid_commit patch Date: Fri, 29 Nov 2024 12:21:11 +0100 Message-ID: MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============5325588823860964796==" List-Id: --===============5325588823860964796== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Hi Michael, I had previously tested your rapid_commit patch on my vm network system. Howe= ver as that network does not have a problem with the rapid_commit option it w= as not the best test. I said that i would test it out on my production system after the visit to my= family. I have now got around to that. My new ISP, that bought out the very good technically based ISP I used to hav= e, is one of the ones that has a DHCP server that just times out if you have = the rapid_commit option in the dhcpcd config file. So I updated my production IPFire to Core-Update 190 Development Build: maste= r/1e2abd66 After rebooting the connection was lost as the rapid_commit option is on by d= efault. I then ran the setup program and unchecked the rapid commit box on the red in= terface and then rebooted and the connection=C2=A0 was running. Another reboot still had the connection running. So I can confirm that the=C2=A0 patch is working . The only thing that came to mind is when we do the update people who have rem= oved or commented out the rapid_commit option in dhcpcd.conf will lose their = connection. So either * we need to have the update check if the rapid_commit option has been comm= ented out or has been removed and then disable the rapid commit option for th= e red interface. or * we need to make sure that it is well communicated that anyone who has dis= abled the rapid_commit option needs to run the setup program after the upgrad= e (maybe before the reboot) to disable the option for the red interface. We can discuss this at our next conf call meeting on 3rd or 4th Dec. Regards, Adolf. --===============5325588823860964796==--