Johan Marcusson

Ideas for future gNMI/gRPC support: instead of polling the ARP table via SNMP at regular intervals you could subscribe to events when new ARP entries are added and query via:...

This is a feature that we see as important for SUNET as well. I imagine admins will want to change interfaces back and forth between static access VLAN and dot1x...

Should we really store that entire string? Seems like NAPALM gets it via "show version" and then: ```_, os_version = line.split("Cisco IOS Software, ")``` If you ask about version via...

The code looks nice! I did a few tests in our lab setup and it seems to work with manually adding a second link between a core and a dist...

Should be fine with a force push, I'm confident no one else from here did changes in this feature branch

Needs some documentation to describe the URL format, but looks simple and clean!

I had a quick look at https://hub.docker.com/_/redis?tab=tags&page=1&ordering=last_updated and since I don't think we have any special requirements really we could probably go with "redis:6" or "redis:6.2"

Ok to merge after the github conflict thing is looked at

We should also differentiate PUT and PATCH methods, today only PUT is used but functions as PATCH should Maybe PUT to reflesh repos should be a POST also? And of...

As long as code coverage does not decrease I guess all tests are still running, seems ok according to the codecov report above but src/cnaas_nms/conftest.py should probably be excluded from...