Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 403133 - sys-cluster/keepalived - patches for libipvs
Summary: sys-cluster/keepalived - patches for libipvs
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Cluster Team
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2012-02-11 08:48 UTC by Alexander Holler
Modified: 2012-07-11 09:39 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
needed for ipv6 (0001-libipvs-Fix-initialization-of-netlink-needed-for-IPv.patch,1.01 KB, patch)
2012-02-11 08:49 UTC, Alexander Holler
Details | Diff
correctly report ipvs-backup-thread (0001-libipvs-Fix-reporting-of-the-state-of-the-backup-dae.patch,1.88 KB, patch)
2012-02-11 08:50 UTC, Alexander Holler
Details | Diff
fix modprobe (0001-Fix-modprobe-arguments.patch,781 bytes, patch)
2012-02-11 08:51 UTC, Alexander Holler
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Holler 2012-02-11 08:48:51 UTC
Hello,

I've written two patches for libipvs which is used in ipvsadm and keepalived.

One is necessary to see running ipvs-backup-threads in ipvsadm, and one is necessary to use ipv6 with keepalived.

Both patches are based on keepalived 1.22 but should apply to older version too.
Both patches should be applied to ipvsadm too (its having it's own copy of libipvs).

I've added a third patch too, which is necessary to start keepalived when the module for ipvs (ip_vs) was not loaded before.


Reproducible: Always
Comment 1 Alexander Holler 2012-02-11 08:49:38 UTC
Created attachment 301533 [details, diff]
needed for ipv6
Comment 2 Alexander Holler 2012-02-11 08:50:32 UTC
Created attachment 301535 [details, diff]
correctly report ipvs-backup-thread
Comment 3 Alexander Holler 2012-02-11 08:51:12 UTC
Created attachment 301537 [details, diff]
fix modprobe
Comment 4 Alexander Holler 2012-02-11 08:53:18 UTC
I'm not sure if the names of attachments are saved, they are:

0001-libipvs-Fix-initialization-of-netlink-needed-for-IPv.patch
0001-libipvs-Fix-reporting-of-the-state-of-the-backup-dae.patch
0001-Fix-modprobe-arguments.patch

The first twoshouldbe applied to ipvsadm too, I will file a separate bug.
Comment 5 Alexander Holler 2012-02-11 09:05:49 UTC
I've reported my two patches upstream too (the third was already reported by someone else), but it seems upstream currently doesn't have much time. So I suggest to update keepalived to 1.2.2 and to apply at least those 3 patches.
Comment 6 Greg 2012-02-11 13:53:30 UTC
I can confirm these issues, and that the attached patches resolve the issues described.
Comment 7 Ultrabug gentoo-dev 2012-07-11 09:39:33 UTC
Thanks Alexander for your work and patience. Thanks also to Greg for reporting a successful test.

It's fixed in tree now.

+*keepalived-1.2.2-r4 (11 Jul 2012)
+
+  11 Jul 2012; Ultrabug <ultrabug@gentoo.org> +keepalived-1.2.2-r4.ebuild,
+  +files/keepalived-1.2.2-fix-ipv4-addr-compare.patch,
+  +files/keepalived-1.2.2-libipvs-fix-backup-daemon.patch,
+  +files/keepalived-1.2.2-libipvs-fix-ipv6.patch:
+  Fix #403133 by Alexander Holler, thanks to Greg for testing. Fix #425082
+  thanks to Ruslan Lutsenko. Add ipv6 USE flag.
+