Sorry, I just overlooked them.

I merged them and pushed them just now.

-Michael

> On 14 Sep 2025, at 11:11, Adolf Belka <[email protected]> wrote:
> 
> Hi Michael,
> 
> On 08/09/2025 15:07, Adolf Belka wrote:
>> Hi All,
>> On 03/09/2025 20:21, Adolf Belka wrote:
>>> Hallo All.
>>> 
>>> I have found another issue with OpenVPN on CU197 Testing.
>>> 
>>> The RW graphs Statistics graphs are not getting updated.
>>> 
>>> After some searching around, I found that it was because the collectd.vpn 
>>> plugin was still saying /var/log/ovpnserver.log
>>> 
>>> I discovered that there was an update to the collectd.vpn file but collectd 
>>> was not shipped in CU197 Testing so it didn't get included.
>>> 
>>> I have submitted a patch to ship collectd in CU197 Testing.
>> I have dropped that patch in patchwork.
>> I realised that doing that would update the ovpnserver.log to openvpn-rw.log 
>> but it would also remove any n2n entries in collectd.vpn
>> I have submitted two new patches. One for update.sh in CU197, which after 
>> running the ovpnmain.cgi file checks if collectd.vpn has the old log file 
>> name in it and if so replaces it with the new version.
>> The other patch is to do the same thing in backup.pl for any restores from 
>> old backups before CU197.
>> This makes sure that the RW entry is updated but leaves any existing n2n 
>> entries alone in collectd.vpn.
>> The above was tested and resulted in working graphs for the RW connection.
> 
> The two new patches mentioned above
> 
> https://lists.ipfire.org/development/[email protected]/T/#t
> 
> have not been merged yet. Have they just been missed or is there a question 
> about them?
> 
> Regards,
> 
> Adolf.
> 
>> Regards,
>> Adolf.
>>> 
>>> Evaluated the change in my vm and the rw graphs worked again.
>>> 
>>> Regards,
>>> 
>>> Adolf.



Reply via email to