-
Notifications
You must be signed in to change notification settings - Fork 239
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adds duplicated code as function for inventory helper. #727
Adds duplicated code as function for inventory helper. #727
Conversation
I think if we want to go down this path and support this we should add a Basically that Thoughts? |
That makes sense. It may take me a little bit longer to get back on that particular update. I also need to check on what is going on with the tests that are failing here now after locally testing. |
Re failing tests, you may need to rebase as it's due to some breaking changes github actions did. Kirk fixed those already a few days ago though so hopefully you don't have to do anything about them. |
I've still got this on my radar. I've just come down with a bit of a cold that has knocked me out of working much lately... |
No worries, get well! |
I'm getting this in front of me a bit more. I thought I had something but the tests are not working. So I am going to try to restart the process on another branch. |
@jvanderaa What is the status of this? Is this dead now i.e. should be closed? |
Yeah, probably good to close out. Sorry for leaving it for so long @ktbyers |
@jvanderaa No worries at all. |
Fixes #723
Details
https://github.com/wvandeun/nornir_netbox/blob/develop/nornir_netbox/plugins/inventory/netbox.py#L39-L48
https://github.com/carlmontanari/nornir_ansible/blob/master/nornir_ansible/plugins/inventory/ansible.py#L456-L472