ansible_modules icon indicating copy to clipboard operation
ansible_modules copied to clipboard

[Feature]: Add tenant option to netbox_location

Open jsalatiel opened this issue 2 years ago • 8 comments

NetBox version

v3.2.5

Feature type

Change to existing Module

Proposed functionality

Accept the parameter tenant on the netbox.netbox.netbox_location module.

Use case

Location should be able to be linked to a tenant like it can be done on the webinterface.

External dependencies

No response

jsalatiel avatar Jul 20 '22 13:07 jsalatiel

Is it location or is it site? Because at least the docs at https://docs.netbox.dev/en/stable/core-functionality/tenancy/ don't list location as being able to be assigned to a tenant.

sc68cal avatar Jul 20 '22 20:07 sc68cal

It's location. I always have to set manually on the webui because the module does not support it.

jsalatiel avatar Jul 21 '22 01:07 jsalatiel

image

jsalatiel avatar Jul 21 '22 01:07 jsalatiel

From my understanding, all of these modules should support tenant.

image

jsalatiel avatar Jul 21 '22 01:07 jsalatiel

I went and checked the code - it appears the docs are out of date. So yeah, that's fine, go ahead and add the tenant attribute

sc68cal avatar Jul 21 '22 14:07 sc68cal

Thank you. Hoping to see it working on next release ,:)

jsalatiel avatar Jul 21 '22 14:07 jsalatiel

Do you plan on implementing this feature? It should be straight forward, since there are other objects that have the tenant attribute, so it should be easy to look at how it is implemented by other objects.

sc68cal avatar Jul 21 '22 14:07 sc68cal

Unfortunately I do not code. I hope someone else can implement that.

jsalatiel avatar Jul 21 '22 14:07 jsalatiel

Will be added in the next collection release.

rodvand avatar Sep 18 '22 19:09 rodvand