On Fri, Apr 19, 2013 at 10:21:24PM +0200, Lukas Tribus wrote:
> Hi Geoff!
> 
> > I've told them to ensure the weight is '1' on all servers that
> > should be serving content, but as we all know something like
> > that can be missed pretty easily
> 
> Agreed.
> 
> 
> > I'm not sure if this is duplicating functionality that wasn't
> > functional or not, but here are the changes I've made to the
> > latest snapshot to support the functionality I was requesting.
> 
> I believe the mailer damaged the diff. I manually applied your
> changes with some slight modifications (renaming the class name
> from maintain2 to softstop and removed "(MAINT)") and I like it.
> 
> Attaching the patch for further review.

Makes sense, especially since what commonly happens is the following
scenario :
  - soft stop
  - wait
  - disable
  - operate
  - enable and go to bed

But the server keeps its 0 weight and does not take any traffic.

I have applied your patch and I like it as well.

Thanks!
Willy


Reply via email to