Bugfix-release ipvsman_0.9.1-4 is out now.

[Dr. Volker Jaenisch]

Hello Just!

Bugfix-release ipvsman_0.9.1-4 is out now.

Just Marc schrieb:
> Cool tool.  congratulations.  It has lots of potential.  I love the 
> realtime weight changing.   I was the first to download your updated 
> version! :)
Thank you very much.
>
> I have a few observations:
Many thanks for all this feedback!
> - In order to make it work, I had to create the dirs:  
> /var/log/ipvsman and  /var/lib/ipvsman/  the script should either 
> create them by itself or ask the user politely do to so, without 
> giving a scary traceback.
Are you running directly from source or from a python instalation or 
from a debian package?
The debian package and the python installer should create that 
directories. I will have a look at this.
> - Playing around, I got this:
> Traceback (most recent call last):
>  File "./ipvsman", line 179, in ?
>    gui.main()
>  File "lib/ipvsman/gui.py", line 434, in main
>    self.ui.run_wrapper( self.run )
>  File "/var/lib/python-support/python2.4/urwid/curses_display.py", 
> line 202, in run_wrapper
>    return fn()
>  File "lib/ipvsman/gui.py", line 483, in run
>    self.help()
> TypeError: 'AttrWrap' object is not callable
>
> and
>
> Traceback (most recent call last):
>  File "./ipvsman", line 179, in ?
>    gui.main()
>  File "lib/ipvsman/gui.py", line 434, in main
>    self.ui.run_wrapper( self.run )
>  File "/var/lib/python-support/python2.4/urwid/curses_display.py", 
> line 202, in run_wrapper
>    return fn()
>  File "lib/ipvsman/gui.py", line 522, in run
>    self.showLog()
>  File "lib/ipvsman/gui.py", line 315, in showLog
>    self.makeLog( )
>  File "lib/ipvsman/gui.py", line 303, in makeLog
>    self.logText = urwid.ListBox( LineWalker( utils.findActualLogFile() 
> ) )
>  File "lib/ipvsman/gui.py", line 29, in __init__
>    self.file = open(name)
> TypeError: coercing to Unicode: need string or buffer, NoneType found
Both bugs found an fixed in 0.9.1-4
> - I would suggest a timed auto-refresh of the stats, with a 'd' 
> command to set the delay in seconds, like top
I will put this on the ToDo list. I personally do not like autorefresh - 
at one of our customer sites
some dozen admins work with ipvsman in parallel. This makes lots of 
refreshs.
The best solution will be to make a commandline switch -a for autorefresh.
> - The table headers (real   weight   conns ... etc) is not aligned to 
> the data.
Is already on the ToDo ;-)
> - For the 'totals' line below, write 'Totals:' before printing the 
> values to make it clear
Will do.
> - The word 'laden' should be changed to load :)
Ups. Excuse my blindness for german words :-). Fixed in 0.9.1-4.

Beste Regards

Volker



-- 
====================================================
   inqbus it-consulting      +49 ( 341 )  5643800
   Dr.  Volker Jaenisch      http://www.inqbus.de
   Herloßsohnstr.    12      0 4 1 5 5    Leipzig
   N  O  T -  F Ä L L E      +49 ( 170 )  3113748
====================================================