joshinryz

This also does not seem to work, no error is reported - but the case does not update. Using a non-existing case # does throw an error that it doesn't...

I resolved this, appears the regex was not matching - this actual happens on most of the calls. I removed the "Begins with" , as the script is actually receiving...

While we are on the topic. Non-sequence lines also get ignored. Should this be handled by the ACL module? ``` ip access-list test statistics per entry 10 permit any any...

@NilashishC do you have a sense when this may get worked on ?

I haven't really - previously we used a higher level function like tower to cache inventory . If you want to work on such a feature , happy to review...

@BrandonALXEllisSS - do you want to submit a PR for the documentation changes? Otherwise I can put something together and update it. Thanks for noticing this.

Brandon - im leaving this open for visibility , however - I'm going to be working on an ansible collection for this prolly soon, and will handle it there once...

Hey @draxxy9 , I never wrote this to support CNs, as best practice they are meant to be OUs. Let me review and see how hard it would be to...

Discussed this with others , I'd love some idea of why we should support CNs. Usually this is a bad practice from my experience.

Example change to remedy: ``` if isinstance(self._before, dict): self._before = {k:self._before[k] for k in self._before if not any(regex.match("{}: {}".format(k, self._before[k])) for regex in self._skip_lines) } else: self._before = [ line...