[Openais] [PATCH update]: add additional note about bindnetaddt to corosync.conf.example

2011-08-01 Thread Florian Haas
Ahoj Jan,

Thanks for reviewing. Please take a look at this followup patch to see if
the information in there is useful. If everything is OK for you, I'll fix up
my git tree again so you can pull later today.

Cheers,
Florian

___
Openais mailing list
Openais@lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/openais


[Openais] [PATCH] corosync.conf.example: add note about host addresses in bindnetaddr

2011-08-01 Thread Florian Haas
https://lists.linux-foundation.org/pipermail/openais/2011-July/016563.html

Jan Friesse pointed out that bindnetaddr should be set to a host
address (as opposed to a network address) on hosts where multiple
NICs live on the same subnet. Add a comment to that effect to
the example configuration file.

Signed-off-by: Florian Haas 
---
 conf/corosync.conf.example |   16 
 1 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/conf/corosync.conf.example b/conf/corosync.conf.example
index c849dba..ac1718f 100644
--- a/conf/corosync.conf.example
+++ b/conf/corosync.conf.example
@@ -17,11 +17,19 @@ totem {
interface {
 # Rings must be consecutively numbered, starting at 0.
ringnumber: 0
-   # This is the *network* address of the interface to
-   # bind to. This ensures that you can use identical
-   # instances of this configuration file across all your
-   # cluster nodes, without having to modify this option.
+   # This is normally the *network* address of the
+   # interface to bind to. This ensures that you can use
+   # identical instances of this configuration file
+   # across all your cluster nodes, without having to
+   # modify this option.
bindnetaddr: 192.168.1.0
+   # However, if you have multiple physical network
+   # interfaces configured for the same subnet, then the
+   # network address alone is not sufficient to identify
+   # the interface Corosync should bind to. In that case,
+   # configure the *host* address of the interface
+   # instead:
+   # bindnetaddr: 192.168.1.1
# When selecting a multicast address, consider RFC
# 2365 (which, among other things, specifies that
# 239.255.x.x addresses are left to the discretion of
-- 
1.7.1

___
Openais mailing list
Openais@lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/openais


Re: [Openais] [PATCH] corosync.conf.example: add note about host addresses in bindnetaddr

2011-08-01 Thread Steven Dake
These patches look good.

Reviewed-by: Steven Dake 

Regards
-steve

On 07/31/2011 11:56 PM, Florian Haas wrote:
> https://lists.linux-foundation.org/pipermail/openais/2011-July/016563.html
> 
> Jan Friesse pointed out that bindnetaddr should be set to a host
> address (as opposed to a network address) on hosts where multiple
> NICs live on the same subnet. Add a comment to that effect to
> the example configuration file.
> 
> Signed-off-by: Florian Haas 
> ---
>  conf/corosync.conf.example |   16 
>  1 files changed, 12 insertions(+), 4 deletions(-)
> 
> diff --git a/conf/corosync.conf.example b/conf/corosync.conf.example
> index c849dba..ac1718f 100644
> --- a/conf/corosync.conf.example
> +++ b/conf/corosync.conf.example
> @@ -17,11 +17,19 @@ totem {
>   interface {
>  # Rings must be consecutively numbered, starting at 0.
>   ringnumber: 0
> - # This is the *network* address of the interface to
> - # bind to. This ensures that you can use identical
> - # instances of this configuration file across all your
> - # cluster nodes, without having to modify this option.
> + # This is normally the *network* address of the
> + # interface to bind to. This ensures that you can use
> + # identical instances of this configuration file
> + # across all your cluster nodes, without having to
> + # modify this option.
>   bindnetaddr: 192.168.1.0
> + # However, if you have multiple physical network
> + # interfaces configured for the same subnet, then the
> + # network address alone is not sufficient to identify
> + # the interface Corosync should bind to. In that case,
> + # configure the *host* address of the interface
> + # instead:
> + # bindnetaddr: 192.168.1.1
>   # When selecting a multicast address, consider RFC
>   # 2365 (which, among other things, specifies that
>   # 239.255.x.x addresses are left to the discretion of

___
Openais mailing list
Openais@lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/openais


[Openais] [GIT PULL] Changes to example configuration file (corosync.conf.example)

2011-08-01 Thread Florian Haas
Steve,

please consider pulling the following changes since commit
d4fb83e971b6fa9af0447ce0a70345fb20064dc1:

  main: let poll really stop before totempg_finalize (2011-07-26
10:07:08 +0200)

from the the git repository at:
  git://github.com/fghaas/corosync master

All changes have undergone review on the list. Thanks to Dan Frincu and
Jan Friesse for their valuable feedback. A patch-by-patch summary and
diffstat are below, as usual.

Cheers,
Florian


Florian Haas (4):
  corosync.conf.example: change bindnetaddr
  corosync.conf.example: change mcastaddr
  corosync.conf.example: include comments
  corosync.conf.example: add note about host addresses in bindnetaddr

 conf/corosync.conf.example |   54
+--
 1 files changed, 51 insertions(+), 3 deletions(-)




signature.asc
Description: OpenPGP digital signature
___
Openais mailing list
Openais@lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/openais