napalm-automation-community / napalm-dellos10

NAPALM Driver for Dell EMC OS10 Enterprise
http://dellos10-napalm.readthedocs.io/en/latest/
Apache License 2.0
10 stars 11 forks source link

Fix error handling in merge operations #27

Closed tmshlvck closed 3 years ago

tmshlvck commented 3 years ago

Merge operations can cause errors that are reported by various messages containing string "Error".

The error handling program path must not call rollback() function which is unimplemented and raises NotImplementedError exception that is subsequently caught in Napalm code and fails to propagate to the custom code or Napalm CLI. Therefore the cause of the merge error and even the fact that the merge failed is lost.