ACL Processing for nmdb-import-cisco

[marshall-sg]

Is your enhancement request related to a problem? Please describe.
The nmdb-import-cisco tool should be able to appropriately parse ACLs. Currently only the ASA one does this.

Describe the solution you'd like
The nmdb-import-cisco tool process ACL lists similar to how the ASA one does.

Describe alternatives you've considered
Attempted to process with the ASA parser, but it appears not to work appropriately. Manual insertions was the only other option via the nmdb-insert-ac tool.

[cctechwiz]

Code complete.

Troubleshooting regression test issues that master branch is throwing an error running nmdb-import-cisco.

$ nmdb-import-cisco --device-id test ../Running.Config -v 8
ERROR: option '--data-path' cannot be specified more than once

All nmdb-import-cisco* parsers are doing this when provided with a --device-id flag. If you omit that flag, it runs properly. Running --help does not show --device-id as an options for those parsers.

Checking the parser tool code, the device-id was in fact removed in modifyToolOptions. This makes me very curious as to when my branch was made and how it doesn't reflect that change since there are no commits since V2.0 that makes that change.

---

Ok. This was introduced much earlier on this branch. So there is no way currently that the master branch will run against the regression tests at all. I will need to regress against another point, if at all.

[marshall-sg]

I think you need to regress with c5472c1 installed. That appears to be the last commit I was changing stuff that would impact the regression testing.

[marshall-sg]

I moved it back from done. It needs an approved and merged pull request before it can be considered done.