-
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
Unable to assign VM/Device in one site to cluster that has no site #15717
Comments
I'm looking for 'easy' first bug fixes to help with for NetBox. This issue appears to have been accepted, so does that mean the powers-that-be are good with allowing VM's assigned to a Site to be associated with a Cluster that doesn't have a Site? Seems fairly basic, I'm just wondering if and how I could help. Looks like a simple tweak to:
Should it only be allowed for Clusters without a Site assigned, should Clusters be allowed to be assigned to multiple Sites, etc? |
A Cluster, like in my case can span the globe. Different providers in different regions. The VM's that the cluster runs on, in this case kubernetes do have a site, being their DC. |
I can submit a PR allowing device/vm's with a site, to be assigned to a non-site cluster as described above, if allowed :) |
Don't know what this allow business is? Generally with Open Source projects, you raise the PR and submit. That's generally the only time you'll have input. Obviously I'm all for the issue being fixed!! |
I'm new here but I believe I have to be assigned to the issue before I can/should submit the PR. I have it ready to submit (diff at link below), it appears Devices already have this logic, so I updated VM's to match, added a new test for VM's, and all the similar tests to Devices since they didn't already exist. |
Where'd that come from? If that's the policy to contribute to this repo, I can't imagine anyone coming back, let alone starting in the first place. 2-3 days avg turn around, it'd be a week before you even had the PR started. All that time wasted just for a line of code. When in the end all that is needed is to submit the PR, if it's not good to go, it'll be denied. |
It comes from the maintainers of NetBox. NetBox is an enterprise grade piece of software that has many applications in business critical environments, and the process of handling PRs is their chosen way to control the direction the development takes. In the environments NetBox is used in speed is definitely not the first priority, but stability and maturity are. On the other hand it's also a way to keep people from investing unnecessary work in PRs that eventually wouldn't be accepted anyway, so don't just see it as a random act of bureaucracy. |
@jon-nfc please take some time to read our contributing policy. These policies have been put in place to protect everyone's time. If you find yourself unable or unwilling to abide by it, please respectfully refrain from engaging with the project. |
@nopg I am going to assign this to you. Feel free to submit the PR when you are ready. |
Deployment Type
Self-hosted
NetBox Version
v3.7.5
Python Version
3.11
Steps to Reproduce
Error message
The selected cluster (<cluster name>) is not assigned to this site (<site name>)
Expected Behavior
If a cluster has no site set, then a VM/device from any site should be assignable to it. Without being able to do this, it's not possible to track a cluster that spans multiple DC.
Observed Behavior
Can't assign a device/vm with a defined site to a cluster with no defined site (
null
)The text was updated successfully, but these errors were encountered: