Re: [Patch 3/7] tabled: add the status display

[Pete Zaitcev <zaitcev@redhat.com>]

On Wed, 20 Jan 2010 14:51:59 -0500
Jeff Garzik <jeff@garzik.org> wrote:

> On 01/14/2010 11:12 PM, Pete Zaitcev wrote:
> > I find it too tiresome to monitor the status with kill -USR1.
> > This is more fun. The patch seems not too ugly regarding the factoring
> > of the cli_xxx methods.
> >
> > Signed-Off-By: Pete Zaitcev<zaitcev@redhat.com>
> >
> > ---
> >   doc/etc.tabled.conf |    3
> >   server/Makefile.am  |    4
> >   server/config.c     |   19 +++
> >   server/replica.c    |   64 +++++++++++-
> >   server/server.c     |  104 +++++++++++++-------
> >   server/status.c     |  218 ++++++++++++++++++++++++++++++++++++++++++
> >   server/tabled.h     |   12 +-
> >   7 files changed, 379 insertions(+), 45 deletions(-)
> 
> applied...  but we need some way to either turn this off, or limit to a 
> list of incoming IP addresses.  This is an admin interface, which should 
> not be automatically available to all end-user applications.

I thought it was obvious that StatusPort defaults to off (zero).

	if (tabled_srv.status_port)
		net_open_known(tabled_srv.status_port, true);

-- Pete