Hi Dejan, 2013/7/15 Dejan Muhamedagic <[email protected]>> > On Mon, Jul 15, 2013 at 01:40:47PM +0200, Dejan Muhamedagic wrote: > > Hi Sami, > > > > > On Fri, Jun 28, 2013 at 03:40:02PM +0300, Sami Kähkönen wrote: > > > Hi, > > > Here is a small patch to enable multiple IP support for eDir88 RA. To > > > summarize, eDirectory supports multiple IP numbers in config file > > > separated > > > by comma. > > > Example line in nds.conf: > > > n4u.server.interfaces=168.0.0.1@524,10.0.0.1@524 > > > > > > Current resource agent is unable to cope with such configurations. > > > > > > This patch creates an array of IP:port configurations and checks them > > > individually. Tested in SLES 11 SP2 HA environment with one and multiple > > > IP's. All comments and additional testing are welcome. > > > > > > Patch also in github: https://github.com/skahkonen/resource-agents > > > > Sorry for the delay and many thanks for the patch. It looks good > > to me. If you can, it would be good to make a pull request at > > github. > > I did that myself and put together the indentation patch. See > https://github.com/ClusterLabs/resource-agents/pull/283
You beat me to it by few minutes, as I was about to make the pull request myself after checking all ocf_log -statements. Thanks, - Sami > Cheers, > > Dejan _______________________________________________________ Linux-HA-Dev: [email protected] http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev Home Page: http://linux-ha.org/
