[acassen/keepalived] 99f87e: Don't zero the mem_allocated count during reload

pqarmitage <[email protected]>
Newsgroups gmane.linux.keepalived.devel
Message-ID <5829866be2215_1a1e3fd62ad7f13c565f@hookshot-fe3-cp1-prd.iad.github.net.mail>
  Branch: refs/heads/master
  Home:   https://github.com/acassen/keepalived
  Commit: 99f87e2e582e94f493fa39a62d1f3338f621319d
      https://github.com/acassen/keepalived/commit/99f87e2e582e94f493fa39a62d1f3338f621319d
  Author: Quentin Armitage <[email protected]>
  Date:   2016-11-13 (Sun, 13 Nov 2016)

  Changed paths:
    M keepalived/check/check_daemon.c
    M keepalived/vrrp/vrrp_daemon.c

  Log Message:
  -----------
  Don't zero the mem_allocated count during reload

We want to know if there is a leak during reload, so don't zero the
counter.

Signed-off-by: Quentin Armitage <[email protected]>


  Commit: 98aefe14c405daaa0695eae3baf7af205031b07b
      https://github.com/acassen/keepalived/commit/98aefe14c405daaa0695eae3baf7af205031b07b
  Author: Quentin Armitage <[email protected]>
  Date:   2016-11-13 (Sun, 13 Nov 2016)

  Changed paths:
    M keepalived/include/vrrp_ipset.h
    M keepalived/include/vrrp_iptables.h
    M keepalived/vrrp/vrrp.c
    M keepalived/vrrp/vrrp_daemon.c
    M keepalived/vrrp/vrrp_ipaddress.c
    M keepalived/vrrp/vrrp_ipset.c
    M keepalived/vrrp/vrrp_iptables.c

  Log Message:
  -----------
  Ensure iptables/sets entries and ip routes/rules not lost on reload

There were several places in the code that were causing existing
iptables/ipsets entries to be lost on reload, and also new entries
for additional ip addresses were deleted after being added. In
addition, ip rules/routes for existing entries were being removed.

Signed-off-by: Quentin Armitage <[email protected]>


  Commit: 6f03ddeda50b966b3265f8d983e8a91ece23adf6
      https://github.com/acassen/keepalived/commit/6f03ddeda50b966b3265f8d983e8a91ece23adf6
  Author: Quentin Armitage <[email protected]>
  Date:   2016-11-13 (Sun, 13 Nov 2016)

  Changed paths:
    M keepalived/include/vrrp_ipaddress.h
    M keepalived/vrrp/vrrp.c
    M keepalived/vrrp/vrrp_ipaddress.c

  Log Message:
  -----------
  Ensure GARPs/GNAs are sent after reload if VIP/eVIP addresses added

Although there have been versions of keepalived when GARPs/GNAs were
sent after a reload, this was due to a bug in determining if the VRRP
instance had existed before. Resolving that bug (commit aaea8a4),
caused keepalived to stop sending GARPs after a reload. This commit
now specifically adds code to send GARPs on a VRRP instance for all
addresses on that instance. It would be better if GARPs were sent only
for the added addresses, and that may be resolved in a future commit.

Signed-off-by: Quentin Armitage <[email protected]>


  Commit: 03adb03401fee173ac8a4819fb70af2584361a44
      https://github.com/acassen/keepalived/commit/03adb03401fee173ac8a4819fb70af2584361a44
  Author: pqarmitage <[email protected]>
  Date:   2016-11-14 (Mon, 14 Nov 2016)

  Changed paths:
    M keepalived/check/check_daemon.c
    M keepalived/include/vrrp_ipaddress.h
    M keepalived/include/vrrp_ipset.h
    M keepalived/include/vrrp_iptables.h
    M keepalived/vrrp/vrrp.c
    M keepalived/vrrp/vrrp_daemon.c
    M keepalived/vrrp/vrrp_ipaddress.c
    M keepalived/vrrp/vrrp_ipset.c
    M keepalived/vrrp/vrrp_iptables.c

  Log Message:
  -----------
  Merge pull request #452 from pqarmitage/fixes

Fix sending GARPs after reload, and also not losing iptables or ip rules/routes entries


Compare: https://github.com/acassen/keepalived/compare/5227e1ee4203...03adb03401fe

------------------------------------------------------------------------------
Developer Access Program for Intel Xeon Phi Processors
Access to Intel Xeon Phi processor-based developer platforms.
With one year of Intel Parallel Studio XE.
Training and support from Colfax.
Order your platform today. http://sdm.link/xeonphi

_______________________________________________
Keepalived-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/keepalived-devel
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.