All of lore.kernel.org
 help / color / mirror / Atom feed
From: Steve Dickson <SteveD@redhat.com>
To: "Ragnar Kjørstad" <nfs@ragnark.vestdata.no>
Cc: nfs@lists.sourceforge.net
Subject: Re: [PATCH] STATD - SM_NOTIFY have wrong ID_NAME on multihost servers.
Date: Wed, 24 Nov 2004 17:08:11 -0500	[thread overview]
Message-ID: <41A5064B.1030009@RedHat.com> (raw)
In-Reply-To: <20041124173525.GI19342@vestdata.no>



Ragnar Kj=F8rstad wrote:

>On Wed, Nov 24, 2004 at 07:00:08AM -0500, Steve Dickson wrote:
> =20
>
>>>If that's the case, are we allowed to just send the IP instead of the
>>>name?
>>>
>>>
>>>     =20
>>>
>>No, since the spec says a the id_mon needs to be character string....
>>   =20
>>
>
>"10.0.0.1" is a character string as much as any - unless the
>specification says something more about this?
> =20
>
Well here is what the spec says wrt to mon_names:
"For maximum portability and interworking, it is recommended that=20
applications
and users define host names containing only the characters of the Portabl=
e
Filename Characters defined in ISO/IEC 9945-1:1990."

So doing some like would probably cause interopability issues with other
OSes....

>In short, this is exactly like what your patch does except that:
>- if there are multiple matching interfaces (e.g. aliases) we
>  send messages for all
> =20
>
Yes see your point, but is aliasing all that common in non-HA environment=
s?
The relatively simple  patch I'm proposing is a small step in the right=20
direction
which would make statd work correctly on multihome servers.
Should we strive to make statd more HA friendly, YES!  but I would
suggest one step at time.... Meaning we should propose this match to Neil
for inclusion in upstream and then build on it... imho...

SteveD.





-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now. 
http://productguide.itmanagersjournal.com/
_______________________________________________
NFS maillist  -  NFS@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs

  parent reply	other threads:[~2004-11-24 22:05 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-11-23 20:28 [PATCH] STATD - SM_NOTIFY have wrong ID_NAME on multihost servers Steve Dickson
2004-11-23 23:26 ` Ragnar Kjørstad
2004-11-24  0:46   ` Steve Dickson
2004-11-24  2:00     ` Ragnar Kjørstad
2004-11-24 12:00       ` Steve Dickson
2004-11-24 17:35         ` Ragnar Kjørstad
2004-11-24 18:43           ` Marc Eshel
2004-11-24 20:35             ` Ragnar Kjørstad
2004-11-24 22:08           ` Steve Dickson [this message]
2004-11-24 23:25             ` Ragnar Kjørstad
2004-11-26 13:01               ` Steve Dickson
2004-11-24  4:42     ` Marc Eshel

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=41A5064B.1030009@RedHat.com \
    --to=steved@redhat.com \
    --cc=nfs@lists.sourceforge.net \
    --cc=nfs@ragnark.vestdata.no \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.