Skip to content
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

Feature: Improve the extendability of the netbox inventory plugin #62

Closed
loganbest opened this issue Jan 13, 2020 · 1 comment · Fixed by #99
Closed

Feature: Improve the extendability of the netbox inventory plugin #62

loganbest opened this issue Jan 13, 2020 · 1 comment · Fixed by #99

Comments

@loganbest
Copy link
Contributor

ISSUE TYPE
  • Feature Idea
SUMMARY

Separating the list of lookup functions from the logic that executes the lookups. This will improve the extendability of the inventory plugin by allowing the addition of more lookups without having to re-implement or change the refresh_lookups method.

Original Ansible Core PR:
ansible/ansible#59662

@ChrisPortman since you're the original author of this PR, could you submit a new one to this repo? We're pulling the latest devel version of the Inventory plugin out of Ansible Core and into this collection for future development.

@loganbest
Copy link
Contributor Author

I would also like to note that subclassing will not be officially supported by this Inventory plugin. Any issues that arise from subclassing that cannot be reproduced using the plugin alone will be up to the individual to resolve.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants