Why in the balls is there no timeout field?

wrgeorge1983 / Pystol

Collection of network management tools written in Python

GNU Lesser General Public License v2.1

6 stars 0 forks source link

Why in the balls is there no timeout field? #10

Open wrgeorge1983 opened 9 years ago

wrgeorge1983 commented 9 years ago

And/or why don't I use it!?

wrgeorge1983 commented 9 years ago

There IS one. Implemented in sshexecute and sshutil classes, but operations regularly exceed this time?

wrgeorge1983 commented 9 years ago

This probably relates to not using good exceptions, and improperly catching the ones we do use.

Also, e.g. sshutil.py:Switch.populate_lite There's several entries that could each (even with better handling of the above) take a long time. Sensible for this to have it's own timeout since the per-item timeouts provide for setup-time that shouldn't be necessary here.