load_replace_candidate broke in v0.5.3
nbtnc1 opened this issue · comments
Thomas Nicholson commented
Environment
- Python version: 3.8.5
- napalm version: 3.3.1
- napalm-panos version: 0.5.3
Expected Behavior
Configuration should have loaded on to the device successfully.
Observed Behavior
Error while loading config from {path}.
Steps to Reproduce
- Attempt to load a config on to the device using load_replace_candidate
It looks like you removed the else statement from here: https://github.com/napalm-automation-community/napalm-panos/blob/v0.5.3/napalm_panos/panos.py#L206
Thomas Nicholson commented
Also need an else on the other if too - https://github.com/napalm-automation-community/napalm-panos/blob/develop/napalm_panos/panos.py#L209
Ken Celenza commented
Will take me a few hours to test, but the logic that was there was a bit complicated, I tried to make it simpler. #97
Ken Celenza commented
Updated, tested, and released 0.5.4. I yanked 0.5.3 (rather than delete) from pypi.