-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Support the assignment of virtual machines as cluster hosts #1494
Comments
I looked into this possibility while fleshing out the initial data model and it's just a nightmare to implement. The VirtualMachine model already has a foreign key to the Cluster model to indicate assignment as a VM. Adding a second relationship between the two introduces a whole slew of validation issues: preventing a VM from hosting its own cluster, preventing infinite recursion, preventing physical devices from hosting Docker-based clusters, etc. NetBox's role is as an IPAM/DCIM application and I believe the model introduced in the beta has satisfied that role. You can still create VM-based clusters and track IP assignments to Docker containers, you just can't model the assignment of VMs as cluster hosts. (The assignment of hosts to a cluster is optional.) IMO this is a reasonable compromise and keeps us from creeping into VM orchestration territory, for which most organizations will use a separate tool anyway. |
OK, understood. Many objects in Netbox have either "description" or "comments" field. How about adding one to Cluster? Then such cases can be documented ad-hoc. |
Fixed the |
@jeremystretch |
Issue type
[X] Feature request
[ ] Bug report
[ ] Documentation
Environment
Description
A VM runs on pool of resources called a "cluster". At the moment, a cluster can only consist of physical devices. I would like a cluster to have VMs as hosts too [^1]
Use cases:
[^1] It would probably be OK if a cluster were limited to take either physical devices or virtual machines; that is, you have to select "physical cluster" or "virtual cluster".
That might make the UI less confusing, although it blocks the (relatively unusual) case of a mixed cluster.
The text was updated successfully, but these errors were encountered: