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

Assign virtualization Clusters to a Location #7699

Closed
bluikko opened this issue Oct 31, 2021 · 4 comments · Fixed by #17848
Closed

Assign virtualization Clusters to a Location #7699

bluikko opened this issue Oct 31, 2021 · 4 comments · Fixed by #17848
Assignees
Labels
breaking change This change modifies or removes some previously documented functionality complexity: low Requires minimal effort to implement netbox status: accepted This issue has been accepted for implementation type: feature Introduction of new functionality to the application
Milestone

Comments

@bluikko
Copy link
Contributor

bluikko commented Oct 31, 2021

NetBox version

v3.0.8

Feature type

Data model extension

Proposed functionality

Virtualization clusters should be assignable to locations.

When the location function was added, clusters missed this option.

The open issue #6429 is similar and is marked as blocked by #6440 for better handling of "multi-model parent assignments". The same issue may affect this feature but I feel the use case is quite strong.

Use case

Because Clusters are only assignable to Sites and not to Locations, the whole Locations feature is currently unusable for documentation that consist of both physical Devices and Virtual Machines.

It makes integrations very difficult if the physical location of Devices uses the Locations feature but location of Virtual Machines uses the Sites feature. So in practice both Devices and Virtual Machines then need to use Site for the physical location.

Adding Location for Clusters would allow to locate both Devices and Virtual Machines using Location.

Database changes

Virtualization Clusters would need a parent Location.

External dependencies

none

@bluikko bluikko added the type: feature Introduction of new functionality to the application label Oct 31, 2021
@jeremystretch jeremystretch added the status: under review Further discussion is needed to determine this issue's scope and/or implementation label Nov 11, 2021
@github-actions
Copy link
Contributor

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. NetBox is governed by a small group of core maintainers which means not all opened issues may receive direct feedback. Please see our contributing guide.

@github-actions github-actions bot added the pending closure Requires immediate attention to avoid being closed for inactivity label Jan 11, 2022
@jeremystretch jeremystretch added needs milestone Awaiting prioritization for inclusion with a future NetBox release and removed status: under review Further discussion is needed to determine this issue's scope and/or implementation pending closure Requires immediate attention to avoid being closed for inactivity labels Feb 1, 2022
@jeremystretch
Copy link
Member

Similarly, #9969 proposes also extending cluster assignments to site groups.

@ocient-jlarson
Copy link

This would be incredibly useful. Location shows up in Ansible hostvars very nicely for devices. After some poking I couldn't figure out how to get location passed in for VMs and then found this. In our use case "Site" is a datacenter and the "Location" could be separate locations within the datacenter "cages" between prod/nonprod. Was planning to use this for tagging on our monitoring to keep it consistent between devices/virtual machines.

@ocient-cliimatta
Copy link

Yes, this makes it very difficult to use a location's config context on a Virtual machine, even though "Cluster" is available.

I assume that location would need to be assignable to a cluster for this to work? Currently it looks like only Sites are supported at that level.

@jeremystretch jeremystretch added the status: backlog Awaiting selection for work label May 21, 2024
@arthanson arthanson added the complexity: low Requires minimal effort to implement label May 22, 2024
@jeffgdotorg jeffgdotorg added this to the v4.1 milestone May 31, 2024
@jeffgdotorg jeffgdotorg removed the needs milestone Awaiting prioritization for inclusion with a future NetBox release label Jun 7, 2024
@arthanson arthanson assigned arthanson and unassigned arthanson Jul 9, 2024
@jeremystretch jeremystretch removed this from the v4.1 milestone Jul 12, 2024
@jeffgdotorg jeffgdotorg added the needs milestone Awaiting prioritization for inclusion with a future NetBox release label Jul 14, 2024
@jeremystretch jeremystretch added this to the v4.2 milestone Sep 26, 2024
@jeremystretch jeremystretch removed the needs milestone Awaiting prioritization for inclusion with a future NetBox release label Sep 26, 2024
@jeremystretch jeremystretch added v4.2 and removed v4.2 labels Oct 18, 2024
@arthanson arthanson self-assigned this Oct 23, 2024
@jeremystretch jeremystretch added status: accepted This issue has been accepted for implementation and removed status: backlog Awaiting selection for work labels Oct 30, 2024
@jeremystretch jeremystretch added the netbox label Nov 1, 2024 — with Linear
@jeremystretch jeremystretch added the breaking change This change modifies or removes some previously documented functionality label Nov 22, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
breaking change This change modifies or removes some previously documented functionality complexity: low Requires minimal effort to implement netbox status: accepted This issue has been accepted for implementation type: feature Introduction of new functionality to the application
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants