nornir-automation / nornir

Pluggable multi-threaded framework with inventory management to help operate collections of devices
https://nornir.readthedocs.io/
Apache License 2.0
1.38k stars 234 forks source link

Add Python 3.12 to test matrix #961

Closed ogenstad closed 2 months ago

ktbyers commented 2 months ago

@ogenstad I would probably try rolling nornir-napalm to 0.5.0 (since your error seems related to outdated ncclient which is possibly being caused by older nornir-napalm):

https://github.com/nornir-automation/nornir/blob/main/pyproject.toml#L46

And it should be updated anyways.

ogenstad commented 2 months ago

@ktbyers, thanks. Looks like it got a bit further but hit the next problem. Might be that we just need to upgrade nbval, I'll try to return to it a bit later.

coveralls commented 2 months ago

Pull Request Test Coverage Report for Build 9890392497

Details


Totals Coverage Status
Change from base Build 9886686772: 0.0%
Covered Lines: 539
Relevant Lines: 951

💛 - Coveralls
ktbyers commented 2 months ago

LGTM