-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Cannot filter virtual machine interfaces #1557
Comments
We should probably just ditch the |
That seems reasonable, although it begs the question: should the
`/virtualization/` endpoint exist at all? Maybe virtual-machines should
just be part of dcim?
On 6 Oct 2017 20:59, "Jeremy Stretch" <notifications@github.com> wrote:
We should probably just ditch the /api/virtualization/interfaces/ endpoint
entirely. It's redundant to the DCIM-based endpoint.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#1557 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABOQV-2EDiGqpA6uaJ8KE09JOSs4n9TAks5spoalgaJpZM4PtvyY>
.
|
VirtualMachines and Clusters are their own discrete models with dedicated tables in the database, so they need their own endpoints. This is in contrast to the Interface model, which is actually shared by Devices and VirtualMachines. |
After pondering this a bit more, it's cleaner to maintain a separate endpoint for VM interfaces. In addition to being more efficient (since the DCIM endpoint only pulls in devices and the virtualization endpoint only pulls in VMs), there's enough difference between the two interface representations to justify separate filters. |
Issue type
[ ] Feature request
[x] Bug report
[ ] Documentation
Environment
Description
Not sure if this counts as a bug or a feature request, but as far as I can tell there's no way to filter the
/virtualisation/interfaces/
API endpoint?The only way to determine which interfaces a VM has is to get all the interfaces and iterate over them clientside.
The text was updated successfully, but these errors were encountered: