Merge pull request #17157 from netbox-community/develop

Release v4.0.9

.github/ISSUE_TEMPLATE/bug_report.yaml

@@ -26,7 +26,7 @@ body:
     attributes:
       label: NetBox Version
       description: What version of NetBox are you currently running?
-      placeholder: v4.0.8
+      placeholder: v4.0.9
     validations:
       required: true
   - type: dropdown

.github/ISSUE_TEMPLATE/feature_request.yaml

@@ -14,7 +14,7 @@ body:
     attributes:
       label: NetBox version
       description: What version of NetBox are you currently running?
-      placeholder: v4.0.8
+      placeholder: v4.0.9
     validations:
       required: true
   - type: dropdown

CONTRIBUTING.md

@@ -40,7 +40,7 @@ NetBox users are welcome to participate in either role, on stage or in the crowd
 
 * First, ensure that you're running the [latest stable version](https://github.com/netbox-community/netbox/releases) of NetBox. If you're running an older version, it's likely that the bug has already been fixed.
 
-* Next, search our [issues list](https://github.com/netbox-community/netbox/issues?q=is%3Aissue) to see if the bug you've found has already been reported. If you come across a bug report that seems to match, please click "add a reaction" in the bottom left corner of the issue and add a thumbs up (:thumbsup:). This will help draw more attention to it. Any comments you can add to provide additional information or context would also be much appreciated.
+* Next, search our [issues list](https://github.com/netbox-community/netbox/issues?q=is%3Aissue) to see if the bug you've found has already been reported. If you come across a bug report that seems to match, please click "add a reaction" in the bottom left corner of the issue and add a thumbs up ( :thumbsup: ). This will help draw more attention to it. Any comments you can add to provide additional information or context would also be much appreciated.
 
 * If you can't find any existing issues (open or closed) that seem to match yours, you're welcome to [submit a new bug report](https://github.com/netbox-community/netbox/issues/new?label=type%3A+bug&template=bug_report.yaml). Be sure to complete the entire report template, including detailed steps that someone triaging your issue can follow to confirm the reported behavior. (If we're not able to replicate the bug based on the information provided, we'll ask for additional detail.)
 
@@ -56,7 +56,9 @@ intake policy](https://github.com/netbox-community/netbox/wiki/Issue-Intake-Poli
 
 ## :bulb: Feature Requests
 
-* First, check the GitHub [issues list](https://github.com/netbox-community/netbox/issues?q=is%3Aissue) to see if the feature you have in mind has already been proposed. If you happen to find an open feature request that matches your idea, click "add a reaction" in the top right corner of the issue and add a thumbs up (:thumbsup:). This ensures that the issue has a better chance of receiving attention. Also feel free to add a comment with any additional justification for the feature.
+* First, check the GitHub [issues list](https://github.com/netbox-community/netbox/issues?q=is%3Aissue) to see if the feature you have in mind has already been proposed. If you happen to find an open feature request that matches your idea, click "add a reaction" in the top right corner of the issue and add a thumbs up ( :thumbsup: ). This ensures that the issue has a better chance of receiving attention. Also feel free to add a comment with any additional justification for the feature.
+
+* Please don't submit duplicate issues! Sometimes we reject feature requests, for various reasons. Even if you disagree with those reasons, please **do not** submit a duplicate feature request. It is very disrepectful of the maintainers' time, and you may be barred from opening future issues.
 
 * If you have a rough idea that's not quite ready for formal submission yet, start a [GitHub discussion](https://github.com/netbox-community/netbox/discussions) instead. This is a great way to test the viability and narrow down the scope of a new feature prior to submitting a formal proposal, and can serve to generate interest in your idea from other community members.
 

contrib/generated_schema.json

@@ -377,6 +377,7 @@
                         "ieee802.11ad",
                         "ieee802.11ax",
                         "ieee802.11ay",
+                        "ieee802.11be",
                         "ieee802.15.1",
                         "other-wireless",
                         "gsm",

docs/configuration/system.md

@@ -87,17 +87,6 @@ addresses (and [`DEBUG`](#debug) is true).
 
 ---
 
-## ISOLATED_DEPLOYMENT
-
-Default: False
-
-Set this configuration parameter to True for NetBox deployments which do not have Internet access. This will disable miscellaneous functionality which depends on access to the Internet.
-
-!!! note
-    If Internet access is available via a proxy, set [`HTTP_PROXIES`](#http_proxies) instead.
-
----
-
 ## JINJA2_FILTERS
 
 Default: `{}`
@@ -154,7 +143,7 @@ LOGGING = {
 
 ## MEDIA_ROOT
 
-Default: `$INSTALL_ROOT/netbox/media/`
+Default: $INSTALL_ROOT/netbox/media/
 
 The file path to the location where media files (such as image attachments) are stored. By default, this is the `netbox/media/` directory within the base NetBox installation path.
 

docs/customization/custom-fields.md

@@ -74,8 +74,6 @@ If a default value is specified for a selection field, it must exactly match one
 
 An object or multi-object custom field can be used to refer to a particular NetBox object or objects as the "value" for a custom field. These custom fields must define an `object_type`, which determines the type of object to which custom field instances point.
 
-By default, an object choice field will make all objects of that type available for selection in the drop-down. The list choices can be filtered to show only objects with certain values by providing a `query_params` dict in the Related Object Filter field, as a JSON value. More information about `query_params` can be found [here](./custom-scripts.md#objectvar).
-
 ## Custom Fields in Templates
 
 Several features within NetBox, such as export templates and webhooks, utilize Jinja2 templating. For convenience, objects which support custom field assignment expose custom field data through the `cf` property. This is a bit cleaner than accessing custom field data through the actual field (`custom_field_data`).

docs/customization/custom-validation.md

@@ -86,6 +86,8 @@ CUSTOM_VALIDATORS = {
 
 #### Referencing Related Object Attributes
 
+!!! info "This feature was introduced in NetBox v4.0."
+
 The attributes of a related object can be referenced by specifying a dotted path. For example, to reference the name of a region to which a site is assigned, use `region.name`:
 
 ```python
@@ -102,6 +104,8 @@ CUSTOM_VALIDATORS = {
 
 #### Validating Request Parameters
 
+!!! info "This feature was introduced in NetBox v4.0."
+
 In addition to validating object attributes, custom validators can also match against parameters of the current request (where available). For example, the following rule will permit only the user named "admin" to modify an object:
 
 ```json

docs/development/models.md

@@ -18,7 +18,7 @@ Depending on its classification, each NetBox model may support various features
 | [Custom links](../customization/custom-links.md)           | `CustomLinksMixin`      | `custom_links`     | These models support the assignment of custom links                                     |
 | [Custom validation](../customization/custom-validation.md) | `CustomValidationMixin` | -                  | Supports the enforcement of custom validation rules                                     |
 | [Export templates](../customization/export-templates.md)   | `ExportTemplatesMixin`  | `export_templates` | Users can create custom export templates for these models                               |
-| [Job results](../features/background-jobs.md)              | `JobsMixin`             | `jobs`             | Background jobs can be scheduled for these models                                       |
+| [Job results](../features/background-jobs.md)              | `JobsMixin`             | `jobs`             | Users can create custom export templates for these models                               |
 | [Journaling](../features/journaling.md)                    | `JournalingMixin`       | `journaling`       | These models support persistent historical commentary                                   |
 | [Synchronized data](../integrations/synchronized-data.md)  | `SyncedDataMixin`       | `synced_data`      | Certain model data can be automatically synchronized from a remote data source          |
 | [Tagging](../models/extras/tag.md)                         | `TagsMixin`             | `tags`             | The models can be tagged with user-defined tags                                         |
@@ -34,9 +34,7 @@ These are considered the "core" application models which are used to model netwo
 * [circuits.Provider](../models/circuits/provider.md)
 * [circuits.ProviderAccount](../models/circuits/provideraccount.md)
 * [circuits.ProviderNetwork](../models/circuits/providernetwork.md)
-* [core.DataFile](../models/core/datafile.md)
 * [core.DataSource](../models/core/datasource.md)
-* [core.Job](../models/core/job.md)
 * [dcim.Cable](../models/dcim/cable.md)
 * [dcim.Device](../models/dcim/device.md)
 * [dcim.DeviceType](../models/dcim/devicetype.md)
@@ -46,14 +44,12 @@ These are considered the "core" application models which are used to model netwo
 * [dcim.PowerPanel](../models/dcim/powerpanel.md)
 * [dcim.Rack](../models/dcim/rack.md)
 * [dcim.RackReservation](../models/dcim/rackreservation.md)
-* [dcim.RackType](../models/dcim/racktype.md)
 * [dcim.Site](../models/dcim/site.md)
 * [dcim.VirtualChassis](../models/dcim/virtualchassis.md)
 * [dcim.VirtualDeviceContext](../models/dcim/virtualdevicecontext.md)
 * [ipam.Aggregate](../models/ipam/aggregate.md)
 * [ipam.ASN](../models/ipam/asn.md)
 * [ipam.FHRPGroup](../models/ipam/fhrpgroup.md)
-* [ipam.FHRPGroupAssignment](../models/ipam/fhrpgroupassignment.md)
 * [ipam.IPAddress](../models/ipam/ipaddress.md)
 * [ipam.IPRange](../models/ipam/iprange.md)
 * [ipam.Prefix](../models/ipam/prefix.md)
@@ -80,7 +76,6 @@ These are considered the "core" application models which are used to model netwo
 
 Organization models are used to organize and classify primary models.
 
-* [circuits.CircuitGroup](../models/circuits/circuitgroup.md)
 * [circuits.CircuitType](../models/circuits/circuittype.md)
 * [dcim.DeviceRole](../models/dcim/devicerole.md)
 * [dcim.Manufacturer](../models/dcim/manufacturer.md)
@@ -93,7 +88,6 @@ Organization models are used to organize and classify primary models.
 * [tenancy.ContactRole](../models/tenancy/contactrole.md)
 * [virtualization.ClusterGroup](../models/virtualization/clustergroup.md)
 * [virtualization.ClusterType](../models/virtualization/clustertype.md)
-* [vpn.TunnelGroup](../models/vpn/tunnelgroup.md)
 
 ### Nested Group Models
 
@@ -137,10 +131,3 @@ These function as templates to effect the replication of device and virtual mach
 * [dcim.PowerOutletTemplate](../models/dcim/poweroutlettemplate.md)
 * [dcim.PowerPortTemplate](../models/dcim/powerporttemplate.md)
 * [dcim.RearPortTemplate](../models/dcim/rearporttemplate.md)
-
-### Connection Models
-
-Connection models are used to model the connections, or connection endpoints between models.
-
-* [circuits.CircuitTermination](../models/circuits/circuittermination.md)
-* [vpn.TunnelTermination](../models/vpn/tunneltermination.md)

docs/features/event-rules.md

@@ -1,10 +1,9 @@
 # Event Rules
 
-NetBox includes the ability to automatically perform certain functions in response to internal events. These include:
+NetBox includes the ability to execute certain functions in response to internal object changes. These include:
 
-* Executing a [custom script](../customization/custom-scripts.md)
-* Sending a [webhook](../integrations/webhooks.md)
-* Generating [user notifications](../features/notifications.md)
+* [Scripts](../customization/custom-scripts.md) execution
+* [Webhooks](../integrations/webhooks.md) execution
 
 For example, suppose you want to automatically configure a monitoring system to start monitoring a device when its operational status is changed to active, and remove it from monitoring for any other status. You can create a webhook in NetBox for the device model and craft its content and destination URL to effect the desired change on the receiving system. You can then associate an event rule with this webhook and the webhook will be sent automatically by NetBox whenever the configured constraints are met.
 

docs/features/facilities.md

@@ -56,10 +56,6 @@ A site typically represents a building within a region and/or site group. Each s
 
 A location can be any logical subdivision within a building, such as a floor or room. Like regions and site groups, locations can be nested into a self-recursive hierarchy for maximum flexibility. And like sites, each location has an operational status assigned to it.
 
-## Rack Types
-
-A rack type represents a unique specification of a rack which exists in the real world. Each rack type can be setup with weight, height, and unit ordering. New racks of this type can then be created in NetBox, and any associated specifications will be automatically replicated from the device type.
-
 ## Racks
 
 Finally, NetBox models each equipment rack as a discrete object within a site and location. These are physical objects into which devices are installed. Each rack can be assigned an operational status, type, facility ID, and other attributes related to inventory tracking. Each rack also must define a height (in rack units) and width, and may optionally specify its physical dimensions.

docs/features/notifications.md

@@ -1,10 +0,0 @@
-# Notifications
-
-!!! info "This feature was introduced in NetBox v4.1."
-
-NetBox includes a system for generating user notifications, which can be marked as read or deleted by individual users. There are two built-in mechanisms for generating a notification:
-
-* A user can subscribe to an object. When that object is modified, a notification is created to inform the user of the change.
-* An [event rule](./event-rules.md) can be defined to automatically generate a notification for one or more users in response to specific system events.
-
-Additionally, NetBox plugins can generate notifications for their own purposes.

docs/models/circuits/circuitgroup.md

@@ -1,15 +0,0 @@
-# Circuit Groups
-
-!!! info "This feature was introduced in NetBox v4.1."
-
-[Circuits](./circuit.md) can be arranged into administrative groups for organization. The assignment of a circuit to a group is optional.
-
-## Fields
-
-### Name
-
-A unique human-friendly name.
-
-### Slug
-
-A unique URL-friendly identifier. (This value can be used for filtering.)

docs/models/circuits/circuitgroupassignment.md

@@ -1,25 +0,0 @@
-# Circuit Group Assignments
-
-Circuits can be assigned to [circuit groups](./circuitgroup.md) for correlation purposes. For instance, three circuits, each belonging to a different provider, may each be assigned to the same circuit group. Each assignment may optionally include a priority designation.
-
-## Fields
-
-### Group
-
-The [circuit group](./circuitgroup.md) being assigned.
-
-### Circuit
-
-The [circuit](./circuit.md) that is being assigned to the group.
-
-### Priority
-
-The circuit's operation priority relative to its peers within the group. The assignment of a priority is optional. Choices include:
-
-* Primary
-* Secondary
-* Tertiary
-* Inactive
-
-!!! tip
-    Additional priority choices may be defined by setting `CircuitGroupAssignment.priority` under the [`FIELD_CHOICES`](../../configuration/data-validation.md#field_choices) configuration parameter.

docs/models/dcim/modulebay.md

@@ -14,10 +14,6 @@ Module bays represent a space or slot within a device in which a field-replaceab
 
 The device to which this module bay belongs.
 
-### Module
-
-The module to which this bay belongs (optional).
-
 ### Name
 
 The module bay's name. Must be unique to the parent device.

docs/models/dcim/moduletype.md

@@ -39,9 +39,3 @@ An alternative part number to uniquely identify the module type.
 ### Weight
 
 The numeric weight of the module, including a unit designation (e.g. 3 kilograms or 1 pound).
-
-### Airflow
-
-!!! info "The `airflow` field was introduced in NetBox v4.1."
-
-The direction in which air circulates through the device chassis for cooling.

docs/models/dcim/rack.md

@@ -20,10 +20,6 @@ The [location](./location.md) within a site where the rack has been installed (o
 
 The rack's name or identifier. Must be unique to the rack's location, if assigned.
 
-### Rack Type
-
-The [physical type](./racktype.md) of this rack. The rack type defines physical attributes such as height and weight.
-
 ### Status
 
 Operational status.
@@ -47,5 +43,44 @@ The unique physical serial number assigned to this rack.
 
 A unique, locally-administered label used to identify hardware resources.
 
-!!! note
-    Some additional fields pertaining to physical attributes such as height and weight can also be defined on each rack, but should generally be defined instead on the [rack type](./racktype.md).
+### Type
+
+A rack can be designated as one of the following types:
+
+* 2-post frame
+* 4-post frame
+* 4-post cabinet
+* Wall-mounted frame
+* Wall-mounted cabinet
+
+### Width
+
+The canonical distance between the two vertical rails on a face. (This is typically 19 inches, however other standard widths exist.)
+
+### Height
+
+The height of the rack, measured in units.
+
+### Starting Unit
+
+The number of the numerically lowest unit in the rack. This value defaults to one, but may be higher in certain situations. For example, you may want to model only a select range of units within a shared physical rack (e.g. U13 through U24).
+
+### Outer Dimensions
+
+The external width and depth of the rack can be tracked to aid in floorplan calculations. These measurements must be designated in either millimeters or inches.
+
+### Mounting Depth
+
+The maximum depth of a mounted device that the rack can accommodate, in millimeters. For four-post frames or cabinets, this is the horizontal distance between the front and rear vertical rails. (Note that this measurement does _not_ include space between the rails and the cabinet doors.)
+
+### Weight
+
+The numeric weight of the rack, including a unit designation (e.g. 10 kilograms or 20 pounds).
+
+### Maximum Weight
+
+The maximum total weight capacity for all installed devices, inclusive of the rack itself.
+
+### Descending Units
+
+If selected, the rack's elevation will display unit 1 at the top of the rack. (Most racks use ascending numbering, with unit 1 assigned to the bottommost position.)

docs/models/dcim/racktype.md

@@ -1,66 +0,0 @@
-# Rack Types
-
-!!! info "This feature was introduced in NetBox v4.1."
-
-A rack type defines the physical characteristics of a particular model of [rack](./rack.md).
-
-## Fields
-
-### Manufacturer
-
-The [manufacturer](./manufacturer.md) which produces this type of rack.
-
-### Model
-
-The model number assigned to this rack type by its manufacturer. Must be unique to the manufacturer.
-
-### Slug
-
-A unique URL-friendly representation of the model identifier. (This value can be used for filtering.)
-
-### Form Factor
-
-A rack can be designated as one of the following form factors:
-
-* 2-post frame
-* 4-post frame
-* 4-post cabinet
-* Wall-mounted frame
-* Wall-mounted cabinet
-
-### Width
-
-The canonical distance between the two vertical rails on a face. (This is typically 19 inches, however other standard widths exist.)
-
-### Height
-
-The height of the rack, measured in units.
-
-### Starting Unit
-
-The number of the numerically lowest unit in the rack. This value defaults to one, but may be higher in certain situations. For example, you may want to model only a select range of units within a shared physical rack (e.g. U13 through U24).
-
-### Outer Dimensions
-
-The external width and depth of the rack can be tracked to aid in floorplan calculations. These measurements must be designated in either millimeters or inches.
-
-### Mounting Depth
-
-The maximum depth of a mounted device that the rack can accommodate, in millimeters. For four-post frames or cabinets, this is the horizontal distance between the front and rear vertical rails. (Note that this measurement does _not_ include space between the rails and the cabinet doors.)
-
-### Weight
-
-The numeric weight of the rack, including a unit designation (e.g. 10 kilograms or 20 pounds).
-
-### Maximum Weight
-
-The maximum total weight capacity for all installed devices, inclusive of the rack itself.
-
-### Descending Units
-
-If selected, the rack's elevation will display unit 1 at the top of the rack. (Most racks use ascending numbering, with unit 1 assigned to the bottommost position.)
-
-### Airflow
-
-The direction in which air circulates through the rack for cooling.
-

docs/models/extras/customfield.md

@@ -42,15 +42,6 @@ The type of data this field holds. This must be one of the following:
 
 For object and multiple-object fields only. Designates the type of NetBox object being referenced.
 
-### Related Object Filter
-
-!!! info "This field was introduced in NetBox v4.1."
-
-For object and multi-object custom fields, a filter may be defined to limit the available objects when populating a field value. This filter maps object attributes to values. For example, `{"status": "active"}` will include only objects with a status of "active."
-
-!!! warning
-    This setting is employed for convenience only, and should not be relied upon to enforce data integrity.
-
 ### Weight
 
 A numeric weight used to override alphabetic ordering of fields by name. Custom fields with a lower weight will be listed before those with a higher weight. (Note that weight applies within the context of a custom field group, if defined.)
@@ -116,7 +107,3 @@ For numeric custom fields only. The maximum valid value (optional).
 ### Validation Regex
 
 For string-based custom fields only. A regular expression used to validate the field's value (optional).
-
-### Uniqueness Validation
-
-If enabled, each object must have a unique value set for this custom field (per object type).

docs/models/extras/eventrule.md

@@ -18,22 +18,17 @@ The type(s) of object in NetBox that will trigger the rule.
 
 If not selected, the event rule will not be processed.
 
-### Events Types
+### Events
 
-The event types which will trigger the rule. At least one event type must be selected.
+The events which will trigger the rule. At least one event type must be selected.
 
-| Name           | Description                                 |
-|----------------|---------------------------------------------|
-| Object created | A new object has been created               |
-| Object updated | An existing object has been modified        |
-| Object deleted | An object has been deleted                  |
-| Job started    | A background job is initiated               |
-| Job completed  | A background job completes successfully     |
-| Job failed     | A background job fails                      |
-| Job errored    | A background job is aborted due to an error |
-
-!!! tip "Custom Event Types"
-    The above list includes only built-in event types. NetBox plugins can also register their own custom event types.
+| Name       | Description                          |
+|------------|--------------------------------------|
+| Creations  | A new object has been created        |
+| Updates    | An existing object has been modified |
+| Deletions  | An object has been deleted           |
+| Job starts | A job for an object starts           |
+| Job ends   | A job for an object terminates       |
 
 ### Conditions
 

docs/models/extras/notification.md

@@ -1,17 +0,0 @@
-# Notification
-
-A notification alerts a user that a specific action has taken place in NetBox, such as an object being modified or a background job completing. A notification may be generated via a user's [subscription](./subscription.md) to a particular object, or by an event rule targeting a [notification group](./notificationgroup.md) of which the user is a member.
-
-## Fields
-
-### User
-
-The recipient of the notification.
-
-### Object
-
-The object to which the notification relates.
-
-### Event Type
-
-The type of event indicated by the notification.

docs/models/extras/notificationgroup.md

@@ -1,17 +0,0 @@
-# Notification Group
-
-A set of NetBox users and/or groups of users identified as recipients for certain [notifications](./notification.md).
-
-## Fields
-
-### Name
-
-The name of the notification group.
-
-### Users
-
-One or more users directly designated as members of the notification group.
-
-### Groups
-
-All users of any selected groups are considered as members of the notification group.

docs/models/extras/subscription.md

@@ -1,15 +0,0 @@
-# Subscription
-
-A record indicating that a user is to be notified of any changes to a particular NetBox object. A notification maps exactly one user to exactly one object.
-
-When an object to which a user is subscribed changes, a [notification](./notification.md) is generated for the user.
-
-## Fields
-
-### User
-
-The subscribed user.
-
-### Object
-
-The object to which the user is subscribed.

docs/models/ipam/vlangroup.md

@@ -14,11 +14,9 @@ A unique human-friendly name.
 
 A unique URL-friendly identifier. (This value can be used for filtering.)
 
-### VLAN ID Ranges
+### Minimum & Maximum VLAN IDs
 
-!!! info "This field replaced the legacy `min_vid` and `max_vid` fields in NetBox v4.1."
-
-The set of VLAN IDs which are encompassed by the group. By default, this will be the entire range of valid IEEE 802.1Q VLAN IDs (1 to 4094, inclusive). VLANs created within a group must have a VID that falls within one of these ranges. Ranges may not overlap.
+A minimum and maximum child VLAN ID must be set for each group. (These default to 1 and 4094 respectively.) VLANs created within a group must have a VID that falls between these values (inclusive).
 
 ### Scope
 

docs/models/virtualization/virtualmachine.md

@@ -50,13 +50,4 @@ The amount of running memory provisioned, in megabytes.
 
 ### Disk
 
-The amount of disk storage provisioned, in megabytes.
-
-!!! warning
-    This field may be directly modified only on virtual machines which do not define discrete [virtual disks](./virtualdisk.md). Otherwise, it will report the sum of all attached disks.
-
-### Serial Number
-
-!!! info "This field was introduced in NetBox v4.1."
-
-Optional serial number assigned to this virtual machine. Unlike devices, uniqueness is not enforced for virtual machine serial numbers.
+The amount of disk storage provisioned, in gigabytes.

docs/models/wireless/wirelesslink.md

@@ -20,12 +20,6 @@ The operational status of the link. Options include:
 
 The service set identifier (SSID) for the wireless link (optional).
 
-### Distance
-
-!!! info "This field was introduced in NetBox v4.1."
-
-The distance between the link's two endpoints, including a unit designation (e.g. 100 meters or 25 feet).
-
 ### Authentication Type
 
 The type of wireless authentication in use. Options include:

docs/plugins/development/background-jobs.md

@@ -1,99 +0,0 @@
-# Background Jobs
-
-!!! info "This feature was introduced in NetBox v4.1."
-
-NetBox plugins can defer certain operations by enqueuing [background jobs](../../features/background-jobs.md), which are executed asynchronously by background workers. This is helpful for decoupling long-running processes from the user-facing request-response cycle.
-
-For example, your plugin might need to fetch data from a remote system. Depending on the amount of data and the responsiveness of the remote server, this could take a few minutes. Deferring this task to a queued job ensures that it can be completed in the background, without interrupting the user. The data it fetches can be made available once the job has completed.
-
-## Job Runners
-
-A background job implements a basic [Job](../../models/core/job.md) executor for all kinds of tasks. It has logic implemented to handle the management of the associated job object, rescheduling of periodic jobs in the given interval and error handling. Adding custom jobs is done by subclassing NetBox's `JobRunner` class.
-
-::: netbox.jobs.JobRunner
-
-#### Example
-
-```python title="jobs.py"
-from netbox.jobs import JobRunner
-
-
-class MyTestJob(JobRunner):
-    class Meta:
-        name = "My Test Job"
-
-    def run(self, *args, **kwargs):
-        obj = self.job.object
-        # your logic goes here
-```
-
-You can schedule the background job from within your code (e.g. from a model's `save()` method or a view) by calling `MyTestJob.enqueue()`. This method passes through all arguments to `Job.enqueue()`. However, no `name` argument must be passed, as the background job name will be used instead.
-
-### Attributes
-
-`JobRunner` attributes are defined under a class named `Meta` within the job. These are optional, but encouraged.
-
-#### `name`
-
-This is the human-friendly names of your background job. If omitted, the class name will be used.
-
-### Scheduled Jobs
-
-As described above, jobs can be scheduled for immediate execution or at any later time using the `enqueue()` method. However, for management purposes, the `enqueue_once()` method allows a job to be scheduled exactly once avoiding duplicates. If a job is already scheduled for a particular instance, a second one won't be scheduled, respecting thread safety. An example use case would be to schedule a periodic task that is bound to an instance in general, but not to any event of that instance (such as updates). The parameters of the `enqueue_once()` method are identical to those of `enqueue()`.
-
-!!! tip
-    It is not forbidden to `enqueue()` additional jobs while an interval schedule is active. An example use of this would be to schedule a periodic daily synchronization, but also trigger additional synchronizations on demand when the user presses a button.
-
-#### Example
-
-```python title="jobs.py"
-from netbox.jobs import JobRunner
-
-
-class MyHousekeepingJob(JobRunner):
-    class Meta:
-        name = "Housekeeping"
-
-    def run(self, *args, **kwargs):
-        # your logic goes here
-```
-
-```python title="__init__.py"
-from netbox.plugins import PluginConfig
-
-class MyPluginConfig(PluginConfig):
-    def ready(self):
-        from .jobs import MyHousekeepingJob
-        MyHousekeepingJob.setup(interval=60)
-```
-
-## Task queues
-
-Three task queues of differing priority are defined by default:
-
-* High
-* Default
-* Low
-
-Any tasks in the "high" queue are completed before the default queue is checked, and any tasks in the default queue are completed before those in the "low" queue.
-
-Plugins can also add custom queues for their own needs by setting the `queues` attribute under the PluginConfig class. An example is included below:
-
-```python
-class MyPluginConfig(PluginConfig):
-    name = 'myplugin'
-    ...
-    queues = [
-        'foo',
-        'bar',
-    ]
-```
-
-The `PluginConfig` above creates two custom queues with the following names `my_plugin.foo` and `my_plugin.bar`. (The plugin's name is prepended to each queue to avoid conflicts between plugins.)
-
-!!! warning "Configuring the RQ worker process"
-    By default, NetBox's RQ worker process only services the high, default, and low queues. Plugins which introduce custom queues should advise users to either reconfigure the default worker, or run a dedicated worker specifying the necessary queues. For example:
-    
-    ```
-    python manage.py rqworker my_plugin.foo my_plugin.bar
-    ```

docs/plugins/development/background-tasks.md

@@ -0,0 +1,30 @@
+# Background Tasks
+
+NetBox supports the queuing of tasks that need to be performed in the background, decoupled from the request-response cycle, using the [Python RQ](https://python-rq.org/) library. Three task queues of differing priority are defined by default:
+
+* High
+* Default
+* Low
+
+Any tasks in the "high" queue are completed before the default queue is checked, and any tasks in the default queue are completed before those in the "low" queue.
+
+Plugins can also add custom queues for their own needs by setting the `queues` attribute under the PluginConfig class. An example is included below:
+
+```python
+class MyPluginConfig(PluginConfig):
+    name = 'myplugin'
+    ...
+    queues = [
+        'foo',
+        'bar',
+    ]
+```
+
+The PluginConfig above creates two custom queues with the following names `my_plugin.foo` and `my_plugin.bar`. (The plugin's name is prepended to each queue to avoid conflicts between plugins.)
+
+!!! warning "Configuring the RQ worker process"
+    By default, NetBox's RQ worker process only services the high, default, and low queues. Plugins which introduce custom queues should advise users to either reconfigure the default worker, or run a dedicated worker specifying the necessary queues. For example:
+    
+    ```
+    python manage.py rqworker my_plugin.foo my_plugin.bar
+    ```

docs/plugins/development/event-types.md

@@ -1,18 +0,0 @@
-# Event Types
-
-!!! info "This feature was introduced in NetBox v4.1."
-
-Plugins can register their own custom event types for use with NetBox [event rules](../../models/extras/eventrule.md). This is accomplished by calling the `register()` method on an instance of the `EventType` class. This can be done anywhere within the plugin. An example is provided below.
-
-```python
-from django.utils.translation import gettext_lazy as _
-from netbox.events import EventType, EVENT_TYPE_KIND_SUCCESS
-
-EventType(
-    name='ticket_opened',
-    text=_('Ticket opened'),
-    kind=EVENT_TYPE_KIND_SUCCESS
-).register()
-```
-
-::: netbox.events.EventType

docs/plugins/development/index.md

@@ -47,7 +47,6 @@ project-name/
     - __init__.py
     - filtersets.py
     - graphql.py
-    - jobs.py
     - models.py
     - middleware.py
     - navigation.py

docs/plugins/development/models.md

@@ -130,8 +130,6 @@ For more information about database migrations, see the [Django documentation](h
 
 ::: netbox.models.features.ExportTemplatesMixin
 
-::: netbox.models.features.JobsMixin
-
 ::: netbox.models.features.JournalingMixin
 
 ::: netbox.models.features.TagsMixin

docs/plugins/development/rest-api.md

@@ -27,7 +27,7 @@ Serializers are responsible for converting Python objects to JSON data suitable
 
 #### Example
 
-To create a serializer for a plugin model, subclass `NetBoxModelSerializer` in `api/serializers.py`. Specify the model class and the fields to include within the serializer's `Meta` class.
+To create a serializer for a plugin model, subclass `NetBoxModelSerializer` in `api/serializers.py`. Specify the model class and the fields to include within the serializer's `Meta` class. It is generally advisable to include a `url` attribute on each serializer. This will render the direct link to access the object being rendered.
 
 ```python
 # api/serializers.py
@@ -36,7 +36,9 @@ from netbox.api.serializers import NetBoxModelSerializer
 from my_plugin.models import MyModel
 
 class MyModelSerializer(NetBoxModelSerializer):
-    foo = SiteSerializer(nested=True, allow_null=True)
+    url = serializers.HyperlinkedIdentityField(
+        view_name='plugins-api:myplugin-api:mymodel-detail'
+    )
 
     class Meta:
         model = MyModel
@@ -61,7 +63,9 @@ from netbox.api.serializers import WritableNestedSerializer
 from my_plugin.models import MyModel
 
 class NestedMyModelSerializer(WritableNestedSerializer):
-    foo = SiteSerializer(nested=True, allow_null=True)
+    url = serializers.HyperlinkedIdentityField(
+        view_name='plugins-api:myplugin-api:mymodel-detail'
+    )
 
     class Meta:
         model = MyModel

docs/plugins/development/views.md

@@ -191,25 +191,19 @@ class MyView(generic.ObjectView):
 
 ### Extra Template Content
 
-Plugins can inject custom content into certain areas of core NetBox views. This is accomplished by subclassing `PluginTemplateExtension`, optionally designating one or more particular NetBox models, and defining the desired method(s) to render custom content. Five methods are available:
+Plugins can inject custom content into certain areas of core NetBox views. This is accomplished by subclassing `PluginTemplateExtension`, designating a particular NetBox model, and defining the desired method(s) to render custom content. Five methods are available:
 
 | Method              | View        | Description                                         |
 |---------------------|-------------|-----------------------------------------------------|
-| `navbar()`          | All         | Inject content inside the top navigation bar        |
-| `list_buttons()`    | List view   | Add buttons to the top of the page                  |
-| `buttons()`         | Object view | Add buttons to the top of the page                  |
-| `alerts()`          | Object view | Inject content at the top of the page               |
 | `left_page()`       | Object view | Inject content on the left side of the page         |
 | `right_page()`      | Object view | Inject content on the right side of the page        |
 | `full_width_page()` | Object view | Inject content across the entire bottom of the page |
-
-!!! info "The `navbar()` and `alerts()` methods were introduced in NetBox v4.1."
+| `buttons()`         | Object view | Add buttons to the top of the page                  |
+| `list_buttons()`    | List view   | Add buttons to the top of the page                  |
 
 Additionally, a `render()` method is available for convenience. This method accepts the name of a template to render, and any additional context data you want to pass. Its use is optional, however.
 
-To control where the custom content is injected, plugin authors can specify an iterable of models by overriding the `models` attribute on the subclass. Extensions which do not specify a set of models will be invoked on every view, where supported.
-
-When a PluginTemplateExtension is instantiated, context data is assigned to `self.context`. Available data includes:
+When a PluginTemplateExtension is instantiated, context data is assigned to `self.context`. Available data include:
 
 * `object` - The object being viewed (object views only)
 * `model` - The model of the list view (list views only)
@@ -226,7 +220,7 @@ from netbox.plugins import PluginTemplateExtension
 from .models import Animal
 
 class SiteAnimalCount(PluginTemplateExtension):
-    models = ['dcim.site']
+    model = 'dcim.site'
 
     def right_page(self):
         return self.render('netbox_animal_sounds/inc/animal_count.html', extra_context={

docs/release-notes/version-4.0.md

@@ -1,6 +1,23 @@
 # NetBox v4.0
 
-## v4.0.9 (FUTURE)
+## v4.0.9 (2024-08-14)
+
+### Enhancements
+
+* [#16692](https://github.com/netbox-community/netbox/issues/16692) - Enable modifying VLAN assignment while bulk editing prefixes
+* [#17006](https://github.com/netbox-community/netbox/issues/17006) - Add IEEE 802.11be interface type
+
+### Bug Fixes
+
+* [#13459](https://github.com/netbox-community/netbox/issues/13459) - Correct OpenAPI schema type for `TreeNodeMultipleChoiceFilter`
+* [#16073](https://github.com/netbox-community/netbox/issues/16073) - Respect default values for custom fields during bulk import of objects
+* [#16176](https://github.com/netbox-community/netbox/issues/16176) - Restore ability to select multiple terminating devices when connecting a cable
+* [#16871](https://github.com/netbox-community/netbox/issues/16871) - Sanitize device ID query parameter when bulk editing components to prevent exception
+* [#17038](https://github.com/netbox-community/netbox/issues/17038) - Fix AttributeError exception when attempting to export system status data
+* [#17064](https://github.com/netbox-community/netbox/issues/17064) - Fix misaligned text within rendered Markdown code blocks
+* [#17124](https://github.com/netbox-community/netbox/issues/17124) - `BaseTable` should follow reverse one-to-one relationships when prefetching related objects
+* [#17131](https://github.com/netbox-community/netbox/issues/17131) - Fix exception when creating object-type custom field without selecting related object type
+* [#17144](https://github.com/netbox-community/netbox/issues/17144) - Avoid showing duplicated pop-up messages
 
 ---
 

docs/release-notes/version-4.1.md

@@ -1,104 +0,0 @@
-# NetBox v4.1
-
-## v4.1-beta1 (2024-08-05)
-
-!!! danger "Not for Production Use"
-    This is a beta release of NetBox intended for testing and evaluation. **Do not use this software in production.** Also be aware that no upgrade path is provided to future releases.
-
-### Breaking Changes
-
-* Several filters deprecated in v4.0 have been removed (see [#15410](https://github.com/netbox-community/netbox/issues/15410)).
-* The unit size for `VirtualMachine.disk` and `VirtualDisk.size` been changed from 1 gigabyte to 1 megabyte. Existing values have been updated accordingly.
-* The `min_vid` and `max_vid` fields on the VLAN group model have been replaced with `vid_ranges`, an array of starting and ending integer pairs.
-* The five individual event type fields on the EventRule model have been replaced by a single `event_types` array field, indicating each assigned event type by name.
-* The UI views & API endpoints associate with change records have been moved from `/extras` to `/core`.
-* The `validate()` method on CustomValidator subclasses now **must** accept the request argument (deprecated in v4.0 by #14279).
-
-### New Features
-
-#### Circuit Groups ([#7025](https://github.com/netbox-community/netbox/issues/7025))
-
-Circuits can now be assigned to groups for administrative purposes. Each circuit may be assigned to multiple groups, and each assignment may optionally indicate a priority (primary, secondary, or tertiary).
-
-#### VLAN Group ID Ranges ([#9627](https://github.com/netbox-community/netbox/issues/9627))
-
-The VLAN group model has been enhanced to support multiple VLAN ID (VID) ranges, whereas previously it could track only a single beginning and ending VID. VID ranges are stored as an array of beginning and ending (inclusive) integer pairs.
-
-#### Nested Device Modules ([#10500](https://github.com/netbox-community/netbox/issues/10500))
-
-Module bays can now be nested to effect a hierarchical arrangement of modules within a device. A module installed within a device's module bay may itself have module bays into which child modules may be installed.
-
-#### Rack Types ([#12826](https://github.com/netbox-community/netbox/issues/12826))
-
-A new rack type model has been introduced, which functions similarly to the device type model. Users can now define a common make and model of rack, the attributes of which are automatically populated when creating a new rack of that type. Backward compatibility for racks with individually defined characteristics is fully retained.
-
-#### Plugins Catalog Integration ([#14731](https://github.com/netbox-community/netbox/issues/14731))
-
-The NetBox UI now integrates directly with the canonical [plugins catalog](https://netboxlabs.com/netbox-plugins/) hosted by NetBox Labs. In addition to locally installed plugins, users can explore available plugins and check for newer releases.
-
-#### User Notifications ([#15621](https://github.com/netbox-community/netbox/issues/15621))
-
-NetBox now includes a user notification system. Users can subscribe to individual objects and be alerted to changes live within the web interface. Additionally, event rules can now trigger notifications to specific users and/or groups. Plugins can also employ this notification system for their own purposes.
-
-### Enhancements
-
-* [#7537](https://github.com/netbox-community/netbox/issues/7537) - Add a serial number field for virtual machines
-* [#8198](https://github.com/netbox-community/netbox/issues/8198) - Enable uniqueness enforcement for custom field values
-* [#8984](https://github.com/netbox-community/netbox/issues/8984) - Enable filtering of custom script output by log level
-* [#11969](https://github.com/netbox-community/netbox/issues/11969) - Support for tracking airflow on racks and module types
-* [#14656](https://github.com/netbox-community/netbox/issues/14656) - Dynamically render custom field edit form depending on the selected field type
-* [#15106](https://github.com/netbox-community/netbox/issues/15106) - Add distance tracking for wireless links
-* [#15156](https://github.com/netbox-community/netbox/issues/15156) - Add `display_url` field to all REST API serializers
-* [#16580](https://github.com/netbox-community/netbox/issues/16580) - Enable individual views to enforce `LOGIN_REQUIRED` selectively (remove `AUTH_EXEMPT_PATHS`)
-* [#16782](https://github.com/netbox-community/netbox/issues/16782) - Enable filtering of selection choices for object type custom fields
-* [#16907](https://github.com/netbox-community/netbox/issues/16907) - Updated user interface styling
-* [#17051](https://github.com/netbox-community/netbox/issues/17051) - Introduced `ISOLATED_DEPLOYMENT` config parameter
-
-### Plugins
-
-* [#15692](https://github.com/netbox-community/netbox/issues/15692) - Introduce improved plugin support for background jobs
-* [#16359](https://github.com/netbox-community/netbox/issues/16359) - Enable plugins to embed content in the top navigation bar
-* [#16726](https://github.com/netbox-community/netbox/issues/16726) - Extend `PluginTemplateExtension` to enable registering multiple models
-* [#16776](https://github.com/netbox-community/netbox/issues/16776) - Added an `alerts()` method to `PluginTemplateExtension` for embedding important information about specific objects
-* [#16886](https://github.com/netbox-community/netbox/issues/16886) - Introduced a mechanism for plugins to register custom event types (for use with user notifications)
-
-### Other Changes
-
-* [#14692](https://github.com/netbox-community/netbox/issues/14692) - Change atomic unit for virtual disks from 1GB to 1MB
-* [#14861](https://github.com/netbox-community/netbox/issues/14861) - The URL path for UI views concerning virtual disks has been standardized to `/virtualization/virtual-disks/`
-* [#15410](https://github.com/netbox-community/netbox/issues/15410) - Removed various deprecated filters
-* [#15908](https://github.com/netbox-community/netbox/issues/15908) - Indicate product edition in release data
-* [#16388](https://github.com/netbox-community/netbox/issues/16388) - Move all change logging resources from `extras` to `core`
-* [#16884](https://github.com/netbox-community/netbox/issues/16884) - Remove the ID column from the default table configuration for changelog records
-* [#16988](https://github.com/netbox-community/netbox/issues/16988) - Relocated rack items in navigation menu
-
-### REST API Changes
-
-* The `/api/extras/object-changes/` endpoint has moved to `/api/core/object-changes/`
-* Added the following endpoints:
-    * `/api/circuits/circuit-groups/`
-    * `/api/circuits/circuit-group-assignments/`
-    * `/api/dcim/rack-types/`
-* circuits.Circuit
-    * Added the `assignments` field, which lists all group assignments
-* dcim.ModuleBay
-    * Added the optional `module` foreign key field
-* dcim.ModuleBayTemplate
-    * Added the optional `module_type` foreign key field
-* dcim.ModuleType
-    * Added the optional `airflow` choice field
-* dcim.Rack
-    * Added the optional `rack_type` foreign key field
-    * Added the optional `airflow` choice field
-* extras.CustomField
-    * Added the `related_object_filter` JSON field for object and multi-object custom fields
-* extras.EventRule
-    * Removed the `type_create`, `type_update`, `type_delete`, `type_job_start`, and `type_job_end` boolean fields
-    * Added the `event_types` array field
-* ipam.VLANGroup
-    * Removed the `min_vid` and `max_vid` fields
-    * Added the `vid_ranges` field, and array of starting & ending VLAN IDs
-* virtualization.VirtualMachine
-    * Added the optional `serial` field
-* wireless.WirelessLink
-    * Added the optional `distance` and `distance_unit` fields

mkdocs.yml

@@ -86,7 +86,6 @@ nav:
         - Change Logging: 'features/change-logging.md'
         - Journaling: 'features/journaling.md'
         - Event Rules: 'features/event-rules.md'
-        - Notifications: 'features/notifications.md'
         - Background Jobs: 'features/background-jobs.md'
         - Auth & Permissions: 'features/authentication-permissions.md'
         - API & Integration: 'features/api-integration.md'
@@ -143,11 +142,10 @@ nav:
             - Forms: 'plugins/development/forms.md'
             - Filters & Filter Sets: 'plugins/development/filtersets.md'
             - Search: 'plugins/development/search.md'
-            - Event Types: 'plugins/development/event-types.md'
             - Data Backends: 'plugins/development/data-backends.md'
             - REST API: 'plugins/development/rest-api.md'
             - GraphQL API: 'plugins/development/graphql-api.md'
-            - Background Jobs: 'plugins/development/background-jobs.md'
+            - Background Tasks: 'plugins/development/background-tasks.md'
             - Dashboard Widgets: 'plugins/development/dashboard-widgets.md'
             - Staged Changes: 'plugins/development/staged-changes.md'
             - Exceptions: 'plugins/development/exceptions.md'
@@ -165,8 +163,6 @@ nav:
     - Data Model:
         - Circuits:
             - Circuit: 'models/circuits/circuit.md'
-            - CircuitGroup: 'models/circuits/circuitgroup.md'
-            - CircuitGroupAssignment: 'models/circuits/circuitgroupassignment.md'
             - Circuit Termination: 'models/circuits/circuittermination.md'
             - Circuit Type: 'models/circuits/circuittype.md'
             - Provider: 'models/circuits/provider.md'
@@ -210,7 +206,6 @@ nav:
             - Rack: 'models/dcim/rack.md'
             - RackReservation: 'models/dcim/rackreservation.md'
             - RackRole: 'models/dcim/rackrole.md'
-            - RackType: 'models/dcim/racktype.md'
             - RearPort: 'models/dcim/rearport.md'
             - RearPortTemplate: 'models/dcim/rearporttemplate.md'
             - Region: 'models/dcim/region.md'
@@ -230,11 +225,8 @@ nav:
             - ExportTemplate: 'models/extras/exporttemplate.md'
             - ImageAttachment: 'models/extras/imageattachment.md'
             - JournalEntry: 'models/extras/journalentry.md'
-            - Notification: 'models/extras/notification.md'
-            - NotificationGroup: 'models/extras/notificationgroup.md'
             - SavedFilter: 'models/extras/savedfilter.md'
             - StagedChange: 'models/extras/stagedchange.md'
-            - Subscription: 'models/extras/subscription.md'
             - Tag: 'models/extras/tag.md'
             - Webhook: 'models/extras/webhook.md'
         - IPAM:
@@ -304,7 +296,6 @@ nav:
         - git Cheat Sheet: 'development/git-cheat-sheet.md'
     - Release Notes:
         - Summary: 'release-notes/index.md'
-        - Version 4.1: 'release-notes/version-4.1.md'
         - Version 4.0: 'release-notes/version-4.0.md'
         - Version 3.7: 'release-notes/version-3.7.md'
         - Version 3.6: 'release-notes/version-3.6.md'

netbox/account/urls.py

@@ -9,8 +9,6 @@ urlpatterns = [
     # Account views
     path('profile/', views.ProfileView.as_view(), name='profile'),
     path('bookmarks/', views.BookmarkListView.as_view(), name='bookmarks'),
-    path('notifications/', views.NotificationListView.as_view(), name='notifications'),
-    path('subscriptions/', views.SubscriptionListView.as_view(), name='subscriptions'),
     path('preferences/', views.UserConfigView.as_view(), name='preferences'),
     path('password/', views.ChangePasswordView.as_view(), name='change_password'),
     path('api-tokens/', views.UserTokenListView.as_view(), name='usertoken_list'),

netbox/account/views.py

@@ -19,10 +19,8 @@ from django.views.generic import View
 from social_core.backends.utils import load_backends
 
 from account.models import UserToken
-from core.models import ObjectChange
-from core.tables import ObjectChangeTable
-from extras.models import Bookmark
-from extras.tables import BookmarkTable, NotificationTable, SubscriptionTable
+from extras.models import Bookmark, ObjectChange
+from extras.tables import BookmarkTable, ObjectChangeTable
 from netbox.authentication import get_auth_backend_display, get_saml_idps
 from netbox.config import get_config
 from netbox.views import generic
@@ -111,7 +109,7 @@ class LoginView(View):
             # Authenticate user
             auth_login(request, form.get_user())
             logger.info(f"User {request.user} successfully authenticated")
-            messages.success(request, f"Logged in as {request.user}.")
+            messages.success(request, _("Logged in as {user}.").format(user=request.user))
 
             # Ensure the user has a UserConfig defined. (This should normally be handled by
             # create_userconfig() on user creation.)
@@ -161,7 +159,7 @@ class LogoutView(View):
         username = request.user
         auth_logout(request)
         logger.info(f"User {username} has logged out")
-        messages.info(request, "You have logged out.")
+        messages.info(request, _("You have logged out."))
 
         # Delete session key & language cookies (if set) upon logout
         response = HttpResponseRedirect(resolve_url(settings.LOGOUT_REDIRECT_URL))
@@ -236,7 +234,7 @@ class ChangePasswordView(LoginRequiredMixin, View):
     def get(self, request):
         # LDAP users cannot change their password here
         if getattr(request.user, 'ldap_username', None):
-            messages.warning(request, "LDAP-authenticated user credentials cannot be changed within NetBox.")
+            messages.warning(request, _("LDAP-authenticated user credentials cannot be changed within NetBox."))
             return redirect('account:profile')
 
         form = PasswordChangeForm(user=request.user)
@@ -251,7 +249,7 @@ class ChangePasswordView(LoginRequiredMixin, View):
         if form.is_valid():
             form.save()
             update_session_auth_hash(request, form.user)
-            messages.success(request, "Your password has been changed successfully.")
+            messages.success(request, _("Your password has been changed successfully."))
             return redirect('account:profile')
 
         return render(request, self.template_name, {
@@ -277,36 +275,6 @@ class BookmarkListView(LoginRequiredMixin, generic.ObjectListView):
         }
 
 
-#
-# Notifications & subscriptions
-#
-
-class NotificationListView(LoginRequiredMixin, generic.ObjectListView):
-    table = NotificationTable
-    template_name = 'account/notifications.html'
-
-    def get_queryset(self, request):
-        return request.user.notifications.all()
-
-    def get_extra_context(self, request):
-        return {
-            'active_tab': 'notifications',
-        }
-
-
-class SubscriptionListView(LoginRequiredMixin, generic.ObjectListView):
-    table = SubscriptionTable
-    template_name = 'account/subscriptions.html'
-
-    def get_queryset(self, request):
-        return request.user.subscriptions.all()
-
-    def get_extra_context(self, request):
-        return {
-            'active_tab': 'subscriptions',
-        }
-
-
 #
 # User views for token management
 #

netbox/circuits/api/nested_serializers.py

@@ -20,10 +20,11 @@ __all__ = [
 #
 
 class NestedProviderNetworkSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='circuits-api:providernetwork-detail')
 
     class Meta:
         model = ProviderNetwork
-        fields = ['id', 'url', 'display_url', 'display', 'name']
+        fields = ['id', 'url', 'display', 'name']
 
 
 #
@@ -34,11 +35,12 @@ class NestedProviderNetworkSerializer(WritableNestedSerializer):
     exclude_fields=('circuit_count',),
 )
 class NestedProviderSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='circuits-api:provider-detail')
     circuit_count = RelatedObjectCountField('circuits')
 
     class Meta:
         model = Provider
-        fields = ['id', 'url', 'display_url', 'display', 'name', 'slug', 'circuit_count']
+        fields = ['id', 'url', 'display', 'name', 'slug', 'circuit_count']
 
 
 #
@@ -46,10 +48,11 @@ class NestedProviderSerializer(WritableNestedSerializer):
 #
 
 class NestedProviderAccountSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='circuits-api:provideraccount-detail')
 
     class Meta:
         model = ProviderAccount
-        fields = ['id', 'url', 'display_url', 'display', 'name', 'account']
+        fields = ['id', 'url', 'display', 'name', 'account']
 
 
 #
@@ -60,23 +63,26 @@ class NestedProviderAccountSerializer(WritableNestedSerializer):
     exclude_fields=('circuit_count',),
 )
 class NestedCircuitTypeSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='circuits-api:circuittype-detail')
     circuit_count = RelatedObjectCountField('circuits')
 
     class Meta:
         model = CircuitType
-        fields = ['id', 'url', 'display_url', 'display', 'name', 'slug', 'circuit_count']
+        fields = ['id', 'url', 'display', 'name', 'slug', 'circuit_count']
 
 
 class NestedCircuitSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='circuits-api:circuit-detail')
 
     class Meta:
         model = Circuit
-        fields = ['id', 'url', 'display_url', 'display', 'cid']
+        fields = ['id', 'url', 'display', 'cid']
 
 
 class NestedCircuitTerminationSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='circuits-api:circuittermination-detail')
     circuit = NestedCircuitSerializer()
 
     class Meta:
         model = CircuitTermination
-        fields = ['id', 'url', 'display_url', 'display', 'circuit', 'term_side', 'cable', '_occupied']
+        fields = ['id', 'url', 'display', 'circuit', 'term_side', 'cable', '_occupied']

netbox/circuits/api/serializers_/circuits.py

@@ -1,7 +1,7 @@
 from rest_framework import serializers
 
-from circuits.choices import CircuitPriorityChoices, CircuitStatusChoices
-from circuits.models import Circuit, CircuitGroup, CircuitGroupAssignment, CircuitTermination, CircuitType
+from circuits.choices import CircuitStatusChoices
+from circuits.models import Circuit, CircuitTermination, CircuitType
 from dcim.api.serializers_.cables import CabledObjectSerializer
 from dcim.api.serializers_.sites import SiteSerializer
 from netbox.api.fields import ChoiceField, RelatedObjectCountField
@@ -12,14 +12,13 @@ from .providers import ProviderAccountSerializer, ProviderNetworkSerializer, Pro
 
 __all__ = (
     'CircuitSerializer',
-    'CircuitGroupAssignmentSerializer',
-    'CircuitGroupSerializer',
     'CircuitTerminationSerializer',
     'CircuitTypeSerializer',
 )
 
 
 class CircuitTypeSerializer(NetBoxModelSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='circuits-api:circuittype-detail')
 
     # Related object counts
     circuit_count = RelatedObjectCountField('circuits')
@@ -27,53 +26,27 @@ class CircuitTypeSerializer(NetBoxModelSerializer):
     class Meta:
         model = CircuitType
         fields = [
-            'id', 'url', 'display_url', 'display', 'name', 'slug', 'color', 'description', 'tags', 'custom_fields',
-            'created', 'last_updated', 'circuit_count',
+            'id', 'url', 'display', 'name', 'slug', 'color', 'description', 'tags', 'custom_fields', 'created',
+            'last_updated', 'circuit_count',
         ]
         brief_fields = ('id', 'url', 'display', 'name', 'slug', 'description', 'circuit_count')
 
 
 class CircuitCircuitTerminationSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='circuits-api:circuittermination-detail')
     site = SiteSerializer(nested=True, allow_null=True)
     provider_network = ProviderNetworkSerializer(nested=True, allow_null=True)
 
     class Meta:
         model = CircuitTermination
         fields = [
-            'id', 'url', 'display_url', 'display', 'site', 'provider_network', 'port_speed', 'upstream_speed',
-            'xconnect_id', 'description',
+            'id', 'url', 'display', 'site', 'provider_network', 'port_speed', 'upstream_speed', 'xconnect_id',
+            'description',
         ]
 
 
-class CircuitGroupSerializer(NetBoxModelSerializer):
-    tenant = TenantSerializer(nested=True, required=False, allow_null=True)
-    circuit_count = RelatedObjectCountField('assignments')
-
-    class Meta:
-        model = CircuitGroup
-        fields = [
-            'id', 'url', 'display_url', 'display', 'name', 'slug', 'description', 'tenant',
-            'tags', 'custom_fields', 'created', 'last_updated', 'circuit_count'
-        ]
-        brief_fields = ('id', 'url', 'display', 'name')
-
-
-class CircuitGroupAssignmentSerializer_(NetBoxModelSerializer):
-    """
-    Base serializer for group assignments under CircuitSerializer.
-    """
-    group = CircuitGroupSerializer(nested=True)
-    priority = ChoiceField(choices=CircuitPriorityChoices, allow_blank=True, required=False)
-
-    class Meta:
-        model = CircuitGroupAssignment
-        fields = [
-            'id', 'url', 'display_url', 'display', 'group', 'priority', 'tags', 'created', 'last_updated',
-        ]
-        brief_fields = ('id', 'url', 'display', 'group', 'priority')
-
-
 class CircuitSerializer(NetBoxModelSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='circuits-api:circuit-detail')
     provider = ProviderSerializer(nested=True)
     provider_account = ProviderAccountSerializer(nested=True, required=False, allow_null=True, default=None)
     status = ChoiceField(choices=CircuitStatusChoices, required=False)
@@ -81,19 +54,19 @@ class CircuitSerializer(NetBoxModelSerializer):
     tenant = TenantSerializer(nested=True, required=False, allow_null=True)
     termination_a = CircuitCircuitTerminationSerializer(read_only=True, allow_null=True)
     termination_z = CircuitCircuitTerminationSerializer(read_only=True, allow_null=True)
-    assignments = CircuitGroupAssignmentSerializer_(nested=True, many=True, required=False)
 
     class Meta:
         model = Circuit
         fields = [
-            'id', 'url', 'display_url', 'display', 'cid', 'provider', 'provider_account', 'type', 'status', 'tenant',
-            'install_date', 'termination_date', 'commit_rate', 'description', 'termination_a', 'termination_z',
-            'comments', 'tags', 'custom_fields', 'created', 'last_updated', 'assignments',
+            'id', 'url', 'display', 'cid', 'provider', 'provider_account', 'type', 'status', 'tenant', 'install_date',
+            'termination_date', 'commit_rate', 'description', 'termination_a', 'termination_z', 'comments', 'tags',
+            'custom_fields', 'created', 'last_updated',
         ]
         brief_fields = ('id', 'url', 'display', 'cid', 'description')
 
 
 class CircuitTerminationSerializer(NetBoxModelSerializer, CabledObjectSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='circuits-api:circuittermination-detail')
     circuit = CircuitSerializer(nested=True)
     site = SiteSerializer(nested=True, required=False, allow_null=True)
     provider_network = ProviderNetworkSerializer(nested=True, required=False, allow_null=True)
@@ -101,19 +74,8 @@ class CircuitTerminationSerializer(NetBoxModelSerializer, CabledObjectSerializer
     class Meta:
         model = CircuitTermination
         fields = [
-            'id', 'url', 'display_url', 'display', 'circuit', 'term_side', 'site', 'provider_network', 'port_speed',
-            'upstream_speed', 'xconnect_id', 'pp_info', 'description', 'mark_connected', 'cable', 'cable_end',
-            'link_peers', 'link_peers_type', 'tags', 'custom_fields', 'created', 'last_updated', '_occupied',
+            'id', 'url', 'display', 'circuit', 'term_side', 'site', 'provider_network', 'port_speed', 'upstream_speed',
+            'xconnect_id', 'pp_info', 'description', 'mark_connected', 'cable', 'cable_end', 'link_peers',
+            'link_peers_type', 'tags', 'custom_fields', 'created', 'last_updated', '_occupied',
         ]
         brief_fields = ('id', 'url', 'display', 'circuit', 'term_side', 'description', 'cable', '_occupied')
-
-
-class CircuitGroupAssignmentSerializer(CircuitGroupAssignmentSerializer_):
-    circuit = CircuitSerializer(nested=True)
-
-    class Meta:
-        model = CircuitGroupAssignment
-        fields = [
-            'id', 'url', 'display_url', 'display', 'group', 'circuit', 'priority', 'tags', 'created', 'last_updated',
-        ]
-        brief_fields = ('id', 'url', 'display', 'group', 'circuit', 'priority')

netbox/circuits/api/serializers_/providers.py

@@ -15,6 +15,7 @@ __all__ = (
 
 
 class ProviderSerializer(NetBoxModelSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='circuits-api:provider-detail')
     accounts = SerializedPKRelatedField(
         queryset=ProviderAccount.objects.all(),
         serializer=NestedProviderAccountSerializer,
@@ -35,32 +36,34 @@ class ProviderSerializer(NetBoxModelSerializer):
     class Meta:
         model = Provider
         fields = [
-            'id', 'url', 'display_url', 'display', 'name', 'slug', 'accounts', 'description', 'comments',
-            'asns', 'tags', 'custom_fields', 'created', 'last_updated', 'circuit_count',
+            'id', 'url', 'display', 'name', 'slug', 'accounts', 'description', 'comments', 'asns', 'tags',
+            'custom_fields', 'created', 'last_updated', 'circuit_count',
         ]
         brief_fields = ('id', 'url', 'display', 'name', 'slug', 'description', 'circuit_count')
 
 
 class ProviderAccountSerializer(NetBoxModelSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='circuits-api:provideraccount-detail')
     provider = ProviderSerializer(nested=True)
     name = serializers.CharField(allow_blank=True, max_length=100, required=False, default='')
 
     class Meta:
         model = ProviderAccount
         fields = [
-            'id', 'url', 'display_url', 'display', 'provider', 'name', 'account', 'description', 'comments', 'tags',
-            'custom_fields', 'created', 'last_updated',
+            'id', 'url', 'display', 'provider', 'name', 'account', 'description', 'comments', 'tags', 'custom_fields',
+            'created', 'last_updated',
         ]
         brief_fields = ('id', 'url', 'display', 'name', 'account', 'description')
 
 
 class ProviderNetworkSerializer(NetBoxModelSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='circuits-api:providernetwork-detail')
     provider = ProviderSerializer(nested=True)
 
     class Meta:
         model = ProviderNetwork
         fields = [
-            'id', 'url', 'display_url', 'display', 'provider', 'name', 'service_id', 'description', 'comments', 'tags',
+            'id', 'url', 'display', 'provider', 'name', 'service_id', 'description', 'comments', 'tags',
             'custom_fields', 'created', 'last_updated',
         ]
         brief_fields = ('id', 'url', 'display', 'name', 'description')

netbox/circuits/api/urls.py

@@ -14,8 +14,6 @@ router.register('provider-networks', views.ProviderNetworkViewSet)
 router.register('circuit-types', views.CircuitTypeViewSet)
 router.register('circuits', views.CircuitViewSet)
 router.register('circuit-terminations', views.CircuitTerminationViewSet)
-router.register('circuit-groups', views.CircuitGroupViewSet)
-router.register('circuit-group-assignments', views.CircuitGroupAssignmentViewSet)
 
 app_name = 'circuits-api'
 urlpatterns = router.urls

netbox/circuits/api/views.py

@@ -55,26 +55,6 @@ class CircuitTerminationViewSet(PassThroughPortMixin, NetBoxModelViewSet):
     filterset_class = filtersets.CircuitTerminationFilterSet
 
 
-#
-# Circuit Groups
-#
-
-class CircuitGroupViewSet(NetBoxModelViewSet):
-    queryset = CircuitGroup.objects.all()
-    serializer_class = serializers.CircuitGroupSerializer
-    filterset_class = filtersets.CircuitGroupFilterSet
-
-
-#
-# Circuit Group Assignments
-#
-
-class CircuitGroupAssignmentViewSet(NetBoxModelViewSet):
-    queryset = CircuitGroupAssignment.objects.all()
-    serializer_class = serializers.CircuitGroupAssignmentSerializer
-    filterset_class = filtersets.CircuitGroupAssignmentFilterSet
-
-
 #
 # Provider accounts
 #

netbox/circuits/choices.py

@@ -76,19 +76,3 @@ class CircuitTerminationPortSpeedChoices(ChoiceSet):
         (1544, 'T1 (1.544 Mbps)'),
         (2048, 'E1 (2.048 Mbps)'),
     ]
-
-
-class CircuitPriorityChoices(ChoiceSet):
-    key = 'CircuitGroupAssignment.priority'
-
-    PRIORITY_PRIMARY = 'primary'
-    PRIORITY_SECONDARY = 'secondary'
-    PRIORITY_TERTIARY = 'tertiary'
-    PRIORITY_INACTIVE = 'inactive'
-
-    CHOICES = [
-        (PRIORITY_PRIMARY, _('Primary')),
-        (PRIORITY_SECONDARY, _('Secondary')),
-        (PRIORITY_TERTIARY, _('Tertiary')),
-        (PRIORITY_INACTIVE, _('Inactive')),
-    ]

netbox/circuits/filtersets.py

@@ -13,8 +13,6 @@ from .models import *
 
 __all__ = (
     'CircuitFilterSet',
-    'CircuitGroupAssignmentFilterSet',
-    'CircuitGroupFilterSet',
     'CircuitTerminationFilterSet',
     'CircuitTypeFilterSet',
     'ProviderNetworkFilterSet',
@@ -305,43 +303,3 @@ class CircuitTerminationFilterSet(NetBoxModelFilterSet, CabledObjectFilterSet):
             Q(pp_info__icontains=value) |
             Q(description__icontains=value)
         ).distinct()
-
-
-class CircuitGroupFilterSet(OrganizationalModelFilterSet, TenancyFilterSet):
-
-    class Meta:
-        model = CircuitGroup
-        fields = ('id', 'name', 'slug', 'description')
-
-
-class CircuitGroupAssignmentFilterSet(NetBoxModelFilterSet):
-    q = django_filters.CharFilter(
-        method='search',
-        label=_('Search'),
-    )
-    circuit_id = django_filters.ModelMultipleChoiceFilter(
-        queryset=Circuit.objects.all(),
-        label=_('Circuit'),
-    )
-    group_id = django_filters.ModelMultipleChoiceFilter(
-        queryset=CircuitGroup.objects.all(),
-        label=_('Circuit group (ID)'),
-    )
-    group = django_filters.ModelMultipleChoiceFilter(
-        field_name='group__slug',
-        queryset=CircuitGroup.objects.all(),
-        to_field_name='slug',
-        label=_('Circuit group (slug)'),
-    )
-
-    class Meta:
-        model = CircuitGroupAssignment
-        fields = ('id', 'circuit', 'group', 'priority')
-
-    def search(self, queryset, name, value):
-        if not value.strip():
-            return queryset
-        return queryset.filter(
-            Q(circuit__cid__icontains=value) |
-            Q(group__name__icontains=value)
-        )

netbox/circuits/forms/bulk_edit.py

@@ -1,7 +1,7 @@
 from django import forms
 from django.utils.translation import gettext_lazy as _
 
-from circuits.choices import CircuitCommitRateChoices, CircuitPriorityChoices, CircuitStatusChoices
+from circuits.choices import CircuitCommitRateChoices, CircuitStatusChoices
 from circuits.models import *
 from dcim.models import Site
 from ipam.models import ASN
@@ -14,8 +14,6 @@ from utilities.forms.widgets import BulkEditNullBooleanSelect, DatePicker, Numbe
 
 __all__ = (
     'CircuitBulkEditForm',
-    'CircuitGroupAssignmentBulkEditForm',
-    'CircuitGroupBulkEditForm',
     'CircuitTerminationBulkEditForm',
     'CircuitTypeBulkEditForm',
     'ProviderBulkEditForm',
@@ -221,40 +219,3 @@ class CircuitTerminationBulkEditForm(NetBoxModelBulkEditForm):
         FieldSet('port_speed', 'upstream_speed', name=_('Termination Details')),
     )
     nullable_fields = ('description')
-
-
-class CircuitGroupBulkEditForm(NetBoxModelBulkEditForm):
-    description = forms.CharField(
-        label=_('Description'),
-        max_length=200,
-        required=False
-    )
-    tenant = DynamicModelChoiceField(
-        label=_('Tenant'),
-        queryset=Tenant.objects.all(),
-        required=False
-    )
-
-    model = CircuitGroup
-    nullable_fields = (
-        'description', 'tenant',
-    )
-
-
-class CircuitGroupAssignmentBulkEditForm(NetBoxModelBulkEditForm):
-    circuit = DynamicModelChoiceField(
-        label=_('Circuit'),
-        queryset=Circuit.objects.all(),
-        required=False
-    )
-    priority = forms.ChoiceField(
-        label=_('Priority'),
-        choices=add_blank_choice(CircuitPriorityChoices),
-        required=False
-    )
-
-    model = CircuitGroupAssignment
-    fieldsets = (
-        FieldSet('circuit', 'priority'),
-    )
-    nullable_fields = ('priority',)

netbox/circuits/forms/bulk_import.py

@@ -11,8 +11,6 @@ from utilities.forms.fields import CSVChoiceField, CSVModelChoiceField, SlugFiel
 
 __all__ = (
     'CircuitImportForm',
-    'CircuitGroupAssignmentImportForm',
-    'CircuitGroupImportForm',
     'CircuitTerminationImportForm',
     'CircuitTerminationImportRelatedForm',
     'CircuitTypeImportForm',
@@ -152,24 +150,3 @@ class CircuitTerminationImportForm(NetBoxModelImportForm, BaseCircuitTermination
             'circuit', 'term_side', 'site', 'provider_network', 'port_speed', 'upstream_speed', 'xconnect_id',
             'pp_info', 'description', 'tags'
         ]
-
-
-class CircuitGroupImportForm(NetBoxModelImportForm):
-    tenant = CSVModelChoiceField(
-        label=_('Tenant'),
-        queryset=Tenant.objects.all(),
-        required=False,
-        to_field_name='name',
-        help_text=_('Assigned tenant')
-    )
-
-    class Meta:
-        model = CircuitGroup
-        fields = ('name', 'slug', 'description', 'tenant', 'tags')
-
-
-class CircuitGroupAssignmentImportForm(NetBoxModelImportForm):
-
-    class Meta:
-        model = CircuitGroupAssignment
-        fields = ('circuit', 'group', 'priority')

netbox/circuits/forms/filtersets.py

@@ -1,7 +1,7 @@
 from django import forms
 from django.utils.translation import gettext as _
 
-from circuits.choices import CircuitCommitRateChoices, CircuitPriorityChoices, CircuitStatusChoices, CircuitTerminationSideChoices
+from circuits.choices import CircuitCommitRateChoices, CircuitStatusChoices, CircuitTerminationSideChoices
 from circuits.models import *
 from dcim.models import Region, Site, SiteGroup
 from ipam.models import ASN
@@ -13,8 +13,6 @@ from utilities.forms.widgets import DatePicker, NumberWithOptions
 
 __all__ = (
     'CircuitFilterForm',
-    'CircuitGroupAssignmentFilterForm',
-    'CircuitGroupFilterForm',
     'CircuitTerminationFilterForm',
     'CircuitTypeFilterForm',
     'ProviderFilterForm',
@@ -232,36 +230,3 @@ class CircuitTerminationFilterForm(NetBoxModelFilterSetForm):
         label=_('Provider')
     )
     tag = TagFilterField(model)
-
-
-class CircuitGroupFilterForm(TenancyFilterForm, NetBoxModelFilterSetForm):
-    model = CircuitGroup
-    fieldsets = (
-        FieldSet('q', 'filter_id', 'tag'),
-        FieldSet('tenant_group_id', 'tenant_id', name=_('Tenant')),
-    )
-    tag = TagFilterField(model)
-
-
-class CircuitGroupAssignmentFilterForm(NetBoxModelFilterSetForm):
-    model = CircuitGroupAssignment
-    fieldsets = (
-        FieldSet('q', 'filter_id', 'tag'),
-        FieldSet('circuit_id', 'group_id', 'priority', name=_('Assignment')),
-    )
-    circuit_id = DynamicModelMultipleChoiceField(
-        queryset=Circuit.objects.all(),
-        required=False,
-        label=_('Circuit')
-    )
-    group_id = DynamicModelMultipleChoiceField(
-        queryset=CircuitGroup.objects.all(),
-        required=False,
-        label=_('Group')
-    )
-    priority = forms.MultipleChoiceField(
-        label=_('Priority'),
-        choices=CircuitPriorityChoices,
-        required=False
-    )
-    tag = TagFilterField(model)

netbox/circuits/forms/model_forms.py

@@ -12,8 +12,6 @@ from utilities.forms.widgets import DatePicker, NumberWithOptions
 
 __all__ = (
     'CircuitForm',
-    'CircuitGroupAssignmentForm',
-    'CircuitGroupForm',
     'CircuitTerminationForm',
     'CircuitTypeForm',
     'ProviderForm',
@@ -173,36 +171,3 @@ class CircuitTerminationForm(NetBoxModelForm):
                 options=CircuitTerminationPortSpeedChoices
             ),
         }
-
-
-class CircuitGroupForm(TenancyForm, NetBoxModelForm):
-    slug = SlugField()
-
-    fieldsets = (
-        FieldSet('name', 'slug', 'description', 'tags', name=_('Circuit Group')),
-        FieldSet('tenant_group', 'tenant', name=_('Tenancy')),
-    )
-
-    class Meta:
-        model = CircuitGroup
-        fields = [
-            'name', 'slug', 'description', 'tenant_group', 'tenant', 'tags',
-        ]
-
-
-class CircuitGroupAssignmentForm(NetBoxModelForm):
-    group = DynamicModelChoiceField(
-        label=_('Group'),
-        queryset=CircuitGroup.objects.all(),
-    )
-    circuit = DynamicModelChoiceField(
-        label=_('Circuit'),
-        queryset=Circuit.objects.all(),
-        selector=True
-    )
-
-    class Meta:
-        model = CircuitGroupAssignment
-        fields = [
-            'group', 'circuit', 'priority', 'tags',
-        ]

netbox/circuits/graphql/filters.py

@@ -7,8 +7,6 @@ from netbox.graphql.filter_mixins import autotype_decorator, BaseFilterMixin
 __all__ = (
     'CircuitTerminationFilter',
     'CircuitFilter',
-    'CircuitGroupAssignmentFilter',
-    'CircuitGroupFilter',
     'CircuitTypeFilter',
     'ProviderFilter',
     'ProviderAccountFilter',
@@ -34,18 +32,6 @@ class CircuitTypeFilter(BaseFilterMixin):
     pass
 
 
-@strawberry_django.filter(models.CircuitGroup, lookups=True)
-@autotype_decorator(filtersets.CircuitGroupFilterSet)
-class CircuitGroupFilter(BaseFilterMixin):
-    pass
-
-
-@strawberry_django.filter(models.CircuitGroupAssignment, lookups=True)
-@autotype_decorator(filtersets.CircuitGroupAssignmentFilterSet)
-class CircuitGroupAssignmentFilter(BaseFilterMixin):
-    pass
-
-
 @strawberry_django.filter(models.Provider, lookups=True)
 @autotype_decorator(filtersets.ProviderFilterSet)
 class ProviderFilter(BaseFilterMixin):

netbox/circuits/graphql/schema.py

@@ -24,16 +24,6 @@ class CircuitsQuery:
         return models.CircuitType.objects.get(pk=id)
     circuit_type_list: List[CircuitTypeType] = strawberry_django.field()
 
-    @strawberry.field
-    def circuit_group(self, id: int) -> CircuitGroupType:
-        return models.CircuitGroup.objects.get(pk=id)
-    circuit_group_list: List[CircuitGroupType] = strawberry_django.field()
-
-    @strawberry.field
-    def circuit_group_assignment(self, id: int) -> CircuitGroupAssignmentType:
-        return models.CircuitGroupAssignment.objects.get(pk=id)
-    circuit_group_assignment_list: List[CircuitGroupAssignmentType] = strawberry_django.field()
-
     @strawberry.field
     def provider(self, id: int) -> ProviderType:
         return models.Provider.objects.get(pk=id)

netbox/circuits/graphql/types.py

@@ -6,15 +6,13 @@ import strawberry_django
 from circuits import models
 from dcim.graphql.mixins import CabledObjectMixin
 from extras.graphql.mixins import ContactsMixin, CustomFieldsMixin, TagsMixin
-from netbox.graphql.types import BaseObjectType, NetBoxObjectType, ObjectType, OrganizationalObjectType
+from netbox.graphql.types import NetBoxObjectType, ObjectType, OrganizationalObjectType
 from tenancy.graphql.types import TenantType
 from .filters import *
 
 __all__ = (
     'CircuitTerminationType',
     'CircuitType',
-    'CircuitGroupAssignmentType',
-    'CircuitGroupType',
     'CircuitTypeType',
     'ProviderType',
     'ProviderAccountType',
@@ -93,22 +91,3 @@ class CircuitType(NetBoxObjectType, ContactsMixin):
     tenant: TenantType | None
 
     terminations: List[CircuitTerminationType]
-
-
-@strawberry_django.type(
-    models.CircuitGroup,
-    fields='__all__',
-    filters=CircuitGroupFilter
-)
-class CircuitGroupType(OrganizationalObjectType):
-    tenant: TenantType | None
-
-
-@strawberry_django.type(
-    models.CircuitGroupAssignment,
-    fields='__all__',
-    filters=CircuitGroupAssignmentFilter
-)
-class CircuitGroupAssignmentType(TagsMixin, BaseObjectType):
-    group: Annotated["CircuitGroupType", strawberry.lazy('circuits.graphql.types')]
-    circuit: Annotated["CircuitType", strawberry.lazy('circuits.graphql.types')]

netbox/circuits/migrations/0044_circuit_groups.py

@@ -1,90 +0,0 @@
-# Generated by Django 5.0.7 on 2024-07-22 06:27
-
-import django.db.models.deletion
-import taggit.managers
-import utilities.json
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('circuits', '0043_circuittype_color'),
-        ('extras', '0118_notifications'),
-        ('tenancy', '0015_contactassignment_rename_content_type'),
-    ]
-
-    operations = [
-        migrations.CreateModel(
-            name='CircuitGroup',
-            fields=[
-                ('id', models.BigAutoField(auto_created=True, primary_key=True, serialize=False)),
-                ('created', models.DateTimeField(auto_now_add=True, null=True)),
-                ('last_updated', models.DateTimeField(auto_now=True, null=True)),
-                (
-                    'custom_field_data',
-                    models.JSONField(blank=True, default=dict, encoder=utilities.json.CustomFieldJSONEncoder),
-                ),
-                ('name', models.CharField(max_length=100, unique=True)),
-                ('slug', models.SlugField(max_length=100, unique=True)),
-                ('description', models.CharField(blank=True, max_length=200)),
-                ('tags', taggit.managers.TaggableManager(through='extras.TaggedItem', to='extras.Tag')),
-                (
-                    'tenant',
-                    models.ForeignKey(
-                        blank=True,
-                        null=True,
-                        on_delete=django.db.models.deletion.PROTECT,
-                        related_name='circuit_groups',
-                        to='tenancy.tenant',
-                    ),
-                ),
-            ],
-            options={
-                'verbose_name': 'Circuit group',
-                'verbose_name_plural': 'Circuit group',
-                'ordering': ('name',),
-            },
-        ),
-        migrations.CreateModel(
-            name='CircuitGroupAssignment',
-            fields=[
-                ('id', models.BigAutoField(auto_created=True, primary_key=True, serialize=False)),
-                ('created', models.DateTimeField(auto_now_add=True, null=True)),
-                ('last_updated', models.DateTimeField(auto_now=True, null=True)),
-                (
-                    'custom_field_data',
-                    models.JSONField(blank=True, default=dict, encoder=utilities.json.CustomFieldJSONEncoder),
-                ),
-                ('priority', models.CharField(blank=True, max_length=50)),
-                (
-                    'circuit',
-                    models.ForeignKey(
-                        on_delete=django.db.models.deletion.CASCADE,
-                        related_name='assignments',
-                        to='circuits.circuit',
-                    ),
-                ),
-                (
-                    'group',
-                    models.ForeignKey(
-                        on_delete=django.db.models.deletion.CASCADE,
-                        related_name='assignments',
-                        to='circuits.circuitgroup',
-                    ),
-                ),
-                ('tags', taggit.managers.TaggableManager(through='extras.TaggedItem', to='extras.Tag')),
-            ],
-            options={
-                'verbose_name': 'Circuit group assignment',
-                'verbose_name_plural': 'Circuit group assignments',
-                'ordering': ('group', 'circuit', 'priority', 'pk'),
-            },
-        ),
-        migrations.AddConstraint(
-            model_name='circuitgroupassignment',
-            constraint=models.UniqueConstraint(
-                fields=('circuit', 'group'), name='circuits_circuitgroupassignment_unique_circuit_group'
-            ),
-        ),
-    ]

netbox/circuits/models/circuits.py

@@ -6,13 +6,11 @@ from django.utils.translation import gettext_lazy as _
 from circuits.choices import *
 from dcim.models import CabledObjectModel
 from netbox.models import ChangeLoggedModel, OrganizationalModel, PrimaryModel
-from netbox.models.features import ContactsMixin, CustomFieldsMixin, CustomLinksMixin, ExportTemplatesMixin, ImageAttachmentsMixin, TagsMixin
+from netbox.models.features import ContactsMixin, CustomFieldsMixin, CustomLinksMixin, ImageAttachmentsMixin, TagsMixin
 from utilities.fields import ColorField
 
 __all__ = (
     'Circuit',
-    'CircuitGroup',
-    'CircuitGroupAssignment',
     'CircuitTermination',
     'CircuitType',
 )
@@ -153,75 +151,6 @@ class Circuit(ContactsMixin, ImageAttachmentsMixin, PrimaryModel):
             raise ValidationError({'provider_account': "The assigned account must belong to the assigned provider."})
 
 
-class CircuitGroup(OrganizationalModel):
-    """
-    An administrative grouping of Circuits.
-    """
-    tenant = models.ForeignKey(
-        to='tenancy.Tenant',
-        on_delete=models.PROTECT,
-        related_name='circuit_groups',
-        blank=True,
-        null=True
-    )
-
-    class Meta:
-        ordering = ('name',)
-        verbose_name = _('circuit group')
-        verbose_name_plural = _('circuit groups')
-
-    def __str__(self):
-        return self.name
-
-    def get_absolute_url(self):
-        return reverse('circuits:circuitgroup', args=[self.pk])
-
-
-class CircuitGroupAssignment(CustomFieldsMixin, ExportTemplatesMixin, TagsMixin, ChangeLoggedModel):
-    """
-    Assignment of a Circuit to a CircuitGroup with an optional priority.
-    """
-    circuit = models.ForeignKey(
-        Circuit,
-        on_delete=models.CASCADE,
-        related_name='assignments'
-    )
-    group = models.ForeignKey(
-        CircuitGroup,
-        on_delete=models.CASCADE,
-        related_name='assignments'
-    )
-    priority = models.CharField(
-        verbose_name=_('priority'),
-        max_length=50,
-        choices=CircuitPriorityChoices,
-        blank=True
-    )
-    prerequisite_models = (
-        'circuits.Circuit',
-        'circuits.CircuitGroup',
-    )
-
-    class Meta:
-        ordering = ('group', 'circuit', 'priority', 'pk')
-        constraints = (
-            models.UniqueConstraint(
-                fields=('circuit', 'group'),
-                name='%(app_label)s_%(class)s_unique_circuit_group'
-            ),
-        )
-        verbose_name = _('Circuit group assignment')
-        verbose_name_plural = _('Circuit group assignments')
-
-    def __str__(self):
-        if self.priority:
-            return f"{self.group} ({self.get_priority_display()})"
-        return str(self.group)
-
-    def get_absolute_url(self):
-        return reverse('circuits:circuitgroupassignment', args=[self.pk])
-
-
 class CircuitTermination(
     CustomFieldsMixin,
     CustomLinksMixin,

netbox/circuits/search.py

@@ -13,17 +13,6 @@ class CircuitIndex(SearchIndex):
     display_attrs = ('provider', 'provider_account', 'type', 'status', 'tenant', 'description')
 
 
-@register_search
-class CircuitGroupIndex(SearchIndex):
-    model = models.CircuitGroup
-    fields = (
-        ('name', 100),
-        ('slug', 110),
-        ('description', 500),
-    )
-    display_attrs = ('description',)
-
-
 @register_search
 class CircuitTerminationIndex(SearchIndex):
     model = models.CircuitTermination

netbox/circuits/tables/circuits.py

@@ -9,8 +9,6 @@ from netbox.tables import NetBoxTable, columns
 from .columns import CommitRateColumn
 
 __all__ = (
-    'CircuitGroupAssignmentTable',
-    'CircuitGroupTable',
     'CircuitTable',
     'CircuitTerminationTable',
     'CircuitTypeTable',
@@ -77,22 +75,18 @@ class CircuitTable(TenancyColumnsMixin, ContactsColumnMixin, NetBoxTable):
         verbose_name=_('Commit Rate')
     )
     comments = columns.MarkdownColumn(
-        verbose_name=_('Comments')
+        verbose_name=_('Comments'),
     )
     tags = columns.TagColumn(
         url_name='circuits:circuit_list'
     )
-    assignments = columns.ManyToManyColumn(
-        verbose_name=_('Assignments'),
-        linkify_item=True
-    )
 
     class Meta(NetBoxTable.Meta):
         model = Circuit
         fields = (
             'pk', 'id', 'cid', 'provider', 'provider_account', 'type', 'status', 'tenant', 'tenant_group',
             'termination_a', 'termination_z', 'install_date', 'termination_date', 'commit_rate', 'description',
-            'comments', 'contacts', 'tags', 'created', 'last_updated', 'assignments',
+            'comments', 'contacts', 'tags', 'created', 'last_updated',
         )
         default_columns = (
             'pk', 'cid', 'provider', 'type', 'status', 'tenant', 'termination_a', 'termination_z', 'description',
@@ -125,50 +119,3 @@ class CircuitTerminationTable(NetBoxTable):
             'xconnect_id', 'pp_info', 'description', 'created', 'last_updated', 'actions',
         )
         default_columns = ('pk', 'id', 'circuit', 'provider', 'term_side', 'description')
-
-
-class CircuitGroupTable(NetBoxTable):
-    name = tables.Column(
-        verbose_name=_('Name'),
-        linkify=True
-    )
-    circuit_group_assignment_count = columns.LinkedCountColumn(
-        viewname='circuits:circuitgroupassignment_list',
-        url_params={'group_id': 'pk'},
-        verbose_name=_('Circuits')
-    )
-    tags = columns.TagColumn(
-        url_name='circuits:circuitgroup_list'
-    )
-
-    class Meta(NetBoxTable.Meta):
-        model = CircuitGroup
-        fields = (
-            'pk', 'name', 'description', 'circuit_group_assignment_count', 'tags',
-            'created', 'last_updated', 'actions',
-        )
-        default_columns = ('pk', 'name', 'description', 'circuit_group_assignment_count')
-
-
-class CircuitGroupAssignmentTable(NetBoxTable):
-    group = tables.Column(
-        verbose_name=_('Group'),
-        linkify=True
-    )
-    circuit = tables.Column(
-        verbose_name=_('Circuit'),
-        linkify=True
-    )
-    priority = tables.Column(
-        verbose_name=_('Priority'),
-    )
-    tags = columns.TagColumn(
-        url_name='circuits:circuitgroupassignment_list'
-    )
-
-    class Meta(NetBoxTable.Meta):
-        model = CircuitGroupAssignment
-        fields = (
-            'pk', 'id', 'group', 'circuit', 'priority', 'created', 'last_updated', 'actions', 'tags',
-        )
-        default_columns = ('pk', 'group', 'circuit', 'priority')

netbox/circuits/tests/test_api.py

@@ -206,38 +206,6 @@ class CircuitTerminationTest(APIViewTestCases.APIViewTestCase):
         }
 
 
-class CircuitGroupTest(APIViewTestCases.APIViewTestCase):
-    model = CircuitGroup
-    brief_fields = ['display', 'id', 'name', 'url']
-    bulk_update_data = {
-        'description': 'New description',
-    }
-
-    @classmethod
-    def setUpTestData(cls):
-        circuit_groups = (
-            CircuitGroup(name="Circuit Group 1", slug='circuit-group-1'),
-            CircuitGroup(name="Circuit Group 2", slug='circuit-group-2'),
-            CircuitGroup(name="Circuit Group 3", slug='circuit-group-3'),
-        )
-        CircuitGroup.objects.bulk_create(circuit_groups)
-
-        cls.create_data = [
-            {
-                'name': 'Circuit Group 4',
-                'slug': 'circuit-group-4',
-            },
-            {
-                'name': 'Circuit Group 5',
-                'slug': 'circuit-group-5',
-            },
-            {
-                'name': 'Circuit Group 6',
-                'slug': 'circuit-group-6',
-            },
-        ]
-
-
 class ProviderAccountTest(APIViewTestCases.APIViewTestCase):
     model = ProviderAccount
     brief_fields = ['account', 'description', 'display', 'id', 'name', 'url']
@@ -281,77 +249,6 @@ class ProviderAccountTest(APIViewTestCases.APIViewTestCase):
         }
 
 
-class CircuitGroupAssignmentTest(APIViewTestCases.APIViewTestCase):
-    model = CircuitGroupAssignment
-    brief_fields = ['circuit', 'display', 'group', 'id', 'priority', 'url']
-    bulk_update_data = {
-        'priority': CircuitPriorityChoices.PRIORITY_INACTIVE,
-    }
-
-    @classmethod
-    def setUpTestData(cls):
-
-        circuit_groups = (
-            CircuitGroup(name='Circuit Group 1', slug='circuit-group-1'),
-            CircuitGroup(name='Circuit Group 2', slug='circuit-group-2'),
-            CircuitGroup(name='Circuit Group 3', slug='circuit-group-3'),
-            CircuitGroup(name='Circuit Group 4', slug='circuit-group-4'),
-            CircuitGroup(name='Circuit Group 5', slug='circuit-group-5'),
-            CircuitGroup(name='Circuit Group 6', slug='circuit-group-6'),
-        )
-        CircuitGroup.objects.bulk_create(circuit_groups)
-
-        provider = Provider.objects.create(name='Provider 1', slug='provider-1')
-        circuittype = CircuitType.objects.create(name='Circuit Type 1', slug='circuit-type-1')
-
-        circuits = (
-            Circuit(cid='Circuit 1', provider=provider, type=circuittype),
-            Circuit(cid='Circuit 2', provider=provider, type=circuittype),
-            Circuit(cid='Circuit 3', provider=provider, type=circuittype),
-            Circuit(cid='Circuit 4', provider=provider, type=circuittype),
-            Circuit(cid='Circuit 5', provider=provider, type=circuittype),
-            Circuit(cid='Circuit 6', provider=provider, type=circuittype),
-        )
-        Circuit.objects.bulk_create(circuits)
-
-        assignments = (
-            CircuitGroupAssignment(
-                group=circuit_groups[0],
-                circuit=circuits[0],
-                priority=CircuitPriorityChoices.PRIORITY_PRIMARY
-            ),
-            CircuitGroupAssignment(
-                group=circuit_groups[1],
-                circuit=circuits[1],
-                priority=CircuitPriorityChoices.PRIORITY_SECONDARY
-            ),
-            CircuitGroupAssignment(
-                group=circuit_groups[2],
-                circuit=circuits[2],
-                priority=CircuitPriorityChoices.PRIORITY_TERTIARY
-            ),
-        )
-        CircuitGroupAssignment.objects.bulk_create(assignments)
-
-        cls.create_data = [
-            {
-                'group': circuit_groups[3].pk,
-                'circuit': circuits[3].pk,
-                'priority': CircuitPriorityChoices.PRIORITY_PRIMARY,
-            },
-            {
-                'group': circuit_groups[4].pk,
-                'circuit': circuits[4].pk,
-                'priority': CircuitPriorityChoices.PRIORITY_SECONDARY,
-            },
-            {
-                'group': circuit_groups[5].pk,
-                'circuit': circuits[5].pk,
-                'priority': CircuitPriorityChoices.PRIORITY_TERTIARY,
-            },
-        ]
-
-
 class ProviderNetworkTest(APIViewTestCases.APIViewTestCase):
     model = ProviderNetwork
     brief_fields = ['description', 'display', 'id', 'name', 'url']

netbox/circuits/tests/test_filtersets.py

@@ -451,122 +451,6 @@ class CircuitTerminationTestCase(TestCase, ChangeLoggedFilterSetTests):
         self.assertEqual(self.filterset(params, self.queryset).qs.count(), 7)
 
 
-class CircuitGroupTestCase(TestCase, ChangeLoggedFilterSetTests):
-    queryset = CircuitGroup.objects.all()
-    filterset = CircuitGroupFilterSet
-
-    @classmethod
-    def setUpTestData(cls):
-        tenant_groups = (
-            TenantGroup(name='Tenant group 1', slug='tenant-group-1'),
-            TenantGroup(name='Tenant group 2', slug='tenant-group-2'),
-            TenantGroup(name='Tenant group 3', slug='tenant-group-3'),
-        )
-        for tenantgroup in tenant_groups:
-            tenantgroup.save()
-
-        tenants = (
-            Tenant(name='Tenant 1', slug='tenant-1', group=tenant_groups[0]),
-            Tenant(name='Tenant 2', slug='tenant-2', group=tenant_groups[1]),
-            Tenant(name='Tenant 3', slug='tenant-3', group=tenant_groups[2]),
-        )
-        Tenant.objects.bulk_create(tenants)
-
-        CircuitGroup.objects.bulk_create((
-            CircuitGroup(name='Circuit Group 1', slug='circuit-group-1', description='foobar1', tenant=tenants[0]),
-            CircuitGroup(name='Circuit Group 2', slug='circuit-group-2', description='foobar2', tenant=tenants[1]),
-            CircuitGroup(name='Circuit Group 3', slug='circuit-group-3', tenant=tenants[1]),
-        ))
-
-    def test_q(self):
-        params = {'q': 'foobar1'}
-        self.assertEqual(self.filterset(params, self.queryset).qs.count(), 1)
-
-    def test_name(self):
-        params = {'name': ['Circuit Group 1']}
-        self.assertEqual(self.filterset(params, self.queryset).qs.count(), 1)
-
-    def test_slug(self):
-        params = {'slug': ['circuit-group-1']}
-        self.assertEqual(self.filterset(params, self.queryset).qs.count(), 1)
-
-    def test_description(self):
-        params = {'description': ['foobar1', 'foobar2']}
-        self.assertEqual(self.filterset(params, self.queryset).qs.count(), 2)
-
-    def test_tenant(self):
-        tenants = Tenant.objects.all()[:2]
-        params = {'tenant_id': [tenants[0].pk, tenants[1].pk]}
-        self.assertEqual(self.filterset(params, self.queryset).qs.count(), 3)
-        params = {'tenant': [tenants[0].slug, tenants[1].slug]}
-        self.assertEqual(self.filterset(params, self.queryset).qs.count(), 3)
-
-    def test_tenant_group(self):
-        tenant_groups = TenantGroup.objects.all()[:2]
-        params = {'tenant_group_id': [tenant_groups[0].pk, tenant_groups[1].pk]}
-        self.assertEqual(self.filterset(params, self.queryset).qs.count(), 3)
-        params = {'tenant_group': [tenant_groups[0].slug, tenant_groups[1].slug]}
-        self.assertEqual(self.filterset(params, self.queryset).qs.count(), 3)
-
-
-class CircuitGroupAssignmentTestCase(TestCase, ChangeLoggedFilterSetTests):
-    queryset = CircuitGroupAssignment.objects.all()
-    filterset = CircuitGroupAssignmentFilterSet
-
-    @classmethod
-    def setUpTestData(cls):
-
-        circuit_groups = (
-            CircuitGroup(name='Circuit Group 1', slug='circuit-group-1'),
-            CircuitGroup(name='Circuit Group 2', slug='circuit-group-2'),
-            CircuitGroup(name='Circuit Group 3', slug='circuit-group-3'),
-            CircuitGroup(name='Circuit Group 4', slug='circuit-group-4'),
-        )
-        CircuitGroup.objects.bulk_create(circuit_groups)
-
-        provider = Provider.objects.create(name='Provider 1', slug='provider-1')
-        circuittype = CircuitType.objects.create(name='Circuit Type 1', slug='circuit-type-1')
-
-        circuits = (
-            Circuit(cid='Circuit 1', provider=provider, type=circuittype),
-            Circuit(cid='Circuit 2', provider=provider, type=circuittype),
-            Circuit(cid='Circuit 3', provider=provider, type=circuittype),
-            Circuit(cid='Circuit 4', provider=provider, type=circuittype),
-        )
-        Circuit.objects.bulk_create(circuits)
-
-        assignments = (
-            CircuitGroupAssignment(
-                group=circuit_groups[0],
-                circuit=circuits[0],
-                priority=CircuitPriorityChoices.PRIORITY_PRIMARY
-            ),
-            CircuitGroupAssignment(
-                group=circuit_groups[1],
-                circuit=circuits[1],
-                priority=CircuitPriorityChoices.PRIORITY_SECONDARY
-            ),
-            CircuitGroupAssignment(
-                group=circuit_groups[2],
-                circuit=circuits[2],
-                priority=CircuitPriorityChoices.PRIORITY_TERTIARY
-            ),
-        )
-        CircuitGroupAssignment.objects.bulk_create(assignments)
-
-    def test_group_id(self):
-        groups = CircuitGroup.objects.filter(name__in=['Circuit Group 1', 'Circuit Group 2'])
-        params = {'group_id': [groups[0].pk, groups[1].pk]}
-        self.assertEqual(self.filterset(params, self.queryset).qs.count(), 2)
-        params = {'group': [groups[0].slug, groups[1].slug]}
-        self.assertEqual(self.filterset(params, self.queryset).qs.count(), 2)
-
-    def test_circuit_id(self):
-        circuits = Circuit.objects.filter(cid__in=['Circuit 1', 'Circuit 2'])
-        params = {'circuit_id': [circuits[0].pk, circuits[1].pk]}
-        self.assertEqual(self.filterset(params, self.queryset).qs.count(), 2)
-
-
 class ProviderNetworkTestCase(TestCase, ChangeLoggedFilterSetTests):
     queryset = ProviderNetwork.objects.all()
     filterset = ProviderNetworkFilterSet

netbox/circuits/tests/test_views.py

@@ -404,109 +404,3 @@ class CircuitTerminationTestCase(ViewTestCases.PrimaryObjectViewTestCase):
 
         response = self.client.get(reverse('circuits:circuittermination_trace', kwargs={'pk': circuittermination.pk}))
         self.assertHttpStatus(response, 200)
-
-
-class CircuitGroupTestCase(ViewTestCases.OrganizationalObjectViewTestCase):
-    model = CircuitGroup
-
-    @classmethod
-    def setUpTestData(cls):
-
-        circuit_groups = (
-            CircuitGroup(name='Circuit Group 1', slug='circuit-group-1'),
-            CircuitGroup(name='Circuit Group 2', slug='circuit-group-2'),
-            CircuitGroup(name='Circuit Group 3', slug='circuit-group-3'),
-        )
-        CircuitGroup.objects.bulk_create(circuit_groups)
-
-        tags = create_tags('Alpha', 'Bravo', 'Charlie')
-
-        cls.form_data = {
-            'name': 'Circuit Group X',
-            'slug': 'circuit-group-x',
-            'description': 'A new Circuit Group',
-            'tags': [t.pk for t in tags],
-        }
-
-        cls.csv_data = (
-            "name,slug",
-            "Circuit Group 4,circuit-group-4",
-            "Circuit Group 5,circuit-group-5",
-            "Circuit Group 6,circuit-group-6",
-        )
-
-        cls.csv_update_data = (
-            "id,name,description",
-            f"{circuit_groups[0].pk},Circuit Group 7,New description7",
-            f"{circuit_groups[1].pk},Circuit Group 8,New description8",
-            f"{circuit_groups[2].pk},Circuit Group 9,New description9",
-        )
-
-        cls.bulk_edit_data = {
-            'description': 'Foo',
-        }
-
-
-class CircuitGroupAssignmentTestCase(
-    ViewTestCases.CreateObjectViewTestCase,
-    ViewTestCases.EditObjectViewTestCase,
-    ViewTestCases.DeleteObjectViewTestCase,
-    ViewTestCases.ListObjectsViewTestCase,
-    ViewTestCases.BulkEditObjectsViewTestCase,
-    ViewTestCases.BulkDeleteObjectsViewTestCase
-):
-    model = CircuitGroupAssignment
-
-    @classmethod
-    def setUpTestData(cls):
-
-        circuit_groups = (
-            CircuitGroup(name='Circuit Group 1', slug='circuit-group-1'),
-            CircuitGroup(name='Circuit Group 2', slug='circuit-group-2'),
-            CircuitGroup(name='Circuit Group 3', slug='circuit-group-3'),
-            CircuitGroup(name='Circuit Group 4', slug='circuit-group-4'),
-        )
-        CircuitGroup.objects.bulk_create(circuit_groups)
-
-        provider = Provider.objects.create(name='Provider 1', slug='provider-1')
-        circuittype = CircuitType.objects.create(name='Circuit Type 1', slug='circuit-type-1')
-
-        circuits = (
-            Circuit(cid='Circuit 1', provider=provider, type=circuittype),
-            Circuit(cid='Circuit 2', provider=provider, type=circuittype),
-            Circuit(cid='Circuit 3', provider=provider, type=circuittype),
-            Circuit(cid='Circuit 4', provider=provider, type=circuittype),
-        )
-        Circuit.objects.bulk_create(circuits)
-
-        assignments = (
-            CircuitGroupAssignment(
-                group=circuit_groups[0],
-                circuit=circuits[0],
-                priority=CircuitPriorityChoices.PRIORITY_PRIMARY
-            ),
-            CircuitGroupAssignment(
-                group=circuit_groups[1],
-                circuit=circuits[1],
-                priority=CircuitPriorityChoices.PRIORITY_SECONDARY
-            ),
-            CircuitGroupAssignment(
-                group=circuit_groups[2],
-                circuit=circuits[2],
-                priority=CircuitPriorityChoices.PRIORITY_TERTIARY
-            ),
-        )
-        CircuitGroupAssignment.objects.bulk_create(assignments)
-
-        tags = create_tags('Alpha', 'Bravo', 'Charlie')
-
-        cls.form_data = {
-            'group': circuit_groups[3].pk,
-            'circuit': circuits[3].pk,
-            'priority': CircuitPriorityChoices.PRIORITY_INACTIVE,
-            'tags': [t.pk for t in tags],
-        }
-
-        cls.bulk_edit_data = {
-            'priority': CircuitPriorityChoices.PRIORITY_INACTIVE,
-        }

netbox/circuits/urls.py

@@ -55,19 +55,4 @@ urlpatterns = [
     path('circuit-terminations/delete/', views.CircuitTerminationBulkDeleteView.as_view(), name='circuittermination_bulk_delete'),
     path('circuit-terminations/<int:pk>/', include(get_model_urls('circuits', 'circuittermination'))),
 
-    # Circuit Groups
-    path('circuit-groups/', views.CircuitGroupListView.as_view(), name='circuitgroup_list'),
-    path('circuit-groups/add/', views.CircuitGroupEditView.as_view(), name='circuitgroup_add'),
-    path('circuit-groups/import/', views.CircuitGroupBulkImportView.as_view(), name='circuitgroup_import'),
-    path('circuit-groups/edit/', views.CircuitGroupBulkEditView.as_view(), name='circuitgroup_bulk_edit'),
-    path('circuit-groups/delete/', views.CircuitGroupBulkDeleteView.as_view(), name='circuitgroup_bulk_delete'),
-    path('circuit-groups/<int:pk>/', include(get_model_urls('circuits', 'circuitgroup'))),
-
-    # Circuit Group Assignments
-    path('circuit-group-assignments/', views.CircuitGroupAssignmentListView.as_view(), name='circuitgroupassignment_list'),
-    path('circuit-group-assignments/add/', views.CircuitGroupAssignmentEditView.as_view(), name='circuitgroupassignment_add'),
-    path('circuit-group-assignments/import/', views.CircuitGroupAssignmentBulkImportView.as_view(), name='circuitgroupassignment_import'),
-    path('circuit-group-assignments/edit/', views.CircuitGroupAssignmentBulkEditView.as_view(), name='circuitgroupassignment_bulk_edit'),
-    path('circuit-group-assignments/delete/', views.CircuitGroupAssignmentBulkDeleteView.as_view(), name='circuitgroupassignment_bulk_delete'),
-    path('circuit-group-assignments/<int:pk>/', include(get_model_urls('circuits', 'circuitgroupassignment'))),
 ]

netbox/circuits/views.py

@@ -1,6 +1,7 @@
 from django.contrib import messages
 from django.db import transaction
 from django.shortcuts import get_object_or_404, redirect, render
+from django.utils.translation import gettext_lazy as _
 
 from dcim.views import PathTraceView
 from netbox.views import generic
@@ -326,7 +327,9 @@ class CircuitSwapTerminations(generic.ObjectEditView):
 
         # Circuit must have at least one termination to swap
         if not circuit.termination_a and not circuit.termination_z:
-            messages.error(request, "No terminations have been defined for circuit {}.".format(circuit))
+            messages.error(request, _(
+                "No terminations have been defined for circuit {circuit}."
+            ).format(circuit=circuit))
             return redirect('circuits:circuit', pk=circuit.pk)
 
         return render(request, 'circuits/circuit_terminations_swap.html', {
@@ -374,7 +377,7 @@ class CircuitSwapTerminations(generic.ObjectEditView):
                 circuit.termination_z = None
                 circuit.save()
 
-            messages.success(request, f"Swapped terminations for circuit {circuit}.")
+            messages.success(request, _("Swapped terminations for circuit {circuit}.").format(circuit=circuit))
             return redirect('circuits:circuit', pk=circuit.pk)
 
         return render(request, 'circuits/circuit_terminations_swap.html', {
@@ -440,100 +443,3 @@ class CircuitTerminationBulkDeleteView(generic.BulkDeleteView):
 
 # Trace view
 register_model_view(CircuitTermination, 'trace', kwargs={'model': CircuitTermination})(PathTraceView)
-
-
-#
-# Circuit Groups
-#
-
-class CircuitGroupListView(generic.ObjectListView):
-    queryset = CircuitGroup.objects.annotate(
-        circuit_group_assignment_count=count_related(CircuitGroupAssignment, 'group')
-    )
-    filterset = filtersets.CircuitGroupFilterSet
-    filterset_form = forms.CircuitGroupFilterForm
-    table = tables.CircuitGroupTable
-
-
-@register_model_view(CircuitGroup)
-class CircuitGroupView(GetRelatedModelsMixin, generic.ObjectView):
-    queryset = CircuitGroup.objects.all()
-
-    def get_extra_context(self, request, instance):
-        return {
-            'related_models': self.get_related_models(request, instance),
-        }
-
-
-@register_model_view(CircuitGroup, 'edit')
-class CircuitGroupEditView(generic.ObjectEditView):
-    queryset = CircuitGroup.objects.all()
-    form = forms.CircuitGroupForm
-
-
-@register_model_view(CircuitGroup, 'delete')
-class CircuitGroupDeleteView(generic.ObjectDeleteView):
-    queryset = CircuitGroup.objects.all()
-
-
-class CircuitGroupBulkImportView(generic.BulkImportView):
-    queryset = CircuitGroup.objects.all()
-    model_form = forms.CircuitGroupImportForm
-
-
-class CircuitGroupBulkEditView(generic.BulkEditView):
-    queryset = CircuitGroup.objects.all()
-    filterset = filtersets.CircuitGroupFilterSet
-    table = tables.CircuitGroupTable
-    form = forms.CircuitGroupBulkEditForm
-
-
-class CircuitGroupBulkDeleteView(generic.BulkDeleteView):
-    queryset = CircuitGroup.objects.all()
-    filterset = filtersets.CircuitGroupFilterSet
-    table = tables.CircuitGroupTable
-
-
-#
-# Circuit Groups
-#
-
-class CircuitGroupAssignmentListView(generic.ObjectListView):
-    queryset = CircuitGroupAssignment.objects.all()
-    filterset = filtersets.CircuitGroupAssignmentFilterSet
-    filterset_form = forms.CircuitGroupAssignmentFilterForm
-    table = tables.CircuitGroupAssignmentTable
-
-
-@register_model_view(CircuitGroupAssignment)
-class CircuitGroupAssignmentView(generic.ObjectView):
-    queryset = CircuitGroupAssignment.objects.all()
-
-
-@register_model_view(CircuitGroupAssignment, 'edit')
-class CircuitGroupAssignmentEditView(generic.ObjectEditView):
-    queryset = CircuitGroupAssignment.objects.all()
-    form = forms.CircuitGroupAssignmentForm
-
-
-@register_model_view(CircuitGroupAssignment, 'delete')
-class CircuitGroupAssignmentDeleteView(generic.ObjectDeleteView):
-    queryset = CircuitGroupAssignment.objects.all()
-
-
-class CircuitGroupAssignmentBulkImportView(generic.BulkImportView):
-    queryset = CircuitGroupAssignment.objects.all()
-    model_form = forms.CircuitGroupAssignmentImportForm
-
-
-class CircuitGroupAssignmentBulkEditView(generic.BulkEditView):
-    queryset = CircuitGroupAssignment.objects.all()
-    filterset = filtersets.CircuitGroupAssignmentFilterSet
-    table = tables.CircuitGroupAssignmentTable
-    form = forms.CircuitGroupAssignmentBulkEditForm
-
-
-class CircuitGroupAssignmentBulkDeleteView(generic.BulkDeleteView):
-    queryset = CircuitGroupAssignment.objects.all()
-    filterset = filtersets.CircuitGroupAssignmentFilterSet
-    table = tables.CircuitGroupAssignmentTable

netbox/core/api/nested_serializers.py

@@ -14,20 +14,23 @@ __all__ = (
 
 
 class NestedDataSourceSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='core-api:datasource-detail')
 
     class Meta:
         model = DataSource
-        fields = ['id', 'url', 'display_url', 'display', 'name']
+        fields = ['id', 'url', 'display', 'name']
 
 
 class NestedDataFileSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='core-api:datafile-detail')
 
     class Meta:
         model = DataFile
-        fields = ['id', 'url', 'display_url', 'display', 'path']
+        fields = ['id', 'url', 'display', 'path']
 
 
 class NestedJobSerializer(serializers.ModelSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='core-api:job-detail')
     status = ChoiceField(choices=JobStatusChoices)
     user = UserSerializer(
         nested=True,
@@ -36,4 +39,4 @@ class NestedJobSerializer(serializers.ModelSerializer):
 
     class Meta:
         model = Job
-        fields = ['url', 'display_url', 'created', 'completed', 'user', 'status']
+        fields = ['url', 'created', 'completed', 'user', 'status']

netbox/core/api/serializers.py

@@ -1,4 +1,3 @@
-from .serializers_.change_logging import *
 from .serializers_.data import *
 from .serializers_.jobs import *
 from .nested_serializers import *

netbox/core/api/serializers_/data.py

@@ -13,6 +13,9 @@ __all__ = (
 
 
 class DataSourceSerializer(NetBoxModelSerializer):
+    url = serializers.HyperlinkedIdentityField(
+        view_name='core-api:datasource-detail'
+    )
     type = ChoiceField(
         choices=get_data_backend_choices()
     )
@@ -27,13 +30,16 @@ class DataSourceSerializer(NetBoxModelSerializer):
     class Meta:
         model = DataSource
         fields = [
-            'id', 'url', 'display_url', 'display', 'name', 'type', 'source_url', 'enabled', 'status', 'description', 'comments',
+            'id', 'url', 'display', 'name', 'type', 'source_url', 'enabled', 'status', 'description', 'comments',
             'parameters', 'ignore_rules', 'custom_fields', 'created', 'last_updated', 'file_count',
         ]
         brief_fields = ('id', 'url', 'display', 'name', 'description')
 
 
 class DataFileSerializer(NetBoxModelSerializer):
+    url = serializers.HyperlinkedIdentityField(
+        view_name='core-api:datafile-detail'
+    )
     source = DataSourceSerializer(
         nested=True,
         read_only=True
@@ -42,6 +48,6 @@ class DataFileSerializer(NetBoxModelSerializer):
     class Meta:
         model = DataFile
         fields = [
-            'id', 'url', 'display_url', 'display', 'source', 'path', 'last_updated', 'size', 'hash',
+            'id', 'url', 'display', 'source', 'path', 'last_updated', 'size', 'hash',
         ]
         brief_fields = ('id', 'url', 'display', 'path')

netbox/core/api/serializers_/jobs.py

@@ -12,6 +12,7 @@ __all__ = (
 
 
 class JobSerializer(BaseModelSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='core-api:job-detail')
     user = UserSerializer(
         nested=True,
         read_only=True
@@ -24,7 +25,7 @@ class JobSerializer(BaseModelSerializer):
     class Meta:
         model = Job
         fields = [
-            'id', 'url', 'display_url', 'display', 'object_type', 'object_id', 'name', 'status', 'created', 'scheduled', 'interval',
+            'id', 'url', 'display', 'object_type', 'object_id', 'name', 'status', 'created', 'scheduled', 'interval',
             'started', 'completed', 'user', 'data', 'error', 'job_id',
         ]
         brief_fields = ('url', 'created', 'completed', 'user', 'status')

netbox/core/api/urls.py

@@ -5,10 +5,12 @@ from . import views
 router = NetBoxRouter()
 router.APIRootView = views.CoreRootView
 
+# Data sources
 router.register('data-sources', views.DataSourceViewSet)
 router.register('data-files', views.DataFileViewSet)
+
+# Jobs
 router.register('jobs', views.JobViewSet)
-router.register('object-changes', views.ObjectChangeViewSet)
 
 app_name = 'core-api'
 urlpatterns = router.urls

netbox/core/api/views.py

@@ -7,10 +7,7 @@ from rest_framework.routers import APIRootView
 from rest_framework.viewsets import ReadOnlyModelViewSet
 
 from core import filtersets
-from core.choices import DataSourceStatusChoices
-from core.jobs import SyncDataSourceJob
 from core.models import *
-from netbox.api.metadata import ContentTypeMetadata
 from netbox.api.viewsets import NetBoxModelViewSet, NetBoxReadOnlyModelViewSet
 from . import serializers
 
@@ -38,11 +35,7 @@ class DataSourceViewSet(NetBoxModelViewSet):
         if not request.user.has_perm('core.sync_datasource', obj=datasource):
             raise PermissionDenied(_("This user does not have permission to synchronize this data source."))
 
-        # Enqueue the sync job & update the DataSource's status
-        SyncDataSourceJob.enqueue(instance=datasource, user=request.user)
-        datasource.status = DataSourceStatusChoices.QUEUED
-        DataSource.objects.filter(pk=datasource.pk).update(status=datasource.status)
-
+        datasource.enqueue_sync_job(request)
         serializer = serializers.DataSourceSerializer(datasource, context={'request': request})
 
         return Response(serializer.data)
@@ -61,13 +54,3 @@ class JobViewSet(ReadOnlyModelViewSet):
     queryset = Job.objects.all()
     serializer_class = serializers.JobSerializer
     filterset_class = filtersets.JobFilterSet
-
-
-class ObjectChangeViewSet(ReadOnlyModelViewSet):
-    """
-    Retrieve a list of recent changes.
-    """
-    metadata_class = ContentTypeMetadata
-    queryset = ObjectChange.objects.valid_models()
-    serializer_class = serializers.ObjectChangeSerializer
-    filterset_class = filtersets.ObjectChangeFilterSet

netbox/core/apps.py

@@ -18,7 +18,7 @@ class CoreConfig(AppConfig):
     def ready(self):
         from core.api import schema  # noqa
         from netbox.models.features import register_models
-        from . import data_backends, events, search
+        from . import data_backends, search
 
         # Register models
         register_models(*self.get_models())

netbox/core/choices.py

@@ -59,31 +59,8 @@ class JobStatusChoices(ChoiceSet):
         (STATUS_FAILED, _('Failed'), 'red'),
     )
 
-    ENQUEUED_STATE_CHOICES = (
-        STATUS_PENDING,
-        STATUS_SCHEDULED,
-        STATUS_RUNNING,
-    )
-
     TERMINAL_STATE_CHOICES = (
         STATUS_COMPLETED,
         STATUS_ERRORED,
         STATUS_FAILED,
     )
-
-
-#
-# ObjectChanges
-#
-
-class ObjectChangeActionChoices(ChoiceSet):
-
-    ACTION_CREATE = 'create'
-    ACTION_UPDATE = 'update'
-    ACTION_DELETE = 'delete'
-
-    CHOICES = (
-        (ACTION_CREATE, _('Created'), 'green'),
-        (ACTION_UPDATE, _('Updated'), 'blue'),
-        (ACTION_DELETE, _('Deleted'), 'red'),
-    )

netbox/core/events.py

@@ -1,33 +0,0 @@
-from django.utils.translation import gettext as _
-
-from netbox.events import EventType, EVENT_TYPE_KIND_DANGER, EVENT_TYPE_KIND_SUCCESS, EVENT_TYPE_KIND_WARNING
-
-__all__ = (
-    'JOB_COMPLETED',
-    'JOB_ERRORED',
-    'JOB_FAILED',
-    'JOB_STARTED',
-    'OBJECT_CREATED',
-    'OBJECT_DELETED',
-    'OBJECT_UPDATED',
-)
-
-# Object events
-OBJECT_CREATED = 'object_created'
-OBJECT_UPDATED = 'object_updated'
-OBJECT_DELETED = 'object_deleted'
-
-# Job events
-JOB_STARTED = 'job_started'
-JOB_COMPLETED = 'job_completed'
-JOB_FAILED = 'job_failed'
-JOB_ERRORED = 'job_errored'
-
-# Register core events
-EventType(OBJECT_CREATED, _('Object created')).register()
-EventType(OBJECT_UPDATED, _('Object updated')).register()
-EventType(OBJECT_DELETED, _('Object deleted'), destructive=True).register()
-EventType(JOB_STARTED, _('Job started')).register()
-EventType(JOB_COMPLETED, _('Job completed'), kind=EVENT_TYPE_KIND_SUCCESS).register()
-EventType(JOB_FAILED, _('Job failed'), kind=EVENT_TYPE_KIND_WARNING).register()
-EventType(JOB_ERRORED, _('Job errored'), kind=EVENT_TYPE_KIND_DANGER).register()

netbox/core/filtersets.py

@@ -1,4 +1,3 @@
-from django.contrib.contenttypes.models import ContentType
 from django.db.models import Q
 from django.utils.translation import gettext as _
 
@@ -6,8 +5,6 @@ import django_filters
 
 from netbox.filtersets import BaseFilterSet, ChangeLoggedModelFilterSet, NetBoxModelFilterSet
 from netbox.utils import get_data_backend_choices
-from users.models import User
-from utilities.filters import ContentTypeFilter
 from .choices import *
 from .models import *
 
@@ -16,7 +13,6 @@ __all__ = (
     'DataFileFilterSet',
     'DataSourceFilterSet',
     'JobFilterSet',
-    'ObjectChangeFilterSet',
 )
 
 
@@ -130,43 +126,6 @@ class JobFilterSet(BaseFilterSet):
         )
 
 
-class ObjectChangeFilterSet(BaseFilterSet):
-    q = django_filters.CharFilter(
-        method='search',
-        label=_('Search'),
-    )
-    time = django_filters.DateTimeFromToRangeFilter()
-    changed_object_type = ContentTypeFilter()
-    changed_object_type_id = django_filters.ModelMultipleChoiceFilter(
-        queryset=ContentType.objects.all()
-    )
-    user_id = django_filters.ModelMultipleChoiceFilter(
-        queryset=User.objects.all(),
-        label=_('User (ID)'),
-    )
-    user = django_filters.ModelMultipleChoiceFilter(
-        field_name='user__username',
-        queryset=User.objects.all(),
-        to_field_name='username',
-        label=_('User name'),
-    )
-
-    class Meta:
-        model = ObjectChange
-        fields = (
-            'id', 'user', 'user_name', 'request_id', 'action', 'changed_object_type_id', 'changed_object_id',
-            'related_object_type', 'related_object_id', 'object_repr',
-        )
-
-    def search(self, queryset, name, value):
-        if not value.strip():
-            return queryset
-        return queryset.filter(
-            Q(user_name__icontains=value) |
-            Q(object_repr__icontains=value)
-        )
-
-
 class ConfigRevisionFilterSet(BaseFilterSet):
     q = django_filters.CharFilter(
         method='search',

netbox/core/forms/filtersets.py

@@ -1,4 +1,5 @@
 from django import forms
+from django.contrib.auth import get_user_model
 from django.utils.translation import gettext_lazy as _
 
 from core.choices import *
@@ -6,11 +7,8 @@ from core.models import *
 from netbox.forms import NetBoxModelFilterSetForm
 from netbox.forms.mixins import SavedFiltersMixin
 from netbox.utils import get_data_backend_choices
-from users.models import User
-from utilities.forms import BOOLEAN_WITH_BLANK_CHOICES, FilterForm, add_blank_choice
-from utilities.forms.fields import (
-    ContentTypeChoiceField, ContentTypeMultipleChoiceField, DynamicModelMultipleChoiceField,
-)
+from utilities.forms import BOOLEAN_WITH_BLANK_CHOICES, FilterForm
+from utilities.forms.fields import ContentTypeChoiceField, DynamicModelMultipleChoiceField
 from utilities.forms.rendering import FieldSet
 from utilities.forms.widgets import DateTimePicker
 
@@ -19,7 +17,6 @@ __all__ = (
     'DataFileFilterForm',
     'DataSourceFilterForm',
     'JobFilterForm',
-    'ObjectChangeFilterForm',
 )
 
 
@@ -121,46 +118,12 @@ class JobFilterForm(SavedFiltersMixin, FilterForm):
         widget=DateTimePicker()
     )
     user = DynamicModelMultipleChoiceField(
-        queryset=User.objects.all(),
+        queryset=get_user_model().objects.all(),
         required=False,
         label=_('User')
     )
 
 
-class ObjectChangeFilterForm(SavedFiltersMixin, FilterForm):
-    model = ObjectChange
-    fieldsets = (
-        FieldSet('q', 'filter_id'),
-        FieldSet('time_before', 'time_after', name=_('Time')),
-        FieldSet('action', 'user_id', 'changed_object_type_id', name=_('Attributes')),
-    )
-    time_after = forms.DateTimeField(
-        required=False,
-        label=_('After'),
-        widget=DateTimePicker()
-    )
-    time_before = forms.DateTimeField(
-        required=False,
-        label=_('Before'),
-        widget=DateTimePicker()
-    )
-    action = forms.ChoiceField(
-        label=_('Action'),
-        choices=add_blank_choice(ObjectChangeActionChoices),
-        required=False
-    )
-    user_id = DynamicModelMultipleChoiceField(
-        queryset=User.objects.all(),
-        required=False,
-        label=_('User')
-    )
-    changed_object_type_id = ContentTypeMultipleChoiceField(
-        queryset=ObjectType.objects.with_feature('change_logging'),
-        required=False,
-        label=_('Object Type'),
-    )
-
-
 class ConfigRevisionFilterForm(SavedFiltersMixin, FilterForm):
     fieldsets = (
         FieldSet('q', 'filter_id'),

netbox/core/graphql/filters.py

@@ -6,7 +6,6 @@ from netbox.graphql.filter_mixins import autotype_decorator, BaseFilterMixin
 __all__ = (
     'DataFileFilter',
     'DataSourceFilter',
-    'ObjectChangeFilter',
 )
 
 
@@ -20,9 +19,3 @@ class DataFileFilter(BaseFilterMixin):
 @autotype_decorator(filtersets.DataSourceFilterSet)
 class DataSourceFilter(BaseFilterMixin):
     pass
-
-
-@strawberry_django.filter(models.ObjectChange, lookups=True)
-@autotype_decorator(filtersets.ObjectChangeFilterSet)
-class ObjectChangeFilter(BaseFilterMixin):
-    pass

netbox/core/graphql/mixins.py

@@ -1,24 +0,0 @@
-from typing import Annotated, List
-
-import strawberry
-import strawberry_django
-from django.contrib.contenttypes.models import ContentType
-
-from core.models import ObjectChange
-
-__all__ = (
-    'ChangelogMixin',
-)
-
-
-@strawberry.type
-class ChangelogMixin:
-
-    @strawberry_django.field
-    def changelog(self, info) -> List[Annotated["ObjectChangeType", strawberry.lazy('.types')]]:
-        content_type = ContentType.objects.get_for_model(self)
-        object_changes = ObjectChange.objects.filter(
-            changed_object_type=content_type,
-            changed_object_id=self.pk
-        )
-        return object_changes.restrict(info.context.request.user, 'view')

netbox/core/graphql/types.py

@@ -10,7 +10,6 @@ from .filters import *
 __all__ = (
     'DataFileType',
     'DataSourceType',
-    'ObjectChangeType',
 )
 
 
@@ -31,12 +30,3 @@ class DataFileType(BaseObjectType):
 class DataSourceType(NetBoxObjectType):
 
     datafiles: List[Annotated["DataFileType", strawberry.lazy('core.graphql.types')]]
-
-
-@strawberry_django.type(
-    models.ObjectChange,
-    fields='__all__',
-    filters=ObjectChangeFilter
-)
-class ObjectChangeType(BaseObjectType):
-    pass

netbox/core/jobs.py

@@ -1,33 +1,33 @@
 import logging
 
-from netbox.jobs import JobRunner
 from netbox.search.backends import search_backend
-from .choices import DataSourceStatusChoices
+from .choices import *
 from .exceptions import SyncError
 from .models import DataSource
+from rq.timeouts import JobTimeoutException
 
 logger = logging.getLogger(__name__)
 
 
-class SyncDataSourceJob(JobRunner):
+def sync_datasource(job, *args, **kwargs):
     """
     Call sync() on a DataSource.
     """
+    datasource = DataSource.objects.get(pk=job.object_id)
 
-    class Meta:
-        name = 'Synchronization'
+    try:
+        job.start()
+        datasource.sync()
 
-    def run(self, *args, **kwargs):
-        datasource = DataSource.objects.get(pk=self.job.object_id)
+        # Update the search cache for DataFiles belonging to this source
+        search_backend.cache(datasource.datafiles.iterator())
 
-        try:
-            datasource.sync()
+        job.terminate()
 
-            # Update the search cache for DataFiles belonging to this source
-            search_backend.cache(datasource.datafiles.iterator())
-
-        except Exception as e:
-            DataSource.objects.filter(pk=datasource.pk).update(status=DataSourceStatusChoices.FAILED)
-            if type(e) is SyncError:
-                logging.error(e)
+    except Exception as e:
+        job.terminate(status=JobStatusChoices.STATUS_ERRORED, error=repr(e))
+        DataSource.objects.filter(pk=datasource.pk).update(status=DataSourceStatusChoices.FAILED)
+        if type(e) in (SyncError, JobTimeoutException):
+            logging.error(e)
+        else:
             raise e

netbox/core/management/commands/nbshell.py

@@ -5,10 +5,10 @@ import sys
 from django import get_version
 from django.apps import apps
 from django.conf import settings
+from django.contrib.auth import get_user_model
 from django.core.management.base import BaseCommand
 
 from core.models import ObjectType
-from users.models import User
 
 APPS = ('circuits', 'core', 'dcim', 'extras', 'ipam', 'tenancy', 'users', 'virtualization', 'vpn', 'wireless')
 
@@ -18,7 +18,7 @@ BANNER_TEXT = """### NetBox interactive shell ({node})
     node=platform.node(),
     python=platform.python_version(),
     django=get_version(),
-    netbox=settings.RELEASE.name
+    netbox=settings.VERSION
 )
 
 
@@ -61,7 +61,7 @@ class Command(BaseCommand):
 
         # Additional objects to include
         namespace['ObjectType'] = ObjectType
-        namespace['User'] = User
+        namespace['User'] = get_user_model()
 
         # Load convenience commands
         namespace.update({

netbox/core/migrations/0011_move_objectchange.py

@@ -1,45 +0,0 @@
-import django.db.models.deletion
-from django.conf import settings
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('contenttypes', '0002_remove_content_type_name'),
-        ('core', '0010_gfk_indexes'),
-        migrations.swappable_dependency(settings.AUTH_USER_MODEL),
-    ]
-
-    operations = [
-        migrations.SeparateDatabaseAndState(
-            state_operations=[
-                migrations.CreateModel(
-                    name='ObjectChange',
-                    fields=[
-                        ('id', models.BigAutoField(auto_created=True, primary_key=True, serialize=False)),
-                        ('time', models.DateTimeField(auto_now_add=True, db_index=True)),
-                        ('user_name', models.CharField(editable=False, max_length=150)),
-                        ('request_id', models.UUIDField(db_index=True, editable=False)),
-                        ('action', models.CharField(max_length=50)),
-                        ('changed_object_id', models.PositiveBigIntegerField()),
-                        ('related_object_id', models.PositiveBigIntegerField(blank=True, null=True)),
-                        ('object_repr', models.CharField(editable=False, max_length=200)),
-                        ('prechange_data', models.JSONField(blank=True, editable=False, null=True)),
-                        ('postchange_data', models.JSONField(blank=True, editable=False, null=True)),
-                        ('changed_object_type', models.ForeignKey(on_delete=django.db.models.deletion.PROTECT, related_name='+', to='contenttypes.contenttype')),
-                        ('related_object_type', models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.PROTECT, related_name='+', to='contenttypes.contenttype')),
-                        ('user', models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.SET_NULL, related_name='changes', to=settings.AUTH_USER_MODEL)),
-                    ],
-                    options={
-                        'verbose_name': 'object change',
-                        'verbose_name_plural': 'object changes',
-                        'ordering': ['-time'],
-                        'indexes': [models.Index(fields=['changed_object_type', 'changed_object_id'], name='core_object_changed_c227ce_idx'), models.Index(fields=['related_object_type', 'related_object_id'], name='core_object_related_3375d6_idx')],
-                    },
-                ),
-            ],
-            # Table has been renamed from 'extras' app
-            database_operations=[],
-        ),
-    ]

netbox/core/migrations/0012_job_object_type_optional.py

@@ -1,24 +0,0 @@
-import django.db.models.deletion
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('contenttypes', '0002_remove_content_type_name'),
-        ('core', '0011_move_objectchange'),
-    ]
-
-    operations = [
-        migrations.AlterField(
-            model_name='job',
-            name='object_type',
-            field=models.ForeignKey(
-                blank=True,
-                null=True,
-                on_delete=django.db.models.deletion.CASCADE,
-                related_name='jobs',
-                to='contenttypes.contenttype'
-            ),
-        ),
-    ]

netbox/core/models/__init__.py

@@ -1,6 +1,5 @@
-from .contenttypes import *
-from .change_logging import *
 from .config import *
+from .contenttypes import *
 from .data import *
 from .files import *
 from .jobs import *

netbox/core/models/data.py

@@ -1,10 +1,10 @@
 import hashlib
 import logging
 import os
+import yaml
 from fnmatch import fnmatchcase
 from urllib.parse import urlparse
 
-import yaml
 from django.conf import settings
 from django.contrib.contenttypes.fields import GenericForeignKey
 from django.core.exceptions import ValidationError
@@ -12,6 +12,7 @@ from django.core.validators import RegexValidator
 from django.db import models
 from django.urls import reverse
 from django.utils import timezone
+from django.utils.module_loading import import_string
 from django.utils.translation import gettext as _
 
 from netbox.constants import CENSOR_TOKEN, CENSOR_TOKEN_CHANGED
@@ -21,6 +22,8 @@ from netbox.registry import registry
 from utilities.querysets import RestrictedQuerySet
 from ..choices import *
 from ..exceptions import SyncError
+from ..signals import post_sync, pre_sync
+from .jobs import Job
 
 __all__ = (
     'AutoSyncRecord',
@@ -150,6 +153,21 @@ class DataSource(JobsMixin, PrimaryModel):
 
         return objectchange
 
+    def enqueue_sync_job(self, request):
+        """
+        Enqueue a background job to synchronize the DataSource by calling sync().
+        """
+        # Set the status to "syncing"
+        self.status = DataSourceStatusChoices.QUEUED
+        DataSource.objects.filter(pk=self.pk).update(status=self.status)
+
+        # Enqueue a sync job
+        return Job.enqueue(
+            import_string('core.jobs.sync_datasource'),
+            instance=self,
+            user=request.user
+        )
+
     def get_backend(self):
         backend_params = self.parameters or {}
         return self.backend_class(self.source_url, **backend_params)
@@ -158,8 +176,6 @@ class DataSource(JobsMixin, PrimaryModel):
         """
         Create/update/delete child DataFiles as necessary to synchronize with the remote source.
         """
-        from core.signals import post_sync, pre_sync
-
         if self.status == DataSourceStatusChoices.SYNCING:
             raise SyncError(_("Cannot initiate sync; syncing already in progress."))
 

netbox/core/models/jobs.py

@@ -13,6 +13,7 @@ from django.utils.translation import gettext as _
 from core.choices import JobStatusChoices
 from core.models import ObjectType
 from core.signals import job_end, job_start
+from extras.constants import EVENT_JOB_END, EVENT_JOB_START
 from netbox.config import get_config
 from netbox.constants import RQ_QUEUE_DEFAULT
 from utilities.querysets import RestrictedQuerySet
@@ -31,8 +32,6 @@ class Job(models.Model):
         to='contenttypes.ContentType',
         related_name='jobs',
         on_delete=models.CASCADE,
-        blank=True,
-        null=True
     )
     object_id = models.PositiveBigIntegerField(
         blank=True,
@@ -199,34 +198,25 @@ class Job(models.Model):
         job_end.send(self)
 
     @classmethod
-    def enqueue(cls, func, instance=None, name='', user=None, schedule_at=None, interval=None, immediate=False, **kwargs):
+    def enqueue(cls, func, instance, name='', user=None, schedule_at=None, interval=None, **kwargs):
         """
         Create a Job instance and enqueue a job using the given callable
 
         Args:
             func: The callable object to be enqueued for execution
-            instance: The NetBox object to which this job pertains (optional)
+            instance: The NetBox object to which this job pertains
             name: Name for the job (optional)
             user: The user responsible for running the job
             schedule_at: Schedule the job to be executed at the passed date and time
             interval: Recurrence interval (in minutes)
-            immediate: Run the job immediately without scheduling it in the background. Should be used for interactive
-                management commands only.
         """
-        if schedule_at and immediate:
-            raise ValueError(_("enqueue() cannot be called with values for both schedule_at and immediate."))
-
-        if instance:
-            object_type = ObjectType.objects.get_for_model(instance, for_concrete_model=False)
-            object_id = instance.pk
-        else:
-            object_type = object_id = None
-        rq_queue_name = get_queue_for_model(object_type.model if object_type else None)
+        object_type = ObjectType.objects.get_for_model(instance, for_concrete_model=False)
+        rq_queue_name = get_queue_for_model(object_type.model)
         queue = django_rq.get_queue(rq_queue_name)
         status = JobStatusChoices.STATUS_SCHEDULED if schedule_at else JobStatusChoices.STATUS_PENDING
         job = Job.objects.create(
             object_type=object_type,
-            object_id=object_id,
+            object_id=instance.pk,
             name=name,
             status=status,
             scheduled=schedule_at,
@@ -235,16 +225,8 @@ class Job(models.Model):
             job_id=uuid.uuid4()
         )
 
-        # Run the job immediately, rather than enqueuing it as a background task. Note that this is a synchronous
-        # (blocking) operation, and execution will pause until the job completes.
-        if immediate:
-            func(job_id=str(job.job_id), job=job, **kwargs)
-
-        # Schedule the job to run at a specific date & time.
-        elif schedule_at:
+        if schedule_at:
             queue.enqueue_at(schedule_at, func, job_id=str(job.job_id), job=job, **kwargs)
-
-        # Schedule the job to run asynchronously at this first available opportunity.
         else:
             queue.enqueue(func, job_id=str(job.job_id), job=job, **kwargs)
 

netbox/core/plugins.py

@@ -1,214 +0,0 @@
-import datetime
-import importlib
-import importlib.util
-from dataclasses import dataclass, field
-from typing import Optional
-
-import requests
-from django.conf import settings
-from django.core.cache import cache
-
-from netbox.plugins import PluginConfig
-from utilities.datetime import datetime_from_timestamp
-
-USER_AGENT_STRING = f'NetBox/{settings.RELEASE.version} {settings.RELEASE.edition}'
-CACHE_KEY_CATALOG_FEED = 'plugins-catalog-feed'
-
-
-@dataclass
-class PluginAuthor:
-    """
-    Identifying information for the author of a plugin.
-    """
-    name: str
-    org_id: str = ''
-    url: str = ''
-
-
-@dataclass
-class PluginVersion:
-    """
-    Details for a specific versioned release of a plugin.
-    """
-    date: datetime.datetime = None
-    version: str = ''
-    netbox_min_version: str = ''
-    netbox_max_version: str = ''
-    has_model: bool = False
-    is_certified: bool = False
-    is_feature: bool = False
-    is_integration: bool = False
-    is_netboxlabs_supported: bool = False
-
-
-@dataclass
-class Plugin:
-    """
-    The representation of a NetBox plugin in the catalog API.
-    """
-    id: str = ''
-    status: str = ''
-    title_short: str = ''
-    title_long: str = ''
-    tag_line: str = ''
-    description_short: str = ''
-    slug: str = ''
-    author: Optional[PluginAuthor] = None
-    created_at: datetime.datetime = None
-    updated_at: datetime.datetime = None
-    license_type: str = ''
-    homepage_url: str = ''
-    package_name_pypi: str = ''
-    config_name: str = ''
-    is_certified: bool = False
-    release_latest: PluginVersion = field(default_factory=PluginVersion)
-    release_recent_history: list[PluginVersion] = field(default_factory=list)
-    is_local: bool = False  # extra field for locally installed plugins
-    is_installed: bool = False
-    installed_version: str = ''
-
-
-def get_local_plugins(plugins=None):
-    """
-    Return a dictionary of all locally-installed plugins, mapped by name.
-    """
-    plugins = plugins or {}
-    local_plugins = {}
-
-    # Gather all locally-installed plugins
-    for plugin_name in settings.PLUGINS:
-        plugin = importlib.import_module(plugin_name)
-        plugin_config: PluginConfig = plugin.config
-
-        local_plugins[plugin_config.name] = Plugin(
-            slug=plugin_config.name,
-            title_short=plugin_config.verbose_name,
-            tag_line=plugin_config.description,
-            description_short=plugin_config.description,
-            is_local=True,
-            is_installed=True,
-            installed_version=plugin_config.version,
-        )
-
-    # Update catalog entries for local plugins, or add them to the list if not listed
-    for k, v in local_plugins.items():
-        if k in plugins:
-            plugins[k].is_local = True
-            plugins[k].is_installed = True
-        else:
-            plugins[k] = v
-
-    return plugins
-
-
-def get_catalog_plugins():
-    """
-    Return a dictionary of all entries in the plugins catalog, mapped by name.
-    """
-    session = requests.Session()
-
-    # Disable catalog fetching for isolated deployments
-    if settings.ISOLATED_DEPLOYMENT:
-        return {}
-
-    def get_pages():
-        # TODO: pagination is currently broken in API
-        payload = {'page': '1', 'per_page': '50'}
-        first_page = session.get(
-            settings.PLUGIN_CATALOG_URL,
-            headers={'User-Agent': USER_AGENT_STRING},
-            proxies=settings.HTTP_PROXIES,
-            timeout=3,
-            params=payload
-        ).json()
-        yield first_page
-        num_pages = first_page['metadata']['pagination']['last_page']
-
-        for page in range(2, num_pages + 1):
-            payload['page'] = page
-            next_page = session.get(
-                settings.PLUGIN_CATALOG_URL,
-                headers={'User-Agent': USER_AGENT_STRING},
-                proxies=settings.HTTP_PROXIES,
-                timeout=3,
-                params=payload
-            ).json()
-            yield next_page
-
-    def make_plugin_dict():
-        plugins = {}
-
-        for page in get_pages():
-            for data in page['data']:
-
-                # Populate releases
-                releases = []
-                for version in data['release_recent_history']:
-                    releases.append(
-                        PluginVersion(
-                            date=datetime_from_timestamp(version['date']),
-                            version=version['version'],
-                            netbox_min_version=version['netbox_min_version'],
-                            netbox_max_version=version['netbox_max_version'],
-                            has_model=version['has_model'],
-                            is_certified=version['is_certified'],
-                            is_feature=version['is_feature'],
-                            is_integration=version['is_integration'],
-                            is_netboxlabs_supported=version['is_netboxlabs_supported'],
-                        )
-                    )
-                releases = sorted(releases, key=lambda x: x.date, reverse=True)
-                latest_release = PluginVersion(
-                    date=datetime_from_timestamp(data['release_latest']['date']),
-                    version=data['release_latest']['version'],
-                    netbox_min_version=data['release_latest']['netbox_min_version'],
-                    netbox_max_version=data['release_latest']['netbox_max_version'],
-                    has_model=data['release_latest']['has_model'],
-                    is_certified=data['release_latest']['is_certified'],
-                    is_feature=data['release_latest']['is_feature'],
-                    is_integration=data['release_latest']['is_integration'],
-                    is_netboxlabs_supported=data['release_latest']['is_netboxlabs_supported'],
-                )
-
-                # Populate author (if any)
-                if data['author']:
-                    author = PluginAuthor(
-                        name=data['author']['name'],
-                        org_id=data['author']['org_id'],
-                        url=data['author']['url'],
-                    )
-                else:
-                    author = None
-
-                # Populate plugin data
-                plugins[data['slug']] = Plugin(
-                    id=data['id'],
-                    status=data['status'],
-                    title_short=data['title_short'],
-                    title_long=data['title_long'],
-                    tag_line=data['tag_line'],
-                    description_short=data['description_short'],
-                    slug=data['slug'],
-                    author=author,
-                    created_at=datetime_from_timestamp(data['created_at']),
-                    updated_at=datetime_from_timestamp(data['updated_at']),
-                    license_type=data['license_type'],
-                    homepage_url=data['homepage_url'],
-                    package_name_pypi=data['package_name_pypi'],
-                    config_name=data['config_name'],
-                    is_certified=data['is_certified'],
-                    release_latest=latest_release,
-                    release_recent_history=releases,
-                )
-
-        return plugins
-
-    catalog_plugins = cache.get(CACHE_KEY_CATALOG_FEED, default={})
-    if not catalog_plugins:
-        try:
-            catalog_plugins = make_plugin_dict()
-            cache.set(CACHE_KEY_CATALOG_FEED, catalog_plugins, 3600)
-        except requests.exceptions.RequestException:
-            pass
-
-    return catalog_plugins

netbox/core/querysets.py

@@ -1,26 +0,0 @@
-from django.apps import apps
-from django.contrib.contenttypes.models import ContentType
-from django.db.utils import ProgrammingError
-
-from utilities.querysets import RestrictedQuerySet
-
-__all__ = (
-    'ObjectChangeQuerySet',
-)
-
-
-class ObjectChangeQuerySet(RestrictedQuerySet):
-
-    def valid_models(self):
-        # Exclude any change records which refer to an instance of a model that's no longer installed. This
-        # can happen when a plugin is removed but its data remains in the database, for example.
-        try:
-            content_types = ContentType.objects.get_for_models(*apps.get_models()).values()
-        except ProgrammingError:
-            # Handle the case where the database schema has not yet been initialized
-            content_types = ContentType.objects.none()
-
-        content_type_ids = set(
-            ct.pk for ct in content_types
-        )
-        return self.filter(changed_object_type_id__in=content_type_ids)

netbox/core/signals.py

@@ -1,26 +1,9 @@
-import logging
+from django.db.models.signals import post_save
+from django.dispatch import Signal, receiver
 
-from django.contrib.contenttypes.models import ContentType
-from django.core.exceptions import ValidationError
-from django.db.models.fields.reverse_related import ManyToManyRel
-from django.db.models.signals import m2m_changed, post_save, pre_delete
-from django.dispatch import receiver, Signal
-from django.utils.translation import gettext_lazy as _
-from django_prometheus.models import model_deletes, model_inserts, model_updates
-
-from core.choices import ObjectChangeActionChoices
-from core.events import *
-from core.models import ObjectChange
-from extras.events import enqueue_event
-from extras.utils import run_validators
-from netbox.config import get_config
-from netbox.context import current_request, events_queue
-from netbox.models.features import ChangeLoggingMixin
-from utilities.exceptions import AbortRequest
 from .models import ConfigRevision
 
 __all__ = (
-    'clear_events',
     'job_end',
     'job_start',
     'post_sync',
@@ -35,152 +18,6 @@ job_end = Signal()
 pre_sync = Signal()
 post_sync = Signal()
 
-# Event signals
-clear_events = Signal()
-
-
-#
-# Change logging & event handling
-#
-
-@receiver((post_save, m2m_changed))
-def handle_changed_object(sender, instance, **kwargs):
-    """
-    Fires when an object is created or updated.
-    """
-    m2m_changed = False
-
-    if not hasattr(instance, 'to_objectchange'):
-        return
-
-    # Get the current request, or bail if not set
-    request = current_request.get()
-    if request is None:
-        return
-
-    # Determine the type of change being made
-    if kwargs.get('created'):
-        event_type = OBJECT_CREATED
-    elif 'created' in kwargs:
-        event_type = OBJECT_UPDATED
-    elif kwargs.get('action') in ['post_add', 'post_remove'] and kwargs['pk_set']:
-        # m2m_changed with objects added or removed
-        m2m_changed = True
-        event_type = OBJECT_UPDATED
-    else:
-        return
-
-    # Create/update an ObjectChange record for this change
-    action = {
-        OBJECT_CREATED: ObjectChangeActionChoices.ACTION_CREATE,
-        OBJECT_UPDATED: ObjectChangeActionChoices.ACTION_UPDATE,
-        OBJECT_DELETED: ObjectChangeActionChoices.ACTION_DELETE,
-    }[event_type]
-    objectchange = instance.to_objectchange(action)
-    # If this is a many-to-many field change, check for a previous ObjectChange instance recorded
-    # for this object by this request and update it
-    if m2m_changed and (
-        prev_change := ObjectChange.objects.filter(
-            changed_object_type=ContentType.objects.get_for_model(instance),
-            changed_object_id=instance.pk,
-            request_id=request.id
-        ).first()
-    ):
-        prev_change.postchange_data = objectchange.postchange_data
-        prev_change.save()
-    elif objectchange and objectchange.has_changes:
-        objectchange.user = request.user
-        objectchange.request_id = request.id
-        objectchange.save()
-
-    # Ensure that we're working with fresh M2M assignments
-    if m2m_changed:
-        instance.refresh_from_db()
-
-    # Enqueue the object for event processing
-    queue = events_queue.get()
-    enqueue_event(queue, instance, request.user, request.id, event_type)
-    events_queue.set(queue)
-
-    # Increment metric counters
-    if event_type == OBJECT_CREATED:
-        model_inserts.labels(instance._meta.model_name).inc()
-    elif event_type == OBJECT_UPDATED:
-        model_updates.labels(instance._meta.model_name).inc()
-
-
-@receiver(pre_delete)
-def handle_deleted_object(sender, instance, **kwargs):
-    """
-    Fires when an object is deleted.
-    """
-    # Run any deletion protection rules for the object. Note that this must occur prior
-    # to queueing any events for the object being deleted, in case a validation error is
-    # raised, causing the deletion to fail.
-    model_name = f'{sender._meta.app_label}.{sender._meta.model_name}'
-    validators = get_config().PROTECTION_RULES.get(model_name, [])
-    try:
-        run_validators(instance, validators)
-    except ValidationError as e:
-        raise AbortRequest(
-            _("Deletion is prevented by a protection rule: {message}").format(message=e)
-        )
-
-    # Get the current request, or bail if not set
-    request = current_request.get()
-    if request is None:
-        return
-
-    # Record an ObjectChange if applicable
-    if hasattr(instance, 'to_objectchange'):
-        if hasattr(instance, 'snapshot') and not getattr(instance, '_prechange_snapshot', None):
-            instance.snapshot()
-        objectchange = instance.to_objectchange(ObjectChangeActionChoices.ACTION_DELETE)
-        objectchange.user = request.user
-        objectchange.request_id = request.id
-        objectchange.save()
-
-    # Django does not automatically send an m2m_changed signal for the reverse direction of a
-    # many-to-many relationship (see https://code.djangoproject.com/ticket/17688), so we need to
-    # trigger one manually. We do this by checking for any reverse M2M relationships on the
-    # instance being deleted, and explicitly call .remove() on the remote M2M field to delete
-    # the association. This triggers an m2m_changed signal with the `post_remove` action type
-    # for the forward direction of the relationship, ensuring that the change is recorded.
-    for relation in instance._meta.related_objects:
-        if type(relation) is not ManyToManyRel:
-            continue
-        related_model = relation.related_model
-        related_field_name = relation.remote_field.name
-        if not issubclass(related_model, ChangeLoggingMixin):
-            # We only care about triggering the m2m_changed signal for models which support
-            # change logging
-            continue
-        for obj in related_model.objects.filter(**{related_field_name: instance.pk}):
-            obj.snapshot()  # Ensure the change record includes the "before" state
-            getattr(obj, related_field_name).remove(instance)
-
-    # Enqueue the object for event processing
-    queue = events_queue.get()
-    enqueue_event(queue, instance, request.user, request.id, OBJECT_DELETED)
-    events_queue.set(queue)
-
-    # Increment metric counters
-    model_deletes.labels(instance._meta.model_name).inc()
-
-
-@receiver(clear_events)
-def clear_events_queue(sender, **kwargs):
-    """
-    Delete any queued events (e.g. because of an aborted bulk transaction)
-    """
-    logger = logging.getLogger('events')
-    logger.info(f"Clearing {len(events_queue.get())} queued events ({sender})")
-    events_queue.set({})
-
-
-#
-# DataSource handlers
-#
 
 @receiver(post_sync)
 def auto_sync(instance, **kwargs):

netbox/core/tables/__init__.py

@@ -1,4 +1,3 @@
-from .change_logging import *
 from .config import *
 from .data import *
 from .jobs import *

netbox/core/tables/change_logging.py

@@ -1,53 +0,0 @@
-import django_tables2 as tables
-from django.utils.translation import gettext_lazy as _
-
-from core.models import ObjectChange
-from netbox.tables import NetBoxTable, columns
-from .template_code import *
-
-__all__ = (
-    'ObjectChangeTable',
-)
-
-
-class ObjectChangeTable(NetBoxTable):
-    time = columns.DateTimeColumn(
-        verbose_name=_('Time'),
-        timespec='minutes',
-        linkify=True
-    )
-    user_name = tables.Column(
-        verbose_name=_('Username')
-    )
-    full_name = tables.TemplateColumn(
-        accessor=tables.A('user'),
-        template_code=OBJECTCHANGE_FULL_NAME,
-        verbose_name=_('Full Name'),
-        orderable=False
-    )
-    action = columns.ChoiceFieldColumn(
-        verbose_name=_('Action'),
-    )
-    changed_object_type = columns.ContentTypeColumn(
-        verbose_name=_('Type')
-    )
-    object_repr = tables.TemplateColumn(
-        accessor=tables.A('changed_object'),
-        template_code=OBJECTCHANGE_OBJECT,
-        verbose_name=_('Object'),
-        orderable=False
-    )
-    request_id = tables.TemplateColumn(
-        template_code=OBJECTCHANGE_REQUEST_ID,
-        verbose_name=_('Request ID')
-    )
-    actions = columns.ActionsColumn(
-        actions=()
-    )
-
-    class Meta(NetBoxTable.Meta):
-        model = ObjectChange
-        fields = (
-            'pk', 'time', 'user_name', 'full_name', 'action', 'changed_object_type', 'object_repr', 'request_id',
-            'actions',
-        )

netbox/core/tables/plugins.py

@@ -1,80 +1,39 @@
 import django_tables2 as tables
 from django.utils.translation import gettext_lazy as _
-
-from netbox.tables import BaseTable, columns
+from netbox.tables import BaseTable
 
 __all__ = (
-    'CatalogPluginTable',
-    'PluginVersionTable',
+    'PluginTable',
 )
 
 
-class PluginVersionTable(BaseTable):
+class PluginTable(BaseTable):
+    name = tables.Column(
+        accessor=tables.A('verbose_name'),
+        verbose_name=_('Name')
+    )
     version = tables.Column(
         verbose_name=_('Version')
     )
-    last_updated = columns.DateTimeColumn(
-        accessor=tables.A('date'),
-        timespec='minutes',
-        verbose_name=_('Last Updated')
-    )
-    min_version = tables.Column(
-        accessor=tables.A('netbox_min_version'),
-        verbose_name=_('Minimum NetBox Version')
-    )
-    max_version = tables.Column(
-        accessor=tables.A('netbox_max_version'),
-        verbose_name=_('Maximum NetBox Version')
-    )
-
-    class Meta(BaseTable.Meta):
-        empty_text = _('No plugin data found')
-        fields = (
-            'version', 'last_updated', 'min_version', 'max_version',
-        )
-        default_columns = (
-            'version', 'last_updated', 'min_version', 'max_version',
-        )
-        orderable = False
-
-
-class CatalogPluginTable(BaseTable):
-    title_short = tables.Column(
-        linkify=('core:plugin', [tables.A('slug')]),
-        verbose_name=_('Name')
+    package = tables.Column(
+        accessor=tables.A('name'),
+        verbose_name=_('Package')
     )
     author = tables.Column(
-        accessor=tables.A('author__name'),
         verbose_name=_('Author')
     )
-    is_local = columns.BooleanColumn(
-        verbose_name=_('Local')
+    author_email = tables.Column(
+        verbose_name=_('Author Email')
     )
-    is_installed = columns.BooleanColumn(
-        verbose_name=_('Installed')
-    )
-    is_certified = columns.BooleanColumn(
-        verbose_name=_('Certified')
-    )
-    created_at = columns.DateTimeColumn(
-        verbose_name=_('Published')
-    )
-    updated_at = columns.DateTimeColumn(
-        verbose_name=_('Updated')
-    )
-    installed_version = tables.Column(
-        verbose_name=_('Installed version')
+    description = tables.Column(
+        verbose_name=_('Description')
     )
 
     class Meta(BaseTable.Meta):
-        empty_text = _('No plugin data found')
+        empty_text = _('No plugins found')
         fields = (
-            'title_short', 'author', 'is_local', 'is_installed', 'is_certified', 'created_at', 'updated_at',
-            'installed_version',
+            'name', 'version', 'package', 'author', 'author_email', 'description',
         )
         default_columns = (
-            'title_short', 'author', 'is_local', 'is_installed', 'is_certified', 'created_at', 'updated_at',
+            'name', 'version', 'package', 'description',
         )
-        # List installed plugins first, then certified plugins, then
-        # everything else (with each tranche ordered alphabetically)
-        order_by = ('-is_installed', '-is_certified', 'name')

netbox/core/tables/template_code.py

@@ -1,16 +0,0 @@
-OBJECTCHANGE_FULL_NAME = """
-{% load helpers %}
-{{ value.get_full_name|placeholder }}
-"""
-
-OBJECTCHANGE_OBJECT = """
-{% if value and value.get_absolute_url %}
-    <a href="{{ value.get_absolute_url }}">{{ record.object_repr }}</a>
-{% else %}
-    {{ record.object_repr }}
-{% endif %}
-"""
-
-OBJECTCHANGE_REQUEST_ID = """
-<a href="{% url 'core:objectchange_list' %}?request_id={{ value }}">{{ value }}</a>
-"""

netbox/core/tests/test_filtersets.py

@@ -1,13 +1,7 @@
-import uuid
 from datetime import datetime, timezone
 
-from django.contrib.contenttypes.models import ContentType
 from django.test import TestCase
-
-from dcim.models import Site
-from ipam.models import IPAddress
-from users.models import User
-from utilities.testing import BaseFilterSetTests, ChangeLoggedFilterSetTests
+from utilities.testing import ChangeLoggedFilterSetTests
 from ..choices import *
 from ..filtersets import *
 from ..models import *
@@ -138,99 +132,3 @@ class DataFileTestCase(TestCase, ChangeLoggedFilterSetTests):
             'a78168c7c97115bafd96450ed03ea43acec495094c5caa28f0d02e20e3a76cc2',
         ]}
         self.assertEqual(self.filterset(params, self.queryset).qs.count(), 2)
-
-
-class ObjectChangeTestCase(TestCase, BaseFilterSetTests):
-    queryset = ObjectChange.objects.all()
-    filterset = ObjectChangeFilterSet
-    ignore_fields = ('prechange_data', 'postchange_data')
-
-    @classmethod
-    def setUpTestData(cls):
-        users = (
-            User(username='user1'),
-            User(username='user2'),
-            User(username='user3'),
-        )
-        User.objects.bulk_create(users)
-
-        site = Site.objects.create(name='Test Site 1', slug='test-site-1')
-        ipaddress = IPAddress.objects.create(address='192.0.2.1/24')
-
-        object_changes = (
-            ObjectChange(
-                user=users[0],
-                user_name=users[0].username,
-                request_id=uuid.uuid4(),
-                action=ObjectChangeActionChoices.ACTION_CREATE,
-                changed_object=site,
-                object_repr=str(site),
-                postchange_data={'name': site.name, 'slug': site.slug}
-            ),
-            ObjectChange(
-                user=users[0],
-                user_name=users[0].username,
-                request_id=uuid.uuid4(),
-                action=ObjectChangeActionChoices.ACTION_UPDATE,
-                changed_object=site,
-                object_repr=str(site),
-                postchange_data={'name': site.name, 'slug': site.slug}
-            ),
-            ObjectChange(
-                user=users[1],
-                user_name=users[1].username,
-                request_id=uuid.uuid4(),
-                action=ObjectChangeActionChoices.ACTION_DELETE,
-                changed_object=site,
-                object_repr=str(site),
-                postchange_data={'name': site.name, 'slug': site.slug}
-            ),
-            ObjectChange(
-                user=users[1],
-                user_name=users[1].username,
-                request_id=uuid.uuid4(),
-                action=ObjectChangeActionChoices.ACTION_CREATE,
-                changed_object=ipaddress,
-                object_repr=str(ipaddress),
-                postchange_data={'address': ipaddress.address, 'status': ipaddress.status}
-            ),
-            ObjectChange(
-                user=users[2],
-                user_name=users[2].username,
-                request_id=uuid.uuid4(),
-                action=ObjectChangeActionChoices.ACTION_UPDATE,
-                changed_object=ipaddress,
-                object_repr=str(ipaddress),
-                postchange_data={'address': ipaddress.address, 'status': ipaddress.status}
-            ),
-            ObjectChange(
-                user=users[2],
-                user_name=users[2].username,
-                request_id=uuid.uuid4(),
-                action=ObjectChangeActionChoices.ACTION_DELETE,
-                changed_object=ipaddress,
-                object_repr=str(ipaddress),
-                postchange_data={'address': ipaddress.address, 'status': ipaddress.status}
-            ),
-        )
-        ObjectChange.objects.bulk_create(object_changes)
-
-    def test_q(self):
-        params = {'q': 'Site 1'}
-        self.assertEqual(self.filterset(params, self.queryset).qs.count(), 3)
-
-    def test_user(self):
-        params = {'user_id': User.objects.filter(username__in=['user1', 'user2']).values_list('pk', flat=True)}
-        self.assertEqual(self.filterset(params, self.queryset).qs.count(), 4)
-        params = {'user': ['user1', 'user2']}
-        self.assertEqual(self.filterset(params, self.queryset).qs.count(), 4)
-
-    def test_user_name(self):
-        params = {'user_name': ['user1', 'user2']}
-        self.assertEqual(self.filterset(params, self.queryset).qs.count(), 4)
-
-    def test_changed_object_type(self):
-        params = {'changed_object_type': 'dcim.site'}
-        self.assertEqual(self.filterset(params, self.queryset).qs.count(), 3)
-        params = {'changed_object_type_id': [ContentType.objects.get(app_label='dcim', model='site').pk]}
-        self.assertEqual(self.filterset(params, self.queryset).qs.count(), 3)

netbox/core/tests/test_models.py

@@ -1,7 +1,7 @@
 from django.test import TestCase
 
 from core.models import DataSource
-from core.choices import ObjectChangeActionChoices
+from extras.choices import ObjectChangeActionChoices
 from netbox.constants import CENSOR_TOKEN, CENSOR_TOKEN_CHANGED
 
 

netbox/core/tests/test_views.py

@@ -1,4 +1,4 @@
-import urllib.parse
+import logging
 import uuid
 from datetime import datetime
 
@@ -10,11 +10,8 @@ from django_rq.workers import get_worker
 from rq.job import Job as RQ_Job, JobStatus
 from rq.registry import DeferredJobRegistry, FailedJobRegistry, FinishedJobRegistry, StartedJobRegistry
 
-from core.choices import ObjectChangeActionChoices
-from core.models import *
-from dcim.models import Site
-from users.models import User
 from utilities.testing import TestCase, ViewTestCases, create_tags
+from ..models import *
 
 
 class DataSourceTestCase(ViewTestCases.PrimaryObjectViewTestCase):
@@ -102,43 +99,6 @@ class DataFileTestCase(
         DataFile.objects.bulk_create(data_files)
 
 
-# TODO: Convert to StandardTestCases.Views
-class ObjectChangeTestCase(TestCase):
-    user_permissions = (
-        'core.view_objectchange',
-    )
-
-    @classmethod
-    def setUpTestData(cls):
-
-        site = Site(name='Site 1', slug='site-1')
-        site.save()
-
-        # Create three ObjectChanges
-        user = User.objects.create_user(username='testuser2')
-        for i in range(1, 4):
-            oc = site.to_objectchange(action=ObjectChangeActionChoices.ACTION_UPDATE)
-            oc.user = user
-            oc.request_id = uuid.uuid4()
-            oc.save()
-
-    def test_objectchange_list(self):
-
-        url = reverse('core:objectchange_list')
-        params = {
-            "user": User.objects.first().pk,
-        }
-
-        response = self.client.get('{}?{}'.format(url, urllib.parse.urlencode(params)))
-        self.assertHttpStatus(response, 200)
-
-    def test_objectchange(self):
-
-        objectchange = ObjectChange.objects.first()
-        response = self.client.get(objectchange.get_absolute_url())
-        self.assertHttpStatus(response, 200)
-
-
 class BackgroundTaskTestCase(TestCase):
     user_permissions = ()
 

netbox/core/urls.py

@@ -25,10 +25,6 @@ urlpatterns = (
     path('jobs/<int:pk>/', views.JobView.as_view(), name='job'),
     path('jobs/<int:pk>/delete/', views.JobDeleteView.as_view(), name='job_delete'),
 
-    # Change logging
-    path('changelog/', views.ObjectChangeListView.as_view(), name='objectchange_list'),
-    path('changelog/<int:pk>/', include(get_model_urls('core', 'objectchange'))),
-
     # Background Tasks
     path('background-queues/', views.BackgroundQueueListView.as_view(), name='background_queue_list'),
     path('background-queues/<int:queue_index>/<str:status>/', views.BackgroundTaskListView.as_view(), name='background_task_list'),
@@ -49,8 +45,4 @@ urlpatterns = (
 
     # System
     path('system/', views.SystemView.as_view(), name='system'),
-
-    # Plugins
-    path('plugins/', views.PluginListView.as_view(), name='plugin_list'),
-    path('plugins/<str:name>/', views.PluginView.as_view(), name='plugin'),
 )

netbox/core/views.py

@@ -29,17 +29,12 @@ from netbox.config import get_config, PARAMS
 from netbox.views import generic
 from netbox.views.generic.base import BaseObjectView
 from netbox.views.generic.mixins import TableMixin
-from utilities.data import shallow_compare_dict
 from utilities.forms import ConfirmationForm
 from utilities.htmx import htmx_partial
 from utilities.query import count_related
 from utilities.views import ContentTypePermissionRequiredMixin, GetRelatedModelsMixin, register_model_view
 from . import filtersets, forms, tables
-from .choices import DataSourceStatusChoices
-from .jobs import SyncDataSourceJob
 from .models import *
-from .plugins import get_catalog_plugins, get_local_plugins
-from .tables import CatalogPluginTable, PluginVersionTable
 
 
 #
@@ -79,13 +74,12 @@ class DataSourceSyncView(BaseObjectView):
 
     def post(self, request, pk):
         datasource = get_object_or_404(self.queryset, pk=pk)
+        job = datasource.enqueue_sync_job(request)
 
-        # Enqueue the sync job & update the DataSource's status
-        job = SyncDataSourceJob.enqueue(instance=datasource, user=request.user)
-        datasource.status = DataSourceStatusChoices.QUEUED
-        DataSource.objects.filter(pk=datasource.pk).update(status=datasource.status)
-
-        messages.success(request, f"Queued job #{job.pk} to sync {datasource}")
+        messages.success(
+            request,
+            _("Queued job #{id} to sync {datasource}").format(id=job.pk, datasource=datasource)
+        )
         return redirect(datasource.get_absolute_url())
 
 
@@ -181,75 +175,6 @@ class JobBulkDeleteView(generic.BulkDeleteView):
     table = tables.JobTable
 
 
-#
-# Change logging
-#
-
-class ObjectChangeListView(generic.ObjectListView):
-    queryset = ObjectChange.objects.valid_models()
-    filterset = filtersets.ObjectChangeFilterSet
-    filterset_form = forms.ObjectChangeFilterForm
-    table = tables.ObjectChangeTable
-    template_name = 'core/objectchange_list.html'
-    actions = {
-        'export': {'view'},
-    }
-
-
-@register_model_view(ObjectChange)
-class ObjectChangeView(generic.ObjectView):
-    queryset = ObjectChange.objects.valid_models()
-
-    def get_extra_context(self, request, instance):
-        related_changes = ObjectChange.objects.valid_models().restrict(request.user, 'view').filter(
-            request_id=instance.request_id
-        ).exclude(
-            pk=instance.pk
-        )
-        related_changes_table = tables.ObjectChangeTable(
-            data=related_changes[:50],
-            orderable=False
-        )
-
-        objectchanges = ObjectChange.objects.valid_models().restrict(request.user, 'view').filter(
-            changed_object_type=instance.changed_object_type,
-            changed_object_id=instance.changed_object_id,
-        )
-
-        next_change = objectchanges.filter(time__gt=instance.time).order_by('time').first()
-        prev_change = objectchanges.filter(time__lt=instance.time).order_by('-time').first()
-
-        if not instance.prechange_data and instance.action in ['update', 'delete'] and prev_change:
-            non_atomic_change = True
-            prechange_data = prev_change.postchange_data_clean
-        else:
-            non_atomic_change = False
-            prechange_data = instance.prechange_data_clean
-
-        if prechange_data and instance.postchange_data:
-            diff_added = shallow_compare_dict(
-                prechange_data or dict(),
-                instance.postchange_data_clean or dict(),
-                exclude=['last_updated'],
-            )
-            diff_removed = {
-                x: prechange_data.get(x) for x in diff_added
-            } if prechange_data else {}
-        else:
-            diff_added = None
-            diff_removed = None
-
-        return {
-            'diff_added': diff_added,
-            'diff_removed': diff_removed,
-            'next_change': next_change,
-            'prev_change': prev_change,
-            'related_changes_table': related_changes_table,
-            'related_changes_count': related_changes.count(),
-            'non_atomic_change': non_atomic_change
-        }
-
-
 #
 # Config Revisions
 #
@@ -313,7 +238,7 @@ class ConfigRevisionRestoreView(ContentTypePermissionRequiredMixin, View):
 
         candidate_config = get_object_or_404(ConfigRevision, pk=pk)
         candidate_config.activate()
-        messages.success(request, f"Restored configuration revision #{pk}")
+        messages.success(request, _("Restored configuration revision #{id}").format(id=pk))
 
         return redirect(candidate_config.get_absolute_url())
 
@@ -457,9 +382,9 @@ class BackgroundTaskDeleteView(BaseRQView):
             # Remove job id from queue and delete the actual job
             queue.connection.lrem(queue.key, 0, job.id)
             job.delete()
-            messages.success(request, f'Deleted job {job_id}')
+            messages.success(request, _('Job {id} has been deleted.').format(id=job_id))
         else:
-            messages.error(request, f'Error deleting job: {form.errors[0]}')
+            messages.error(request, _('Error deleting job {id}: {error}').format(id=job_id, error=form.errors[0]))
 
         return redirect(reverse('core:background_queue_list'))
 
@@ -472,13 +397,13 @@ class BackgroundTaskRequeueView(BaseRQView):
         try:
             job = RQ_Job.fetch(job_id, connection=get_redis_connection(config['connection_config']),)
         except NoSuchJobError:
-            raise Http404(_("Job {job_id} not found").format(job_id=job_id))
+            raise Http404(_("Job {id} not found.").format(id=job_id))
 
         queue_index = QUEUES_MAP[job.origin]
         queue = get_queue_by_index(queue_index)
 
         requeue_job(job_id, connection=queue.connection, serializer=queue.serializer)
-        messages.success(request, f'You have successfully requeued: {job_id}')
+        messages.success(request, _('Job {id} has been re-enqueued.').format(id=job_id))
         return redirect(reverse('core:background_task', args=[job_id]))
 
 
@@ -490,7 +415,7 @@ class BackgroundTaskEnqueueView(BaseRQView):
         try:
             job = RQ_Job.fetch(job_id, connection=get_redis_connection(config['connection_config']),)
         except NoSuchJobError:
-            raise Http404(_("Job {job_id} not found").format(job_id=job_id))
+            raise Http404(_("Job {id} not found.").format(id=job_id))
 
         queue_index = QUEUES_MAP[job.origin]
         queue = get_queue_by_index(queue_index)
@@ -513,7 +438,7 @@ class BackgroundTaskEnqueueView(BaseRQView):
             registry = ScheduledJobRegistry(queue.name, queue.connection)
             registry.remove(job)
 
-        messages.success(request, f'You have successfully enqueued: {job_id}')
+        messages.success(request, _('Job {id} has been enqueued.').format(id=job_id))
         return redirect(reverse('core:background_task', args=[job_id]))
 
 
@@ -530,11 +455,11 @@ class BackgroundTaskStopView(BaseRQView):
         queue_index = QUEUES_MAP[job.origin]
         queue = get_queue_by_index(queue_index)
 
-        stopped, _ = stop_jobs(queue, job_id)
-        if len(stopped) == 1:
-            messages.success(request, f'You have successfully stopped {job_id}')
+        stopped_jobs = stop_jobs(queue, job_id)[0]
+        if len(stopped_jobs) == 1:
+            messages.success(request, _('Job {id} has been stopped.').format(id=job_id))
         else:
-            messages.error(request, f'Failed to stop {job_id}')
+            messages.error(request, _('Failed to stop job {id}').format(id=job_id))
 
         return redirect(reverse('core:background_task', args=[job_id]))
 
@@ -589,7 +514,7 @@ class WorkerView(BaseRQView):
 
 
 #
-# System
+# Plugins
 #
 
 class SystemView(UserPassesTestMixin, View):
@@ -613,7 +538,7 @@ class SystemView(UserPassesTestMixin, View):
         except (ProgrammingError, IndexError):
             pass
         stats = {
-            'netbox_release': settings.RELEASE,
+            'netbox_version': settings.VERSION,
             'django_version': DJANGO_VERSION,
             'python_version': platform.python_version(),
             'postgresql_version': psql_version,
@@ -622,6 +547,12 @@ class SystemView(UserPassesTestMixin, View):
             'rq_worker_count': Worker.count(get_connection('default')),
         }
 
+        # Plugins
+        plugins = [
+            # Look up app config by package name
+            apps.get_app_config(plugin.rsplit('.', 1)[-1]) for plugin in settings.PLUGINS
+        ]
+
         # Configuration
         try:
             config = ConfigRevision.objects.get(pk=cache.get('config_version'))
@@ -631,11 +562,12 @@ class SystemView(UserPassesTestMixin, View):
 
         # Raw data export
         if 'export' in request.GET:
-            stats['netbox_release'] = stats['netbox_release'].asdict()
             params = [param.name for param in PARAMS]
             data = {
                 **stats,
-                'plugins': settings.PLUGINS,
+                'plugins': {
+                    plugin.name: plugin.version for plugin in plugins
+                },
                 'config': {
                     k: getattr(config, k) for k in sorted(params)
                 },
@@ -644,71 +576,11 @@ class SystemView(UserPassesTestMixin, View):
             response['Content-Disposition'] = 'attachment; filename="netbox.json"'
             return response
 
+        plugins_table = tables.PluginTable(plugins, orderable=False)
+        plugins_table.configure(request)
+
         return render(request, 'core/system.html', {
             'stats': stats,
+            'plugins_table': plugins_table,
             'config': config,
         })
-
-
-#
-# Plugins
-#
-
-class BasePluginView(UserPassesTestMixin, View):
-    CACHE_KEY_CATALOG_ERROR = 'plugins-catalog-error'
-
-    def test_func(self):
-        return self.request.user.is_staff
-
-    def get_cached_plugins(self, request):
-        catalog_plugins = {}
-        catalog_plugins_error = cache.get(self.CACHE_KEY_CATALOG_ERROR, default=False)
-        if not catalog_plugins_error:
-            catalog_plugins = get_catalog_plugins()
-            if not catalog_plugins:
-                # Cache for 5 minutes to avoid spamming connection
-                cache.set(self.CACHE_KEY_CATALOG_ERROR, True, 300)
-                messages.warning(request, _("Plugins catalog could not be loaded"))
-
-        return get_local_plugins(catalog_plugins)
-
-
-class PluginListView(BasePluginView):
-
-    def get(self, request):
-        q = request.GET.get('q', None)
-
-        plugins = self.get_cached_plugins(request).values()
-        if q:
-            plugins = [obj for obj in plugins if q.casefold() in obj.title_short.casefold()]
-
-        table = CatalogPluginTable(plugins, user=request.user)
-        table.configure(request)
-
-        # If this is an HTMX request, return only the rendered table HTML
-        if htmx_partial(request):
-            return render(request, 'htmx/table.html', {
-                'table': table,
-            })
-
-        return render(request, 'core/plugin_list.html', {
-            'table': table,
-        })
-
-
-class PluginView(BasePluginView):
-
-    def get(self, request, name):
-
-        plugins = self.get_cached_plugins(request)
-        if name not in plugins:
-            raise Http404(_("Plugin {name} not found").format(name=name))
-        plugin = plugins[name]
-
-        table = PluginVersionTable(plugin.release_recent_history, user=request.user)
-        table.configure(request)
-
-        return render(request, 'core/plugin.html', {
-            'plugin': plugin,
-            'table': table,
-        })

netbox/dcim/api/nested_serializers.py

@@ -57,31 +57,34 @@ __all__ = [
     exclude_fields=('site_count',),
 )
 class NestedRegionSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:region-detail')
     site_count = serializers.IntegerField(read_only=True)
     _depth = serializers.IntegerField(source='level', read_only=True)
 
     class Meta:
         model = models.Region
-        fields = ['id', 'url', 'display_url', 'display', 'name', 'slug', 'site_count', '_depth']
+        fields = ['id', 'url', 'display', 'name', 'slug', 'site_count', '_depth']
 
 
 @extend_schema_serializer(
     exclude_fields=('site_count',),
 )
 class NestedSiteGroupSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:sitegroup-detail')
     site_count = serializers.IntegerField(read_only=True)
     _depth = serializers.IntegerField(source='level', read_only=True)
 
     class Meta:
         model = models.SiteGroup
-        fields = ['id', 'url', 'display_url', 'display', 'name', 'slug', 'site_count', '_depth']
+        fields = ['id', 'url', 'display', 'name', 'slug', 'site_count', '_depth']
 
 
 class NestedSiteSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:site-detail')
 
     class Meta:
         model = models.Site
-        fields = ['id', 'url', 'display_url', 'display', 'name', 'slug']
+        fields = ['id', 'url', 'display', 'name', 'slug']
 
 
 #
@@ -92,42 +95,46 @@ class NestedSiteSerializer(WritableNestedSerializer):
     exclude_fields=('rack_count',),
 )
 class NestedLocationSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:location-detail')
     rack_count = serializers.IntegerField(read_only=True)
     _depth = serializers.IntegerField(source='level', read_only=True)
 
     class Meta:
         model = models.Location
-        fields = ['id', 'url', 'display_url', 'display', 'name', 'slug', 'rack_count', '_depth']
+        fields = ['id', 'url', 'display', 'name', 'slug', 'rack_count', '_depth']
 
 
 @extend_schema_serializer(
     exclude_fields=('rack_count',),
 )
 class NestedRackRoleSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:rackrole-detail')
     rack_count = RelatedObjectCountField('racks')
 
     class Meta:
         model = models.RackRole
-        fields = ['id', 'url', 'display_url', 'display', 'name', 'slug', 'rack_count']
+        fields = ['id', 'url', 'display', 'name', 'slug', 'rack_count']
 
 
 @extend_schema_serializer(
     exclude_fields=('device_count',),
 )
 class NestedRackSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:rack-detail')
     device_count = RelatedObjectCountField('devices')
 
     class Meta:
         model = models.Rack
-        fields = ['id', 'url', 'display_url', 'display', 'name', 'device_count']
+        fields = ['id', 'url', 'display', 'name', 'device_count']
 
 
 class NestedRackReservationSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:rackreservation-detail')
     user = serializers.SerializerMethodField(read_only=True)
 
     class Meta:
         model = models.RackReservation
-        fields = ['id', 'url', 'display_url', 'display', 'user', 'units']
+        fields = ['id', 'url', 'display', 'user', 'units']
 
     def get_user(self, obj):
         return obj.user.username
@@ -141,31 +148,34 @@ class NestedRackReservationSerializer(WritableNestedSerializer):
     exclude_fields=('devicetype_count',),
 )
 class NestedManufacturerSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:manufacturer-detail')
     devicetype_count = RelatedObjectCountField('device_types')
 
     class Meta:
         model = models.Manufacturer
-        fields = ['id', 'url', 'display_url', 'display', 'name', 'slug', 'devicetype_count']
+        fields = ['id', 'url', 'display', 'name', 'slug', 'devicetype_count']
 
 
 @extend_schema_serializer(
     exclude_fields=('device_count',),
 )
 class NestedDeviceTypeSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:devicetype-detail')
     manufacturer = NestedManufacturerSerializer(read_only=True)
     device_count = RelatedObjectCountField('instances')
 
     class Meta:
         model = models.DeviceType
-        fields = ['id', 'url', 'display_url', 'display', 'manufacturer', 'model', 'slug', 'device_count']
+        fields = ['id', 'url', 'display', 'manufacturer', 'model', 'slug', 'device_count']
 
 
 class NestedModuleTypeSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:moduletype-detail')
     manufacturer = NestedManufacturerSerializer(read_only=True)
 
     class Meta:
         model = models.ModuleType
-        fields = ['id', 'url', 'display_url', 'display', 'manufacturer', 'model']
+        fields = ['id', 'url', 'display', 'manufacturer', 'model']
 
 
 #
@@ -173,74 +183,84 @@ class NestedModuleTypeSerializer(WritableNestedSerializer):
 #
 
 class NestedConsolePortTemplateSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:consoleporttemplate-detail')
 
     class Meta:
         model = models.ConsolePortTemplate
-        fields = ['id', 'url', 'display_url', 'display', 'name']
+        fields = ['id', 'url', 'display', 'name']
 
 
 class NestedConsoleServerPortTemplateSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:consoleserverporttemplate-detail')
 
     class Meta:
         model = models.ConsoleServerPortTemplate
-        fields = ['id', 'url', 'display_url', 'display', 'name']
+        fields = ['id', 'url', 'display', 'name']
 
 
 class NestedPowerPortTemplateSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:powerporttemplate-detail')
 
     class Meta:
         model = models.PowerPortTemplate
-        fields = ['id', 'url', 'display_url', 'display', 'name']
+        fields = ['id', 'url', 'display', 'name']
 
 
 class NestedPowerOutletTemplateSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:poweroutlettemplate-detail')
 
     class Meta:
         model = models.PowerOutletTemplate
-        fields = ['id', 'url', 'display_url', 'display', 'name']
+        fields = ['id', 'url', 'display', 'name']
 
 
 class NestedInterfaceTemplateSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:interfacetemplate-detail')
 
     class Meta:
         model = models.InterfaceTemplate
-        fields = ['id', 'url', 'display_url', 'display', 'name']
+        fields = ['id', 'url', 'display', 'name']
 
 
 class NestedRearPortTemplateSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:rearporttemplate-detail')
 
     class Meta:
         model = models.RearPortTemplate
-        fields = ['id', 'url', 'display_url', 'display', 'name']
+        fields = ['id', 'url', 'display', 'name']
 
 
 class NestedFrontPortTemplateSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:frontporttemplate-detail')
 
     class Meta:
         model = models.FrontPortTemplate
-        fields = ['id', 'url', 'display_url', 'display', 'name']
+        fields = ['id', 'url', 'display', 'name']
 
 
 class NestedModuleBayTemplateSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:modulebaytemplate-detail')
 
     class Meta:
         model = models.ModuleBayTemplate
-        fields = ['id', 'url', 'display_url', 'display', 'name']
+        fields = ['id', 'url', 'display', 'name']
 
 
 class NestedDeviceBayTemplateSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:devicebaytemplate-detail')
 
     class Meta:
         model = models.DeviceBayTemplate
-        fields = ['id', 'url', 'display_url', 'display', 'name']
+        fields = ['id', 'url', 'display', 'name']
 
 
 class NestedInventoryItemTemplateSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:inventoryitemtemplate-detail')
     _depth = serializers.IntegerField(source='level', read_only=True)
 
     class Meta:
         model = models.InventoryItemTemplate
-        fields = ['id', 'url', 'display_url', 'display', 'name', '_depth']
+        fields = ['id', 'url', 'display', 'name', '_depth']
 
 
 #
@@ -251,154 +271,171 @@ class NestedInventoryItemTemplateSerializer(WritableNestedSerializer):
     exclude_fields=('device_count', 'virtualmachine_count'),
 )
 class NestedDeviceRoleSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:devicerole-detail')
     device_count = RelatedObjectCountField('devices')
     virtualmachine_count = RelatedObjectCountField('virtual_machines')
 
     class Meta:
         model = models.DeviceRole
-        fields = ['id', 'url', 'display_url', 'display', 'name', 'slug', 'device_count', 'virtualmachine_count']
+        fields = ['id', 'url', 'display', 'name', 'slug', 'device_count', 'virtualmachine_count']
 
 
 @extend_schema_serializer(
     exclude_fields=('device_count', 'virtualmachine_count'),
 )
 class NestedPlatformSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:platform-detail')
     device_count = RelatedObjectCountField('devices')
     virtualmachine_count = RelatedObjectCountField('virtual_machines')
 
     class Meta:
         model = models.Platform
-        fields = ['id', 'url', 'display_url', 'display', 'name', 'slug', 'device_count', 'virtualmachine_count']
+        fields = ['id', 'url', 'display', 'name', 'slug', 'device_count', 'virtualmachine_count']
 
 
 class NestedDeviceSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:device-detail')
 
     class Meta:
         model = models.Device
-        fields = ['id', 'url', 'display_url', 'display', 'name']
+        fields = ['id', 'url', 'display', 'name']
 
 
 class ModuleNestedModuleBaySerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:modulebay-detail')
 
     class Meta:
         model = models.ModuleBay
-        fields = ['id', 'url', 'display_url', 'display', 'name']
+        fields = ['id', 'url', 'display', 'name']
 
 
 class ModuleBayNestedModuleSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:module-detail')
 
     class Meta:
         model = models.Module
-        fields = ['id', 'url', 'display_url', 'display', 'serial']
+        fields = ['id', 'url', 'display', 'serial']
 
 
 class NestedModuleSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:module-detail')
     device = NestedDeviceSerializer(read_only=True)
     module_bay = ModuleNestedModuleBaySerializer(read_only=True)
     module_type = NestedModuleTypeSerializer(read_only=True)
 
     class Meta:
         model = models.Module
-        fields = ['id', 'url', 'display_url', 'display', 'device', 'module_bay', 'module_type']
+        fields = ['id', 'url', 'display', 'device', 'module_bay', 'module_type']
 
 
 class NestedConsoleServerPortSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:consoleserverport-detail')
     device = NestedDeviceSerializer(read_only=True)
     _occupied = serializers.BooleanField(required=False, read_only=True)
 
     class Meta:
         model = models.ConsoleServerPort
-        fields = ['id', 'url', 'display_url', 'display', 'device', 'name', 'cable', '_occupied']
+        fields = ['id', 'url', 'display', 'device', 'name', 'cable', '_occupied']
 
 
 class NestedConsolePortSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:consoleport-detail')
     device = NestedDeviceSerializer(read_only=True)
     _occupied = serializers.BooleanField(required=False, read_only=True)
 
     class Meta:
         model = models.ConsolePort
-        fields = ['id', 'url', 'display_url', 'display', 'device', 'name', 'cable', '_occupied']
+        fields = ['id', 'url', 'display', 'device', 'name', 'cable', '_occupied']
 
 
 class NestedPowerOutletSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:poweroutlet-detail')
     device = NestedDeviceSerializer(read_only=True)
     _occupied = serializers.BooleanField(required=False, read_only=True)
 
     class Meta:
         model = models.PowerOutlet
-        fields = ['id', 'url', 'display_url', 'display', 'device', 'name', 'cable', '_occupied']
+        fields = ['id', 'url', 'display', 'device', 'name', 'cable', '_occupied']
 
 
 class NestedPowerPortSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:powerport-detail')
     device = NestedDeviceSerializer(read_only=True)
     _occupied = serializers.BooleanField(required=False, read_only=True)
 
     class Meta:
         model = models.PowerPort
-        fields = ['id', 'url', 'display_url', 'display', 'device', 'name', 'cable', '_occupied']
+        fields = ['id', 'url', 'display', 'device', 'name', 'cable', '_occupied']
 
 
 class NestedInterfaceSerializer(WritableNestedSerializer):
     device = NestedDeviceSerializer(read_only=True)
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:interface-detail')
     _occupied = serializers.BooleanField(required=False, read_only=True)
 
     class Meta:
         model = models.Interface
-        fields = ['id', 'url', 'display_url', 'display', 'device', 'name', 'cable', '_occupied']
+        fields = ['id', 'url', 'display', 'device', 'name', 'cable', '_occupied']
 
 
 class NestedRearPortSerializer(WritableNestedSerializer):
     device = NestedDeviceSerializer(read_only=True)
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:rearport-detail')
     _occupied = serializers.BooleanField(required=False, read_only=True)
 
     class Meta:
         model = models.RearPort
-        fields = ['id', 'url', 'display_url', 'display', 'device', 'name', 'cable', '_occupied']
+        fields = ['id', 'url', 'display', 'device', 'name', 'cable', '_occupied']
 
 
 class NestedFrontPortSerializer(WritableNestedSerializer):
     device = NestedDeviceSerializer(read_only=True)
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:frontport-detail')
     _occupied = serializers.BooleanField(required=False, read_only=True)
 
     class Meta:
         model = models.FrontPort
-        fields = ['id', 'url', 'display_url', 'display', 'device', 'name', 'cable', '_occupied']
+        fields = ['id', 'url', 'display', 'device', 'name', 'cable', '_occupied']
 
 
 class NestedModuleBaySerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:modulebay-detail')
     installed_module = ModuleBayNestedModuleSerializer(required=False, allow_null=True)
 
     class Meta:
         model = models.ModuleBay
-        fields = ['id', 'url', 'display_url', 'display', 'installed_module', 'name']
+        fields = ['id', 'url', 'display', 'installed_module', 'name']
 
 
 class NestedDeviceBaySerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:devicebay-detail')
     device = NestedDeviceSerializer(read_only=True)
 
     class Meta:
         model = models.DeviceBay
-        fields = ['id', 'url', 'display_url', 'display', 'device', 'name']
+        fields = ['id', 'url', 'display', 'device', 'name']
 
 
 class NestedInventoryItemSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:inventoryitem-detail')
     device = NestedDeviceSerializer(read_only=True)
     _depth = serializers.IntegerField(source='level', read_only=True)
 
     class Meta:
         model = models.InventoryItem
-        fields = ['id', 'url', 'display_url', 'display', 'device', 'name', '_depth']
+        fields = ['id', 'url', 'display', 'device', 'name', '_depth']
 
 
 @extend_schema_serializer(
     exclude_fields=('inventoryitem_count',),
 )
 class NestedInventoryItemRoleSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:inventoryitemrole-detail')
     inventoryitem_count = RelatedObjectCountField('inventory_items')
 
     class Meta:
         model = models.InventoryItemRole
-        fields = ['id', 'url', 'display_url', 'display', 'name', 'slug', 'inventoryitem_count']
+        fields = ['id', 'url', 'display', 'name', 'slug', 'inventoryitem_count']
 
 
 #
@@ -406,10 +443,11 @@ class NestedInventoryItemRoleSerializer(WritableNestedSerializer):
 #
 
 class NestedCableSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:cable-detail')
 
     class Meta:
         model = models.Cable
-        fields = ['id', 'url', 'display_url', 'display', 'label']
+        fields = ['id', 'url', 'display', 'label']
 
 
 #
@@ -420,12 +458,13 @@ class NestedCableSerializer(WritableNestedSerializer):
     exclude_fields=('member_count',),
 )
 class NestedVirtualChassisSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:virtualchassis-detail')
     master = NestedDeviceSerializer()
     member_count = serializers.IntegerField(read_only=True)
 
     class Meta:
         model = models.VirtualChassis
-        fields = ['id', 'url', 'display_url', 'display', 'name', 'master', 'member_count']
+        fields = ['id', 'url', 'display', 'name', 'master', 'member_count']
 
 
 #
@@ -436,24 +475,27 @@ class NestedVirtualChassisSerializer(WritableNestedSerializer):
     exclude_fields=('powerfeed_count',),
 )
 class NestedPowerPanelSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:powerpanel-detail')
     powerfeed_count = RelatedObjectCountField('powerfeeds')
 
     class Meta:
         model = models.PowerPanel
-        fields = ['id', 'url', 'display_url', 'display', 'name', 'powerfeed_count']
+        fields = ['id', 'url', 'display', 'name', 'powerfeed_count']
 
 
 class NestedPowerFeedSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:powerfeed-detail')
     _occupied = serializers.BooleanField(required=False, read_only=True)
 
     class Meta:
         model = models.PowerFeed
-        fields = ['id', 'url', 'display_url', 'display', 'name', 'cable', '_occupied']
+        fields = ['id', 'url', 'display', 'name', 'cable', '_occupied']
 
 
 class NestedVirtualDeviceContextSerializer(WritableNestedSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:virtualdevicecontext-detail')
     device = NestedDeviceSerializer()
 
     class Meta:
         model = models.VirtualDeviceContext
-        fields = ['id', 'url', 'display_url', 'display', 'name', 'identifier', 'device']
+        fields = ['id', 'url', 'display', 'name', 'identifier', 'device']

netbox/dcim/api/serializers_/cables.py

@@ -7,7 +7,7 @@ from dcim.choices import *
 from dcim.constants import *
 from dcim.models import Cable, CablePath, CableTermination
 from netbox.api.fields import ChoiceField, ContentTypeField
-from netbox.api.serializers import BaseModelSerializer, GenericObjectSerializer, NetBoxModelSerializer
+from netbox.api.serializers import GenericObjectSerializer, NetBoxModelSerializer
 from tenancy.api.serializers_.tenants import TenantSerializer
 from utilities.api import get_serializer_for_model
 
@@ -21,6 +21,7 @@ __all__ = (
 
 
 class CableSerializer(NetBoxModelSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:cable-detail')
     a_terminations = GenericObjectSerializer(many=True, required=False)
     b_terminations = GenericObjectSerializer(many=True, required=False)
     status = ChoiceField(choices=LinkStatusChoices, required=False)
@@ -30,26 +31,27 @@ class CableSerializer(NetBoxModelSerializer):
     class Meta:
         model = Cable
         fields = [
-            'id', 'url', 'display_url', 'display', 'type', 'a_terminations', 'b_terminations', 'status', 'tenant',
-            'label', 'color', 'length', 'length_unit', 'description', 'comments', 'tags', 'custom_fields', 'created',
-            'last_updated',
+            'id', 'url', 'display', 'type', 'a_terminations', 'b_terminations', 'status', 'tenant', 'label', 'color',
+            'length', 'length_unit', 'description', 'comments', 'tags', 'custom_fields', 'created', 'last_updated',
         ]
         brief_fields = ('id', 'url', 'display', 'label', 'description')
 
 
-class TracedCableSerializer(BaseModelSerializer):
+class TracedCableSerializer(serializers.ModelSerializer):
     """
     Used only while tracing a cable path.
     """
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:cable-detail')
 
     class Meta:
         model = Cable
         fields = [
-            'id', 'url', 'display_url', 'type', 'status', 'label', 'color', 'length', 'length_unit', 'description',
+            'id', 'url', 'type', 'status', 'label', 'color', 'length', 'length_unit', 'description',
         ]
 
 
 class CableTerminationSerializer(NetBoxModelSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:cabletermination-detail')
     termination_type = ContentTypeField(
         queryset=ContentType.objects.filter(CABLE_TERMINATION_MODELS)
     )
@@ -58,8 +60,8 @@ class CableTerminationSerializer(NetBoxModelSerializer):
     class Meta:
         model = CableTermination
         fields = [
-            'id', 'url', 'display', 'cable', 'cable_end', 'termination_type', 'termination_id',
-            'termination', 'created', 'last_updated',
+            'id', 'url', 'display', 'cable', 'cable_end', 'termination_type', 'termination_id', 'termination',
+            'created', 'last_updated',
         ]
 
     @extend_schema_field(serializers.JSONField(allow_null=True))

netbox/dcim/api/serializers_/device_components.py

@@ -41,6 +41,7 @@ __all__ = (
 
 
 class ConsoleServerPortSerializer(NetBoxModelSerializer, CabledObjectSerializer, ConnectedEndpointsSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:consoleserverport-detail')
     device = DeviceSerializer(nested=True)
     module = ModuleSerializer(
         nested=True,
@@ -62,7 +63,7 @@ class ConsoleServerPortSerializer(NetBoxModelSerializer, CabledObjectSerializer,
     class Meta:
         model = ConsoleServerPort
         fields = [
-            'id', 'url', 'display_url', 'display', 'device', 'module', 'name', 'label', 'type', 'speed', 'description',
+            'id', 'url', 'display', 'device', 'module', 'name', 'label', 'type', 'speed', 'description',
             'mark_connected', 'cable', 'cable_end', 'link_peers', 'link_peers_type', 'connected_endpoints',
             'connected_endpoints_type', 'connected_endpoints_reachable', 'tags', 'custom_fields', 'created',
             'last_updated', '_occupied',
@@ -71,6 +72,7 @@ class ConsoleServerPortSerializer(NetBoxModelSerializer, CabledObjectSerializer,
 
 
 class ConsolePortSerializer(NetBoxModelSerializer, CabledObjectSerializer, ConnectedEndpointsSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:consoleport-detail')
     device = DeviceSerializer(nested=True)
     module = ModuleSerializer(
         nested=True,
@@ -92,7 +94,7 @@ class ConsolePortSerializer(NetBoxModelSerializer, CabledObjectSerializer, Conne
     class Meta:
         model = ConsolePort
         fields = [
-            'id', 'url', 'display_url', 'display', 'device', 'module', 'name', 'label', 'type', 'speed', 'description',
+            'id', 'url', 'display', 'device', 'module', 'name', 'label', 'type', 'speed', 'description',
             'mark_connected', 'cable', 'cable_end', 'link_peers', 'link_peers_type', 'connected_endpoints',
             'connected_endpoints_type', 'connected_endpoints_reachable', 'tags', 'custom_fields', 'created',
             'last_updated', '_occupied',
@@ -101,6 +103,7 @@ class ConsolePortSerializer(NetBoxModelSerializer, CabledObjectSerializer, Conne
 
 
 class PowerPortSerializer(NetBoxModelSerializer, CabledObjectSerializer, ConnectedEndpointsSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:powerport-detail')
     device = DeviceSerializer(nested=True)
     module = ModuleSerializer(
         nested=True,
@@ -118,8 +121,8 @@ class PowerPortSerializer(NetBoxModelSerializer, CabledObjectSerializer, Connect
     class Meta:
         model = PowerPort
         fields = [
-            'id', 'url', 'display_url', 'display', 'device', 'module', 'name', 'label', 'type', 'maximum_draw',
-            'allocated_draw', 'description', 'mark_connected', 'cable', 'cable_end', 'link_peers', 'link_peers_type',
+            'id', 'url', 'display', 'device', 'module', 'name', 'label', 'type', 'maximum_draw', 'allocated_draw',
+            'description', 'mark_connected', 'cable', 'cable_end', 'link_peers', 'link_peers_type',
             'connected_endpoints', 'connected_endpoints_type', 'connected_endpoints_reachable', 'tags', 'custom_fields',
             'created', 'last_updated', '_occupied',
         ]
@@ -127,6 +130,7 @@ class PowerPortSerializer(NetBoxModelSerializer, CabledObjectSerializer, Connect
 
 
 class PowerOutletSerializer(NetBoxModelSerializer, CabledObjectSerializer, ConnectedEndpointsSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:poweroutlet-detail')
     device = DeviceSerializer(nested=True)
     module = ModuleSerializer(
         nested=True,
@@ -155,8 +159,8 @@ class PowerOutletSerializer(NetBoxModelSerializer, CabledObjectSerializer, Conne
     class Meta:
         model = PowerOutlet
         fields = [
-            'id', 'url', 'display_url', 'display', 'device', 'module', 'name', 'label', 'type', 'power_port',
-            'feed_leg', 'description', 'mark_connected', 'cable', 'cable_end', 'link_peers', 'link_peers_type',
+            'id', 'url', 'display', 'device', 'module', 'name', 'label', 'type', 'power_port', 'feed_leg',
+            'description', 'mark_connected', 'cable', 'cable_end', 'link_peers', 'link_peers_type',
             'connected_endpoints', 'connected_endpoints_type', 'connected_endpoints_reachable', 'tags', 'custom_fields',
             'created', 'last_updated', '_occupied',
         ]
@@ -164,6 +168,7 @@ class PowerOutletSerializer(NetBoxModelSerializer, CabledObjectSerializer, Conne
 
 
 class InterfaceSerializer(NetBoxModelSerializer, CabledObjectSerializer, ConnectedEndpointsSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:interface-detail')
     device = DeviceSerializer(nested=True)
     vdcs = SerializedPKRelatedField(
         queryset=VirtualDeviceContext.objects.all(),
@@ -219,11 +224,11 @@ class InterfaceSerializer(NetBoxModelSerializer, CabledObjectSerializer, Connect
     class Meta:
         model = Interface
         fields = [
-            'id', 'url', 'display_url', 'display', 'device', 'vdcs', 'module', 'name', 'label', 'type', 'enabled',
-            'parent', 'bridge', 'lag', 'mtu', 'mac_address', 'speed', 'duplex', 'wwn', 'mgmt_only', 'description',
-            'mode', 'rf_role', 'rf_channel', 'poe_mode', 'poe_type', 'rf_channel_frequency', 'rf_channel_width',
-            'tx_power', 'untagged_vlan', 'tagged_vlans', 'mark_connected', 'cable', 'cable_end', 'wireless_link',
-            'link_peers', 'link_peers_type', 'wireless_lans', 'vrf', 'l2vpn_termination', 'connected_endpoints',
+            'id', 'url', 'display', 'device', 'vdcs', 'module', 'name', 'label', 'type', 'enabled', 'parent', 'bridge',
+            'lag', 'mtu', 'mac_address', 'speed', 'duplex', 'wwn', 'mgmt_only', 'description', 'mode', 'rf_role',
+            'rf_channel', 'poe_mode', 'poe_type', 'rf_channel_frequency', 'rf_channel_width', 'tx_power',
+            'untagged_vlan', 'tagged_vlans', 'mark_connected', 'cable', 'cable_end', 'wireless_link', 'link_peers',
+            'link_peers_type', 'wireless_lans', 'vrf', 'l2vpn_termination', 'connected_endpoints',
             'connected_endpoints_type', 'connected_endpoints_reachable', 'tags', 'custom_fields', 'created',
             'last_updated', 'count_ipaddresses', 'count_fhrp_groups', '_occupied',
         ]
@@ -245,6 +250,7 @@ class InterfaceSerializer(NetBoxModelSerializer, CabledObjectSerializer, Connect
 
 
 class RearPortSerializer(NetBoxModelSerializer, CabledObjectSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:rearport-detail')
     device = DeviceSerializer(nested=True)
     module = ModuleSerializer(
         nested=True,
@@ -257,9 +263,9 @@ class RearPortSerializer(NetBoxModelSerializer, CabledObjectSerializer):
     class Meta:
         model = RearPort
         fields = [
-            'id', 'url', 'display_url', 'display', 'device', 'module', 'name', 'label', 'type', 'color', 'positions',
-            'description', 'mark_connected', 'cable', 'cable_end', 'link_peers', 'link_peers_type', 'tags',
-            'custom_fields', 'created', 'last_updated', '_occupied',
+            'id', 'url', 'display', 'device', 'module', 'name', 'label', 'type', 'color', 'positions', 'description',
+            'mark_connected', 'cable', 'cable_end', 'link_peers', 'link_peers_type', 'tags', 'custom_fields', 'created',
+            'last_updated', '_occupied',
         ]
         brief_fields = ('id', 'url', 'display', 'device', 'name', 'description', 'cable', '_occupied')
 
@@ -268,13 +274,15 @@ class FrontPortRearPortSerializer(WritableNestedSerializer):
     """
     NestedRearPortSerializer but with parent device omitted (since front and rear ports must belong to same device)
     """
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:rearport-detail')
 
     class Meta:
         model = RearPort
-        fields = ['id', 'url', 'display_url', 'display', 'name', 'label', 'description']
+        fields = ['id', 'url', 'display', 'name', 'label', 'description']
 
 
 class FrontPortSerializer(NetBoxModelSerializer, CabledObjectSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:frontport-detail')
     device = DeviceSerializer(nested=True)
     module = ModuleSerializer(
         nested=True,
@@ -288,7 +296,7 @@ class FrontPortSerializer(NetBoxModelSerializer, CabledObjectSerializer):
     class Meta:
         model = FrontPort
         fields = [
-            'id', 'url', 'display_url', 'display', 'device', 'module', 'name', 'label', 'type', 'color', 'rear_port',
+            'id', 'url', 'display', 'device', 'module', 'name', 'label', 'type', 'color', 'rear_port',
             'rear_port_position', 'description', 'mark_connected', 'cable', 'cable_end', 'link_peers',
             'link_peers_type', 'tags', 'custom_fields', 'created', 'last_updated', '_occupied',
         ]
@@ -296,14 +304,8 @@ class FrontPortSerializer(NetBoxModelSerializer, CabledObjectSerializer):
 
 
 class ModuleBaySerializer(NetBoxModelSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:modulebay-detail')
     device = DeviceSerializer(nested=True)
-    module = ModuleSerializer(
-        nested=True,
-        fields=('id', 'url', 'display', 'module_bay'),
-        required=False,
-        allow_null=True,
-        default=None
-    )
     installed_module = ModuleSerializer(
         nested=True,
         fields=('id', 'url', 'display', 'serial', 'description'),
@@ -314,26 +316,28 @@ class ModuleBaySerializer(NetBoxModelSerializer):
     class Meta:
         model = ModuleBay
         fields = [
-            'id', 'url', 'display_url', 'display', 'device', 'module', 'name', 'installed_module', 'label', 'position',
-            'description', 'tags', 'custom_fields', 'created', 'last_updated',
+            'id', 'url', 'display', 'device', 'name', 'installed_module', 'label', 'position', 'description', 'tags',
+            'custom_fields', 'created', 'last_updated',
         ]
         brief_fields = ('id', 'url', 'display', 'installed_module', 'name', 'description')
 
 
 class DeviceBaySerializer(NetBoxModelSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:devicebay-detail')
     device = DeviceSerializer(nested=True)
     installed_device = DeviceSerializer(nested=True, required=False, allow_null=True)
 
     class Meta:
         model = DeviceBay
         fields = [
-            'id', 'url', 'display_url', 'display', 'device', 'name', 'label', 'description', 'installed_device',
-            'tags', 'custom_fields', 'created', 'last_updated',
+            'id', 'url', 'display', 'device', 'name', 'label', 'description', 'installed_device', 'tags',
+            'custom_fields', 'created', 'last_updated',
         ]
         brief_fields = ('id', 'url', 'display', 'device', 'name', 'description')
 
 
 class InventoryItemSerializer(NetBoxModelSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:inventoryitem-detail')
     device = DeviceSerializer(nested=True)
     parent = serializers.PrimaryKeyRelatedField(queryset=InventoryItem.objects.all(), allow_null=True, default=None)
     role = InventoryItemRoleSerializer(nested=True, required=False, allow_null=True)
@@ -349,9 +353,9 @@ class InventoryItemSerializer(NetBoxModelSerializer):
     class Meta:
         model = InventoryItem
         fields = [
-            'id', 'url', 'display_url', 'display', 'device', 'parent', 'name', 'label', 'role', 'manufacturer',
-            'part_id', 'serial', 'asset_tag', 'discovered', 'description', 'component_type', 'component_id',
-            'component', 'tags', 'custom_fields', 'created', 'last_updated', '_depth',
+            'id', 'url', 'display', 'device', 'parent', 'name', 'label', 'role', 'manufacturer', 'part_id', 'serial',
+            'asset_tag', 'discovered', 'description', 'component_type', 'component_id', 'component', 'tags',
+            'custom_fields', 'created', 'last_updated', '_depth',
         ]
         brief_fields = ('id', 'url', 'display', 'device', 'name', 'description', '_depth')
 

netbox/dcim/api/serializers_/devices.py

@@ -29,6 +29,7 @@ __all__ = (
 
 
 class DeviceSerializer(NetBoxModelSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:device-detail')
     device_type = DeviceTypeSerializer(nested=True)
     role = DeviceRoleSerializer(nested=True)
     tenant = TenantSerializer(
@@ -77,13 +78,13 @@ class DeviceSerializer(NetBoxModelSerializer):
     class Meta:
         model = Device
         fields = [
-            'id', 'url', 'display_url', 'display', 'name', 'device_type', 'role', 'tenant', 'platform', 'serial',
-            'asset_tag', 'site', 'location', 'rack', 'position', 'face', 'latitude', 'longitude', 'parent_device',
-            'status', 'airflow', 'primary_ip', 'primary_ip4', 'primary_ip6', 'oob_ip', 'cluster', 'virtual_chassis',
-            'vc_position', 'vc_priority', 'description', 'comments', 'config_template', 'local_context_data', 'tags',
-            'custom_fields', 'created', 'last_updated', 'console_port_count', 'console_server_port_count',
-            'power_port_count', 'power_outlet_count', 'interface_count', 'front_port_count', 'rear_port_count',
-            'device_bay_count', 'module_bay_count', 'inventory_item_count',
+            'id', 'url', 'display', 'name', 'device_type', 'role', 'tenant', 'platform', 'serial', 'asset_tag', 'site',
+            'location', 'rack', 'position', 'face', 'latitude', 'longitude', 'parent_device', 'status', 'airflow',
+            'primary_ip', 'primary_ip4', 'primary_ip6', 'oob_ip', 'cluster', 'virtual_chassis', 'vc_position',
+            'vc_priority', 'description', 'comments', 'config_template', 'local_context_data', 'tags', 'custom_fields',
+            'created', 'last_updated', 'console_port_count', 'console_server_port_count', 'power_port_count',
+            'power_outlet_count', 'interface_count', 'front_port_count', 'rear_port_count', 'device_bay_count',
+            'module_bay_count', 'inventory_item_count',
         ]
         brief_fields = ('id', 'url', 'display', 'name', 'description')
 
@@ -104,13 +105,13 @@ class DeviceWithConfigContextSerializer(DeviceSerializer):
 
     class Meta(DeviceSerializer.Meta):
         fields = [
-            'id', 'url', 'display_url', 'display', 'name', 'device_type', 'role', 'tenant', 'platform', 'serial',
-            'asset_tag', 'site', 'location', 'rack', 'position', 'face', 'latitude', 'longitude', 'parent_device',
-            'status', 'airflow', 'primary_ip', 'primary_ip4', 'primary_ip6', 'oob_ip', 'cluster', 'virtual_chassis',
-            'vc_position', 'vc_priority', 'description', 'comments', 'config_template', 'config_context',
-            'local_context_data', 'tags', 'custom_fields', 'created', 'last_updated', 'console_port_count',
-            'console_server_port_count', 'power_port_count', 'power_outlet_count', 'interface_count',
-            'front_port_count', 'rear_port_count', 'device_bay_count', 'module_bay_count', 'inventory_item_count',
+            'id', 'url', 'display', 'name', 'device_type', 'role', 'tenant', 'platform', 'serial', 'asset_tag', 'site',
+            'location', 'rack', 'position', 'face', 'latitude', 'longitude', 'parent_device', 'status', 'airflow',
+            'primary_ip', 'primary_ip4', 'primary_ip6', 'oob_ip', 'cluster', 'virtual_chassis', 'vc_position',
+            'vc_priority', 'description', 'comments', 'config_template', 'config_context', 'local_context_data', 'tags',
+            'custom_fields', 'created', 'last_updated', 'console_port_count', 'console_server_port_count',
+            'power_port_count', 'power_outlet_count', 'interface_count', 'front_port_count', 'rear_port_count',
+            'device_bay_count', 'module_bay_count', 'inventory_item_count',
         ]
 
     @extend_schema_field(serializers.JSONField(allow_null=True))
@@ -119,6 +120,7 @@ class DeviceWithConfigContextSerializer(DeviceSerializer):
 
 
 class VirtualDeviceContextSerializer(NetBoxModelSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:virtualdevicecontext-detail')
     device = DeviceSerializer(nested=True)
     identifier = serializers.IntegerField(allow_null=True, max_value=32767, min_value=0, required=False, default=None)
     tenant = TenantSerializer(nested=True, required=False, allow_null=True, default=None)
@@ -133,14 +135,15 @@ class VirtualDeviceContextSerializer(NetBoxModelSerializer):
     class Meta:
         model = VirtualDeviceContext
         fields = [
-            'id', 'url', 'display_url', 'display', 'name', 'device', 'identifier', 'tenant', 'primary_ip',
-            'primary_ip4', 'primary_ip6', 'status', 'description', 'comments', 'tags', 'custom_fields',
-            'created', 'last_updated', 'interface_count',
+            'id', 'url', 'display', 'name', 'device', 'identifier', 'tenant', 'primary_ip', 'primary_ip4',
+            'primary_ip6', 'status', 'description', 'comments', 'tags', 'custom_fields', 'created', 'last_updated',
+            'interface_count',
         ]
         brief_fields = ('id', 'url', 'display', 'name', 'identifier', 'device', 'description')
 
 
 class ModuleSerializer(NetBoxModelSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:module-detail')
     device = DeviceSerializer(nested=True)
     module_bay = NestedModuleBaySerializer()
     module_type = ModuleTypeSerializer(nested=True)
@@ -149,7 +152,7 @@ class ModuleSerializer(NetBoxModelSerializer):
     class Meta:
         model = Module
         fields = [
-            'id', 'url', 'display_url', 'display', 'device', 'module_bay', 'module_type', 'status', 'serial',
-            'asset_tag', 'description', 'comments', 'tags', 'custom_fields', 'created', 'last_updated',
+            'id', 'url', 'display', 'device', 'module_bay', 'module_type', 'status', 'serial', 'asset_tag',
+            'description', 'comments', 'tags', 'custom_fields', 'created', 'last_updated',
         ]
         brief_fields = ('id', 'url', 'display', 'device', 'module_bay', 'module_type', 'description')

netbox/dcim/api/serializers_/devicetype_components.py

@@ -32,6 +32,7 @@ __all__ = (
 
 
 class ConsolePortTemplateSerializer(ValidatedModelSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:consoleporttemplate-detail')
     device_type = DeviceTypeSerializer(
         nested=True,
         required=False,
@@ -53,13 +54,14 @@ class ConsolePortTemplateSerializer(ValidatedModelSerializer):
     class Meta:
         model = ConsolePortTemplate
         fields = [
-            'id', 'url', 'display', 'device_type', 'module_type', 'name', 'label', 'type',
-            'description', 'created', 'last_updated',
+            'id', 'url', 'display', 'device_type', 'module_type', 'name', 'label', 'type', 'description', 'created',
+            'last_updated',
         ]
         brief_fields = ('id', 'url', 'display', 'name', 'description')
 
 
 class ConsoleServerPortTemplateSerializer(ValidatedModelSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:consoleserverporttemplate-detail')
     device_type = DeviceTypeSerializer(
         nested=True,
         required=False,
@@ -81,13 +83,14 @@ class ConsoleServerPortTemplateSerializer(ValidatedModelSerializer):
     class Meta:
         model = ConsoleServerPortTemplate
         fields = [
-            'id', 'url', 'display', 'device_type', 'module_type', 'name', 'label', 'type',
-            'description', 'created', 'last_updated',
+            'id', 'url', 'display', 'device_type', 'module_type', 'name', 'label', 'type', 'description', 'created',
+            'last_updated',
         ]
         brief_fields = ('id', 'url', 'display', 'name', 'description')
 
 
 class PowerPortTemplateSerializer(ValidatedModelSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:powerporttemplate-detail')
     device_type = DeviceTypeSerializer(
         nested=True,
         required=False,
@@ -110,13 +113,14 @@ class PowerPortTemplateSerializer(ValidatedModelSerializer):
     class Meta:
         model = PowerPortTemplate
         fields = [
-            'id', 'url', 'display', 'device_type', 'module_type', 'name', 'label', 'type',
-            'maximum_draw', 'allocated_draw', 'description', 'created', 'last_updated',
+            'id', 'url', 'display', 'device_type', 'module_type', 'name', 'label', 'type', 'maximum_draw',
+            'allocated_draw', 'description', 'created', 'last_updated',
         ]
         brief_fields = ('id', 'url', 'display', 'name', 'description')
 
 
 class PowerOutletTemplateSerializer(ValidatedModelSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:poweroutlettemplate-detail')
     device_type = DeviceTypeSerializer(
         nested=True,
         required=False,
@@ -150,13 +154,14 @@ class PowerOutletTemplateSerializer(ValidatedModelSerializer):
     class Meta:
         model = PowerOutletTemplate
         fields = [
-            'id', 'url', 'display', 'device_type', 'module_type', 'name', 'label', 'type',
-            'power_port', 'feed_leg', 'description', 'created', 'last_updated',
+            'id', 'url', 'display', 'device_type', 'module_type', 'name', 'label', 'type', 'power_port', 'feed_leg',
+            'description', 'created', 'last_updated',
         ]
         brief_fields = ('id', 'url', 'display', 'name', 'description')
 
 
 class InterfaceTemplateSerializer(ValidatedModelSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:interfacetemplate-detail')
     device_type = DeviceTypeSerializer(
         nested=True,
         required=False,
@@ -196,13 +201,14 @@ class InterfaceTemplateSerializer(ValidatedModelSerializer):
     class Meta:
         model = InterfaceTemplate
         fields = [
-            'id', 'url', 'display', 'device_type', 'module_type', 'name', 'label', 'type', 'enabled',
-            'mgmt_only', 'description', 'bridge', 'poe_mode', 'poe_type', 'rf_role', 'created', 'last_updated',
+            'id', 'url', 'display', 'device_type', 'module_type', 'name', 'label', 'type', 'enabled', 'mgmt_only',
+            'description', 'bridge', 'poe_mode', 'poe_type', 'rf_role', 'created', 'last_updated',
         ]
         brief_fields = ('id', 'url', 'display', 'name', 'description')
 
 
 class RearPortTemplateSerializer(ValidatedModelSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:rearporttemplate-detail')
     device_type = DeviceTypeSerializer(
         required=False,
         nested=True,
@@ -220,13 +226,14 @@ class RearPortTemplateSerializer(ValidatedModelSerializer):
     class Meta:
         model = RearPortTemplate
         fields = [
-            'id', 'url', 'display', 'device_type', 'module_type', 'name', 'label', 'type', 'color',
-            'positions', 'description', 'created', 'last_updated',
+            'id', 'url', 'display', 'device_type', 'module_type', 'name', 'label', 'type', 'color', 'positions',
+            'description', 'created', 'last_updated',
         ]
         brief_fields = ('id', 'url', 'display', 'name', 'description')
 
 
 class FrontPortTemplateSerializer(ValidatedModelSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:frontporttemplate-detail')
     device_type = DeviceTypeSerializer(
         nested=True,
         required=False,
@@ -245,50 +252,41 @@ class FrontPortTemplateSerializer(ValidatedModelSerializer):
     class Meta:
         model = FrontPortTemplate
         fields = [
-            'id', 'url', 'display', 'device_type', 'module_type', 'name', 'label', 'type', 'color',
-            'rear_port', 'rear_port_position', 'description', 'created', 'last_updated',
+            'id', 'url', 'display', 'device_type', 'module_type', 'name', 'label', 'type', 'color', 'rear_port',
+            'rear_port_position', 'description', 'created', 'last_updated',
         ]
         brief_fields = ('id', 'url', 'display', 'name', 'description')
 
 
 class ModuleBayTemplateSerializer(ValidatedModelSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:modulebaytemplate-detail')
     device_type = DeviceTypeSerializer(
-        nested=True,
-        required=False,
-        allow_null=True,
-        default=None
-    )
-    module_type = ModuleTypeSerializer(
-        nested=True,
-        required=False,
-        allow_null=True,
-        default=None
+        nested=True
     )
 
     class Meta:
         model = ModuleBayTemplate
         fields = [
-            'id', 'url', 'display', 'device_type', 'module_type', 'name', 'label', 'position', 'description',
-            'created', 'last_updated',
+            'id', 'url', 'display', 'device_type', 'name', 'label', 'position', 'description', 'created',
+            'last_updated',
         ]
         brief_fields = ('id', 'url', 'display', 'name', 'description')
 
 
 class DeviceBayTemplateSerializer(ValidatedModelSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:devicebaytemplate-detail')
     device_type = DeviceTypeSerializer(
         nested=True
     )
 
     class Meta:
         model = DeviceBayTemplate
-        fields = [
-            'id', 'url', 'display', 'device_type', 'name', 'label', 'description',
-            'created', 'last_updated'
-        ]
+        fields = ['id', 'url', 'display', 'device_type', 'name', 'label', 'description', 'created', 'last_updated']
         brief_fields = ('id', 'url', 'display', 'name', 'description')
 
 
 class InventoryItemTemplateSerializer(ValidatedModelSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:inventoryitemtemplate-detail')
     device_type = DeviceTypeSerializer(
         nested=True
     )
@@ -315,9 +313,8 @@ class InventoryItemTemplateSerializer(ValidatedModelSerializer):
     class Meta:
         model = InventoryItemTemplate
         fields = [
-            'id', 'url', 'display', 'device_type', 'parent', 'name', 'label', 'role', 'manufacturer',
-            'part_id', 'description', 'component_type', 'component_id', 'component', 'created', 'last_updated',
-            '_depth',
+            'id', 'url', 'display', 'device_type', 'parent', 'name', 'label', 'role', 'manufacturer', 'part_id',
+            'description', 'component_type', 'component_id', 'component', 'created', 'last_updated', '_depth',
         ]
         brief_fields = ('id', 'url', 'display', 'name', 'description', '_depth')
 

netbox/dcim/api/serializers_/devicetypes.py

@@ -17,6 +17,7 @@ __all__ = (
 
 
 class DeviceTypeSerializer(NetBoxModelSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:devicetype-detail')
     manufacturer = ManufacturerSerializer(nested=True)
     default_platform = PlatformSerializer(nested=True, required=False, allow_null=True)
     u_height = serializers.DecimalField(
@@ -50,39 +51,26 @@ class DeviceTypeSerializer(NetBoxModelSerializer):
     class Meta:
         model = DeviceType
         fields = [
-            'id', 'url', 'display_url', 'display', 'manufacturer', 'default_platform', 'model', 'slug', 'part_number',
-            'u_height', 'exclude_from_utilization', 'is_full_depth', 'subdevice_role', 'airflow', 'weight',
-            'weight_unit', 'front_image', 'rear_image', 'description', 'comments', 'tags', 'custom_fields',
-            'created', 'last_updated', 'device_count', 'console_port_template_count',
-            'console_server_port_template_count', 'power_port_template_count', 'power_outlet_template_count',
-            'interface_template_count', 'front_port_template_count', 'rear_port_template_count',
-            'device_bay_template_count', 'module_bay_template_count', 'inventory_item_template_count',
+            'id', 'url', 'display', 'manufacturer', 'default_platform', 'model', 'slug', 'part_number', 'u_height',
+            'exclude_from_utilization', 'is_full_depth', 'subdevice_role', 'airflow', 'weight', 'weight_unit',
+            'front_image', 'rear_image', 'description', 'comments', 'tags', 'custom_fields', 'created', 'last_updated',
+            'device_count', 'console_port_template_count', 'console_server_port_template_count',
+            'power_port_template_count', 'power_outlet_template_count', 'interface_template_count',
+            'front_port_template_count', 'rear_port_template_count', 'device_bay_template_count',
+            'module_bay_template_count', 'inventory_item_template_count',
         ]
         brief_fields = ('id', 'url', 'display', 'manufacturer', 'model', 'slug', 'description', 'device_count')
 
 
 class ModuleTypeSerializer(NetBoxModelSerializer):
-    manufacturer = ManufacturerSerializer(
-        nested=True
-    )
-    weight_unit = ChoiceField(
-        choices=WeightUnitChoices,
-        allow_blank=True,
-        required=False,
-        allow_null=True
-    )
-    airflow = ChoiceField(
-        choices=ModuleAirflowChoices,
-        allow_blank=True,
-        required=False,
-        allow_null=True
-    )
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:moduletype-detail')
+    manufacturer = ManufacturerSerializer(nested=True)
+    weight_unit = ChoiceField(choices=WeightUnitChoices, allow_blank=True, required=False, allow_null=True)
 
     class Meta:
         model = ModuleType
         fields = [
-            'id', 'url', 'display_url', 'display', 'manufacturer', 'model', 'part_number', 'airflow',
-            'weight', 'weight_unit', 'description', 'comments', 'tags', 'custom_fields',
-            'created', 'last_updated',
+            'id', 'url', 'display', 'manufacturer', 'model', 'part_number', 'weight', 'weight_unit', 'description',
+            'comments', 'tags', 'custom_fields', 'created', 'last_updated',
         ]
         brief_fields = ('id', 'url', 'display', 'manufacturer', 'model', 'description')

netbox/dcim/api/serializers_/manufacturers.py

@@ -10,6 +10,7 @@ __all__ = (
 
 
 class ManufacturerSerializer(NetBoxModelSerializer):
+    url = serializers.HyperlinkedIdentityField(view_name='dcim-api:manufacturer-detail')
 
     # Related object counts
     devicetype_count = RelatedObjectCountField('device_types')
@@ -19,7 +20,7 @@ class ManufacturerSerializer(NetBoxModelSerializer):
     class Meta:
         model = Manufacturer
         fields = [
-            'id', 'url', 'display_url', 'display', 'name', 'slug', 'description', 'tags', 'custom_fields',
-            'created', 'last_updated', 'devicetype_count', 'inventoryitem_count', 'platform_count',
+            'id', 'url', 'display', 'name', 'slug', 'description', 'tags', 'custom_fields', 'created', 'last_updated',
+            'devicetype_count', 'inventoryitem_count', 'platform_count',
         ]
         brief_fields = ('id', 'url', 'display', 'name', 'slug', 'description', 'devicetype_count')