-
Notifications
You must be signed in to change notification settings - Fork 134
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
Filters for netbox_devices are undocumented. #570
Comments
Hi, generally, what you say is true. You can look up the supported filters here: https://github.com/e-breuninger/terraform-provider-netbox/blob/master/netbox/data_source_netbox_devices.go#L161-L195 Currently, filters have to be implemented manually and implementing ALL supported filters is very chore-y. We have a very nice contribution in #428 to generate support for all filters supported by go-netbox via reflection. I did not yet get around to implementing it everywhere, though. |
Wow, that i impressively labor intensive. I'll work around this for the moment by using an already implemented filter. |
Adding a filter is actually quite trivial, but the api exposes SO many possible filters that it really is quite annoying to do :D |
I think going with #428 for all datasources would be a major improvement to this provider. |
Absolutely! I just can barely keep up with new merge requests and upstream changes (netbox 4.0 comes to mind...) |
Terraform Version
N/A
Affected Resource(s)
Please list the resources as a list, for example:
Expected Behavior
I would have expected the filter syntax to be the same as that shown in the netbox API playground, but it doesn't seem to be.
Actual Behavior
Some other filter syntax appears to be in use. There should be documentation or examples as to what kind of filters these data sources take.
Steps to Reproduce
I tried with the following filter block:
Important Factoids
N/A
References
N/A
The text was updated successfully, but these errors were encountered: