luci icon indicating copy to clipboard operation
luci copied to clipboard

luci-app-keepalived: Add LuCI for keepalived

Open jempatel opened this issue 3 years ago • 2 comments

LuCI Support for Keepalived

We also need https://github.com/openwrt/packages/pull/19329 We also need https://github.com/openwrt/packages/pull/19374

Screenshots from the new Luci App Keepalived:

Overview (Master): 01-OpenWrt-Overview-LuCI-Master

Overview (Backup): 02-OpenWrt-Overview-LuCI-Backup

Globals: 03-OpenWrt-Globals-LuCI

IP Address: 04-OpenWrt-IP-Address-LuCI

IP Address (Add/Edit): 05-OpenWrt-IP-Address-Add-Edit-LuCI

Static IP Address (Add/Edit): 06-OpenWrt-Static-IP-Address-Add-Edit-LuCI

Route: 07-OpenWrt-Route-LuCI

Route (Add/Edit): 08-OpenWrt-Route-Add-Edit-LuCI

Static Route (Add/Edit): 09-OpenWrt-Static-Route-Add-Edit-LuCI

URLs: 10-OpenWrt-URLs-LuCI

URLs (Add/Edit): 11-OpenWrt-URLs-Add-Edit-LuCI

Scripts: 12-OpenWrt-Scripts-LuCI

Scripts (Add/Edit): 13-OpenWrt-Scripts-Add-Edit-LuCI

Track Scripts (Add/Edit): 14-OpenWrt-Track-Scripts-Add-Edit-LuCI

Interfaces: 15-OpenWrt-Interfaces-LuCI

Interfaces (Add/Edit): 16-OpenWrt-Interfaces-Add-Edit-LuCI

Instance: 17-OpenWrt-Instance-LuCI

Instance (General): 18-OpenWrt-Instance-General-LuCI

Instance (Peer): 19-OpenWrt-Instance-Peer-LuCI

Instance (Tracking): 20-OpenWrt-Instance-Tracking-LuCI

Instance (GARP): 21-OpenWrt-Instance-GARP-LuCI

Instance (Advanced): 22-OpenWrt-Instance-Advanced-LuCI

Servers: 23-OpenWrt-Servers-LuCI

Real Servers (Add/Edit): 24-OpenWrt-Real-Servers-Add-EditLuCI

Virtual Servers (General): 25-OpenWrt-Virtual-Servers-General-LuCI

Virtual Servers (Advanced): 26-OpenWrt-Virtual-Servers-Advanced-LuCI

Sync Group: 27-OpenWrt-Sync-Group-LuCI

Sync Group (Add/Edit): 28-OpenWrt-Sync-Group-Add-Edit-LuCI

jempatel avatar Sep 08 '22 04:09 jempatel

A new page for peers is added.

Peers Sender/Master Node: OpenWrt-Peers-LuCI

Peers Receiver/Backup Node: OpenWrt-Peers-LuCI2

jempatel avatar Sep 16 '22 21:09 jempatel

@feckert @jow- If all comments are addressed, can we merge this?

jempatel avatar Oct 17 '22 19:10 jempatel

Thanks for your work! Merged :+1:

The only other thing I noticed. The following lines always appear in the log! If I am on the status page.


Fri Oct 28 12:37:24 2022 daemon.info Keepalived_vrrp[17362]: Printing VRRP as json for process(17362) on signal
Fri Oct 28 12:37:29 2022 daemon.info Keepalived_vrrp[17362]: Printing VRRP as json for process(17362) on signal
Fri Oct 28 12:37:29 2022 daemon.info Keepalived_vrrp[17362]: Printing VRRP as json for process(17362) on signal
Fri Oct 28 12:37:34 2022 daemon.info Keepalived_vrrp[17362]: Printing VRRP as json for process(17362) on signal

This spams my log! I think this should get fixed in the keepalived packages? Maybe by change the log level? https://github.com/acassen/keepalived/blob/master/keepalived/core/main.c#L1794

feckert avatar Oct 28 '22 10:10 feckert

Thanks for your work! Merged +1

The only other thing I noticed. The following lines always appear in the log! If I am on the status page.


Fri Oct 28 12:37:24 2022 daemon.info Keepalived_vrrp[17362]: Printing VRRP as json for process(17362) on signal
Fri Oct 28 12:37:29 2022 daemon.info Keepalived_vrrp[17362]: Printing VRRP as json for process(17362) on signal
Fri Oct 28 12:37:29 2022 daemon.info Keepalived_vrrp[17362]: Printing VRRP as json for process(17362) on signal
Fri Oct 28 12:37:34 2022 daemon.info Keepalived_vrrp[17362]: Printing VRRP as json for process(17362) on signal

This spams my log! I think this should get fixed in the keepalived packages? Maybe by change the log level? https://github.com/acassen/keepalived/blob/master/keepalived/core/main.c#L1794

Thanks for the reference. I will keep that on my to-do with improvements, in case If find it after testing from the master branch

jempatel avatar Oct 28 '22 12:10 jempatel