Improve napalm-ios hostname handler

napalm-automation / napalm-ios

Apache License 2.0

31 stars 40 forks source link

Improve napalm-ios hostname handler #207

Closed ktbyers closed 7 years ago

ktbyers commented 7 years ago

Current hostname handler has issues with inconsistent failures. I believe these failures were due to not allocating enough time for operation to complete (12 seconds).

I have changed the pattern to now just search for one of the three items #, >, or the current last character of your prompt.

I tested this on both merge/replace and with hostname changes / no hostname changes.

coveralls commented 7 years ago

Coverage increased (+0.06%) to 69.922% when pulling 02082e0efea7f6274a24055fe19fa25b8889815e on ktbyers:improve_hostname_handler into eaaaec6d800ec16717413e255643fc3493df0252 on napalm-automation:develop.