diff --git a/docs/configuration/optional-settings.md b/docs/configuration/optional-settings.md index 4015badf4..576065434 100644 --- a/docs/configuration/optional-settings.md +++ b/docs/configuration/optional-settings.md @@ -355,6 +355,24 @@ The list of permissions to assign a new user account when created using remote a --- +## RELEASE_CHECK_TIMEOUT + +Default: 86,400 (24 hours) + +The number of seconds to retain the latest version that is fetched from the GitHub API before automatically invalidating it and fetching it from the API again. This must be set to at least one hour (3600 seconds). + +--- + +## RELEASE_CHECK_URL + +Default: None + +The releases of this repository are checked to detect new releases, which are shown on the home page of the web interface. You can change this to your own fork of the NetBox repository, or set it to `None` to disable the check. The URL provided **must** be compatible with the GitHub API. + +Use `'https://api.github.com/repos/netbox-community/netbox/releases'` to check for release in the official NetBox repository. + +--- + ## REPORTS_ROOT Default: $BASE_DIR/netbox/reports/ diff --git a/docs/configuration/required-settings.md b/docs/configuration/required-settings.md index e86b2810a..053e2d3d4 100644 --- a/docs/configuration/required-settings.md +++ b/docs/configuration/required-settings.md @@ -46,9 +46,9 @@ DATABASE = { [Redis](https://redis.io/) is an in-memory data store similar to memcached. While Redis has been an optional component of NetBox since the introduction of webhooks in version 2.4, it is required starting in 2.6 to support NetBox's caching functionality (as well as other planned features). In 2.7, the connection settings were broken down into two sections for -webhooks and caching, allowing the user to connect to different Redis instances/databases per feature. +task queuing and caching, allowing the user to connect to different Redis instances/databases per feature. -Redis is configured using a configuration setting similar to `DATABASE` and these settings are the same for both of the `webhooks` and `caching` subsections: +Redis is configured using a configuration setting similar to `DATABASE` and these settings are the same for both of the `tasks` and `caching` subsections: * `HOST` - Name or IP address of the Redis server (use `localhost` if running locally) * `PORT` - TCP port of the Redis service; leave blank for default port (6379) @@ -61,7 +61,7 @@ Example: ```python REDIS = { - 'webhooks': { + 'tasks': { 'HOST': 'redis.example.com', 'PORT': 1234, 'PASSWORD': 'foobar', @@ -84,9 +84,9 @@ REDIS = { If you are upgrading from a version prior to v2.7, please note that the Redis connection configuration settings have changed. Manual modification to bring the `REDIS` section inline with the above specification is necessary -!!! note - It is highly recommended to keep the webhook and cache databases separate. Using the same database number on the - same Redis instance for both may result in webhook processing data being lost during cache flushing events. +!!! warning + It is highly recommended to keep the task and cache databases separate. Using the same database number on the + same Redis instance for both may result in queued background tasks being lost during cache flushing events. ### Using Redis Sentinel @@ -102,7 +102,7 @@ Example: ```python REDIS = { - 'webhooks': { + 'tasks': { 'SENTINELS': [('mysentinel.redis.example.com', 6379)], 'SENTINEL_SERVICE': 'netbox', 'PASSWORD': '', @@ -126,7 +126,7 @@ REDIS = { !!! note It is possible to have only one or the other Redis configurations to use Sentinel functionality. It is possible - for example to have the webhook use sentinel via `HOST`/`PORT` and for caching to use Sentinel via + for example to have the tasks database use sentinel via `HOST`/`PORT` and for caching to use Sentinel via `SENTINELS`/`SENTINEL_SERVICE`. diff --git a/docs/installation/3-netbox.md b/docs/installation/3-netbox.md index fabad20eb..b9b68be1b 100644 --- a/docs/installation/3-netbox.md +++ b/docs/installation/3-netbox.md @@ -172,7 +172,7 @@ Redis is a in-memory key-value store required as part of the NetBox installation ```python REDIS = { - 'webhooks': { + 'tasks': { 'HOST': 'redis.example.com', 'PORT': 1234, 'PASSWORD': 'foobar', diff --git a/netbox/circuits/forms.py b/netbox/circuits/forms.py index 0b0378a7a..c6f0dfdc4 100644 --- a/netbox/circuits/forms.py +++ b/netbox/circuits/forms.py @@ -113,7 +113,6 @@ class ProviderFilterForm(BootstrapMixin, CustomFieldFilterForm): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/regions/", value_field="slug", filter_for={ 'site': 'region' @@ -125,7 +124,6 @@ class ProviderFilterForm(BootstrapMixin, CustomFieldFilterForm): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/sites/", value_field="slug", ) ) @@ -167,16 +165,10 @@ class CircuitTypeCSVForm(forms.ModelForm): class CircuitForm(BootstrapMixin, TenancyForm, CustomFieldModelForm): provider = DynamicModelChoiceField( - queryset=Provider.objects.all(), - widget=APISelect( - api_url="/api/circuits/providers/" - ) + queryset=Provider.objects.all() ) type = DynamicModelChoiceField( - queryset=CircuitType.objects.all(), - widget=APISelect( - api_url="/api/circuits/circuit-types/" - ) + queryset=CircuitType.objects.all() ) comments = CommentField() tags = TagField( @@ -245,17 +237,11 @@ class CircuitBulkEditForm(BootstrapMixin, AddRemoveTagsForm, CustomFieldBulkEdit ) type = DynamicModelChoiceField( queryset=CircuitType.objects.all(), - required=False, - widget=APISelect( - api_url="/api/circuits/circuit-types/" - ) + required=False ) provider = DynamicModelChoiceField( queryset=Provider.objects.all(), - required=False, - widget=APISelect( - api_url="/api/circuits/providers/" - ) + required=False ) status = forms.ChoiceField( choices=add_blank_choice(CircuitStatusChoices), @@ -265,10 +251,7 @@ class CircuitBulkEditForm(BootstrapMixin, AddRemoveTagsForm, CustomFieldBulkEdit ) tenant = DynamicModelChoiceField( queryset=Tenant.objects.all(), - required=False, - widget=APISelect( - api_url="/api/tenancy/tenants/" - ) + required=False ) commit_rate = forms.IntegerField( required=False, @@ -303,7 +286,6 @@ class CircuitFilterForm(BootstrapMixin, TenancyFilterForm, CustomFieldFilterForm to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/circuits/circuit-types/", value_field="slug", ) ) @@ -312,7 +294,6 @@ class CircuitFilterForm(BootstrapMixin, TenancyFilterForm, CustomFieldFilterForm to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/circuits/providers/", value_field="slug", ) ) @@ -326,7 +307,6 @@ class CircuitFilterForm(BootstrapMixin, TenancyFilterForm, CustomFieldFilterForm to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/regions/", value_field="slug", filter_for={ 'site': 'region' @@ -338,7 +318,6 @@ class CircuitFilterForm(BootstrapMixin, TenancyFilterForm, CustomFieldFilterForm to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/sites/", value_field="slug", ) ) @@ -355,6 +334,9 @@ class CircuitFilterForm(BootstrapMixin, TenancyFilterForm, CustomFieldFilterForm # class CircuitTerminationForm(BootstrapMixin, forms.ModelForm): + site = DynamicModelChoiceField( + queryset=Site.objects.all() + ) class Meta: model = CircuitTermination @@ -368,7 +350,4 @@ class CircuitTerminationForm(BootstrapMixin, forms.ModelForm): } widgets = { 'term_side': forms.HiddenInput(), - 'site': APISelect( - api_url="/api/dcim/sites/" - ) } diff --git a/netbox/dcim/forms.py b/netbox/dcim/forms.py index 70bada0a2..54357a2dc 100644 --- a/netbox/dcim/forms.py +++ b/netbox/dcim/forms.py @@ -71,7 +71,6 @@ class DeviceComponentFilterForm(BootstrapMixin, forms.Form): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url='/api/dcim/regions/', value_field='slug', filter_for={ 'site': 'region' @@ -83,7 +82,6 @@ class DeviceComponentFilterForm(BootstrapMixin, forms.Form): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/sites/", value_field="slug", filter_for={ 'device_id': 'site', @@ -93,10 +91,7 @@ class DeviceComponentFilterForm(BootstrapMixin, forms.Form): device_id = DynamicModelMultipleChoiceField( queryset=Device.objects.all(), required=False, - label='Device', - widget=APISelectMultiple( - api_url='/api/dcim/devices/', - ) + label='Device' ) @@ -327,10 +322,7 @@ class SiteBulkEditForm(BootstrapMixin, AddRemoveTagsForm, CustomFieldBulkEditFor ) tenant = DynamicModelChoiceField( queryset=Tenant.objects.all(), - required=False, - widget=APISelect( - api_url="/api/tenancy/tenants", - ) + required=False ) asn = forms.IntegerField( min_value=BGP_ASN_MIN, @@ -371,7 +363,6 @@ class SiteFilterForm(BootstrapMixin, TenancyFilterForm, CustomFieldFilterForm): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/regions/", value_field="slug", ) ) @@ -384,20 +375,10 @@ class SiteFilterForm(BootstrapMixin, TenancyFilterForm, CustomFieldFilterForm): class RackGroupForm(BootstrapMixin, forms.ModelForm): site = DynamicModelChoiceField( - queryset=Site.objects.all(), - widget=APISelect( - api_url="/api/dcim/sites/", - filter_for={ - 'parent': 'site_id', - } - ) + queryset=Site.objects.all() ) parent = DynamicModelChoiceField( - queryset=RackGroup.objects.all(), - required=False, - widget=APISelect( - api_url="/api/dcim/rack-groups/" - ) + queryset=RackGroup.objects.all() ) slug = SlugField() @@ -442,7 +423,6 @@ class RackGroupFilterForm(BootstrapMixin, forms.Form): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/regions/", value_field="slug", filter_for={ 'site': 'region', @@ -455,7 +435,6 @@ class RackGroupFilterForm(BootstrapMixin, forms.Form): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/sites/", value_field="slug", filter_for={ 'parent': 'site', @@ -507,7 +486,6 @@ class RackForm(BootstrapMixin, TenancyForm, CustomFieldModelForm): site = DynamicModelChoiceField( queryset=Site.objects.all(), widget=APISelect( - api_url="/api/dcim/sites/", filter_for={ 'group': 'site_id', } @@ -515,17 +493,11 @@ class RackForm(BootstrapMixin, TenancyForm, CustomFieldModelForm): ) group = DynamicModelChoiceField( queryset=RackGroup.objects.all(), - required=False, - widget=APISelect( - api_url='/api/dcim/rack-groups/', - ) + required=False ) role = DynamicModelChoiceField( queryset=RackRole.objects.all(), - required=False, - widget=APISelect( - api_url='/api/dcim/rack-roles/', - ) + required=False ) comments = CommentField() tags = TagField( @@ -649,7 +621,6 @@ class RackBulkEditForm(BootstrapMixin, AddRemoveTagsForm, CustomFieldBulkEditFor queryset=Site.objects.all(), required=False, widget=APISelect( - api_url="/api/dcim/sites", filter_for={ 'group': 'site_id', } @@ -657,17 +628,11 @@ class RackBulkEditForm(BootstrapMixin, AddRemoveTagsForm, CustomFieldBulkEditFor ) group = DynamicModelChoiceField( queryset=RackGroup.objects.all(), - required=False, - widget=APISelect( - api_url="/api/dcim/rack-groups", - ) + required=False ) tenant = DynamicModelChoiceField( queryset=Tenant.objects.all(), - required=False, - widget=APISelect( - api_url="/api/tenancy/tenants", - ) + required=False ) status = forms.ChoiceField( choices=add_blank_choice(RackStatusChoices), @@ -677,10 +642,7 @@ class RackBulkEditForm(BootstrapMixin, AddRemoveTagsForm, CustomFieldBulkEditFor ) role = DynamicModelChoiceField( queryset=RackRole.objects.all(), - required=False, - widget=APISelect( - api_url="/api/dcim/rack-roles", - ) + required=False ) serial = forms.CharField( max_length=50, @@ -746,7 +708,6 @@ class RackFilterForm(BootstrapMixin, TenancyFilterForm, CustomFieldFilterForm): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/regions/", value_field="slug", filter_for={ 'site': 'region' @@ -758,7 +719,6 @@ class RackFilterForm(BootstrapMixin, TenancyFilterForm, CustomFieldFilterForm): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/sites/", value_field="slug", filter_for={ 'group_id': 'site' @@ -772,7 +732,6 @@ class RackFilterForm(BootstrapMixin, TenancyFilterForm, CustomFieldFilterForm): required=False, label='Rack group', widget=APISelectMultiple( - api_url="/api/dcim/rack-groups/", null_option=True ) ) @@ -786,7 +745,6 @@ class RackFilterForm(BootstrapMixin, TenancyFilterForm, CustomFieldFilterForm): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/rack-roles/", value_field="slug", null_option=True, ) @@ -805,7 +763,6 @@ class RackElevationFilterForm(RackFilterForm): label='Rack', required=False, widget=APISelectMultiple( - api_url='/api/dcim/racks/', display_field='display_name', ) ) @@ -941,10 +898,7 @@ class RackReservationBulkEditForm(BootstrapMixin, BulkEditForm): ) tenant = DynamicModelChoiceField( queryset=Tenant.objects.all(), - required=False, - widget=APISelect( - api_url="/api/tenancy/tenant", - ) + required=False ) description = forms.CharField( max_length=100, @@ -966,7 +920,6 @@ class RackReservationFilterForm(BootstrapMixin, TenancyFilterForm): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/sites/", value_field="slug", ) ) @@ -975,7 +928,6 @@ class RackReservationFilterForm(BootstrapMixin, TenancyFilterForm): required=False, label='Rack group', widget=APISelectMultiple( - api_url="/api/dcim/rack-groups/", null_option=True, ) ) @@ -1012,10 +964,7 @@ class ManufacturerCSVForm(forms.ModelForm): class DeviceTypeForm(BootstrapMixin, CustomFieldModelForm): manufacturer = DynamicModelChoiceField( - queryset=Manufacturer.objects.all(), - widget=APISelect( - api_url="/api/dcim/manufacturers/", - ) + queryset=Manufacturer.objects.all() ) slug = SlugField( slug_source='model' @@ -1056,10 +1005,7 @@ class DeviceTypeBulkEditForm(BootstrapMixin, AddRemoveTagsForm, CustomFieldBulkE ) manufacturer = DynamicModelChoiceField( queryset=Manufacturer.objects.all(), - required=False, - widget=APISelect( - api_url="/api/dcim/manufacturers" - ) + required=False ) u_height = forms.IntegerField( min_value=1, @@ -1086,7 +1032,6 @@ class DeviceTypeFilterForm(BootstrapMixin, CustomFieldFilterForm): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/manufacturers/", value_field="slug", ) ) @@ -1158,10 +1103,7 @@ class ConsolePortTemplateForm(BootstrapMixin, forms.ModelForm): class ConsolePortTemplateCreateForm(BootstrapMixin, forms.Form): device_type = DynamicModelChoiceField( - queryset=DeviceType.objects.all(), - widget=APISelect( - api_url='/api/dcim/device-types/' - ) + queryset=DeviceType.objects.all() ) name_pattern = ExpandableNameField( label='Name' @@ -1201,10 +1143,7 @@ class ConsoleServerPortTemplateForm(BootstrapMixin, forms.ModelForm): class ConsoleServerPortTemplateCreateForm(BootstrapMixin, forms.Form): device_type = DynamicModelChoiceField( - queryset=DeviceType.objects.all(), - widget=APISelect( - api_url='/api/dcim/device-types/' - ) + queryset=DeviceType.objects.all() ) name_pattern = ExpandableNameField( label='Name' @@ -1244,10 +1183,7 @@ class PowerPortTemplateForm(BootstrapMixin, forms.ModelForm): class PowerPortTemplateCreateForm(BootstrapMixin, forms.Form): device_type = DynamicModelChoiceField( - queryset=DeviceType.objects.all(), - widget=APISelect( - api_url='/api/dcim/device-types/' - ) + queryset=DeviceType.objects.all() ) name_pattern = ExpandableNameField( label='Name' @@ -1317,10 +1253,7 @@ class PowerOutletTemplateForm(BootstrapMixin, forms.ModelForm): class PowerOutletTemplateCreateForm(BootstrapMixin, forms.Form): device_type = DynamicModelChoiceField( - queryset=DeviceType.objects.all(), - widget=APISelect( - api_url='/api/dcim/device-types/' - ) + queryset=DeviceType.objects.all() ) name_pattern = ExpandableNameField( label='Name' @@ -1386,10 +1319,7 @@ class InterfaceTemplateForm(BootstrapMixin, forms.ModelForm): class InterfaceTemplateCreateForm(BootstrapMixin, forms.Form): device_type = DynamicModelChoiceField( - queryset=DeviceType.objects.all(), - widget=APISelect( - api_url='/api/dcim/device-types/' - ) + queryset=DeviceType.objects.all() ) name_pattern = ExpandableNameField( label='Name' @@ -1449,10 +1379,7 @@ class FrontPortTemplateForm(BootstrapMixin, forms.ModelForm): class FrontPortTemplateCreateForm(BootstrapMixin, forms.Form): device_type = DynamicModelChoiceField( - queryset=DeviceType.objects.all(), - widget=APISelect( - api_url='/api/dcim/device-types/' - ) + queryset=DeviceType.objects.all() ) name_pattern = ExpandableNameField( label='Name' @@ -1543,10 +1470,7 @@ class RearPortTemplateForm(BootstrapMixin, forms.ModelForm): class RearPortTemplateCreateForm(BootstrapMixin, forms.Form): device_type = DynamicModelChoiceField( - queryset=DeviceType.objects.all(), - widget=APISelect( - api_url='/api/dcim/device-types/' - ) + queryset=DeviceType.objects.all() ) name_pattern = ExpandableNameField( label='Name' @@ -1592,10 +1516,7 @@ class DeviceBayTemplateForm(BootstrapMixin, forms.ModelForm): class DeviceBayTemplateCreateForm(BootstrapMixin, forms.Form): device_type = DynamicModelChoiceField( - queryset=DeviceType.objects.all(), - widget=APISelect( - api_url='/api/dcim/device-types/' - ) + queryset=DeviceType.objects.all() ) name_pattern = ExpandableNameField( label='Name' @@ -1764,10 +1685,7 @@ class DeviceRoleCSVForm(forms.ModelForm): class PlatformForm(BootstrapMixin, forms.ModelForm): manufacturer = DynamicModelChoiceField( queryset=Manufacturer.objects.all(), - required=False, - widget=APISelect( - api_url="/api/dcim/manufacturers/", - ) + required=False ) slug = SlugField( max_length=64 @@ -1811,7 +1729,6 @@ class DeviceForm(BootstrapMixin, TenancyForm, CustomFieldModelForm): site = DynamicModelChoiceField( queryset=Site.objects.all(), widget=APISelect( - api_url="/api/dcim/sites/", filter_for={ 'rack': 'site_id' } @@ -1821,7 +1738,6 @@ class DeviceForm(BootstrapMixin, TenancyForm, CustomFieldModelForm): queryset=Rack.objects.all(), required=False, widget=APISelect( - api_url='/api/dcim/racks/', display_field='display_name' ) ) @@ -1838,7 +1754,6 @@ class DeviceForm(BootstrapMixin, TenancyForm, CustomFieldModelForm): queryset=Manufacturer.objects.all(), required=False, widget=APISelect( - api_url="/api/dcim/manufacturers/", filter_for={ 'device_type': 'manufacturer_id', 'platform': 'manufacturer_id' @@ -1848,21 +1763,16 @@ class DeviceForm(BootstrapMixin, TenancyForm, CustomFieldModelForm): device_type = DynamicModelChoiceField( queryset=DeviceType.objects.all(), widget=APISelect( - api_url='/api/dcim/device-types/', display_field='model' ) ) device_role = DynamicModelChoiceField( - queryset=DeviceRole.objects.all(), - widget=APISelect( - api_url='/api/dcim/device-roles/' - ) + queryset=DeviceRole.objects.all() ) platform = DynamicModelChoiceField( queryset=Platform.objects.all(), required=False, widget=APISelect( - api_url="/api/dcim/platforms/", additional_query_params={ "manufacturer_id": "null" } @@ -1872,7 +1782,6 @@ class DeviceForm(BootstrapMixin, TenancyForm, CustomFieldModelForm): queryset=ClusterGroup.objects.all(), required=False, widget=APISelect( - api_url="/api/virtualization/cluster-groups/", filter_for={ 'cluster': 'group_id' }, @@ -1883,10 +1792,7 @@ class DeviceForm(BootstrapMixin, TenancyForm, CustomFieldModelForm): ) cluster = DynamicModelChoiceField( queryset=Cluster.objects.all(), - required=False, - widget=APISelect( - api_url='/api/virtualization/clusters/', - ) + required=False ) comments = CommentField() tags = TagField(required=False) @@ -2190,32 +2096,19 @@ class DeviceBulkEditForm(BootstrapMixin, AddRemoveTagsForm, CustomFieldBulkEditF ) device_type = DynamicModelChoiceField( queryset=DeviceType.objects.all(), - required=False, - widget=APISelect( - api_url="/api/dcim/device-types/", - display_field='display_name' - ) + required=False ) device_role = DynamicModelChoiceField( queryset=DeviceRole.objects.all(), - required=False, - widget=APISelect( - api_url="/api/dcim/device-roles/" - ) + required=False ) tenant = DynamicModelChoiceField( queryset=Tenant.objects.all(), - required=False, - widget=APISelect( - api_url="/api/tenancy/tenants/" - ) + required=False ) platform = DynamicModelChoiceField( queryset=Platform.objects.all(), - required=False, - widget=APISelect( - api_url="/api/dcim/platforms/" - ) + required=False ) status = forms.ChoiceField( choices=add_blank_choice(DeviceStatusChoices), @@ -2249,7 +2142,6 @@ class DeviceFilterForm(BootstrapMixin, LocalConfigContextFilterForm, TenancyFilt to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/regions/", value_field="slug", filter_for={ 'site': 'region' @@ -2261,7 +2153,6 @@ class DeviceFilterForm(BootstrapMixin, LocalConfigContextFilterForm, TenancyFilt to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/sites/", value_field="slug", filter_for={ 'rack_group_id': 'site', @@ -2274,7 +2165,6 @@ class DeviceFilterForm(BootstrapMixin, LocalConfigContextFilterForm, TenancyFilt required=False, label='Rack group', widget=APISelectMultiple( - api_url="/api/dcim/rack-groups/", filter_for={ 'rack_id': 'group_id', } @@ -2285,7 +2175,6 @@ class DeviceFilterForm(BootstrapMixin, LocalConfigContextFilterForm, TenancyFilt required=False, label='Rack', widget=APISelectMultiple( - api_url="/api/dcim/racks/", null_option=True, ) ) @@ -2294,7 +2183,6 @@ class DeviceFilterForm(BootstrapMixin, LocalConfigContextFilterForm, TenancyFilt to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/device-roles/", value_field="slug", ) ) @@ -2303,7 +2191,6 @@ class DeviceFilterForm(BootstrapMixin, LocalConfigContextFilterForm, TenancyFilt required=False, label='Manufacturer', widget=APISelectMultiple( - api_url="/api/dcim/manufacturers/", filter_for={ 'device_type_id': 'manufacturer_id', } @@ -2314,7 +2201,6 @@ class DeviceFilterForm(BootstrapMixin, LocalConfigContextFilterForm, TenancyFilt required=False, label='Model', widget=APISelectMultiple( - api_url="/api/dcim/device-types/", display_field="model", ) ) @@ -2323,7 +2209,6 @@ class DeviceFilterForm(BootstrapMixin, LocalConfigContextFilterForm, TenancyFilt to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/platforms/", value_field="slug", null_option=True, ) @@ -2467,10 +2352,7 @@ class ConsolePortForm(BootstrapMixin, forms.ModelForm): class ConsolePortCreateForm(BootstrapMixin, forms.Form): device = DynamicModelChoiceField( - queryset=Device.objects.prefetch_related('device_type__manufacturer'), - widget=APISelect( - api_url="/api/dcim/devices/", - ) + queryset=Device.objects.prefetch_related('device_type__manufacturer') ) name_pattern = ExpandableNameField( label='Name' @@ -2557,10 +2439,7 @@ class ConsoleServerPortForm(BootstrapMixin, forms.ModelForm): class ConsoleServerPortCreateForm(BootstrapMixin, forms.Form): device = DynamicModelChoiceField( - queryset=Device.objects.prefetch_related('device_type__manufacturer'), - widget=APISelect( - api_url="/api/dcim/devices/", - ) + queryset=Device.objects.prefetch_related('device_type__manufacturer') ) name_pattern = ExpandableNameField( label='Name' @@ -2661,10 +2540,7 @@ class PowerPortForm(BootstrapMixin, forms.ModelForm): class PowerPortCreateForm(BootstrapMixin, forms.Form): device = DynamicModelChoiceField( - queryset=Device.objects.prefetch_related('device_type__manufacturer'), - widget=APISelect( - api_url="/api/dcim/devices/", - ) + queryset=Device.objects.prefetch_related('device_type__manufacturer') ) name_pattern = ExpandableNameField( label='Name' @@ -2784,10 +2660,7 @@ class PowerOutletForm(BootstrapMixin, forms.ModelForm): class PowerOutletCreateForm(BootstrapMixin, forms.Form): device = DynamicModelChoiceField( - queryset=Device.objects.prefetch_related('device_type__manufacturer'), - widget=APISelect( - api_url="/api/dcim/devices/", - ) + queryset=Device.objects.prefetch_related('device_type__manufacturer') ) name_pattern = ExpandableNameField( label='Name' @@ -2959,7 +2832,6 @@ class InterfaceForm(InterfaceCommonForm, BootstrapMixin, forms.ModelForm): required=False, label='Untagged VLAN', widget=APISelect( - api_url="/api/ipam/vlans/", display_field='display_name', full=True, additional_query_params={ @@ -2972,7 +2844,6 @@ class InterfaceForm(InterfaceCommonForm, BootstrapMixin, forms.ModelForm): required=False, label='Tagged VLANs', widget=APISelectMultiple( - api_url="/api/ipam/vlans/", display_field='display_name', full=True, additional_query_params={ @@ -3024,10 +2895,7 @@ class InterfaceForm(InterfaceCommonForm, BootstrapMixin, forms.ModelForm): class InterfaceCreateForm(BootstrapMixin, InterfaceCommonForm, forms.Form): device = DynamicModelChoiceField( - queryset=Device.objects.prefetch_related('device_type__manufacturer'), - widget=APISelect( - api_url="/api/dcim/devices/", - ) + queryset=Device.objects.prefetch_related('device_type__manufacturer') ) name_pattern = ExpandableNameField( label='Name' @@ -3077,7 +2945,6 @@ class InterfaceCreateForm(BootstrapMixin, InterfaceCommonForm, forms.Form): queryset=VLAN.objects.all(), required=False, widget=APISelect( - api_url="/api/ipam/vlans/", display_field='display_name', full=True, additional_query_params={ @@ -3089,7 +2956,6 @@ class InterfaceCreateForm(BootstrapMixin, InterfaceCommonForm, forms.Form): queryset=VLAN.objects.all(), required=False, widget=APISelectMultiple( - api_url="/api/ipam/vlans/", display_field='display_name', full=True, additional_query_params={ @@ -3236,7 +3102,6 @@ class InterfaceBulkEditForm(BootstrapMixin, AddRemoveTagsForm, BulkEditForm): queryset=VLAN.objects.all(), required=False, widget=APISelect( - api_url="/api/ipam/vlans/", display_field='display_name', full=True, additional_query_params={ @@ -3248,7 +3113,6 @@ class InterfaceBulkEditForm(BootstrapMixin, AddRemoveTagsForm, BulkEditForm): queryset=VLAN.objects.all(), required=False, widget=APISelectMultiple( - api_url="/api/ipam/vlans/", display_field='display_name', full=True, additional_query_params={ @@ -3350,10 +3214,7 @@ class FrontPortForm(BootstrapMixin, forms.ModelForm): # TODO: Merge with FrontPortTemplateCreateForm to remove duplicate logic class FrontPortCreateForm(BootstrapMixin, forms.Form): device = DynamicModelChoiceField( - queryset=Device.objects.prefetch_related('device_type__manufacturer'), - widget=APISelect( - api_url="/api/dcim/devices/", - ) + queryset=Device.objects.prefetch_related('device_type__manufacturer') ) name_pattern = ExpandableNameField( label='Name' @@ -3533,10 +3394,7 @@ class RearPortForm(BootstrapMixin, forms.ModelForm): class RearPortCreateForm(BootstrapMixin, forms.Form): device = DynamicModelChoiceField( - queryset=Device.objects.prefetch_related('device_type__manufacturer'), - widget=APISelect( - api_url="/api/dcim/devices/", - ) + queryset=Device.objects.prefetch_related('device_type__manufacturer') ) name_pattern = ExpandableNameField( label='Name' @@ -3622,7 +3480,6 @@ class ConnectCableToDeviceForm(BootstrapMixin, forms.ModelForm): label='Site', required=False, widget=APISelect( - api_url='/api/dcim/sites/', filter_for={ 'termination_b_rack': 'site_id', 'termination_b_device': 'site_id', @@ -3634,7 +3491,6 @@ class ConnectCableToDeviceForm(BootstrapMixin, forms.ModelForm): label='Rack', required=False, widget=APISelect( - api_url='/api/dcim/racks/', filter_for={ 'termination_b_device': 'rack_id', }, @@ -3648,7 +3504,6 @@ class ConnectCableToDeviceForm(BootstrapMixin, forms.ModelForm): label='Device', required=False, widget=APISelect( - api_url='/api/dcim/devices/', display_field='display_name', filter_for={ 'termination_b_id': 'device_id', @@ -3748,7 +3603,6 @@ class ConnectCableToCircuitTerminationForm(BootstrapMixin, forms.ModelForm): label='Provider', required=False, widget=APISelect( - api_url='/api/circuits/providers/', filter_for={ 'termination_b_circuit': 'provider_id', } @@ -3759,7 +3613,6 @@ class ConnectCableToCircuitTerminationForm(BootstrapMixin, forms.ModelForm): label='Site', required=False, widget=APISelect( - api_url='/api/dcim/sites/', filter_for={ 'termination_b_circuit': 'site_id', } @@ -3769,7 +3622,6 @@ class ConnectCableToCircuitTerminationForm(BootstrapMixin, forms.ModelForm): queryset=Circuit.objects.all(), label='Circuit', widget=APISelect( - api_url='/api/circuits/circuits/', display_field='cid', filter_for={ 'termination_b_id': 'circuit_id', @@ -3800,7 +3652,6 @@ class ConnectCableToPowerFeedForm(BootstrapMixin, forms.ModelForm): label='Site', required=False, widget=APISelect( - api_url='/api/dcim/sites/', display_field='cid', filter_for={ 'termination_b_rackgroup': 'site_id', @@ -3813,7 +3664,6 @@ class ConnectCableToPowerFeedForm(BootstrapMixin, forms.ModelForm): label='Rack Group', required=False, widget=APISelect( - api_url='/api/dcim/rack-groups/', display_field='cid', filter_for={ 'termination_b_powerpanel': 'rackgroup_id', @@ -3825,7 +3675,6 @@ class ConnectCableToPowerFeedForm(BootstrapMixin, forms.ModelForm): label='Power Panel', required=False, widget=APISelect( - api_url='/api/dcim/power-panels/', filter_for={ 'termination_b_id': 'power_panel_id', } @@ -4050,7 +3899,6 @@ class CableFilterForm(BootstrapMixin, forms.Form): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/sites/", value_field="slug", filter_for={ 'rack_id': 'site', @@ -4063,7 +3911,6 @@ class CableFilterForm(BootstrapMixin, forms.Form): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/tenancy/tenants/", value_field='slug', filter_for={ 'device_id': 'tenant', @@ -4075,7 +3922,6 @@ class CableFilterForm(BootstrapMixin, forms.Form): required=False, label='Rack', widget=APISelectMultiple( - api_url="/api/dcim/racks/", null_option=True, filter_for={ 'device_id': 'rack_id', @@ -4100,10 +3946,7 @@ class CableFilterForm(BootstrapMixin, forms.Form): device_id = DynamicModelMultipleChoiceField( queryset=Device.objects.all(), required=False, - label='Device', - widget=APISelectMultiple( - api_url='/api/dcim/devices/', - ) + label='Device' ) @@ -4133,10 +3976,7 @@ class DeviceBayForm(BootstrapMixin, forms.ModelForm): class DeviceBayCreateForm(BootstrapMixin, forms.Form): device = DynamicModelChoiceField( - queryset=Device.objects.prefetch_related('device_type__manufacturer'), - widget=APISelect( - api_url="/api/dcim/devices/", - ) + queryset=Device.objects.prefetch_related('device_type__manufacturer') ) name_pattern = ExpandableNameField( label='Name' @@ -4234,7 +4074,6 @@ class ConsoleConnectionFilterForm(BootstrapMixin, forms.Form): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/sites/", value_field="slug", filter_for={ 'device_id': 'site', @@ -4244,10 +4083,7 @@ class ConsoleConnectionFilterForm(BootstrapMixin, forms.Form): device_id = DynamicModelMultipleChoiceField( queryset=Device.objects.all(), required=False, - label='Device', - widget=APISelectMultiple( - api_url='/api/dcim/devices/', - ) + label='Device' ) @@ -4257,7 +4093,6 @@ class PowerConnectionFilterForm(BootstrapMixin, forms.Form): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/sites/", value_field="slug", filter_for={ 'device_id': 'site', @@ -4267,10 +4102,7 @@ class PowerConnectionFilterForm(BootstrapMixin, forms.Form): device_id = DynamicModelMultipleChoiceField( queryset=Device.objects.all(), required=False, - label='Device', - widget=APISelectMultiple( - api_url='/api/dcim/devices/', - ) + label='Device' ) @@ -4280,7 +4112,6 @@ class InterfaceConnectionFilterForm(BootstrapMixin, forms.Form): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/sites/", value_field="slug", filter_for={ 'device_id': 'site', @@ -4290,10 +4121,7 @@ class InterfaceConnectionFilterForm(BootstrapMixin, forms.Form): device_id = DynamicModelMultipleChoiceField( queryset=Device.objects.all(), required=False, - label='Device', - widget=APISelectMultiple( - api_url='/api/dcim/devices/', - ) + label='Device' ) @@ -4303,17 +4131,11 @@ class InterfaceConnectionFilterForm(BootstrapMixin, forms.Form): class InventoryItemForm(BootstrapMixin, forms.ModelForm): device = DynamicModelChoiceField( - queryset=Device.objects.prefetch_related('device_type__manufacturer'), - widget=APISelect( - api_url="/api/dcim/devices/" - ) + queryset=Device.objects.prefetch_related('device_type__manufacturer') ) manufacturer = DynamicModelChoiceField( queryset=Manufacturer.objects.all(), - required=False, - widget=APISelect( - api_url="/api/dcim/manufacturers/" - ) + required=False ) tags = TagField( required=False @@ -4328,20 +4150,14 @@ class InventoryItemForm(BootstrapMixin, forms.ModelForm): class InventoryItemCreateForm(BootstrapMixin, forms.Form): device = DynamicModelChoiceField( - queryset=Device.objects.prefetch_related('device_type__manufacturer'), - widget=APISelect( - api_url="/api/dcim/devices/", - ) + queryset=Device.objects.prefetch_related('device_type__manufacturer') ) name_pattern = ExpandableNameField( label='Name' ) manufacturer = DynamicModelChoiceField( queryset=Manufacturer.objects.all(), - required=False, - widget=APISelect( - api_url="/api/dcim/manufacturers/" - ) + required=False ) part_id = forms.CharField( max_length=50, @@ -4393,17 +4209,11 @@ class InventoryItemBulkEditForm(BootstrapMixin, BulkEditForm): ) device = DynamicModelChoiceField( queryset=Device.objects.all(), - required=False, - widget=APISelect( - api_url="/api/dcim/devices/" - ) + required=False ) manufacturer = DynamicModelChoiceField( queryset=Manufacturer.objects.all(), - required=False, - widget=APISelect( - api_url="/api/dcim/manufacturers/" - ) + required=False ) part_id = forms.CharField( max_length=50, @@ -4432,7 +4242,6 @@ class InventoryItemFilterForm(BootstrapMixin, forms.Form): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/regions/", value_field="slug", filter_for={ 'site': 'region' @@ -4444,7 +4253,6 @@ class InventoryItemFilterForm(BootstrapMixin, forms.Form): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/sites/", value_field="slug", filter_for={ 'device_id': 'site' @@ -4454,17 +4262,13 @@ class InventoryItemFilterForm(BootstrapMixin, forms.Form): device_id = DynamicModelMultipleChoiceField( queryset=Device.objects.all(), required=False, - label='Device', - widget=APISelect( - api_url='/api/dcim/devices/', - ) + label='Device' ) manufacturer = DynamicModelMultipleChoiceField( queryset=Manufacturer.objects.all(), to_field_name='slug', required=False, widget=APISelect( - api_url="/api/dcim/manufacturers/", value_field="slug", ) ) @@ -4562,7 +4366,6 @@ class VCMemberSelectForm(BootstrapMixin, forms.Form): queryset=Site.objects.all(), required=False, widget=APISelect( - api_url="/api/dcim/sites/", filter_for={ 'rack': 'site_id', 'device': 'site_id', @@ -4573,7 +4376,6 @@ class VCMemberSelectForm(BootstrapMixin, forms.Form): queryset=Rack.objects.all(), required=False, widget=APISelect( - api_url='/api/dcim/racks/', filter_for={ 'device': 'rack_id' }, @@ -4587,7 +4389,6 @@ class VCMemberSelectForm(BootstrapMixin, forms.Form): virtual_chassis__isnull=True ), widget=APISelect( - api_url='/api/dcim/devices/', display_field='display_name', disabled_indicator='virtual_chassis' ) @@ -4613,7 +4414,6 @@ class VirtualChassisFilterForm(BootstrapMixin, CustomFieldFilterForm): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/regions/", value_field="slug", filter_for={ 'site': 'region' @@ -4625,7 +4425,6 @@ class VirtualChassisFilterForm(BootstrapMixin, CustomFieldFilterForm): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/sites/", value_field="slug", ) ) @@ -4634,7 +4433,6 @@ class VirtualChassisFilterForm(BootstrapMixin, CustomFieldFilterForm): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/tenancy/tenant-groups/", value_field="slug", null_option=True, filter_for={ @@ -4647,7 +4445,6 @@ class VirtualChassisFilterForm(BootstrapMixin, CustomFieldFilterForm): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/tenancy/tenants/", value_field="slug", null_option=True, ) @@ -4663,7 +4460,6 @@ class PowerPanelForm(BootstrapMixin, forms.ModelForm): site = DynamicModelChoiceField( queryset=Site.objects.all(), widget=APISelect( - api_url="/api/dcim/sites/", filter_for={ 'rack_group': 'site_id', } @@ -4671,10 +4467,7 @@ class PowerPanelForm(BootstrapMixin, forms.ModelForm): ) rack_group = DynamicModelChoiceField( queryset=RackGroup.objects.all(), - required=False, - widget=APISelect( - api_url='/api/dcim/rack-groups/', - ) + required=False ) class Meta: @@ -4728,7 +4521,6 @@ class PowerPanelBulkEditForm(BootstrapMixin, BulkEditForm): queryset=Site.objects.all(), required=False, widget=APISelect( - api_url="/api/dcim/sites/", filter_for={ 'rack_group': 'site_id', } @@ -4736,10 +4528,7 @@ class PowerPanelBulkEditForm(BootstrapMixin, BulkEditForm): ) rack_group = DynamicModelChoiceField( queryset=RackGroup.objects.all(), - required=False, - widget=APISelect( - api_url="/api/dcim/rack-groups/" - ) + required=False ) class Meta: @@ -4759,7 +4548,6 @@ class PowerPanelFilterForm(BootstrapMixin, CustomFieldFilterForm): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/regions/", value_field="slug", filter_for={ 'site': 'region' @@ -4771,7 +4559,6 @@ class PowerPanelFilterForm(BootstrapMixin, CustomFieldFilterForm): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/sites/", value_field="slug", filter_for={ 'rack_group_id': 'site', @@ -4783,7 +4570,6 @@ class PowerPanelFilterForm(BootstrapMixin, CustomFieldFilterForm): required=False, label='Rack group (ID)', widget=APISelectMultiple( - api_url="/api/dcim/rack-groups/", null_option=True, ) ) @@ -4798,7 +4584,6 @@ class PowerFeedForm(BootstrapMixin, CustomFieldModelForm): queryset=Site.objects.all(), required=False, widget=APISelect( - api_url='/api/dcim/sites/', filter_for={ 'power_panel': 'site_id', 'rack': 'site_id', @@ -4806,17 +4591,11 @@ class PowerFeedForm(BootstrapMixin, CustomFieldModelForm): ) ) power_panel = DynamicModelChoiceField( - queryset=PowerPanel.objects.all(), - widget=APISelect( - api_url="/api/dcim/power-panels/" - ) + queryset=PowerPanel.objects.all() ) rack = DynamicModelChoiceField( queryset=Rack.objects.all(), - required=False, - widget=APISelect( - api_url="/api/dcim/racks/" - ) + required=False ) comments = CommentField() tags = TagField( @@ -4932,7 +4711,6 @@ class PowerFeedBulkEditForm(BootstrapMixin, AddRemoveTagsForm, CustomFieldBulkEd queryset=PowerPanel.objects.all(), required=False, widget=APISelect( - api_url="/api/dcim/power-panels/", filter_for={ 'rackgroup': 'site_id', } @@ -4940,10 +4718,7 @@ class PowerFeedBulkEditForm(BootstrapMixin, AddRemoveTagsForm, CustomFieldBulkEd ) rack = DynamicModelChoiceField( queryset=Rack.objects.all(), - required=False, - widget=APISelect( - api_url="/api/dcim/racks", - ) + required=False ) status = forms.ChoiceField( choices=add_blank_choice(PowerFeedStatusChoices), @@ -5000,7 +4775,6 @@ class PowerFeedFilterForm(BootstrapMixin, CustomFieldFilterForm): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/regions/", value_field="slug", filter_for={ 'site': 'region' @@ -5012,7 +4786,6 @@ class PowerFeedFilterForm(BootstrapMixin, CustomFieldFilterForm): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/sites/", value_field="slug", filter_for={ 'power_panel_id': 'site', @@ -5025,7 +4798,6 @@ class PowerFeedFilterForm(BootstrapMixin, CustomFieldFilterForm): required=False, label='Power panel', widget=APISelectMultiple( - api_url="/api/dcim/power-panels/", null_option=True, ) ) @@ -5034,7 +4806,6 @@ class PowerFeedFilterForm(BootstrapMixin, CustomFieldFilterForm): required=False, label='Rack', widget=APISelectMultiple( - api_url="/api/dcim/racks/", null_option=True, ) ) diff --git a/netbox/extras/forms.py b/netbox/extras/forms.py index 85d19756a..7ec9d2285 100644 --- a/netbox/extras/forms.py +++ b/netbox/extras/forms.py @@ -201,60 +201,36 @@ class ConfigContextForm(BootstrapMixin, forms.ModelForm): ) sites = DynamicModelMultipleChoiceField( queryset=Site.objects.all(), - required=False, - widget=APISelectMultiple( - api_url="/api/dcim/sites/" - ) + required=False ) roles = DynamicModelMultipleChoiceField( queryset=DeviceRole.objects.all(), - required=False, - widget=APISelectMultiple( - api_url="/api/dcim/device-roles/" - ) + required=False ) platforms = DynamicModelMultipleChoiceField( queryset=Platform.objects.all(), - required=False, - widget=APISelectMultiple( - api_url="/api/dcim/platforms/" - ) + required=False ) cluster_groups = DynamicModelMultipleChoiceField( queryset=ClusterGroup.objects.all(), - required=False, - widget=APISelectMultiple( - api_url="/api/virtualization/cluster-groups/" - ) + required=False ) clusters = DynamicModelMultipleChoiceField( queryset=Cluster.objects.all(), - required=False, - widget=APISelectMultiple( - api_url="/api/virtualization/clusters/" - ) + required=False ) tenant_groups = DynamicModelMultipleChoiceField( queryset=TenantGroup.objects.all(), - required=False, - widget=APISelectMultiple( - api_url="/api/tenancy/tenant-groups/" - ) + required=False ) tenants = DynamicModelMultipleChoiceField( queryset=Tenant.objects.all(), - required=False, - widget=APISelectMultiple( - api_url="/api/tenancy/tenants/" - ) + required=False ) tags = DynamicModelMultipleChoiceField( queryset=Tag.objects.all(), to_field_name='slug', - required=False, - widget=APISelectMultiple( - api_url="/api/extras/tags/" - ) + required=False ) data = JSONField( label='' @@ -302,7 +278,6 @@ class ConfigContextFilterForm(BootstrapMixin, forms.Form): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/regions/", value_field="slug", ) ) @@ -311,7 +286,6 @@ class ConfigContextFilterForm(BootstrapMixin, forms.Form): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/sites/", value_field="slug", ) ) @@ -320,7 +294,6 @@ class ConfigContextFilterForm(BootstrapMixin, forms.Form): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/device-roles/", value_field="slug", ) ) @@ -329,7 +302,6 @@ class ConfigContextFilterForm(BootstrapMixin, forms.Form): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/platforms/", value_field="slug", ) ) @@ -338,24 +310,19 @@ class ConfigContextFilterForm(BootstrapMixin, forms.Form): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/virtualization/cluster-groups/", value_field="slug", ) ) cluster_id = DynamicModelMultipleChoiceField( queryset=Cluster.objects.all(), required=False, - label='Cluster', - widget=APISelectMultiple( - api_url="/api/virtualization/clusters/", - ) + label='Cluster' ) tenant_group = DynamicModelMultipleChoiceField( queryset=TenantGroup.objects.all(), to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/tenancy/tenant-groups/", value_field="slug", ) ) @@ -364,7 +331,6 @@ class ConfigContextFilterForm(BootstrapMixin, forms.Form): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/tenancy/tenants/", value_field="slug", ) ) @@ -373,7 +339,6 @@ class ConfigContextFilterForm(BootstrapMixin, forms.Form): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/extras/tags/", value_field="slug", ) ) diff --git a/netbox/extras/management/commands/rqworker.py b/netbox/extras/management/commands/rqworker.py new file mode 100644 index 000000000..02e93c4ef --- /dev/null +++ b/netbox/extras/management/commands/rqworker.py @@ -0,0 +1,16 @@ +from django.conf import settings +from django_rq.management.commands.rqworker import Command as _Command + + +class Command(_Command): + """ + Subclass django_rq's built-in rqworker to listen on all configured queues if none are specified (instead + of only the 'default' queue). + """ + def handle(self, *args, **options): + + # If no queues have been specified on the command line, listen on all configured queues. + if len(args) < 1: + args = settings.RQ_QUEUES + + super().handle(*args, **options) diff --git a/netbox/extras/registry.py b/netbox/extras/registry.py new file mode 100644 index 000000000..cb58f5135 --- /dev/null +++ b/netbox/extras/registry.py @@ -0,0 +1,21 @@ +class Registry(dict): + """ + Central registry for registration of functionality. Once a store (key) is defined, it cannot be overwritten or + deleted (although its value may be manipulated). + """ + def __getitem__(self, key): + try: + return super().__getitem__(key) + except KeyError: + raise KeyError("Invalid store: {}".format(key)) + + def __setitem__(self, key, value): + if key in self: + raise KeyError("Store already set: {}".format(key)) + super().__setitem__(key, value) + + def __delitem__(self, key): + raise TypeError("Cannot delete stores from registry") + + +registry = Registry() diff --git a/netbox/extras/tests/test_registry.py b/netbox/extras/tests/test_registry.py new file mode 100644 index 000000000..53ba6584a --- /dev/null +++ b/netbox/extras/tests/test_registry.py @@ -0,0 +1,33 @@ +from django.test import TestCase + +from extras.registry import Registry + + +class RegistryTest(TestCase): + + def test_add_store(self): + reg = Registry() + reg['foo'] = 123 + + self.assertEqual(reg['foo'], 123) + + def test_manipulate_store(self): + reg = Registry() + reg['foo'] = [1, 2] + reg['foo'].append(3) + + self.assertListEqual(reg['foo'], [1, 2, 3]) + + def test_overwrite_store(self): + reg = Registry() + reg['foo'] = 123 + + with self.assertRaises(KeyError): + reg['foo'] = 456 + + def test_delete_store(self): + reg = Registry() + reg['foo'] = 123 + + with self.assertRaises(TypeError): + del(reg['foo']) diff --git a/netbox/extras/utils.py b/netbox/extras/utils.py index 5ae16bff9..78214fe41 100644 --- a/netbox/extras/utils.py +++ b/netbox/extras/utils.py @@ -6,6 +6,7 @@ from taggit.managers import _TaggableManager from utilities.querysets import DummyQuerySet from extras.constants import EXTRAS_FEATURES +from extras.registry import registry def is_taggable(obj): @@ -21,33 +22,12 @@ def is_taggable(obj): return False -# -# Dynamic feature registration -# - -class Registry: - """ - The registry is a place to hook into for data storage across components - """ - - def add_store(self, store_name, initial_value=None): - """ - Given the name of some new data parameter and an optional initial value, setup the registry store - """ - if not hasattr(Registry, store_name): - setattr(Registry, store_name, initial_value) - - -registry = Registry() - - @deconstructible class FeatureQuery: """ - Helper class that delays evaluation of the registry contents for the functionaility store + Helper class that delays evaluation of the registry contents for the functionality store until it has been populated. """ - def __init__(self, feature): self.feature = feature @@ -59,24 +39,26 @@ class FeatureQuery: Given an extras feature, return a Q object for content type lookup """ query = Q() - for app_label, models in registry.model_feature_store[self.feature].items(): + for app_label, models in registry['model_features'][self.feature].items(): query |= Q(app_label=app_label, model__in=models) return query -registry.add_store('model_feature_store', {f: collections.defaultdict(list) for f in EXTRAS_FEATURES}) - - def extras_features(*features): """ Decorator used to register extras provided features to a model """ def wrapper(model_class): + # Initialize the model_features store if not already defined + if 'model_features' not in registry: + registry['model_features'] = { + f: collections.defaultdict(list) for f in EXTRAS_FEATURES + } for feature in features: if feature in EXTRAS_FEATURES: app_label, model_name = model_class._meta.label_lower.split('.') - registry.model_feature_store[feature][app_label].append(model_name) + registry['model_features'][feature][app_label].append(model_name) else: raise ValueError('{} is not a valid extras feature!'.format(feature)) return model_class diff --git a/netbox/ipam/forms.py b/netbox/ipam/forms.py index 3c555a8fc..f2d4c9c5a 100644 --- a/netbox/ipam/forms.py +++ b/netbox/ipam/forms.py @@ -78,10 +78,7 @@ class VRFBulkEditForm(BootstrapMixin, AddRemoveTagsForm, CustomFieldBulkEditForm ) tenant = DynamicModelChoiceField( queryset=Tenant.objects.all(), - required=False, - widget=APISelect( - api_url="/api/tenancy/tenants/" - ) + required=False ) enforce_unique = forms.NullBooleanField( required=False, @@ -150,10 +147,7 @@ class RIRFilterForm(BootstrapMixin, forms.Form): class AggregateForm(BootstrapMixin, CustomFieldModelForm): rir = DynamicModelChoiceField( - queryset=RIR.objects.all(), - widget=APISelect( - api_url="/api/ipam/rirs/" - ) + queryset=RIR.objects.all() ) tags = TagField( required=False @@ -196,10 +190,7 @@ class AggregateBulkEditForm(BootstrapMixin, AddRemoveTagsForm, CustomFieldBulkEd rir = DynamicModelChoiceField( queryset=RIR.objects.all(), required=False, - label='RIR', - widget=APISelect( - api_url="/api/ipam/rirs/" - ) + label='RIR' ) date_added = forms.DateField( required=False @@ -236,7 +227,6 @@ class AggregateFilterForm(BootstrapMixin, CustomFieldFilterForm): required=False, label='RIR', widget=APISelectMultiple( - api_url="/api/ipam/rirs/", value_field="slug", ) ) @@ -276,16 +266,12 @@ class PrefixForm(BootstrapMixin, TenancyForm, CustomFieldModelForm): vrf = DynamicModelChoiceField( queryset=VRF.objects.all(), required=False, - label='VRF', - widget=APISelect( - api_url="/api/ipam/vrfs/", - ) + label='VRF' ) site = DynamicModelChoiceField( queryset=Site.objects.all(), required=False, widget=APISelect( - api_url="/api/dcim/sites/", filter_for={ 'vlan_group': 'site_id', 'vlan': 'site_id', @@ -300,7 +286,6 @@ class PrefixForm(BootstrapMixin, TenancyForm, CustomFieldModelForm): required=False, label='VLAN group', widget=APISelect( - api_url='/api/ipam/vlan-groups/', filter_for={ 'vlan': 'group_id' }, @@ -314,16 +299,12 @@ class PrefixForm(BootstrapMixin, TenancyForm, CustomFieldModelForm): required=False, label='VLAN', widget=APISelect( - api_url='/api/ipam/vlans/', display_field='display_name' ) ) role = DynamicModelChoiceField( queryset=Role.objects.all(), - required=False, - widget=APISelect( - api_url="/api/ipam/roles/" - ) + required=False ) tags = TagField(required=False) @@ -447,18 +428,12 @@ class PrefixBulkEditForm(BootstrapMixin, AddRemoveTagsForm, CustomFieldBulkEditF ) site = DynamicModelChoiceField( queryset=Site.objects.all(), - required=False, - widget=APISelect( - api_url="/api/dcim/sites/" - ) + required=False ) vrf = DynamicModelChoiceField( queryset=VRF.objects.all(), required=False, - label='VRF', - widget=APISelect( - api_url="/api/ipam/vrfs/" - ) + label='VRF' ) prefix_length = forms.IntegerField( min_value=PREFIX_LENGTH_MIN, @@ -467,10 +442,7 @@ class PrefixBulkEditForm(BootstrapMixin, AddRemoveTagsForm, CustomFieldBulkEditF ) tenant = DynamicModelChoiceField( queryset=Tenant.objects.all(), - required=False, - widget=APISelect( - api_url="/api/tenancy/tenants/" - ) + required=False ) status = forms.ChoiceField( choices=add_blank_choice(PrefixStatusChoices), @@ -479,10 +451,7 @@ class PrefixBulkEditForm(BootstrapMixin, AddRemoveTagsForm, CustomFieldBulkEditF ) role = DynamicModelChoiceField( queryset=Role.objects.all(), - required=False, - widget=APISelect( - api_url="/api/ipam/roles/" - ) + required=False ) is_pool = forms.NullBooleanField( required=False, @@ -536,7 +505,6 @@ class PrefixFilterForm(BootstrapMixin, TenancyFilterForm, CustomFieldFilterForm) required=False, label='VRF', widget=APISelectMultiple( - api_url="/api/ipam/vrfs/", null_option=True, ) ) @@ -550,7 +518,6 @@ class PrefixFilterForm(BootstrapMixin, TenancyFilterForm, CustomFieldFilterForm) to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/regions/", value_field="slug", filter_for={ 'site': 'region' @@ -562,7 +529,6 @@ class PrefixFilterForm(BootstrapMixin, TenancyFilterForm, CustomFieldFilterForm) to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/sites/", value_field="slug", null_option=True, ) @@ -572,7 +538,6 @@ class PrefixFilterForm(BootstrapMixin, TenancyFilterForm, CustomFieldFilterForm) to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/ipam/roles/", value_field="slug", null_option=True, ) @@ -603,17 +568,13 @@ class IPAddressForm(BootstrapMixin, TenancyForm, ReturnURLForm, CustomFieldModel vrf = DynamicModelChoiceField( queryset=VRF.objects.all(), required=False, - label='VRF', - widget=APISelect( - api_url="/api/ipam/vrfs/" - ) + label='VRF' ) nat_site = DynamicModelChoiceField( queryset=Site.objects.all(), required=False, label='Site', widget=APISelect( - api_url="/api/dcim/sites/", filter_for={ 'nat_rack': 'site_id', 'nat_device': 'site_id' @@ -625,7 +586,6 @@ class IPAddressForm(BootstrapMixin, TenancyForm, ReturnURLForm, CustomFieldModel required=False, label='Rack', widget=APISelect( - api_url='/api/dcim/racks/', display_field='display_name', filter_for={ 'nat_device': 'rack_id' @@ -640,19 +600,17 @@ class IPAddressForm(BootstrapMixin, TenancyForm, ReturnURLForm, CustomFieldModel required=False, label='Device', widget=APISelect( - api_url='/api/dcim/devices/', display_field='display_name', filter_for={ 'nat_inside': 'device_id' } ) ) - nat_vrf = forms.ModelChoiceField( + nat_vrf = DynamicModelChoiceField( queryset=VRF.objects.all(), required=False, label='VRF', widget=APISelect( - api_url="/api/ipam/vrfs/", filter_for={ 'nat_inside': 'vrf_id' } @@ -663,7 +621,6 @@ class IPAddressForm(BootstrapMixin, TenancyForm, ReturnURLForm, CustomFieldModel required=False, label='IP Address', widget=APISelect( - api_url='/api/ipam/ip-addresses/', display_field='address' ) ) @@ -761,10 +718,7 @@ class IPAddressBulkAddForm(BootstrapMixin, TenancyForm, CustomFieldModelForm): vrf = DynamicModelChoiceField( queryset=VRF.objects.all(), required=False, - label='VRF', - widget=APISelect( - api_url="/api/ipam/vrfs/" - ) + label='VRF' ) class Meta: @@ -913,10 +867,7 @@ class IPAddressBulkEditForm(BootstrapMixin, AddRemoveTagsForm, CustomFieldBulkEd vrf = DynamicModelChoiceField( queryset=VRF.objects.all(), required=False, - label='VRF', - widget=APISelect( - api_url="/api/ipam/vrfs/" - ) + label='VRF' ) mask_length = forms.IntegerField( min_value=IPADDRESS_MASK_LENGTH_MIN, @@ -925,10 +876,7 @@ class IPAddressBulkEditForm(BootstrapMixin, AddRemoveTagsForm, CustomFieldBulkEd ) tenant = DynamicModelChoiceField( queryset=Tenant.objects.all(), - required=False, - widget=APISelect( - api_url="/api/tenancy/tenants/" - ) + required=False ) status = forms.ChoiceField( choices=add_blank_choice(IPAddressStatusChoices), @@ -960,10 +908,7 @@ class IPAddressAssignForm(BootstrapMixin, forms.Form): queryset=VRF.objects.all(), required=False, label='VRF', - empty_label='Global', - widget=APISelect( - api_url="/api/ipam/vrfs/" - ) + empty_label='Global' ) q = forms.CharField( required=False, @@ -1007,7 +952,6 @@ class IPAddressFilterForm(BootstrapMixin, TenancyFilterForm, CustomFieldFilterFo required=False, label='VRF', widget=APISelectMultiple( - api_url="/api/ipam/vrfs/", null_option=True, ) ) @@ -1038,10 +982,7 @@ class IPAddressFilterForm(BootstrapMixin, TenancyFilterForm, CustomFieldFilterFo class VLANGroupForm(BootstrapMixin, forms.ModelForm): site = DynamicModelChoiceField( queryset=Site.objects.all(), - required=False, - widget=APISelect( - api_url="/api/dcim/sites/" - ) + required=False ) slug = SlugField() @@ -1078,7 +1019,6 @@ class VLANGroupFilterForm(BootstrapMixin, forms.Form): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/regions/", value_field="slug", filter_for={ 'site': 'region', @@ -1090,7 +1030,6 @@ class VLANGroupFilterForm(BootstrapMixin, forms.Form): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/sites/", value_field="slug", null_option=True, ) @@ -1106,7 +1045,6 @@ class VLANForm(BootstrapMixin, TenancyForm, CustomFieldModelForm): queryset=Site.objects.all(), required=False, widget=APISelect( - api_url="/api/dcim/sites/", filter_for={ 'group': 'site_id' }, @@ -1117,17 +1055,11 @@ class VLANForm(BootstrapMixin, TenancyForm, CustomFieldModelForm): ) group = DynamicModelChoiceField( queryset=VLANGroup.objects.all(), - required=False, - widget=APISelect( - api_url='/api/ipam/vlan-groups/', - ) + required=False ) role = DynamicModelChoiceField( queryset=Role.objects.all(), - required=False, - widget=APISelect( - api_url="/api/ipam/roles/" - ) + required=False ) tags = TagField(required=False) @@ -1222,24 +1154,15 @@ class VLANBulkEditForm(BootstrapMixin, AddRemoveTagsForm, CustomFieldBulkEditFor ) site = DynamicModelChoiceField( queryset=Site.objects.all(), - required=False, - widget=APISelect( - api_url="/api/dcim/sites/" - ) + required=False ) group = DynamicModelChoiceField( queryset=VLANGroup.objects.all(), - required=False, - widget=APISelect( - api_url="/api/ipam/vlan-groups/" - ) + required=False ) tenant = DynamicModelChoiceField( queryset=Tenant.objects.all(), - required=False, - widget=APISelect( - api_url="/api/tenancy/tenants/" - ) + required=False ) status = forms.ChoiceField( choices=add_blank_choice(VLANStatusChoices), @@ -1248,10 +1171,7 @@ class VLANBulkEditForm(BootstrapMixin, AddRemoveTagsForm, CustomFieldBulkEditFor ) role = DynamicModelChoiceField( queryset=Role.objects.all(), - required=False, - widget=APISelect( - api_url="/api/ipam/roles/" - ) + required=False ) description = forms.CharField( max_length=100, @@ -1276,7 +1196,6 @@ class VLANFilterForm(BootstrapMixin, TenancyFilterForm, CustomFieldFilterForm): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/regions/", value_field="slug", filter_for={ 'site': 'region', @@ -1289,7 +1208,6 @@ class VLANFilterForm(BootstrapMixin, TenancyFilterForm, CustomFieldFilterForm): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/sites/", value_field="slug", null_option=True, ) @@ -1299,7 +1217,6 @@ class VLANFilterForm(BootstrapMixin, TenancyFilterForm, CustomFieldFilterForm): required=False, label='VLAN group', widget=APISelectMultiple( - api_url="/api/ipam/vlan-groups/", null_option=True, ) ) @@ -1313,7 +1230,6 @@ class VLANFilterForm(BootstrapMixin, TenancyFilterForm, CustomFieldFilterForm): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/ipam/roles/", value_field="slug", null_option=True, ) diff --git a/netbox/netbox/configuration.example.py b/netbox/netbox/configuration.example.py index 4d17c6362..6e67d228f 100644 --- a/netbox/netbox/configuration.example.py +++ b/netbox/netbox/configuration.example.py @@ -21,11 +21,11 @@ DATABASE = { 'CONN_MAX_AGE': 300, # Max database connection age } -# Redis database settings. The Redis database is used for caching and background processing such as webhooks -# Seperate sections for webhooks and caching allow for connecting to seperate Redis instances/datbases if desired. -# Full connection details are required in both sections, even if they are the same. +# Redis database settings. Redis is used for caching and for queuing background tasks such as webhook events. A separate +# configuration exists for each. Full connection details are required in both sections, and it is strongly recommended +# to use two separate database IDs. REDIS = { - 'webhooks': { + 'tasks': { 'HOST': 'localhost', 'PORT': 6379, # Comment out `HOST` and `PORT` lines and uncomment the following if using Redis Sentinel @@ -187,6 +187,14 @@ REMOTE_AUTH_AUTO_CREATE_USER = True REMOTE_AUTH_DEFAULT_GROUPS = [] REMOTE_AUTH_DEFAULT_PERMISSIONS = [] +# This determines how often the GitHub API is called to check the latest release of NetBox. Must be at least 1 hour. +RELEASE_CHECK_TIMEOUT = 24 * 3600 + +# This repository is used to check whether there is a new release of NetBox available. Set to None to disable the +# version check or use the URL below to check for release in the official NetBox repository. +RELEASE_CHECK_URL = None +# RELEASE_CHECK_URL = 'https://api.github.com/repos/netbox-community/netbox/releases' + # The file path where custom reports will be stored. A trailing slash is not needed. Note that the default value of # this setting is derived from the installed location. # REPORTS_ROOT = '/opt/netbox/netbox/reports' diff --git a/netbox/netbox/releases.py b/netbox/netbox/releases.py new file mode 100644 index 000000000..27279cc16 --- /dev/null +++ b/netbox/netbox/releases.py @@ -0,0 +1,33 @@ +import logging + +from cacheops import CacheMiss, cache +from django.conf import settings +from django_rq import get_queue + +from utilities.background_tasks import get_releases + +logger = logging.getLogger('netbox.releases') + + +def get_latest_release(pre_releases=False): + if settings.RELEASE_CHECK_URL: + logger.debug("Checking for most recent release") + try: + latest_release = cache.get('latest_release') + if latest_release: + logger.debug("Found cached release: {}".format(latest_release)) + return latest_release + except CacheMiss: + # Check for an existing job. This can happen if the RQ worker process is not running. + queue = get_queue('check_releases') + if queue.jobs: + logger.warning("Job to check for new releases is already queued; skipping") + else: + # Get the releases in the background worker, it will fill the cache + logger.info("Initiating background task to retrieve updated releases list") + get_releases.delay(pre_releases=pre_releases) + + else: + logger.debug("Skipping release check; RELEASE_CHECK_URL not defined") + + return 'unknown', None diff --git a/netbox/netbox/settings.py b/netbox/netbox/settings.py index 2d0fdaed0..926501498 100644 --- a/netbox/netbox/settings.py +++ b/netbox/netbox/settings.py @@ -2,11 +2,14 @@ import importlib import logging import os import platform +import re import socket import warnings +from urllib.parse import urlsplit from django.contrib.messages import constants as messages -from django.core.exceptions import ImproperlyConfigured +from django.core.exceptions import ImproperlyConfigured, ValidationError +from django.core.validators import URLValidator from pkg_resources import iter_entry_points @@ -98,6 +101,8 @@ REMOTE_AUTH_DEFAULT_GROUPS = getattr(configuration, 'REMOTE_AUTH_DEFAULT_GROUPS' REMOTE_AUTH_DEFAULT_PERMISSIONS = getattr(configuration, 'REMOTE_AUTH_DEFAULT_PERMISSIONS', []) REMOTE_AUTH_ENABLED = getattr(configuration, 'REMOTE_AUTH_ENABLED', False) REMOTE_AUTH_HEADER = getattr(configuration, 'REMOTE_AUTH_HEADER', 'HTTP_REMOTE_USER') +RELEASE_CHECK_URL = getattr(configuration, 'RELEASE_CHECK_URL', None) +RELEASE_CHECK_TIMEOUT = getattr(configuration, 'RELEASE_CHECK_TIMEOUT', 24 * 3600) REPORTS_ROOT = getattr(configuration, 'REPORTS_ROOT', os.path.join(BASE_DIR, 'reports')).rstrip('/') SCRIPTS_ROOT = getattr(configuration, 'SCRIPTS_ROOT', os.path.join(BASE_DIR, 'scripts')).rstrip('/') SESSION_FILE_PATH = getattr(configuration, 'SESSION_FILE_PATH', None) @@ -107,6 +112,20 @@ SHORT_TIME_FORMAT = getattr(configuration, 'SHORT_TIME_FORMAT', 'H:i:s') TIME_FORMAT = getattr(configuration, 'TIME_FORMAT', 'g:i a') TIME_ZONE = getattr(configuration, 'TIME_ZONE', 'UTC') +# Validate update repo URL and timeout +if RELEASE_CHECK_URL: + try: + URLValidator(RELEASE_CHECK_URL) + except ValidationError: + raise ImproperlyConfigured( + "RELEASE_CHECK_URL must be a valid API URL. Example: " + "https://api.github.com/repos/netbox-community/netbox" + ) + +# Enforce a minimum cache timeout for update checks +if RELEASE_CHECK_TIMEOUT < 3600: + raise ImproperlyConfigured("RELEASE_CHECK_TIMEOUT has to be at least 3600 seconds (1 hour)") + # # Database @@ -163,31 +182,40 @@ if STORAGE_CONFIG and STORAGE_BACKEND is None: # Redis # -if 'webhooks' not in REDIS: - raise ImproperlyConfigured( - "REDIS section in configuration.py is missing webhooks subsection." +# Background task queuing +if 'tasks' in REDIS: + TASKS_REDIS = REDIS['tasks'] +elif 'webhooks' in REDIS: + # TODO: Remove support for 'webhooks' name in v2.9 + warnings.warn( + "The 'webhooks' REDIS configuration section has been renamed to 'tasks'. Please update your configuration as " + "support for the old name will be removed in a future release." ) -if 'caching' not in REDIS: + TASKS_REDIS = REDIS['webhooks'] +else: + raise ImproperlyConfigured( + "REDIS section in configuration.py is missing the 'tasks' subsection." + ) +TASKS_REDIS_HOST = TASKS_REDIS.get('HOST', 'localhost') +TASKS_REDIS_PORT = TASKS_REDIS.get('PORT', 6379) +TASKS_REDIS_SENTINELS = TASKS_REDIS.get('SENTINELS', []) +TASKS_REDIS_USING_SENTINEL = all([ + isinstance(TASKS_REDIS_SENTINELS, (list, tuple)), + len(TASKS_REDIS_SENTINELS) > 0 +]) +TASKS_REDIS_SENTINEL_SERVICE = TASKS_REDIS.get('SENTINEL_SERVICE', 'default') +TASKS_REDIS_PASSWORD = TASKS_REDIS.get('PASSWORD', '') +TASKS_REDIS_DATABASE = TASKS_REDIS.get('DATABASE', 0) +TASKS_REDIS_DEFAULT_TIMEOUT = TASKS_REDIS.get('DEFAULT_TIMEOUT', 300) +TASKS_REDIS_SSL = TASKS_REDIS.get('SSL', False) + +# Caching +if 'caching' in REDIS: + CACHING_REDIS = REDIS['caching'] +else: raise ImproperlyConfigured( "REDIS section in configuration.py is missing caching subsection." ) - -WEBHOOKS_REDIS = REDIS.get('webhooks', {}) -WEBHOOKS_REDIS_HOST = WEBHOOKS_REDIS.get('HOST', 'localhost') -WEBHOOKS_REDIS_PORT = WEBHOOKS_REDIS.get('PORT', 6379) -WEBHOOKS_REDIS_SENTINELS = WEBHOOKS_REDIS.get('SENTINELS', []) -WEBHOOKS_REDIS_USING_SENTINEL = all([ - isinstance(WEBHOOKS_REDIS_SENTINELS, (list, tuple)), - len(WEBHOOKS_REDIS_SENTINELS) > 0 -]) -WEBHOOKS_REDIS_SENTINEL_SERVICE = WEBHOOKS_REDIS.get('SENTINEL_SERVICE', 'default') -WEBHOOKS_REDIS_PASSWORD = WEBHOOKS_REDIS.get('PASSWORD', '') -WEBHOOKS_REDIS_DATABASE = WEBHOOKS_REDIS.get('DATABASE', 0) -WEBHOOKS_REDIS_DEFAULT_TIMEOUT = WEBHOOKS_REDIS.get('DEFAULT_TIMEOUT', 300) -WEBHOOKS_REDIS_SSL = WEBHOOKS_REDIS.get('SSL', False) - - -CACHING_REDIS = REDIS.get('caching', {}) CACHING_REDIS_HOST = CACHING_REDIS.get('HOST', 'localhost') CACHING_REDIS_PORT = CACHING_REDIS.get('PORT', 6379) CACHING_REDIS_SENTINELS = CACHING_REDIS.get('SENTINELS', []) @@ -242,7 +270,6 @@ INSTALLED_APPS = [ 'corsheaders', 'debug_toolbar', 'django_filters', - 'django_rq', 'django_tables2', 'django_prometheus', 'mptt', @@ -259,6 +286,7 @@ INSTALLED_APPS = [ 'users', 'utilities', 'virtualization', + 'django_rq', # Must come after extras to allow overriding management commands 'drf_yasg', ] @@ -554,26 +582,31 @@ SWAGGER_SETTINGS = { # Django RQ (Webhooks backend) # -RQ_QUEUES = { - 'default': { - 'HOST': WEBHOOKS_REDIS_HOST, - 'PORT': WEBHOOKS_REDIS_PORT, - 'DB': WEBHOOKS_REDIS_DATABASE, - 'PASSWORD': WEBHOOKS_REDIS_PASSWORD, - 'DEFAULT_TIMEOUT': WEBHOOKS_REDIS_DEFAULT_TIMEOUT, - 'SSL': WEBHOOKS_REDIS_SSL, - } if not WEBHOOKS_REDIS_USING_SENTINEL else { - 'SENTINELS': WEBHOOKS_REDIS_SENTINELS, - 'MASTER_NAME': WEBHOOKS_REDIS_SENTINEL_SERVICE, - 'DB': WEBHOOKS_REDIS_DATABASE, - 'PASSWORD': WEBHOOKS_REDIS_PASSWORD, +if TASKS_REDIS_USING_SENTINEL: + RQ_PARAMS = { + 'SENTINELS': TASKS_REDIS_SENTINELS, + 'MASTER_NAME': TASKS_REDIS_SENTINEL_SERVICE, + 'DB': TASKS_REDIS_DATABASE, + 'PASSWORD': TASKS_REDIS_PASSWORD, 'SOCKET_TIMEOUT': None, 'CONNECTION_KWARGS': { - 'socket_connect_timeout': WEBHOOKS_REDIS_DEFAULT_TIMEOUT + 'socket_connect_timeout': TASKS_REDIS_DEFAULT_TIMEOUT }, } -} +else: + RQ_PARAMS = { + 'HOST': TASKS_REDIS_HOST, + 'PORT': TASKS_REDIS_PORT, + 'DB': TASKS_REDIS_DATABASE, + 'PASSWORD': TASKS_REDIS_PASSWORD, + 'DEFAULT_TIMEOUT': TASKS_REDIS_DEFAULT_TIMEOUT, + 'SSL': TASKS_REDIS_SSL, + } +RQ_QUEUES = { + 'default': RQ_PARAMS, # Webhooks + 'check_releases': RQ_PARAMS, +} # # Django debug toolbar diff --git a/netbox/netbox/tests/test_releases.py b/netbox/netbox/tests/test_releases.py new file mode 100644 index 000000000..635a6782b --- /dev/null +++ b/netbox/netbox/tests/test_releases.py @@ -0,0 +1,166 @@ +from io import BytesIO +from logging import ERROR +from unittest.mock import Mock, patch + +import requests +from cacheops import CacheMiss, RedisCache +from django.test import SimpleTestCase, override_settings +from packaging.version import Version +from requests import Response + +from utilities.background_tasks import get_releases + + +def successful_github_response(url, *_args, **_kwargs): + r = Response() + r.url = url + r.status_code = 200 + r.reason = 'OK' + r.headers = { + 'Content-Type': 'application/json; charset=utf-8', + } + r.raw = BytesIO(b'''[ + { + "html_url": "https://github.com/netbox-community/netbox/releases/tag/v2.7.8", + "tag_name": "v2.7.8", + "prerelease": false + }, + { + "html_url": "https://github.com/netbox-community/netbox/releases/tag/v2.6-beta1", + "tag_name": "v2.6-beta1", + "prerelease": true + }, + { + "html_url": "https://github.com/netbox-community/netbox/releases/tag/v2.5.9", + "tag_name": "v2.5.9", + "prerelease": false + } + ] + ''') + return r + + +def unsuccessful_github_response(url, *_args, **_kwargs): + r = Response() + r.url = url + r.status_code = 404 + r.reason = 'Not Found' + r.headers = { + 'Content-Type': 'application/json; charset=utf-8', + } + r.raw = BytesIO(b'''{ + "message": "Not Found", + "documentation_url": "https://developer.github.com/v3/repos/releases/#list-releases-for-a-repository" + } + ''') + return r + + +@override_settings(RELEASE_CHECK_URL='https://localhost/unittest/releases', RELEASE_CHECK_TIMEOUT=160876) +class GetReleasesTestCase(SimpleTestCase): + @patch.object(requests, 'get') + @patch.object(RedisCache, 'set') + @patch.object(RedisCache, 'get') + def test_pre_releases(self, dummy_cache_get: Mock, dummy_cache_set: Mock, dummy_request_get: Mock): + dummy_cache_get.side_effect = CacheMiss() + dummy_request_get.side_effect = successful_github_response + + releases = get_releases(pre_releases=True) + + # Check result + self.assertListEqual(releases, [ + (Version('2.7.8'), 'https://github.com/netbox-community/netbox/releases/tag/v2.7.8'), + (Version('2.6b1'), 'https://github.com/netbox-community/netbox/releases/tag/v2.6-beta1'), + (Version('2.5.9'), 'https://github.com/netbox-community/netbox/releases/tag/v2.5.9') + ]) + + # Check if correct request is made + dummy_request_get.assert_called_once_with( + 'https://localhost/unittest/releases', + headers={'Accept': 'application/vnd.github.v3+json'} + ) + + # Check if result is put in cache + dummy_cache_set.assert_called_once_with( + 'latest_release', + max(releases), + 160876 + ) + + @patch.object(requests, 'get') + @patch.object(RedisCache, 'set') + @patch.object(RedisCache, 'get') + def test_no_pre_releases(self, dummy_cache_get: Mock, dummy_cache_set: Mock, dummy_request_get: Mock): + dummy_cache_get.side_effect = CacheMiss() + dummy_request_get.side_effect = successful_github_response + + releases = get_releases(pre_releases=False) + + # Check result + self.assertListEqual(releases, [ + (Version('2.7.8'), 'https://github.com/netbox-community/netbox/releases/tag/v2.7.8'), + (Version('2.5.9'), 'https://github.com/netbox-community/netbox/releases/tag/v2.5.9') + ]) + + # Check if correct request is made + dummy_request_get.assert_called_once_with( + 'https://localhost/unittest/releases', + headers={'Accept': 'application/vnd.github.v3+json'} + ) + + # Check if result is put in cache + dummy_cache_set.assert_called_once_with( + 'latest_release', + max(releases), + 160876 + ) + + @patch.object(requests, 'get') + @patch.object(RedisCache, 'set') + @patch.object(RedisCache, 'get') + def test_failed_request(self, dummy_cache_get: Mock, dummy_cache_set: Mock, dummy_request_get: Mock): + dummy_cache_get.side_effect = CacheMiss() + dummy_request_get.side_effect = unsuccessful_github_response + + with self.assertLogs(level=ERROR) as cm: + releases = get_releases() + + # Check log entry + self.assertEqual(len(cm.output), 1) + log_output = cm.output[0] + last_log_line = log_output.split('\n')[-1] + self.assertRegex(last_log_line, '404 .* Not Found') + + # Check result + self.assertListEqual(releases, []) + + # Check if correct request is made + dummy_request_get.assert_called_once_with( + 'https://localhost/unittest/releases', + headers={'Accept': 'application/vnd.github.v3+json'} + ) + + # Check if failure is put in cache + dummy_cache_set.assert_called_once_with( + 'latest_release_no_retry', + 'https://localhost/unittest/releases', + 900 + ) + + @patch.object(requests, 'get') + @patch.object(RedisCache, 'set') + @patch.object(RedisCache, 'get') + def test_blocked_retry(self, dummy_cache_get: Mock, dummy_cache_set: Mock, dummy_request_get: Mock): + dummy_cache_get.return_value = 'https://localhost/unittest/releases' + dummy_request_get.side_effect = successful_github_response + + releases = get_releases() + + # Check result + self.assertListEqual(releases, []) + + # Check if request is NOT made + dummy_request_get.assert_not_called() + + # Check if cache is not updated + dummy_cache_set.assert_not_called() diff --git a/netbox/netbox/views.py b/netbox/netbox/views.py index b0f5b6022..bc87a825b 100644 --- a/netbox/netbox/views.py +++ b/netbox/netbox/views.py @@ -1,8 +1,10 @@ from collections import OrderedDict +from django.conf import settings from django.db.models import Count, F from django.shortcuts import render from django.views.generic import View +from packaging import version from rest_framework.response import Response from rest_framework.reverse import reverse from rest_framework.views import APIView @@ -25,6 +27,7 @@ from extras.models import ObjectChange, ReportResult from ipam.filters import AggregateFilterSet, IPAddressFilterSet, PrefixFilterSet, VLANFilterSet, VRFFilterSet from ipam.models import Aggregate, IPAddress, Prefix, VLAN, VRF from ipam.tables import AggregateTable, IPAddressTable, PrefixTable, VLANTable, VRFTable +from netbox.releases import get_latest_release from secrets.filters import SecretFilterSet from secrets.models import Secret from secrets.tables import SecretTable @@ -240,11 +243,24 @@ class HomeView(View): } + # Check whether a new release is available. (Only for staff/superusers.) + new_release = None + if request.user.is_staff or request.user.is_superuser: + latest_release, release_url = get_latest_release() + if isinstance(latest_release, version.Version): + current_version = version.parse(settings.VERSION) + if latest_release > current_version: + new_release = { + 'version': str(latest_release), + 'url': release_url, + } + return render(request, self.template_name, { 'search_form': SearchForm(), 'stats': stats, 'report_results': ReportResult.objects.order_by('-created')[:10], - 'changelog': ObjectChange.objects.prefetch_related('user', 'changed_object_type')[:15] + 'changelog': ObjectChange.objects.prefetch_related('user', 'changed_object_type')[:15], + 'new_release': new_release, }) diff --git a/netbox/secrets/forms.py b/netbox/secrets/forms.py index a8db8124b..03ff8fab8 100644 --- a/netbox/secrets/forms.py +++ b/netbox/secrets/forms.py @@ -72,10 +72,7 @@ class SecretRoleCSVForm(forms.ModelForm): class SecretForm(BootstrapMixin, CustomFieldModelForm): device = DynamicModelChoiceField( - queryset=Device.objects.all(), - widget=APISelect( - api_url="/api/dcim/devices/" - ) + queryset=Device.objects.all() ) plaintext = forms.CharField( max_length=SECRET_PLAINTEXT_MAX_LENGTH, @@ -94,10 +91,7 @@ class SecretForm(BootstrapMixin, CustomFieldModelForm): widget=forms.PasswordInput() ) role = DynamicModelChoiceField( - queryset=SecretRole.objects.all(), - widget=APISelect( - api_url="/api/secrets/secret-roles/" - ) + queryset=SecretRole.objects.all() ) tags = TagField( required=False @@ -166,10 +160,7 @@ class SecretBulkEditForm(BootstrapMixin, AddRemoveTagsForm, CustomFieldBulkEditF ) role = DynamicModelChoiceField( queryset=SecretRole.objects.all(), - required=False, - widget=APISelect( - api_url="/api/secrets/secret-roles/" - ) + required=False ) name = forms.CharField( max_length=100, @@ -193,7 +184,6 @@ class SecretFilterForm(BootstrapMixin, CustomFieldFilterForm): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/secrets/secret-roles/", value_field="slug", ) ) diff --git a/netbox/templates/home.html b/netbox/templates/home.html index 6977bba4c..d3885b88f 100644 --- a/netbox/templates/home.html +++ b/netbox/templates/home.html @@ -1,6 +1,19 @@ {% extends '_base.html' %} {% load helpers %} +{% block header %} + {{ block.super }} + {% if new_release %} + {# new_release is set only if the current user is a superuser or staff member #} + + {% endif %} +{% endblock %} + + {% block content %} {% include 'search_form.html' %}
diff --git a/netbox/tenancy/forms.py b/netbox/tenancy/forms.py index 3af848f3d..78c872c6a 100644 --- a/netbox/tenancy/forms.py +++ b/netbox/tenancy/forms.py @@ -60,10 +60,7 @@ class TenantForm(BootstrapMixin, CustomFieldModelForm): slug = SlugField() group = DynamicModelChoiceField( queryset=TenantGroup.objects.all(), - required=False, - widget=APISelect( - api_url="/api/tenancy/tenant-groups/" - ) + required=False ) comments = CommentField() tags = TagField( @@ -105,10 +102,7 @@ class TenantBulkEditForm(BootstrapMixin, AddRemoveTagsForm, CustomFieldBulkEditF ) group = DynamicModelChoiceField( queryset=TenantGroup.objects.all(), - required=False, - widget=APISelect( - api_url="/api/tenancy/tenant-groups/" - ) + required=False ) class Meta: @@ -128,7 +122,6 @@ class TenantFilterForm(BootstrapMixin, CustomFieldFilterForm): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/tenancy/tenant-groups/", value_field="slug", null_option=True, ) @@ -145,7 +138,6 @@ class TenancyForm(forms.Form): queryset=TenantGroup.objects.all(), required=False, widget=APISelect( - api_url="/api/tenancy/tenant-groups/", filter_for={ 'tenant': 'group_id', }, @@ -156,10 +148,7 @@ class TenancyForm(forms.Form): ) tenant = DynamicModelChoiceField( queryset=Tenant.objects.all(), - required=False, - widget=APISelect( - api_url='/api/tenancy/tenants/' - ) + required=False ) def __init__(self, *args, **kwargs): @@ -180,7 +169,6 @@ class TenancyFilterForm(forms.Form): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/tenancy/tenant-groups/", value_field="slug", null_option=True, filter_for={ @@ -193,7 +181,6 @@ class TenancyFilterForm(forms.Form): to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/tenancy/tenants/", value_field="slug", null_option=True, ) diff --git a/netbox/utilities/background_tasks.py b/netbox/utilities/background_tasks.py new file mode 100644 index 000000000..1255846b7 --- /dev/null +++ b/netbox/utilities/background_tasks.py @@ -0,0 +1,52 @@ +import logging + +import requests +from cacheops.simple import cache, CacheMiss +from django.conf import settings +from django_rq import job +from packaging import version + +# Get an instance of a logger +logger = logging.getLogger('netbox.releases') + + +@job('check_releases') +def get_releases(pre_releases=False): + url = settings.RELEASE_CHECK_URL + headers = { + 'Accept': 'application/vnd.github.v3+json', + } + releases = [] + + # Check whether this URL has failed recently and shouldn't be retried yet + try: + if url == cache.get('latest_release_no_retry'): + logger.info("Skipping release check; URL failed recently: {}".format(url)) + return [] + except CacheMiss: + pass + + try: + logger.debug("Fetching new releases from {}".format(url)) + response = requests.get(url, headers=headers) + response.raise_for_status() + total_releases = len(response.json()) + + for release in response.json(): + if 'tag_name' not in release: + continue + if not pre_releases and (release.get('devrelease') or release.get('prerelease')): + continue + releases.append((version.parse(release['tag_name']), release.get('html_url'))) + logger.debug("Found {} releases; {} usable".format(total_releases, len(releases))) + + except requests.exceptions.RequestException: + # The request failed. Set a flag in the cache to disable future checks to this URL for 15 minutes. + logger.exception("Error while fetching {}. Disabling checks for 15 minutes.".format(url)) + cache.set('latest_release_no_retry', url, 900) + return [] + + # Cache the most recent release + cache.set('latest_release', max(releases), settings.RELEASE_CHECK_TIMEOUT) + + return releases diff --git a/netbox/utilities/forms.py b/netbox/utilities/forms.py index 8825102d1..c17ff9299 100644 --- a/netbox/utilities/forms.py +++ b/netbox/utilities/forms.py @@ -10,6 +10,7 @@ from django.conf import settings from django.contrib.postgres.forms.jsonb import JSONField as _JSONField, InvalidJSONInput from django.db.models import Count from django.forms import BoundField +from django.urls import reverse from .choices import unpack_grouped_choices from .constants import * @@ -252,7 +253,7 @@ class APISelect(SelectWithDisabled): """ A select widget populated via an API call - :param api_url: API URL + :param api_url: API endpoint URL. Required if not set automatically by the parent field. :param display_field: (Optional) Field to display for child in selection list. Defaults to `name`. :param value_field: (Optional) Field to use for the option value in selection list. Defaults to `id`. :param disabled_indicator: (Optional) Mark option as disabled if this field equates true. @@ -269,7 +270,7 @@ class APISelect(SelectWithDisabled): """ def __init__( self, - api_url, + api_url=None, display_field=None, value_field=None, disabled_indicator=None, @@ -285,7 +286,8 @@ class APISelect(SelectWithDisabled): super().__init__(*args, **kwargs) self.attrs['class'] = 'netbox-select2-api' - self.attrs['data-url'] = '/{}{}'.format(settings.BASE_PATH, api_url.lstrip('/')) # Inject BASE_PATH + if api_url: + self.attrs['data-url'] = '/{}{}'.format(settings.BASE_PATH, api_url.lstrip('/')) # Inject BASE_PATH if full: self.attrs['data-full'] = full if display_field: @@ -566,6 +568,10 @@ class TagFilterField(forms.MultipleChoiceField): class DynamicModelChoiceMixin: filter = django_filters.ModelChoiceFilter + widget = APISelect + + def __init__(self, *args, **kwargs): + super().__init__(*args, **kwargs) def get_bound_field(self, form, field_name): bound_field = BoundField(form, self, field_name) @@ -579,6 +585,14 @@ class DynamicModelChoiceMixin: else: self.queryset = self.queryset.none() + # Set the data URL on the APISelect widget (if not already set) + widget = bound_field.field.widget + if not widget.attrs.get('data-url'): + app_label = self.queryset.model._meta.app_label + model_name = self.queryset.model._meta.model_name + data_url = reverse('{}-api:{}-list'.format(app_label, model_name)) + widget.attrs['data-url'] = data_url + return bound_field @@ -595,6 +609,7 @@ class DynamicModelMultipleChoiceField(DynamicModelChoiceMixin, forms.ModelMultip A multiple-choice version of DynamicModelChoiceField. """ filter = django_filters.ModelMultipleChoiceFilter + widget = APISelectMultiple class LaxURLField(forms.URLField): diff --git a/netbox/virtualization/forms.py b/netbox/virtualization/forms.py index 9d595b35b..f4c2a36ec 100644 --- a/netbox/virtualization/forms.py +++ b/netbox/virtualization/forms.py @@ -9,7 +9,7 @@ from dcim.models import Device, DeviceRole, Interface, Platform, Rack, Region, S from extras.forms import ( AddRemoveTagsForm, CustomFieldBulkEditForm, CustomFieldModelCSVForm, CustomFieldModelForm, CustomFieldFilterForm, ) -from ipam.models import IPAddress, VLANGroup, VLAN +from ipam.models import IPAddress, VLAN from tenancy.forms import TenancyFilterForm, TenancyForm from tenancy.models import Tenant from utilities.forms import ( @@ -77,24 +77,15 @@ class ClusterGroupCSVForm(forms.ModelForm): class ClusterForm(BootstrapMixin, TenancyForm, CustomFieldModelForm): type = DynamicModelChoiceField( - queryset=ClusterType.objects.all(), - widget=APISelect( - api_url="/api/virtualization/cluster-types/" - ) + queryset=ClusterType.objects.all() ) group = DynamicModelChoiceField( queryset=ClusterGroup.objects.all(), - required=False, - widget=APISelect( - api_url="/api/virtualization/cluster-groups/" - ) + required=False ) site = DynamicModelChoiceField( queryset=Site.objects.all(), - required=False, - widget=APISelect( - api_url="/api/dcim/sites/" - ) + required=False ) comments = CommentField() tags = TagField( @@ -157,31 +148,19 @@ class ClusterBulkEditForm(BootstrapMixin, AddRemoveTagsForm, CustomFieldBulkEdit ) type = DynamicModelChoiceField( queryset=ClusterType.objects.all(), - required=False, - widget=APISelect( - api_url="/api/virtualization/cluster-types/" - ) + required=False ) group = DynamicModelChoiceField( queryset=ClusterGroup.objects.all(), - required=False, - widget=APISelect( - api_url="/api/virtualization/cluster-groups/" - ) + required=False ) tenant = DynamicModelChoiceField( queryset=Tenant.objects.all(), - required=False, - widget=APISelect( - api_url="/api/tenancy/tenants/" - ) + required=False ) site = DynamicModelChoiceField( queryset=Site.objects.all(), - required=False, - widget=APISelect( - api_url="/api/dcim/sites/" - ) + required=False ) comments = CommentField( widget=SmallTextarea, @@ -205,7 +184,6 @@ class ClusterFilterForm(BootstrapMixin, TenancyFilterForm, CustomFieldFilterForm to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/virtualization/cluster-types/", value_field='slug', ) ) @@ -214,7 +192,6 @@ class ClusterFilterForm(BootstrapMixin, TenancyFilterForm, CustomFieldFilterForm to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/regions/", value_field="slug", filter_for={ 'site': 'region' @@ -226,7 +203,6 @@ class ClusterFilterForm(BootstrapMixin, TenancyFilterForm, CustomFieldFilterForm to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/dcim/sites/", value_field='slug', null_option=True, ) @@ -236,7 +212,6 @@ class ClusterFilterForm(BootstrapMixin, TenancyFilterForm, CustomFieldFilterForm to_field_name='slug', required=False, widget=APISelectMultiple( - api_url="/api/virtualization/cluster-groups/", value_field='slug', null_option=True, ) @@ -249,7 +224,6 @@ class ClusterAddDevicesForm(BootstrapMixin, forms.Form): queryset=Region.objects.all(), required=False, widget=APISelect( - api_url="/api/dcim/regions/", filter_for={ "site": "region_id", }, @@ -262,7 +236,6 @@ class ClusterAddDevicesForm(BootstrapMixin, forms.Form): queryset=Site.objects.all(), required=False, widget=APISelect( - api_url='/api/dcim/sites/', filter_for={ "rack": "site_id", "devices": "site_id", @@ -273,7 +246,6 @@ class ClusterAddDevicesForm(BootstrapMixin, forms.Form): queryset=Rack.objects.all(), required=False, widget=APISelect( - api_url='/api/dcim/racks/', filter_for={ "devices": "rack_id" }, @@ -285,7 +257,6 @@ class ClusterAddDevicesForm(BootstrapMixin, forms.Form): devices = DynamicModelMultipleChoiceField( queryset=Device.objects.filter(cluster__isnull=True), widget=APISelectMultiple( - api_url='/api/dcim/devices/', display_field='display_name', disabled_indicator='cluster' ) @@ -334,7 +305,6 @@ class VirtualMachineForm(BootstrapMixin, TenancyForm, CustomFieldModelForm): queryset=ClusterGroup.objects.all(), required=False, widget=APISelect( - api_url='/api/virtualization/cluster-groups/', filter_for={ "cluster": "group_id", }, @@ -344,16 +314,12 @@ class VirtualMachineForm(BootstrapMixin, TenancyForm, CustomFieldModelForm): ) ) cluster = DynamicModelChoiceField( - queryset=Cluster.objects.all(), - widget=APISelect( - api_url='/api/virtualization/clusters/' - ) + queryset=Cluster.objects.all() ) role = DynamicModelChoiceField( queryset=DeviceRole.objects.all(), required=False, widget=APISelect( - api_url="/api/dcim/device-roles/", additional_query_params={ "vm_role": "True" } @@ -361,10 +327,7 @@ class VirtualMachineForm(BootstrapMixin, TenancyForm, CustomFieldModelForm): ) platform = DynamicModelChoiceField( queryset=Platform.objects.all(), - required=False, - widget=APISelect( - api_url='/api/dcim/platforms/' - ) + required=False ) tags = TagField( required=False @@ -499,10 +462,7 @@ class VirtualMachineBulkEditForm(BootstrapMixin, AddRemoveTagsForm, CustomFieldB ) cluster = DynamicModelChoiceField( queryset=Cluster.objects.all(), - required=False, - widget=APISelect( - api_url='/api/virtualization/clusters/' - ) + required=False ) role = DynamicModelChoiceField( queryset=DeviceRole.objects.filter( @@ -510,7 +470,6 @@ class VirtualMachineBulkEditForm(BootstrapMixin, AddRemoveTagsForm, CustomFieldB ), required=False, widget=APISelect( - api_url="/api/dcim/device-roles/", additional_query_params={ "vm_role": "True" } @@ -518,17 +477,11 @@ class VirtualMachineBulkEditForm(BootstrapMixin, AddRemoveTagsForm, CustomFieldB ) tenant = DynamicModelChoiceField( queryset=Tenant.objects.all(), - required=False, - widget=APISelect( - api_url='/api/tenancy/tenants/' - ) + required=False ) platform = DynamicModelChoiceField( queryset=Platform.objects.all(), - required=False, - widget=APISelect( - api_url='/api/dcim/platforms/' - ) + required=False ) vcpus = forms.IntegerField( required=False, @@ -568,7 +521,6 @@ class VirtualMachineFilterForm(BootstrapMixin, TenancyFilterForm, CustomFieldFil to_field_name='slug', required=False, widget=APISelectMultiple( - api_url='/api/virtualization/cluster-groups/', value_field="slug", null_option=True, ) @@ -578,7 +530,6 @@ class VirtualMachineFilterForm(BootstrapMixin, TenancyFilterForm, CustomFieldFil to_field_name='slug', required=False, widget=APISelectMultiple( - api_url='/api/virtualization/cluster-types/', value_field="slug", null_option=True, ) @@ -586,17 +537,13 @@ class VirtualMachineFilterForm(BootstrapMixin, TenancyFilterForm, CustomFieldFil cluster_id = DynamicModelMultipleChoiceField( queryset=Cluster.objects.all(), required=False, - label='Cluster', - widget=APISelectMultiple( - api_url='/api/virtualization/clusters/', - ) + label='Cluster' ) region = DynamicModelMultipleChoiceField( queryset=Region.objects.all(), to_field_name='slug', required=False, widget=APISelectMultiple( - api_url='/api/dcim/regions/', value_field="slug", filter_for={ 'site': 'region' @@ -608,7 +555,6 @@ class VirtualMachineFilterForm(BootstrapMixin, TenancyFilterForm, CustomFieldFil to_field_name='slug', required=False, widget=APISelectMultiple( - api_url='/api/dcim/sites/', value_field="slug", null_option=True, ) @@ -618,7 +564,6 @@ class VirtualMachineFilterForm(BootstrapMixin, TenancyFilterForm, CustomFieldFil to_field_name='slug', required=False, widget=APISelectMultiple( - api_url='/api/dcim/device-roles/', value_field="slug", null_option=True, additional_query_params={ @@ -636,7 +581,6 @@ class VirtualMachineFilterForm(BootstrapMixin, TenancyFilterForm, CustomFieldFil to_field_name='slug', required=False, widget=APISelectMultiple( - api_url='/api/dcim/platforms/', value_field="slug", null_option=True, ) @@ -657,7 +601,6 @@ class InterfaceForm(BootstrapMixin, forms.ModelForm): queryset=VLAN.objects.all(), required=False, widget=APISelect( - api_url="/api/ipam/vlans/", display_field='display_name', full=True, additional_query_params={ @@ -669,7 +612,6 @@ class InterfaceForm(BootstrapMixin, forms.ModelForm): queryset=VLAN.objects.all(), required=False, widget=APISelectMultiple( - api_url="/api/ipam/vlans/", display_field='display_name', full=True, additional_query_params={ @@ -766,7 +708,6 @@ class InterfaceCreateForm(BootstrapMixin, forms.Form): queryset=VLAN.objects.all(), required=False, widget=APISelect( - api_url="/api/ipam/vlans/", display_field='display_name', full=True, additional_query_params={ @@ -778,7 +719,6 @@ class InterfaceCreateForm(BootstrapMixin, forms.Form): queryset=VLAN.objects.all(), required=False, widget=APISelectMultiple( - api_url="/api/ipam/vlans/", display_field='display_name', full=True, additional_query_params={ @@ -836,7 +776,6 @@ class InterfaceBulkEditForm(BootstrapMixin, BulkEditForm): queryset=VLAN.objects.all(), required=False, widget=APISelect( - api_url="/api/ipam/vlans/", display_field='display_name', full=True, additional_query_params={ @@ -848,7 +787,6 @@ class InterfaceBulkEditForm(BootstrapMixin, BulkEditForm): queryset=VLAN.objects.all(), required=False, widget=APISelectMultiple( - api_url="/api/ipam/vlans/", display_field='display_name', full=True, additional_query_params={