Oob ip (devices)

[ITJamie]

Fixes: #8137

a first stab at oob_ip for devices. If we are happy to go this route I will add more commits for virtualmachines (also do we want to add oob_ip to the virtualdevicecontext models (it has primary_ip already)?

[ITJamie]

(please ignore test failures for now, just a POC to confirm the method)

@jeremystretch would you be happy with this approach for oob_ip?
there was previously talk in #11953 (comment) about considering another way.

If you are happy with the method in this PR for devices, I will flesh it out more for vms

edit device view:

IP address view (note checkboxes to show if primary IP or oob IP)

Device view:

)

[jeremystretch]

@ITJamie thanks for taking this on. Per my comment here, my intent was to introduce only a single oob_ip field to the Device model.

[jeremystretch]

Also, could you rebase this on the feature branch please?

[ITJamie]

just to check @jeremystretch are you ok with this overall method or do you want me to redo this with the m2m relationship? i had started a POC of the m2m relationship (#13094) but have been pulled onto another project

[jeremystretch]

@ITJamie We won't have the time or bandwidth to explore a change to the underlying relationship for multiple fields as part of the v3.6 release, so if we're going to ship this as part of the beta we'll need to stick with replicating the existing pattern (the direct foreign key relationship). Thanks!

[ITJamie]

ok ill rework it now to have only a single oob_ip

[ITJamie]

refactored down to a single field

[jeremystretch]

Thanks @ITJamie!

[bitcollector1]

Thanks again for the extra work, it's much appreciated!

Hard to believe how much NetBox has matured since I originally requested this. This team is pretty awesome!!