Closes #21157: Add public models to export template context

Move shared get_context() logic from ConfigTemplate into
RenderTemplateMixin so ExportTemplate also gets access to all
public model classes. This enables export templates to perform
cross-model lookups (e.g. resolving parent Prefix from IPAddress).

.github/ISSUE_TEMPLATE/01-feature_request.yaml

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

.github/ISSUE_TEMPLATE/02-bug_report.yaml

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

.github/ISSUE_TEMPLATE/03-performance.yaml

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

.github/workflows/ci.yml

@@ -53,22 +53,22 @@ jobs:
 
     steps:
     - name: Check out repo
-      uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
+      uses: actions/checkout@v4
 
     - name: Check Python linting & PEP8 compliance
-      uses: astral-sh/ruff-action@0ce1b0bf8b818ef400413f810f8a11cdbda0034b # v4.0.0
+      uses: astral-sh/ruff-action@4919ec5cf1f49eff0871dbcea0da843445b837e6 # v3.6.1
       with:
-        version: "0.15.10"
+        version: "0.15.2"
         args: "check --output-format=github"
         src: "netbox/"
 
     - name: Set up Python ${{ matrix.python-version }}
-      uses: actions/setup-python@a309ff8b426b58ec0e2a45f0f869d46889d02405 # v6.2.0
+      uses: actions/setup-python@v5
       with:
         python-version: ${{ matrix.python-version }}
 
     - name: Use Node.js ${{ matrix.node-version }}
-      uses: actions/setup-node@53b83947a5a98c8d113130e565377fae1a50d02f # v6.3.0
+      uses: actions/setup-node@v4
       with:
         node-version: ${{ matrix.node-version }}
     
@@ -76,7 +76,7 @@ jobs:
       run: npm install -g yarn
     
     - name: Setup Node.js with Yarn Caching
-      uses: actions/setup-node@53b83947a5a98c8d113130e565377fae1a50d02f # v6.3.0
+      uses: actions/setup-node@v4
       with:
         node-version: ${{ matrix.node-version }}
         cache: yarn
@@ -92,7 +92,7 @@ jobs:
         pip install coverage tblib
 
     - name: Build documentation
-      run: zensical build
+      run: mkdocs build
 
     - name: Collect static files
       run: python netbox/manage.py collectstatic --no-input

.github/workflows/claude-code-review.yml

@@ -21,7 +21,7 @@ jobs:
 
     steps:
       - name: Checkout repository
-        uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
+        uses: actions/checkout@v4
         with:
           fetch-depth: 1
 

.github/workflows/claude.yml

@@ -26,38 +26,20 @@ jobs:
       actions: read # Required for Claude to read CI results on PRs
     steps:
       - name: Checkout repository
-        uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
+        uses: actions/checkout@v4
         with:
           fetch-depth: 1
 
-      # Workaround for claude-code-action bug with fork PRs: The action fetches by branch name
-      # (git fetch origin --depth=N <branch>), but fork PR branches don't exist on origin.
-      # Fix: redirect origin to the fork's URL so the action can fetch the branch directly.
-      - name: Configure git remote for fork PRs
+      # Workaround for claude-code-action bug with fork PRs: The action tries to fetch by branch name, which doesn't
+      # exist on origin for forks. Pre-fetch the PR ref so it's available as a local ref.
+      - name: Fetch fork PR ref (if applicable)
+        if: github.event.issue.pull_request != '' && github.event.issue.pull_request != null
         env:
           GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
         run: |
-          # Determine PR number based on event type
-          if [ "${{ github.event_name }}" = "issue_comment" ]; then
-            PR_NUMBER="${{ github.event.issue.number }}"
-          elif [ "${{ github.event_name }}" = "pull_request_review_comment" ] || [ "${{ github.event_name }}" = "pull_request_review" ]; then
-            PR_NUMBER="${{ github.event.pull_request.number }}"
-          else
-            exit 0  # issues event — no PR branch to worry about
-          fi
-
-          # Fetch fork info in one API call; silently skip if this is not a PR
-          PR_INFO=$(gh pr view "${PR_NUMBER}" --json isCrossRepository,headRepositoryOwner,headRepository 2>/dev/null || echo "")
-          if [ -z "$PR_INFO" ]; then
-            exit 0
-          fi
-
-          IS_FORK=$(echo "$PR_INFO" | jq -r '.isCrossRepository')
-          if [ "$IS_FORK" = "true" ]; then
-            FORK_OWNER=$(echo "$PR_INFO" | jq -r '.headRepositoryOwner.login')
-            FORK_REPO=$(echo "$PR_INFO" | jq -r '.headRepository.name')
-            echo "Fork PR detected from ${FORK_OWNER}/${FORK_REPO}: updating origin to fork URL"
-            git remote set-url origin "https://github.com/${FORK_OWNER}/${FORK_REPO}.git"
+          PR_NUMBER=$(gh pr view ${{ github.event.issue.number }} --json number -q .number 2>/dev/null || echo "")
+          if [ -n "$PR_NUMBER" ]; then
+            git fetch origin refs/pull/${PR_NUMBER}/head:refs/remotes/pull/${PR_NUMBER}/head || true
           fi
 
       - name: Run Claude Code

.github/workflows/close-incomplete-issues.yml

@@ -15,7 +15,7 @@ jobs:
     if: github.repository == 'netbox-community/netbox'
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/stale@b5d41d4e1d5dceea10e7104786b73624c18a190f # v10.2.0
+      - uses: actions/stale@v9
         with:
           close-issue-message: >
             This issue is being closed as no further information has been provided. If

.github/workflows/close-stale-issues.yml

@@ -16,7 +16,7 @@ jobs:
     if: github.repository == 'netbox-community/netbox'
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/stale@b5d41d4e1d5dceea10e7104786b73624c18a190f # v10.2.0
+      - uses: actions/stale@v9
         with:
           # General parameters
           operations-per-run: 200

.github/workflows/codeql.yml

@@ -27,16 +27,16 @@ jobs:
           build-mode: none
     steps:
     - name: Checkout repository
-      uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
+      uses: actions/checkout@v4
 
     - name: Initialize CodeQL
-      uses: github/codeql-action/init@b1bff81932f5cdfc8695c7752dcee935dcd061c8 # v4.33.0
+      uses: github/codeql-action/init@v4
       with:
         languages: ${{ matrix.language }}
         build-mode: ${{ matrix.build-mode }}
         config-file: .github/codeql/codeql-config.yml
 
     - name: Perform CodeQL Analysis
-      uses: github/codeql-action/analyze@b1bff81932f5cdfc8695c7752dcee935dcd061c8 # v4.33.0
+      uses: github/codeql-action/analyze@v4
       with:
         category: "/language:${{matrix.language}}"

.github/workflows/lock-threads.yml

@@ -11,14 +11,14 @@ permissions:
   pull-requests: write
   discussions: write
 
-concurrency:
-  group: lock-threads
-
 jobs:
   lock:
     if: github.repository == 'netbox-community/netbox'
     runs-on: ubuntu-latest
     steps:
-      - uses: dessant/lock-threads@7266a7ce5c1df01b1c6db85bf8cd86c737dadbe7 # v6.0.0
+      - uses: dessant/lock-threads@1bf7ec25051fe7c00bdd17e6a7cf3d7bfb7dc771 # v5.0.1
         with:
+          issue-inactive-days: 90
+          pr-inactive-days: 30
           discussion-inactive-days: 180
+          issue-lock-reason: 'resolved'

.github/workflows/update-translation-strings.yml

@@ -20,19 +20,19 @@ jobs:
 
     steps:
     - name: Create app token
-      uses: actions/create-github-app-token@1b10c78c7865c340bc4f6099eb2f838309f1e8c3 # v3.1.1
+      uses: actions/create-github-app-token@v1
       id: app-token
       with:
         app-id: 1076524
         private-key: ${{ secrets.HOUSEKEEPING_SECRET_KEY }}
 
     - name: Check out repo
-      uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
+      uses: actions/checkout@v4
       with:
           token: ${{ steps.app-token.outputs.token }}
 
     - name: Set up Python
-      uses: actions/setup-python@a309ff8b426b58ec0e2a45f0f869d46889d02405 # v6.2.0
+      uses: actions/setup-python@v5
       with:
         python-version: 3.12
 
@@ -48,7 +48,7 @@ jobs:
       run: python netbox/manage.py makemessages -l ${{ env.LOCALE }}
 
     - name: Commit changes
-      uses: EndBug/add-and-commit@290ea2c423ad77ca9c62ae0f5b224379612c0321 # v10.0.0
+      uses: EndBug/add-and-commit@a94899bca583c204427a224a7af87c02f9b325d5 # v9.1.4
       with:
         add: 'netbox/translations/'
         default_author: github_actions

.pre-commit-config.yaml

@@ -21,11 +21,11 @@ repos:
       language: system
       pass_filenames: false
       types: [python]
-    - id: zensical-build
+    - id: mkdocs-build
       name: "Build documentation"
-      description: "Build the documentation with Zensical"
+      description: "Build the documentation with mkdocs"
       files: 'docs/'
-      entry: zensical build
+      entry: mkdocs build
       language: system
       pass_filenames: false
     - id: yarn-validate

.readthedocs.yaml

@@ -1,10 +1,10 @@
 version: 2
 build:
-  os: ubuntu-24.04
+  os: ubuntu-22.04
   tools:
     python: "3.12"
-  commands:
-    - pip install -r requirements.txt
-    - python -m zensical build --config-file mkdocs.yml
-    - mkdir -p $READTHEDOCS_OUTPUT/html/
-    - cp -r netbox/project-static/docs/* $READTHEDOCS_OUTPUT/html/
+mkdocs:
+  configuration: mkdocs.yml
+python:
+   install:
+   - requirements: requirements.txt

CLAUDE.md

@@ -54,8 +54,7 @@ python manage.py nbshell   # NetBox-enhanced shell
 
 ## Architecture Conventions
 - **Apps**: Each Django app owns its models, views, API serializers, filtersets, forms, and tests.
-- **Views**: Use `register_model_view()` to register model views by action (e.g. "add", "list", etc.). List views typically don't need to add `select_related()` or `prefetch_related()` on their querysets: Prefetching is handled dynamically by the table class so that only relevant fields are prefetched.
-- **REST API**: DRF serializers live in `<app>/api/serializers.py`; viewsets in `<app>/api/views.py`; URLs auto-registered in `<app>/api/urls.py`. REST API views typically don't need to add `select_related()` or `prefetch_related()` on their querysets: Prefetching is handled dynamically by the serializer so that only relevant fields are prefetched.
+- **REST API**: DRF serializers live in `<app>/api/serializers.py`; viewsets in `<app>/api/views.py`; URLs auto-registered in `<app>/api/urls.py`.
 - **GraphQL**: Strawberry types in `<app>/graphql/types.py`.
 - **Filtersets**: `<app>/filtersets.py` — used for both UI filtering and API `?filter=` params.
 - **Tables**: `django-tables2` used for all object list views (`<app>/tables.py`).
@@ -69,8 +68,6 @@ python manage.py nbshell   # NetBox-enhanced shell
 - API serializers must include a `url` field (absolute URL of the object).
 - Use `FeatureQuery` for generic relations (config contexts, custom fields, tags, etc.).
 - Avoid adding new dependencies without strong justification.
-- Avoid running `ruff format` on existing files, as this tends to introduce unnecessary style changes.
-- Don't craft Django database migrations manually: Prompt the user to run `manage.py makemigrations` instead.
 
 ## Branch & PR Conventions
 - Branch naming: `<issue-number>-short-description` (e.g., `1234-device-typerror`)

base_requirements.txt

@@ -4,7 +4,7 @@ colorama
 
 # The Python web framework on which NetBox is built
 # https://docs.djangoproject.com/en/stable/releases/
-Django==6.0.*
+Django==5.2.*
 
 # Django middleware which permits cross-domain API requests
 # https://github.com/adamchainz/django-cors-headers/blob/main/CHANGELOG.rst
@@ -35,9 +35,7 @@ django-pglocks
 
 # Prometheus metrics library for Django
 # https://github.com/korfuri/django-prometheus/blob/master/CHANGELOG.md
-# TODO: 2.4.1 is incompatible with Django>=6.0, but a fixed release is expected
-# https://github.com/django-commons/django-prometheus/issues/494
-django-prometheus>=2.4.0,<2.5.0,!=2.4.1
+django-prometheus
 
 # Django caching backend using Redis
 # https://github.com/jazzband/django-redis/blob/master/CHANGELOG.rst
@@ -57,8 +55,7 @@ django-storages
 
 # Abstraction models for rendering and paginating HTML tables
 # https://github.com/jieter/django-tables2/blob/master/CHANGELOG.md
-# See #21902 for upgrading to django-tables2 v2.9+
-django-tables2<2.9
+django-tables2
 
 # User-defined tags for objects
 # https://github.com/jazzband/django-taggit/blob/master/CHANGELOG.rst
@@ -71,7 +68,7 @@ django-timezone-field
 # A REST API framework for Django projects
 # https://www.django-rest-framework.org/community/release-notes/
 # TODO: Re-evaluate the monkey-patch of get_unique_validators() before upgrading
-djangorestframework==3.17.1
+djangorestframework==3.16.1
 
 # Sane and flexible OpenAPI 3 schema generation for Django REST framework.
 # https://github.com/tfranzel/drf-spectacular/blob/master/CHANGELOG.rst
@@ -101,6 +98,10 @@ jsonschema
 # https://python-markdown.github.io/changelog/
 Markdown
 
+# MkDocs
+# https://github.com/mkdocs/mkdocs/releases
+mkdocs<2.0
+
 # MkDocs Material theme (for documentation build)
 # https://squidfunk.github.io/mkdocs-material/changelog/
 mkdocs-material
@@ -174,7 +175,3 @@ tablib
 # Timezone data (required by django-timezone-field on Python 3.9+)
 # https://github.com/python/tzdata/blob/master/NEWS.md
 tzdata
-
-# Documentation builder (succeeds mkdocs)
-# https://github.com/zensical/zensical
-zensical

contrib/generated_schema.json

@@ -416,13 +416,9 @@
                         "800gbase-dr8",
                         "800gbase-sr8",
                         "800gbase-vr8",
-                        "1.6tbase-cr8",
-                        "1.6tbase-dr8",
-                        "1.6tbase-dr8-2",
                         "100base-x-sfp",
                         "1000base-x-gbic",
                         "1000base-x-sfp",
-                        "2.5gbase-x-sfp",
                         "10gbase-x-sfpp",
                         "10gbase-x-xenpak",
                         "10gbase-x-xfp",
@@ -452,9 +448,6 @@
                         "400gbase-x-osfp-rhs",
                         "800gbase-x-osfp",
                         "800gbase-x-qsfpdd",
-                        "1.6tbase-x-osfp1600",
-                        "1.6tbase-x-osfp1600-rhs",
-                        "1.6tbase-x-qsfpdd1600",
                         "1000base-kx",
                         "2.5gbase-kx",
                         "5gbase-kr",
@@ -466,7 +459,6 @@
                         "100gbase-kp4",
                         "100gbase-kr2",
                         "100gbase-kr4",
-                        "1.6tbase-kr8",
                         "ieee802.11a",
                         "ieee802.11g",
                         "ieee802.11n",

docs/_theme/partials/copyright.html

@@ -0,0 +1,18 @@
+<div class="md-copyright">
+  {% if config.copyright %}
+    <div class="md-copyright__highlight">
+      {{ config.copyright }}
+    </div>
+  {% endif %}
+  {% if not config.extra.generator == false %}
+    Made with
+    <a href="https://squidfunk.github.io/mkdocs-material/" target="_blank" rel="noopener">
+      Material for MkDocs
+    </a>
+  {% endif %}
+</div>
+{% if not config.extra.build_public %}
+  <div class="md-copyright">
+    ℹ️ Documentation is being served locally
+  </div>
+{% endif %}

docs/administration/permissions.md

@@ -20,9 +20,7 @@ There are four core actions that can be permitted for each type of object within
 * **Change** - Modify an existing object
 * **Delete** - Delete an existing object
 
-In addition to these, permissions can also grant custom actions that may be required by a specific model or plugin. For example, the `sync` action for data sources allows a user to synchronize data from a remote source, and the `render_config` action for devices and virtual machines allows rendering configuration templates.
-
-Some models have registered actions that appear as checkboxes in the "Actions" section when creating or editing a permission. These are shown in a flat list alongside the built-in CRUD actions. Additional actions (such as those not yet registered by a plugin, or for backwards compatibility) can be entered manually in the "Additional actions" field.
+In addition to these, permissions can also grant custom actions that may be required by a specific model or plugin. For example, the `run` permission for scripts allows a user to execute custom scripts. These can be specified when granting a permission in the "additional actions" field.
 
 !!! note
     Internally, all actions granted by a permission (both built-in and custom) are stored as strings in an array field named `actions`.

docs/configuration/development.md

@@ -4,9 +4,9 @@
 
 Default: `False`
 
-This setting enables debugging and displays a debugging toolbar in the user interface. Debugging should be enabled only during development or troubleshooting.
-
-Note that the debugging toolbar will be displayed only for requests originating from [internal IP addresses](./system.md#internal_ips), if defined. If no internal IPs are defined, the toolbar will be displayed for all requests.
+This setting enables debugging. Debugging should be enabled only during development or troubleshooting. Note that only
+clients which access NetBox from a recognized [internal IP address](./system.md#internal_ips) will see debugging tools in the user
+interface.
 
 !!! warning
     Never enable debugging on a production system, as it can expose sensitive data to unauthenticated users and impose a

docs/configuration/index.md

@@ -21,7 +21,6 @@ Some configuration parameters are primarily controlled via NetBox's admin interf
 * [`BANNER_BOTTOM`](./miscellaneous.md#banner_bottom)
 * [`BANNER_LOGIN`](./miscellaneous.md#banner_login)
 * [`BANNER_TOP`](./miscellaneous.md#banner_top)
-* [`CHANGELOG_RETAIN_CREATE_LAST_UPDATE`](./miscellaneous.md#changelog_retain_create_last_update)
 * [`CHANGELOG_RETENTION`](./miscellaneous.md#changelog_retention)
 * [`CUSTOM_VALIDATORS`](./data-validation.md#custom_validators)
 * [`DEFAULT_USER_PREFERENCES`](./default-values.md#default_user_preferences)

docs/configuration/miscellaneous.md

@@ -73,23 +73,6 @@ This data enables the project maintainers to estimate how many NetBox deployment
 
 ---
 
-## CHANGELOG_RETAIN_CREATE_LAST_UPDATE
-
-!!! tip "Dynamic Configuration Parameter"
-
-Default: `False`
-
-When pruning expired changelog entries (per `CHANGELOG_RETENTION`), retain each non-deleted object's original `create`
-change record and its most recent `update` change record. If an object has a `delete` change record, its changelog
-entries are pruned normally according to `CHANGELOG_RETENTION`.
-
-!!! note
-    For objects without a `delete` change record, the original `create` record and most recent `update` record are
-    exempt from pruning. All other changelog records (including intermediate `update` records and all `delete` records)
-    remain subject to pruning per `CHANGELOG_RETENTION`.
-
----
-
 ## CHANGELOG_RETENTION
 
 !!! tip "Dynamic Configuration Parameter"
@@ -237,14 +220,6 @@ This parameter defines the URL of the repository that will be checked for new Ne
 
 ---
 
-## RQ
-
-Default: `{}` (Empty)
-
-This is a wrapper for passing global configuration parameters to [Django RQ](https://github.com/rq/django-rq) to customize its behavior. It is employed within NetBox primarily to alter conditions during testing.
-
----
-
 ## RQ_DEFAULT_TIMEOUT
 
 Default: `300`

docs/configuration/system.md

@@ -105,13 +105,6 @@ A list of IP addresses recognized as internal to the system, used to control the
 example, the debugging toolbar will be viewable only when a client is accessing NetBox from one of the listed IP
 addresses (and [`DEBUG`](./development.md#debug) is `True`).
 
-!!! info "New in NetBox v4.6"
-    Setting this parameter to an empty list will enable the toolbar for all requests provided debugging is enabled:
-
-    ```python
-    INTERNAL_IPS = []
-    ```
-
 ---
 
 ## ISOLATED_DEPLOYMENT
@@ -248,49 +241,21 @@ STORAGES = {
 
 Within the `STORAGES` dictionary, `"default"` is used for image uploads, "staticfiles" is for static files and `"scripts"` is used for custom scripts.
 
-If using a remote storage such as S3 or an S3-compatible service, define the configuration as `STORAGES[key]["OPTIONS"]` for each storage item as needed. For example:
+If using a remote storage like S3, define the config as `STORAGES[key]["OPTIONS"]` for each storage item as needed. For example:
 
 ```python
-STORAGES = {
-    'default': {
-        'BACKEND': 'storages.backends.s3.S3Storage',
-        'OPTIONS': {
-            'bucket_name': 'netbox',
-            'access_key': 'access key',
+STORAGES = { 
+    "scripts": { 
+        "BACKEND": "storages.backends.s3boto3.S3Boto3Storage", 
+        "OPTIONS": { 
+            'access_key': 'access key', 
             'secret_key': 'secret key',
-            'region_name': 'us-east-1',
-            'endpoint_url': 'https://s3.example.com',
-            'location': 'media/',
-        },
-    },
-    'staticfiles': {
-        'BACKEND': 'storages.backends.s3.S3Storage',
-        'OPTIONS': {
-            'bucket_name': 'netbox',
-            'access_key': 'access key',
-            'secret_key': 'secret key',
-            'region_name': 'us-east-1',
-            'endpoint_url': 'https://s3.example.com',
-            'location': 'static/',
-        },
-    },
-    'scripts': {
-        'BACKEND': 'storages.backends.s3.S3Storage',
-        'OPTIONS': {
-            'bucket_name': 'netbox',
-            'access_key': 'access key',
-            'secret_key': 'secret key',
-            'region_name': 'us-east-1',
-            'endpoint_url': 'https://s3.example.com',
-            'location': 'scripts/',
-            'file_overwrite': True,
-        },
-    },
+            "allow_overwrite": True,
+        }
+    }, 
 }
 ```
 
-`bucket_name` is required for `S3Storage`. When using an S3-compatible service, set `region_name` and `endpoint_url` according to your provider.
-
 The specific configuration settings for each storage backend can be found in the [django-storages documentation](https://django-storages.readthedocs.io/en/latest/index.html).
 
 !!! note
@@ -314,7 +279,6 @@ STORAGES = {
             'bucket_name': os.environ.get('AWS_STORAGE_BUCKET_NAME'),
             'access_key': os.environ.get('AWS_S3_ACCESS_KEY_ID'),
             'secret_key': os.environ.get('AWS_S3_SECRET_ACCESS_KEY'),
-            'region_name': os.environ.get('AWS_S3_REGION_NAME'),
             'endpoint_url': os.environ.get('AWS_S3_ENDPOINT_URL'),
             'location': 'media/',
         }
@@ -325,7 +289,6 @@ STORAGES = {
             'bucket_name': os.environ.get('AWS_STORAGE_BUCKET_NAME'),
             'access_key': os.environ.get('AWS_S3_ACCESS_KEY_ID'),
             'secret_key': os.environ.get('AWS_S3_SECRET_ACCESS_KEY'),
-            'region_name': os.environ.get('AWS_S3_REGION_NAME'),
             'endpoint_url': os.environ.get('AWS_S3_ENDPOINT_URL'),
             'location': 'static/',
         }

docs/customization/custom-fields.md

@@ -63,7 +63,6 @@ NetBox supports limited custom validation for custom field values. Following are
 * Text: Regular expression (optional)
 * Integer: Minimum and/or maximum value (optional)
 * Selection: Must exactly match one of the prescribed choices
-* JSON: Must adhere to the defined validation schema (if any)
 
 ### Custom Selection Fields
 

docs/customization/custom-scripts.md

@@ -215,7 +215,6 @@ if obj.pk and hasattr(obj, 'snapshot'):
     obj.snapshot()
 
 obj.property = "New Value"
-obj._changelog_message = 'Example Message Text' # Optional
 obj.full_clean()
 obj.save()
 ```
@@ -384,18 +383,6 @@ A calendar date. Returns a `datetime.date` object.
 
 A complete date & time. Returns a `datetime.datetime` object.
 
-## Uploading Scripts via the API
-
-Script modules can be uploaded to NetBox via the REST API by sending a `multipart/form-data` POST request to `/api/extras/scripts/upload/`. The caller must have the `extras.add_scriptmodule` and `core.add_managedfile` permissions.
-
-```no-highlight
-curl -X POST \
--H "Authorization: Token $TOKEN" \
--H "Accept: application/json; indent=4" \
--F "file=@/path/to/myscript.py" \
-http://netbox/api/extras/scripts/upload/
-```
-
 ## Running Custom Scripts
 
 !!! note

docs/development/application-registry.md

@@ -32,9 +32,6 @@ Core model features are listed in the [features matrix](./models.md#features-mat
 
 ### `models`
 
-!!! warning "Deprecated"
-    Usage of this key has been deprecated and will be removed in NetBox v4.7. Use `ObjectType.objects.public()` to find registered models.
-
 This key lists all models which have been registered in NetBox which are not designated for private use. (Setting `_netbox_private` to True on a model excludes it from this list.) As with individual features under `model_features`, models are organized by app label.
 
 ### `plugins`

docs/development/getting-started.md

@@ -97,7 +97,7 @@ NetBox uses [`pre-commit`](https://pre-commit.com/) to automatically validate co
 * Run the `ruff` Python linter
 * Run Django's internal system check
 * Check for missing database migrations
-* Validate any changes to the documentation with `zensical`
+* Validate any changes to the documentation with `mkdocs`
 * Validate Typescript & Sass styling with `yarn`
 * Ensure that any modified static front end assets have been recompiled
 

docs/development/models.md

@@ -45,7 +45,6 @@ These are considered the "core" application models which are used to model netwo
 * [core.DataSource](../models/core/datasource.md)
 * [core.Job](../models/core/job.md)
 * [dcim.Cable](../models/dcim/cable.md)
-* [dcim.CableBundle](../models/dcim/cablebundle.md)
 * [dcim.Device](../models/dcim/device.md)
 * [dcim.DeviceType](../models/dcim/devicetype.md)
 * [dcim.Module](../models/dcim/module.md)
@@ -74,7 +73,6 @@ These are considered the "core" application models which are used to model netwo
 * [tenancy.Tenant](../models/tenancy/tenant.md)
 * [virtualization.Cluster](../models/virtualization/cluster.md)
 * [virtualization.VirtualMachine](../models/virtualization/virtualmachine.md)
-* [virtualization.VirtualMachineType](../models/virtualization/virtualmachinetype.md)
 * [vpn.IKEPolicy](../models/vpn/ikepolicy.md)
 * [vpn.IKEProposal](../models/vpn/ikeproposal.md)
 * [vpn.IPSecPolicy](../models/vpn/ipsecpolicy.md)
@@ -94,7 +92,6 @@ Organization models are used to organize and classify primary models.
 * [dcim.DeviceRole](../models/dcim/devicerole.md)
 * [dcim.Manufacturer](../models/dcim/manufacturer.md)
 * [dcim.Platform](../models/dcim/platform.md)
-* [dcim.RackGroup](../models/dcim/rackgroup.md)
 * [dcim.RackRole](../models/dcim/rackrole.md)
 * [ipam.ASNRange](../models/ipam/asnrange.md)
 * [ipam.RIR](../models/ipam/rir.md)

docs/development/release-checklist.md

@@ -47,7 +47,7 @@ If a new Django release is adopted or other major dependencies (Python, PostgreS
 Start the documentation server and navigate to the current version of the installation docs:
 
 ```no-highlight
-zensical serve
+mkdocs serve
 ```
 
 Follow these instructions to perform a new installation of NetBox in a temporary environment. This process must not be automated: The goal of this step is to catch any errors or omissions in the documentation and ensure that it is kept up to date for each release. Make any necessary changes to the documentation before proceeding with the release.

docs/extra.css

@@ -5,6 +5,10 @@ img {
     margin-right: auto;
 }
 
+.md-content img {
+    background-color: rgba(255, 255, 255, 0.64);
+}
+
 /* Tables */
 table {
     margin-bottom: 24px;

docs/features/virtualization.md

@@ -1,44 +1,26 @@
 # Virtualization
 
-Virtual machines, clusters, and standalone hypervisors can be modeled in NetBox alongside physical infrastructure. IP addresses and other resources are assigned to these objects just like physical objects, providing a seamless integration between physical and virtual networks.
+Virtual machines and clusters can be modeled in NetBox alongside physical infrastructure. IP addresses and other resources are assigned to these objects just like physical objects, providing a seamless integration between physical and virtual networks.
 
 ```mermaid
 flowchart TD
     ClusterGroup & ClusterType --> Cluster
-    VirtualMachineType --> VirtualMachine
-    Device --> VirtualMachine
     Cluster --> VirtualMachine
     Platform --> VirtualMachine
     VirtualMachine --> VMInterface
 
-    click Cluster "../../models/virtualization/cluster/"
-    click ClusterGroup "../../models/virtualization/clustergroup/"
-    click ClusterType "../../models/virtualization/clustertype/"
-    click VirtualMachineType "../../models/virtualization/virtualmachinetype/"
-    click Device "../../models/dcim/device/"
-    click Platform "../../models/dcim/platform/"
-    click VirtualMachine "../../models/virtualization/virtualmachine/"
-    click VMInterface "../../models/virtualization/vminterface/"
+click Cluster "../../models/virtualization/cluster/"
+click ClusterGroup "../../models/virtualization/clustergroup/"
+click ClusterType "../../models/virtualization/clustertype/"
+click Platform "../../models/dcim/platform/"
+click VirtualMachine "../../models/virtualization/virtualmachine/"
+click VMInterface "../../models/virtualization/vminterface/"
 ```
 
 ## Clusters
 
-A cluster is one or more physical host devices on which virtual machines can run.
-
-Each cluster must have a type and operational status, and may be assigned to a group. (Both types and groups are user-defined.) Each cluster may designate one or more devices as hosts, however this is optional.
-
-## Virtual Machine Types
-
-A virtual machine type provides reusable classification for virtual machines and can define create-time defaults for platform, vCPUs, and memory. This is useful when multiple virtual machines share a common sizing or profile while still allowing per-instance overrides after creation.
+A cluster is one or more physical host devices on which virtual machines can run. Each cluster must have a type and operational status, and may be assigned to a group. (Both types and groups are user-defined.) Each cluster may designate one or more devices as hosts, however this is optional.
 
 ## Virtual Machines
 
-A virtual machine is a virtualized compute instance. These behave in NetBox very similarly to device objects, but without any physical attributes.
-
-For example, a VM may have interfaces assigned to it with IP addresses and VLANs, however its interfaces cannot be connected via cables (because they are virtual). Each VM may define its compute, memory, and storage resources as well. A VM can optionally be assigned a [virtual machine type](../models/virtualization/virtualmachinetype.md) to classify it and provide default values for selected attributes at creation time.
-
-A VM can be placed in one of three ways:
-
-- Assigned to a site alone for logical grouping.
-- Assigned to a cluster and optionally pinned to a specific host device within that cluster.
-- Assigned directly to a standalone device that does not belong to any cluster.
+A virtual machine is a virtualized compute instance. These behave in NetBox very similarly to device objects, but without any physical attributes. For example, a VM may have interfaces assigned to it with IP addresses and VLANs, however its interfaces cannot be connected via cables (because they are virtual). Each VM may also define its compute, memory, and storage resources as well.

docs/integrations/rest-api.md

@@ -341,7 +341,7 @@ When retrieving devices and virtual machines via the REST API, each will include
 
 ## Pagination
 
-API responses which contain a list of many objects will be paginated for efficiency. NetBox employs offset-based pagination by default, which forms a page by skipping the number of objects indicated by the `offset` URL parameter. The root JSON object returned by a list endpoint contains the following attributes:
+API responses which contain a list of many objects will be paginated for efficiency. The root JSON object returned by a list endpoint contains the following attributes:
 
 * `count`: The total number of all objects matching the query
 * `next`: A hyperlink to the next page of results (if applicable)
@@ -398,49 +398,6 @@ The maximum number of objects that can be returned is limited by the [`MAX_PAGE_
 !!! warning
     Disabling the page size limit introduces a potential for very resource-intensive requests, since one API request can effectively retrieve an entire table from the database.
 
-### Cursor-Based Pagination
-
-For large datasets, offset-based pagination can become inefficient because the database must scan all rows up to the offset. As an alternative, cursor-based pagination uses the `start` query parameter to filter results by primary key (PK), enabling efficient keyset pagination.
-
-To use cursor-based pagination, pass `start` (the minimum PK value) and `limit` (the page size):
-
-```
-http://netbox/api/dcim/devices/?start=0&limit=100
-```
-
-This returns objects with an `id` greater than or equal to zero, ordered by PK, limited to 100 results. Below is an example showing an arbitrary `start` value.
-
-```json
-{
-    "count": null,
-    "next": "http://netbox/api/dcim/devices/?start=356&limit=100",
-    "previous": null,
-    "results": [
-        {
-            "id": 109,
-            "name": "dist-router07",
-            ...
-        },
-        ...
-        {
-            "id": 356,
-            "name": "acc-switch492",
-            ...
-        }
-    ]
-}
-```
-
-To iterate through all results, use the `id` of the last object in each response plus one as the `start` value for the next request. Continue until `next` is null.
-
-!!! info
-    Some important differences from offset-based pagination:
-
-    * `start` and `offset` are **mutually exclusive**; specifying both will result in a 400 error.
-    * Results are always ordered by primary key when using `start`. This is required to ensure deterministic behavior.
-    * `count` is always `null` in cursor mode, as counting all matching rows would partially negate its performance benefit.
-    * `previous` is always `null`: cursor-based pagination supports only forward navigation.
-
 ## Interacting with Objects
 
 ### Retrieving Multiple Objects

docs/integrations/webhooks.md

@@ -26,20 +26,10 @@ The following data is available as context for Jinja2 templates:
 * `event` - The type of event which triggered the webhook: `created`, `updated`, or `deleted`.
 * `timestamp` - The time at which the event occurred (in [ISO 8601](https://en.wikipedia.org/wiki/ISO_8601) format).
 * `object_type` - The NetBox model which triggered the change in the form `app_label.model_name`.
-* `request` - Data about the triggering request (if available).
-    * `request.id` - The UUID associated with the request
-    * `request.method` - The HTTP method (e.g. `GET` or `POST`)
-    * `request.path` - The URL path (ex: `/dcim/sites/123/edit/`)
-    * `request.user` - The name of the authenticated user who made the request (if available)
+* `username` - The name of the user account associated with the change.
+* `request_id` - The unique request ID. This may be used to correlate multiple changes associated with a single request.
 * `data` - A detailed representation of the object in its current state. This is typically equivalent to the model's representation in NetBox's REST API.
 * `snapshots` - Minimal "snapshots" of the object state both before and after the change was made; provided as a dictionary with keys named `prechange` and `postchange`. These are not as extensive as the fully serialized representation, but contain enough information to convey what has changed.
-* ⚠️ `request_id` - The unique request ID. This may be used to correlate multiple changes associated with a single request.
-* ⚠️ `username` - The name of the user account associated with the change.
-
-!!! warning "Deprecation of legacy keys"
-    The `request_id` and `username` keys in the webhook payload above are deprecated and should no longer be used. Support for them will be removed in NetBox v4.7.0.
-
-    Use `request.user` and `request.id` from the `request` object included in the callback context instead.
 
 ### Default Request Body
 
@@ -66,12 +56,6 @@ If no body template is specified, the request body will be populated with a JSON
         "region": null,
         ...
     },
-    "request": {
-        "id": "17af32f0-852a-46ca-a7d4-33ecd0c13de6",
-        "method": "POST",
-        "path": "/dcim/sites/add/",
-        "user": "jstretch"
-    },
     "snapshots": {
         "prechange": null,
         "postchange": {

docs/models/core/datasource.md

@@ -36,16 +36,13 @@ If false, synchronization will be disabled.
 
 ### Ignore Rules
 
-A set of rules (one per line) identifying files or paths to ignore during synchronization. Rules are matched against both the full relative path (e.g. `subdir/file.txt`) and the bare filename, so path-based patterns can be used to exclude entire directories. Some examples are provided below. See Python's [`fnmatch()` documentation](https://docs.python.org/3/library/fnmatch.html) for a complete reference.
+A set of rules (one per line) identifying filenames to ignore during synchronization. Some examples are provided below. See Python's [`fnmatch()` documentation](https://docs.python.org/3/library/fnmatch.html) for a complete reference.
 
-| Rule                  | Description                                          |
-|-----------------------|------------------------------------------------------|
-| `README`              | Ignore any files named `README`                      |
-| `*.txt`               | Ignore any files with a `.txt` extension             |
-| `data???.json`        | Ignore e.g. `data123.json`                           |
-| `subdir/*`            | Ignore all files within `subdir/`                    |
-| `subdir/*/*`          | Ignore all files one level deep within `subdir/`     |
-| `*/dev/*`             | Ignore files inside any directory named `dev/`       |
+| Rule           | Description                              |
+|----------------|------------------------------------------|
+| `README`       | Ignore any files named `README`          |
+| `*.txt`        | Ignore any files with a `.txt` extension |
+| `data???.json` | Ignore e.g. `data123.json`               |
 
 ### Sync Interval
 

docs/models/dcim/cablebundle.md

@@ -1,15 +0,0 @@
-# Cable Bundles
-
-A cable bundle is a logical grouping of individual [cables](./cable.md). Bundles can be used to organize cables that share a common purpose, route, or physical grouping (such as a conduit or harness).
-
-Assigning cables to a bundle is optional and does not affect cable tracing or connectivity. Bundles persist independently of their member cables: deleting a cable clears its bundle assignment but does not delete the bundle itself.
-
-## Fields
-
-### Name
-
-A unique name for the cable bundle.
-
-### Description
-
-A brief description of the bundle's purpose or contents.

docs/models/dcim/devicebay.md

@@ -23,8 +23,3 @@ The device bay's name. Must be unique to the parent device.
 ### Label
 
 An alternative physical label identifying the device bay.
-
-### Enabled
-
-Whether this device bay is enabled. Disabled device bays are not available for installation.
-

docs/models/dcim/devicetype.md

@@ -7,18 +7,6 @@ Device types are instantiated as devices installed within sites and/or equipment
 !!! note
     This parent/child relationship is **not** suitable for modeling chassis-based devices, wherein child members share a common control plane. Instead, line cards and similarly non-autonomous hardware should be modeled as modules or inventory items within a device.
 
-## Automatic Component Renaming
-
-When adding component templates to a device type, the string `{vc_position}` can be used in component template names to reference the
-`vc_position` field of the device being provisioned, when that device is a member of a Virtual Chassis.
-
-For example, an interface template named `Gi{vc_position}/0/0` installed on a Virtual Chassis
-member with position `2` will be rendered as `Gi2/0/0`.
-
-If the device is not a member of a Virtual Chassis, `{vc_position}` defaults to `0`. A custom
-fallback value can be specified using the syntax `{vc_position:X}`, where `X` is the desired default.
-For example, `{vc_position:1}` will render as `1` when no Virtual Chassis position is set.
-
 ## Fields
 
 ### Manufacturer

docs/models/dcim/modulebay.md

@@ -1,6 +1,6 @@
 # Module Bays
 
-Module bays represent a space or slot within a device in which a field-replaceable [module](./module.md) may be installed. A common example is that of a chassis-based switch such as the Cisco Nexus 9000 or Juniper EX9200. Modules, in turn, hold additional components that become available to the parent device.
+Module bays represent a space or slot within a device in which a field-replaceable [module](./module.md) may be installed. A common example is that of a chassis-based switch such as the Cisco Nexus 9000 or Juniper EX9200. Modules in turn hold additional components that become available to the parent device.
 
 !!! note
     If you need to model child devices rather than modules, use a [device bay](./devicebay.md) instead.
@@ -29,8 +29,3 @@ An alternative physical label identifying the module bay.
 ### Position
 
 The numeric position in which this module bay is situated. For example, this would be the number assigned to a slot within a chassis-based switch.
-
-### Enabled
-
-Whether this module bay is enabled. Disabled module bays are not available for installation.
-

docs/models/dcim/moduletype.md

@@ -20,38 +20,8 @@ When adding component templates to a module type, the string `{module}` can be u
 
 For example, you can create a module type with interface templates named `Gi{module}/0/[1-48]`. When a new module of this type is "installed" to a module bay with a position of "3", NetBox will automatically name these interfaces `Gi3/0/[1-48]`.
 
-Similarly, the string `{vc_position}` can be used in component template names to reference the
-`vc_position` field of the device being provisioned, when that device is a member of a Virtual Chassis.
-
-For example, an interface template named `Gi{vc_position}/{module}/0` installed on a Virtual Chassis
-member with position `2` and module bay position `3` will be rendered as `Gi2/3/0`.
-
-If the device is not a member of a Virtual Chassis, `{vc_position}` defaults to `0`. A custom
-fallback value can be specified using the syntax `{vc_position:X}`, where `X` is the desired default.
-For example, `{vc_position:1}` will render as `1` when no Virtual Chassis position is set.
-
 Automatic renaming is supported for all modular component types (those listed above).
 
-### Position Inheritance for Nested Modules
-
-When using nested module bays (modules installed inside other modules), the `{module}` placeholder
-can also be used in the **position** field of module bay templates to inherit the parent bay's
-position. This allows a single module type to produce correctly named components at any nesting
-depth, with a user-controlled separator.
-
-For example, a line card module type might define sub-bay positions as `{module}/1`, `{module}/2`,
-etc. When the line card is installed in a device bay with position `3`, these sub-bay positions
-resolve to `3/1`, `3/2`, etc. An SFP module type with interface template `SFP {module}` installed
-in sub-bay `3/2` then produces interface `SFP 3/2`.
-
-The separator between levels is defined by the user in the position field template itself. Using
-`{module}-1` produces positions like `3-1`, while `{module}.1` produces `3.1`. This provides
-full flexibility without requiring a global separator configuration.
-
-!!! note
-    If the position field does not contain `{module}`, no inheritance occurs and behavior is
-    unchanged from previous versions.
-
 ## Fields
 
 ### Manufacturer

docs/models/dcim/rack.md

@@ -1,6 +1,6 @@
 # Racks
 
-The rack model represents a physical two- or four-post equipment rack in which [devices](./device.md) can be installed. Each rack must be assigned to a [site](./site.md), and may optionally be assigned to a [location](./location.md) within that site. Racks can also be organized by user-defined functional roles or by [rack groups](./rackgroup.md). The name and facility ID of each rack within a location must be unique.
+The rack model represents a physical two- or four-post equipment rack in which [devices](./device.md) can be installed. Each rack must be assigned to a [site](./site.md), and may optionally be assigned to a [location](./location.md) within that site. Racks can also be organized by user-defined functional roles. The name and facility ID of each rack within a location must be unique.
 
 Rack height is measured in *rack units* (U); racks are commonly between 42U and 48U tall, but NetBox allows you to define racks of arbitrary height. A toggle is provided to indicate whether rack units are in ascending (from the ground up) or descending order.
 
@@ -16,10 +16,6 @@ The [site](./site.md) to which the rack is assigned.
 
 The [location](./location.md) within a site where the rack has been installed (optional).
 
-### Rack Group
-
-The [group](./rackgroup.md) used to organize racks by physical placement (optional).
-
 ### Name
 
 The rack's name or identifier. Must be unique to the rack's location, if assigned.

docs/models/dcim/rackgroup.md

@@ -1,15 +0,0 @@
-# Rack Groups
-
-Racks can optionally be assigned to rack groups to reflect their physical placement. Rack groups provide a secondary means of categorization alongside [locations](./location.md), which is particularly useful for datacenter operators who need to group racks by row, aisle, or similar physical arrangement while keeping them assigned to the same location, such as a cage or room. Rack groups are flat and do not form a hierarchy.
-
-Rack groups can also be used to scope [VLAN groups](../ipam/vlangroup.md), which can help model L2 domains spanning rows or pairs of racks.
-
-## Fields
-
-### Name
-
-A unique human-friendly name.
-
-### Slug
-
-A unique URL-friendly identifier. (This value can be used for filtering.)

docs/models/dcim/rackreservation.md

@@ -12,10 +12,6 @@ The [rack](./rack.md) being reserved.
 
 The rack unit or units being reserved. Multiple units can be expressed using commas and/or hyphens. For example, `1,3,5-7` specifies units 1, 3, 5, 6, and 7.
 
-### Total U's
-
-A calculated (read-only) field that reflects the total number of units in the reservation. Can be filtered upon using `unit_count_min` and `unit_count_max` parameters in the UI or API.
-
 ### Status
 
 The current status of the reservation. (This is for documentation only: The status of a reservation has no impact on the installation of devices within a reserved rack unit.)

docs/models/extras/customfield.md

@@ -118,7 +118,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).
-
-### Validation Schema
-
-For JSON custom fields, users have the option of defining a [validation schema](https://json-schema.org). Any value applied to this custom field on a model will be validated against the provided schema, if any.

docs/models/extras/webhook.md

@@ -81,13 +81,10 @@ The following context variables are available to the text and link templates.
 
 | Variable      | Description                                          |
 |---------------|------------------------------------------------------|
-| `event`       | The event type (`create`, `update`, or `delete`)     |
+| `event`       | The event type (`created`, `updated`, or `deleted`)  |
 | `timestamp`   | The time at which the event occurred                 |
 | `object_type` | The type of object impacted (`app_label.model_name`) |
 | `username`    | The name of the user associated with the change      |
 | `request_id`  | The unique request ID                                |
 | `data`        | A complete serialized representation of the object   |
 | `snapshots`   | Pre- and post-change snapshots of the object         |
-
-!!! warning "Deprecation of legacy fields"
-    The `request_id` and `username` fields in the webhook payload above are deprecated and should no longer be used. Support for them will be removed in NetBox v4.7.0. Use `request.user` and `request.id` from the `request` object included in the callback context instead. (Note that `request` is populated in the context only when the webhook is associated with a triggering request.)

docs/models/ipam/asn.md

@@ -14,10 +14,6 @@ The 16- or 32-bit AS number.
 
 The [Regional Internet Registry](./rir.md) or similar authority responsible for the allocation of this particular ASN.
 
-### Role
-
-The user-defined functional [role](./role.md) assigned to this ASN.
-
 ### Sites
 
 The [site(s)](../dcim/site.md) to which this ASN is assigned.

docs/models/ipam/vlangroup.md

@@ -18,10 +18,6 @@ A unique URL-friendly identifier. (This value can be used for filtering.)
 
 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.
 
-### Total VLAN IDs
-
-A read-only integer indicating the total count of VLAN IDs available within the group, calculated from the configured VLAN ID Ranges. For example, a group with ranges `100-199` and `300-399` would have a total of 200 VLAN IDs. This value is automatically computed and updated whenever the VLAN ID ranges are modified.
-
 ### Scope
 
 The domain covered by a VLAN group, defined as one of the supported object types. This conveys the context in which a VLAN group applies.

docs/models/virtualization/virtualmachine.md

@@ -1,27 +1,18 @@
 # Virtual Machines
 
-A virtual machine (VM) represents a virtual compute instance hosted within a cluster or directly on a device. Each VM must be assigned to at least one of: a [site](../dcim/site.md), a [cluster](./cluster.md), or a [device](../dcim/device.md).
+A virtual machine (VM) represents a virtual compute instance hosted within a [cluster](./cluster.md). Each VM must be assigned to a [site](../dcim/site.md) and/or cluster, and may optionally be assigned to a particular host [device](../dcim/device.md) within a cluster.
 
-Virtual machines may have virtual [interfaces](./vminterface.md) assigned to them, but do not support any physical component. When a VM has one or more interfaces with IP addresses assigned, a primary IP for the VM can be designated, for both IPv4 and IPv6.
+Virtual machines may have virtual [interfaces](./vminterface.md) assigned to them, but do not support any physical component. When a VM has one or more interfaces with IP addresses assigned, a primary IP for the device can be designated, for both IPv4 and IPv6.
 
 ## Fields
 
 ### Name
 
-The virtual machine's configured name. Must be unique within its scoping context:
-
-- If assigned to a **cluster**: unique within the cluster and tenant.
-- If assigned to a **device** (no cluster): unique within the device and tenant.
-
-### Type
-
-The [virtual machine type](./virtualmachinetype.md) assigned to the VM. A type classifies a virtual machine and can provide default values for platform, vCPUs, and memory when the VM is created.
-
-Changes made to a virtual machine type do **not** apply retroactively to existing virtual machines.
+The virtual machine's configured name. Must be unique to the assigned cluster and tenant.
 
 ### Role
 
-The functional role assigned to the VM.
+The functional [role](../dcim/devicerole.md) assigned to the VM.
 
 ### Status
 
@@ -30,28 +21,24 @@ The VM's operational status.
 !!! tip
     Additional statuses may be defined by setting `VirtualMachine.status` under the [`FIELD_CHOICES`](../../configuration/data-validation.md#field_choices) configuration parameter.
 
-### Start on Boot
+### Start on boot
 
 The start on boot setting from the hypervisor.
 
 !!! tip
     Additional statuses may be defined by setting `VirtualMachine.start_on_boot` under the [`FIELD_CHOICES`](../../configuration/data-validation.md#field_choices) configuration parameter.
 
-### Site / Cluster / Device
+### Site & Cluster
 
-The location or host for this VM. At least one must be specified:
+The [site](../dcim/site.md) and/or [cluster](./cluster.md) to which the VM is assigned.
 
-- **Site only**: The VM exists at a site but is not assigned to a specific cluster or device.
-- **Cluster only**: The VM belongs to a virtualization cluster. The site is automatically inferred from the cluster's scope.
-- **Device only**: The VM runs directly on a physical host device without a cluster (e.g. containers). The site is automatically inferred from the device's site.
-- **Cluster + Device**: The VM belongs to a cluster and is pinned to a specific host device within that cluster. The device must be a registered host of the assigned cluster.
+### Device
 
-!!! info "New in NetBox v4.6"
-    Virtual machines can now be assigned directly to a device without requiring a cluster. This is particularly useful for modeling VMs running on standalone hosts outside of a cluster.
+The physical host [device](../dcim/device.md) within the assigned site/cluster on which this VM resides.
 
 ### Platform
 
-A VM may be associated with a particular [platform](../dcim/platform.md) to indicate its operating system. If a virtual machine type defines a default platform, it will be applied when the VM is created unless an explicit platform is specified.
+A VM may be associated with a particular [platform](../dcim/platform.md) to indicate its operating system.
 
 ### Primary IPv4 & IPv6 Addresses
 
@@ -62,11 +49,11 @@ Each VM may designate one primary IPv4 address and/or one primary IPv6 address f
 
 ### vCPUs
 
-The number of virtual CPUs provisioned. A VM may be allocated a partial vCPU count (e.g. 1.5 vCPU). If a virtual machine type defines a default vCPU allocation, it will be applied when the VM is created unless an explicit value is specified.
+The number of virtual CPUs provisioned. A VM may be allocated a partial vCPU count (e.g. 1.5 vCPU).
 
 ### Memory
 
-The amount of running memory provisioned, in megabytes. If a virtual machine type defines a default memory allocation, it will be applied when the VM is created unless an explicit value is specified.
+The amount of running memory provisioned, in megabytes.
 
 ### Disk
 
@@ -77,7 +64,4 @@ The amount of disk storage provisioned, in megabytes.
 
 ### Serial Number
 
-Optional serial number assigned to this virtual machine.
-
-!!! info
-    Unlike devices, uniqueness is not enforced for virtual machine serial numbers.
+Optional serial number assigned to this virtual machine. Unlike devices, uniqueness is not enforced for virtual machine serial numbers.

docs/models/virtualization/virtualmachinetype.md

@@ -1,27 +0,0 @@
-# Virtual Machine Types
-
-A virtual machine type defines a reusable classification and default configuration for [virtual machines](./virtualmachine.md).
-
-A type can optionally provide default values for a VM's [platform](../dcim/platform.md), vCPU allocation, and memory allocation. When a virtual machine is created with an assigned type, any unset values among these fields will inherit their defaults from the type. Changes made to a virtual machine type do **not** apply retroactively to existing virtual machines.
-
-## Fields
-
-### Name
-
-A unique human-friendly name.
-
-### Slug
-
-A unique URL-friendly identifier. (This value can be used for filtering.)
-
-### Default Platform
-
-If defined, virtual machines instantiated with this type will automatically inherit the selected platform when no explicit platform is provided.
-
-### Default vCPUs
-
-The default number of vCPUs to assign when creating a virtual machine from this type.
-
-### Default Memory
-
-The default amount of memory, in megabytes, to assign when creating a virtual machine from this type.

docs/plugins/development/permissions.md

@@ -1,24 +0,0 @@
-# Custom Model Actions
-
-Plugins can register custom permission actions for their models. These actions appear as checkboxes in the ObjectPermission form, making it easy for administrators to grant or restrict access to plugin-specific functionality without manually entering action names.
-
-For example, a plugin might define a "sync" action for a model that syncs data from an external source, or a "bypass" action that allows users to bypass certain restrictions.
-
-## Registering Model Actions
-
-The preferred way to register custom actions is via Django's `Meta.permissions` on the model class. NetBox will automatically register these as model actions when the app is loaded:
-
-```python
-from netbox.models import NetBoxModel
-
-class MyModel(NetBoxModel):
-    # ...
-
-    class Meta:
-        permissions = [
-            ('sync', 'Synchronize data from external source'),
-            ('export', 'Export data to external system'),
-        ]
-```
-
-Once registered, these actions appear as checkboxes in a flat list when creating or editing an ObjectPermission.

docs/plugins/development/search.md

@@ -1,14 +1,12 @@
 # Search
 
-Plugins can define and register their own models to extend NetBox's core search functionality. Typically, a plugin will include a file named `search.py`, which holds all search indexes for its models.
+Plugins can define and register their own models to extend NetBox's core search functionality. Typically, a plugin will include a file named `search.py`, which holds all search indexes for its models (see the example below).
 
-```python title="search.py"
+```python
 # search.py
-from netbox.search import SearchIndex, register_search
-
+from netbox.search import SearchIndex
 from .models import MyModel
 
-@register_search
 class MyModelIndex(SearchIndex):
     model = MyModel
     fields = (
@@ -19,11 +17,15 @@ class MyModelIndex(SearchIndex):
     display_attrs = ('site', 'device', 'status', 'description')
 ```
 
-Decorate each `SearchIndex` subclass with `@register_search` to register it with NetBox. When using the default `search.py` module, no additional `indexes = [...]` list is required.
+Fields listed in `display_attrs` will not be cached for search, but will be displayed alongside the object when it appears in global search results. This is helpful for conveying to the user additional information about an object.
 
-Fields listed in `display_attrs` are not cached for matching, but they are displayed alongside the object in global search results to provide additional context.
+To register one or more indexes with NetBox, define a list named `indexes` at the end of this file:
+
+```python
+indexes = [MyModelIndex]
+```
 
 !!! tip
-    The legacy `indexes = [...]` list remains supported via `PluginConfig.search_indexes` for backward compatibility and custom loading patterns.
+    The path to the list of search indexes can be modified by setting `search_indexes` in the PluginConfig instance.
 
 ::: netbox.search.SearchIndex

docs/plugins/development/ui-components.md

@@ -1,9 +1,12 @@
 # UI Components
 
-!!! note "New in NetBox v4.6"
-    All UI components described here were introduced in NetBox v4.6. Be sure to set the minimum NetBox version to 4.6.0 for your plugin before incorporating any of these resources.
+!!! note "New in NetBox v4.5"
+    All UI components described here were introduced in NetBox v4.5. Be sure to set the minimum NetBox version to 4.5.0 for your plugin before incorporating any of these resources.
 
-To simplify the process of designing your plugin's user interface, and to encourage a consistent look and feel throughout the entire application, NetBox provides a set of components that enable programmatic UI design. These make it possible to declare complex page layouts with little or no custom HTML.
+!!! danger "Beta Feature"
+    UI components are considered a beta feature, and are still under active development. Please be aware that the API for resources on this page is subject to change in future releases.
+
+To simply the process of designing your plugin's user interface, and to encourage a consistent look and feel throughout the entire application, NetBox provides a set of components that enable programmatic UI design. These make it possible to declare complex page layouts with little or no custom HTML.
 
 ## Page Layout
 
@@ -72,12 +75,9 @@ class RecentChangesPanel(Panel):
             **super().get_context(context),
             'changes': get_changes()[:10],
         }
-
-    def should_render(self, context):
-        return len(context['changes']) > 0
 ```
 
-NetBox also includes a set of panels suited for specific uses, such as displaying object details or embedding a table of related objects. These are listed below.
+NetBox also includes a set of panels suite for specific uses, such as display object details or embedding a table of related objects. These are listed below.
 
 ::: netbox.ui.panels.Panel
 
@@ -85,6 +85,26 @@ NetBox also includes a set of panels suited for specific uses, such as displayin
 
 ::: netbox.ui.panels.ObjectAttributesPanel
 
+#### Object Attributes
+
+The following classes are available to represent object attributes within an ObjectAttributesPanel. Additionally, plugins can subclass `netbox.ui.attrs.ObjectAttribute` to create custom classes.
+
+| Class                                | Description                                      |
+|--------------------------------------|--------------------------------------------------|
+| `netbox.ui.attrs.AddressAttr`        | A physical or mailing address.                   |
+| `netbox.ui.attrs.BooleanAttr`        | A boolean value                                  |
+| `netbox.ui.attrs.ColorAttr`          | A color expressed in RGB                         |
+| `netbox.ui.attrs.ChoiceAttr`         | A selection from a set of choices                |
+| `netbox.ui.attrs.GPSCoordinatesAttr` | GPS coordinates (latitude and longitude)         |
+| `netbox.ui.attrs.ImageAttr`          | An attached image (displays the image)           |
+| `netbox.ui.attrs.NestedObjectAttr`   | A related nested object                          |
+| `netbox.ui.attrs.NumericAttr`        | An integer or float value                        |
+| `netbox.ui.attrs.RelatedObjectAttr`  | A related object                                 |
+| `netbox.ui.attrs.TemplatedAttr`      | Renders an attribute using a custom template     |
+| `netbox.ui.attrs.TextAttr`           | A string (text) value                            |
+| `netbox.ui.attrs.TimezoneAttr`       | A timezone with annotated offset                 |
+| `netbox.ui.attrs.UtilizationAttr`    | A numeric value expressed as a utilization graph |
+
 ::: netbox.ui.panels.OrganizationalObjectPanel
 
 ::: netbox.ui.panels.NestedGroupObjectPanel
@@ -99,13 +119,9 @@ NetBox also includes a set of panels suited for specific uses, such as displayin
 
 ::: netbox.ui.panels.TemplatePanel
 
-::: netbox.ui.panels.TextCodePanel
-
-::: netbox.ui.panels.ContextTablePanel
-
 ::: netbox.ui.panels.PluginContentPanel
 
-### Panel Actions
+## Panel Actions
 
 Each panel may have actions associated with it. These render as links or buttons within the panel header, opposite the panel's title. For example, a common use case is to include an "Add" action on a panel which displays a list of objects. Below is an example of this.
 
@@ -130,60 +146,3 @@ panels.ObjectsTablePanel(
 ::: netbox.ui.actions.AddObject
 
 ::: netbox.ui.actions.CopyContent
-
-## Object Attributes
-
-The following classes are available to represent object attributes within an ObjectAttributesPanel. Additionally, plugins can subclass `netbox.ui.attrs.ObjectAttribute` to create custom classes.
-
-| Class                                    | Description                                      |
-|------------------------------------------|--------------------------------------------------|
-| `netbox.ui.attrs.AddressAttr`            | A physical or mailing address.                   |
-| `netbox.ui.attrs.BooleanAttr`            | A boolean value                                  |
-| `netbox.ui.attrs.ChoiceAttr`             | A selection from a set of choices                |
-| `netbox.ui.attrs.ColorAttr`              | A color expressed in RGB                         |
-| `netbox.ui.attrs.DateTimeAttr`           | A date or datetime value                         |
-| `netbox.ui.attrs.GenericForeignKeyAttr`  | A related object via a generic foreign key       |
-| `netbox.ui.attrs.GPSCoordinatesAttr`     | GPS coordinates (latitude and longitude)         |
-| `netbox.ui.attrs.ImageAttr`              | An attached image (displays the image)           |
-| `netbox.ui.attrs.NestedObjectAttr`       | A related nested object (includes ancestors)     |
-| `netbox.ui.attrs.NumericAttr`            | An integer or float value                        |
-| `netbox.ui.attrs.RelatedObjectAttr`      | A related object                                 |
-| `netbox.ui.attrs.RelatedObjectListAttr`  | A list of related objects                        |
-| `netbox.ui.attrs.TemplatedAttr`          | Renders an attribute using a custom template     |
-| `netbox.ui.attrs.TextAttr`               | A string (text) value                            |
-| `netbox.ui.attrs.TimezoneAttr`           | A timezone with annotated offset                 |
-| `netbox.ui.attrs.UtilizationAttr`        | A numeric value expressed as a utilization graph |
-
-::: netbox.ui.attrs.ObjectAttribute
-
-::: netbox.ui.attrs.AddressAttr
-
-::: netbox.ui.attrs.BooleanAttr
-
-::: netbox.ui.attrs.ChoiceAttr
-
-::: netbox.ui.attrs.ColorAttr
-
-::: netbox.ui.attrs.DateTimeAttr
-
-::: netbox.ui.attrs.GenericForeignKeyAttr
-
-::: netbox.ui.attrs.GPSCoordinatesAttr
-
-::: netbox.ui.attrs.ImageAttr
-
-::: netbox.ui.attrs.NestedObjectAttr
-
-::: netbox.ui.attrs.NumericAttr
-
-::: netbox.ui.attrs.RelatedObjectAttr
-
-::: netbox.ui.attrs.RelatedObjectListAttr
-
-::: netbox.ui.attrs.TemplatedAttr
-
-::: netbox.ui.attrs.TextAttr
-
-::: netbox.ui.attrs.TimezoneAttr
-
-::: netbox.ui.attrs.UtilizationAttr

docs/plugins/development/webhooks.md

@@ -36,7 +36,6 @@ The resulting webhook payload will look like the following:
         "url": "/api/dcim/sites/2/",
         ...
     },
-    "request": {...},
     "snapshots": {...},
     "context": {
         "foo": 123
@@ -44,11 +43,6 @@ The resulting webhook payload will look like the following:
 }
 ```
 
-!!! warning "Deprecation of legacy keys"
-    The `request_id` and `username` keys in the webhook payload above are deprecated and should no longer be used. Support for them will be removed in NetBox v4.7.0.
-
-    Use `request.user` and `request.id` from the `request` object included in the callback context instead.
-
 !!! note "Consider namespacing webhook data"
     The data returned from all webhook callbacks will be compiled into a single `context` dictionary. Any existing keys within this dictionary will be overwritten by subsequent callbacks which include those keys. To avoid collisions with webhook data provided by other plugins, consider namespacing your plugin's data within a nested dictionary as such:
     

docs/release-notes/index.md

@@ -10,14 +10,6 @@ Minor releases are published in April, August, and December of each calendar yea
 
 This page contains a history of all major and minor releases since NetBox v2.0. For more detail on a specific patch release, please see the release notes page for that specific minor release.
 
-#### [Version 4.6](./version-4.6.md) (May 2026)
-
-* Virtual Machine Types ([#5795](https://github.com/netbox-community/netbox/issues/5795))
-* Cable Bundles ([#20151](https://github.com/netbox-community/netbox/issues/20151))
-* Rack Groups ([#20961](https://github.com/netbox-community/netbox/issues/20961))
-* ETag Support for REST API ([#21356](https://github.com/netbox-community/netbox/issues/21356))
-* Cursor-based Pagination for REST API ([#21363](https://github.com/netbox-community/netbox/issues/21363))
-
 #### [Version 4.5](./version-4.5.md) (January 2026)
 
 * Lookup Modifiers in Filter Forms ([#7604](https://github.com/netbox-community/netbox/issues/7604))

docs/release-notes/version-4.5.md

@@ -1,112 +1,5 @@
 # NetBox v4.5
 
-## v4.5.8 (2026-04-14)
-
-### Enhancements
-
-* [#21430](https://github.com/netbox-community/netbox/issues/21430) - Display the device role's color in the device view
-* [#21795](https://github.com/netbox-community/netbox/issues/21795) - Update `humanize_speed` template filter to support decimal Gbps/Tbps values
-
-### Bug Fixes
-
-* [#21529](https://github.com/netbox-community/netbox/issues/21529) - Exclude non-existent custom fields from object changelog data returned via the REST API
-* [#21542](https://github.com/netbox-community/netbox/issues/21542) - Expand interface speed field to 64-bit integer to prevent overflow for LAG interfaces exceeding ~2.1 Tbps
-* [#21704](https://github.com/netbox-community/netbox/issues/21704) - Fix missing port mappings in device type YAML export
-* [#21783](https://github.com/netbox-community/netbox/issues/21783) - Fix support for bulk import of cables connected to power feeds
-* [#21801](https://github.com/netbox-community/netbox/issues/21801) - Prevent duplicate filename collision when uploading files using S3 storage
-* [#21814](https://github.com/netbox-community/netbox/issues/21814) - Fix custom script "last run" time to reflect job start time rather than creation time
-* [#21835](https://github.com/netbox-community/netbox/issues/21835) - Correct help text for color selection form fields
-* [#21841](https://github.com/netbox-community/netbox/issues/21841) - Restore visibility of the edit button for script modules to non-superusers
-* [#21845](https://github.com/netbox-community/netbox/issues/21845) - Fix CSV export of connection columns rendering template whitespace instead of a formatted value
-* [#21869](https://github.com/netbox-community/netbox/issues/21869) - Remove redundant `ScriptModule` class synchronization triggered on save
-
----
-
-## v4.5.7 (2026-04-03)
-
-### Enhancements
-
-* [#21095](https://github.com/netbox-community/netbox/issues/21095) - Adopt IEC unit labels (e.g. GiB) for virtual machine resources
-* [#21696](https://github.com/netbox-community/netbox/issues/21696) - Add support for django-rq 4.0 and introduce `RQ` configuration parameter
-* [#21701](https://github.com/netbox-community/netbox/issues/21701) - Support uploading custom scripts via the REST API (`/api/extras/scripts/upload/`)
-* [#21760](https://github.com/netbox-community/netbox/issues/21760) - Add a 1C2P:2C1P breakout cable profile
-
-### Performance Improvements
-
-* [#21655](https://github.com/netbox-community/netbox/issues/21655) - Optimize queries for object and multi-object type custom fields
-
-### Bug Fixes
-
-* [#20474](https://github.com/netbox-community/netbox/issues/20474) - Fix installation of modules with placeholder values in component names
-* [#21498](https://github.com/netbox-community/netbox/issues/21498) - Fix server error triggered by event rules referencing deleted objects
-* [#21533](https://github.com/netbox-community/netbox/issues/21533) - Ensure read-only fields are included in REST API responses upon object creation
-* [#21535](https://github.com/netbox-community/netbox/issues/21535) - Fix filtering of object-type custom fields when "is empty" is selected
-* [#21784](https://github.com/netbox-community/netbox/issues/21784) - Fix `AttributeError` exception when sorting a table as an anonymous user
-* [#21808](https://github.com/netbox-community/netbox/issues/21808) - Fix `RelatedObjectDoesNotExist` exception when viewing an interface with a virtual circuit termination
-* [#21810](https://github.com/netbox-community/netbox/issues/21810) - Fix `AttributeError` exception when viewing virtual chassis member
-* [#21825](https://github.com/netbox-community/netbox/issues/21825) - Fix sorting by broken columns in several object lists
-
----
-
-## v4.5.6 (2026-03-31)
-
-### Enhancements
-
-* [#21480](https://github.com/netbox-community/netbox/issues/21480) - Add OSFP224 (1.6T) interface type
-* [#21727](https://github.com/netbox-community/netbox/issues/21727) - Add 2.5GBASE-X SFP modular interface type
-* [#21743](https://github.com/netbox-community/netbox/issues/21743) - Improve object change diff styling and layout
-* [#21793](https://github.com/netbox-community/netbox/issues/21793) - Add 50 Gbps, 800 Gbps, and 1.6 Tbps interface speed options
-
-### Bug Fixes
-
-* [#20467](https://github.com/netbox-community/netbox/issues/20467) - Fix resolution of the `{module}` variable for position fields in nested modules
-* [#21698](https://github.com/netbox-community/netbox/issues/21698) - Adjust custom field URL filter to support non-standard port numbers
-* [#21707](https://github.com/netbox-community/netbox/issues/21707) - Fix grouping of owner fields in provider account add/edit forms
-* [#21749](https://github.com/netbox-community/netbox/issues/21749) - Fix `FieldError` exception when sorting the circuit group assignment table by the member column
-* [#21763](https://github.com/netbox-community/netbox/issues/21763) - Use separate add/remove form fields when editing a site or provider with a large number of ASNs assigned
-
----
-
-## v4.5.5 (2026-03-17)
-
-### Enhancements
-
-* [#21114](https://github.com/netbox-community/netbox/issues/21114) - Support path exclusions for data source synchronization
-* [#21578](https://github.com/netbox-community/netbox/issues/21578) - Support identifying scope object by name or slug when bulk importing scoped objects
-
-### Performance Improvements
-
-* [#21330](https://github.com/netbox-community/netbox/issues/21330) - Optimize the assignment of tags when saving objects
-* [#21402](https://github.com/netbox-community/netbox/issues/21402) - Avoid excessive database queries when rendering unnamed devices via the REST API
-* [#21611](https://github.com/netbox-community/netbox/issues/21611) - Replace inefficient calls to `.count()` with `.exists()`
-
-### Bug Fixes
-
-* [#19867](https://github.com/netbox-community/netbox/issues/19867) - Preserve the "per page" pagination setting when returning from object edit forms
-* [#20077](https://github.com/netbox-community/netbox/issues/20077) - Fix form field focus bug in Microsoft Edge
-* [#20385](https://github.com/netbox-community/netbox/issues/20385) - Enforce `MAX_PAGE_SIZE` limit for GraphQL API requests
-* [#20468](https://github.com/netbox-community/netbox/issues/20468) - Fix range-based filter lookups for integer fields in GraphQL API
-* [#20915](https://github.com/netbox-community/netbox/issues/20915) - Restore user language preference after login via social authentication
-* [#20934](https://github.com/netbox-community/netbox/issues/20934) - Fix dark mode flicker on page load
-* [#21012](https://github.com/netbox-community/netbox/issues/21012) - Add pagination for VLAN table on interface view to prevent silent truncation at 100 entries
-* [#21380](https://github.com/netbox-community/netbox/issues/21380) - Fix display of the background tasks table on mobile
-* [#21440](https://github.com/netbox-community/netbox/issues/21440) - Avoid erroneously clearing primary/OOB IP assignments during bulk import/update
-* [#21468](https://github.com/netbox-community/netbox/issues/21468) - Preserve safe custom HTTP headers when copying requests for background job processing
-* [#21486](https://github.com/netbox-community/netbox/issues/21486) - Fix `AttributeError` exception caused by missing `COOKIES` attribute on `NetBoxFakeRequest`
-* [#21512](https://github.com/netbox-community/netbox/issues/21512) - Fix GraphQL filter field name mismatch for device component types (e.g. `console_ports`)
-* [#21531](https://github.com/netbox-community/netbox/issues/21531) - Fix search functionality for location when combined with other filters
-* [#21556](https://github.com/netbox-community/netbox/issues/21556) - Avoid clearing the platform field when changing device type in the device edit form
-* [#21579](https://github.com/netbox-community/netbox/issues/21579) - Hide the script "Add" button for users lacking the required permission
-* [#21580](https://github.com/netbox-community/netbox/issues/21580) - Hide the virtual machine "Add components" dropdown for users lacking change permission
-* [#21586](https://github.com/netbox-community/netbox/issues/21586) - Fix broken "Add child group" link in site group view (was pointing to the region endpoint)
-* [#21618](https://github.com/netbox-community/netbox/issues/21618) - Fix cable termination points being lost when bulk-editing the cable profile
-* [#21651](https://github.com/netbox-community/netbox/issues/21651) - Disable sorting by the `is_primary` column in the MAC address list view
-* [#21653](https://github.com/netbox-community/netbox/issues/21653) - Fix profile-based cable tracing when a single origin carries multiple positions
-* [#21673](https://github.com/netbox-community/netbox/issues/21673) - Fix display of primary IP address with associated NAT IP on virtual machine view
-* [#21686](https://github.com/netbox-community/netbox/issues/21686) - Clean up cached circuit attributes when reassigning a circuit termination
-
----
-
 ## v4.5.4 (2026-03-03)
 
 ### Enhancements

docs/release-notes/version-4.6.md

@@ -1,123 +0,0 @@
-# NetBox v4.6
-
-## v4.6.0-beta1 (2026-04-14)
-
-### New Features
-
-#### Virtual Machine Types ([#5795](https://github.com/netbox-community/netbox/issues/5795))
-
-A new VirtualMachineType model has been introduced to enable categorization of virtual machines by instance type, analogous to how DeviceType categorizes physical hardware. VM types can be defined once and reused across many virtual machines.
-
-#### Cable Bundles ([#20151](https://github.com/netbox-community/netbox/issues/20151))
-
-A new CableBundle model allows individual cables to be grouped together to represent physical cable runs that are managed as a unit; e.g. a bundle of 48 CAT6 cables between two patch panels. (Please note that this feature is _not_ suitable for modeling individual fiber strands within a single cable.)
-
-#### Rack Groups ([#20961](https://github.com/netbox-community/netbox/issues/20961))
-
-A flat RackGroup model has been reintroduced to provide a lightweight secondary axis of rack organization (e.g. by row or aisle) that is independent of the location hierarchy. Racks carry an optional foreign key to a RackGroup, and RackGroup can also serve as a scope for VLANGroup assignments.
-
-#### ETag Support for REST API ([#21356](https://github.com/netbox-community/netbox/issues/21356))
-
-The REST API now returns an `ETag` header on responses for individual objects, derived from the object's last-updated timestamp. Clients can supply an `If-Match` header on PUT/PATCH requests to guard against conflicting concurrent updates; if the object has been modified since the ETag was issued, the server returns a 412 (Precondition Failed) response.
-
-#### Cursor-based Pagination for REST API ([#21363](https://github.com/netbox-community/netbox/issues/21363))
-
-A new `start` query parameter has been introduced as an efficient alternative to the existing `offset` parameter for paginating large result sets. Rather than scanning the table up to a relative offset, the `start` parameter filters for objects with a primary key equal to or greater than the given value, enabling constant-time pagination regardless of result set size.
-
-### Enhancements
-
-* [#12024](https://github.com/netbox-community/netbox/issues/12024) - Permit virtual machines to be assigned to devices without a cluster
-* [#14329](https://github.com/netbox-community/netbox/issues/14329) - Improve diff highlighting for custom field data in change logs
-* [#15513](https://github.com/netbox-community/netbox/issues/15513) - Add bulk creation support for IP prefixes
-* [#17654](https://github.com/netbox-community/netbox/issues/17654) - Support role assignment for ASNs
-* [#19025](https://github.com/netbox-community/netbox/issues/19025) - Support optional schema validation for JSON custom fields
-* [#19034](https://github.com/netbox-community/netbox/issues/19034) - Annotate total reserved unit count on rack reservations
-* [#19138](https://github.com/netbox-community/netbox/issues/19138) - Include NAT addresses for primary & out-of-band IP addresses in REST API
-* [#19796](https://github.com/netbox-community/netbox/issues/19796) - Support `{module}` position inheritance for nested module bays
-* [#19953](https://github.com/netbox-community/netbox/issues/19953) - Enable debugging support for ConfigTemplate rendering
-* [#20123](https://github.com/netbox-community/netbox/issues/20123) - Introduce options to control adoption/replication of device components via REST API (replicates UI behavior)
-* [#20152](https://github.com/netbox-community/netbox/issues/20152) - Support for marking module and device bays as disabled
-* [#20162](https://github.com/netbox-community/netbox/issues/20162) - Provide an option to execute as a background job when adding components to devices in bulk
-* [#20163](https://github.com/netbox-community/netbox/issues/20163) - Add changelog message support for bulk device component creation
-* [#20698](https://github.com/netbox-community/netbox/issues/20698) - Add read-only `total_vlan_ids` attribute on VLAN group representation in REST & GraphQL APIs
-* [#20916](https://github.com/netbox-community/netbox/issues/20916) - Include stack trace for unhandled exceptions in job logs
-* [#21157](https://github.com/netbox-community/netbox/issues/21157) - Include all public model classes in export template context
-* [#21409](https://github.com/netbox-community/netbox/issues/21409) - Introduce `CHANGELOG_RETAIN_CREATE_LAST_UPDATE` configuration parameter to retain creation & most recent update record in change log for each object
-* [#21575](https://github.com/netbox-community/netbox/issues/21575) - Introduce `{vc_position}` template variable for device component template name/label
-* [#21662](https://github.com/netbox-community/netbox/issues/21662) - Increase `rf_channel_frequency` precision to 3 decimal places
-* [#21702](https://github.com/netbox-community/netbox/issues/21702) - Include a serialized representation of the HTTP request in each webhook
-* [#21720](https://github.com/netbox-community/netbox/issues/21720) - Align HTTP basic auth regex of `EnhancedURLValidator` with Django's `URLValidator`
-* [#21770](https://github.com/netbox-community/netbox/issues/21770) - Enable specifying columns to include/exclude on embedded tables
-* [#21771](https://github.com/netbox-community/netbox/issues/21771) - Add support for partial tag assignment (`add_tags`) and removal (`remove_tags`) via REST API
-* [#21780](https://github.com/netbox-community/netbox/issues/21780) - Add changelog message support to bulk creation of IP addresses
-* [#21865](https://github.com/netbox-community/netbox/issues/21865) - Allow setting empty `INTERNAL_IPS` to enable debug toolbar for all clients
-
-### Performance
-
-* [#21455](https://github.com/netbox-community/netbox/issues/21455) - Ensure PostgreSQL indexes exist to support the default ordering of each model
-
-### Plugins
-
-* [#20924](https://github.com/netbox-community/netbox/issues/20924) - Introduce support for declarative layouts and reusable UI components
-* [#21357](https://github.com/netbox-community/netbox/issues/21357) - Provide an API for plugins to register custom model actions (for permission assignment)
-
-### Deprecations
-
-* [#21284](https://github.com/netbox-community/netbox/issues/21284) - Deprecate the `username` and `request_id` fields in event data
-* [#21304](https://github.com/netbox-community/netbox/issues/21304) - Deprecate the `housekeeping` management command
-* [#21331](https://github.com/netbox-community/netbox/issues/21331) - Deprecate NetBox's custom `querystring` template tag
-* [#21881](https://github.com/netbox-community/netbox/issues/21881) - Deprecate legacy Sentry configuration parameters
-* [#21884](https://github.com/netbox-community/netbox/issues/21884) - Deprecate the obsolete `DEFAULT_ACTION_PERMISSIONS` mapping
-* [#21887](https://github.com/netbox-community/netbox/issues/21887) - Deprecate support for legacy view actions
-* [#21890](https://github.com/netbox-community/netbox/issues/21890) - Deprecate `models` key in application registry
-
-### Other Changes
-
-* [#20984](https://github.com/netbox-community/netbox/issues/20984) - Upgrade to Django 6.0
-* [#21635](https://github.com/netbox-community/netbox/issues/21635) - Migrate documentation site from mkdocs to Zensical
-
-### REST API Changes
-
-* New features:
-    * `ETag` response header and `If-Match` request header support for all individual object endpoints
-    * `start` query parameter for cursor-based pagination on all list endpoints
-    * `add_tags` and `remove_tags` write-only fields on all taggable model serializers
-* New endpoints:
-    * `GET/POST /api/dcim/cable-bundles/`
-    * `GET/PUT/PATCH/DELETE /api/dcim/cable-bundles/<id>/`
-    * `GET/POST /api/dcim/rack-groups/`
-    * `GET/PUT/PATCH/DELETE /api/dcim/rack-groups/<id>/`
-    * `GET/POST /api/virtualization/virtual-machine-types/`
-    * `GET/PUT/PATCH/DELETE /api/virtualization/virtual-machine-types/<id>/`
-* `dcim.Cable`
-    * Add optional foreign key field `bundle`
-* `dcim.Device`
-    * The `primary_ip`, `primary_ip4`, `primary_ip6`, and `oob_ip` nested representations now include `nat_inside` and `nat_outside`
-* `dcim.DeviceBay`
-    * Add boolean field `enabled`
-    * Add read-only boolean field `_occupied`
-* `dcim.DeviceBayTemplate`
-    * Add boolean field `enabled`
-* `dcim.Module`
-    * Add write-only boolean fields `replicate_components` and `adopt_components`
-* `dcim.ModuleBay`
-    * Add boolean field `enabled`
-    * Add read-only boolean field `_occupied`
-* `dcim.ModuleBayTemplate`
-    * Add boolean field `enabled`
-* `dcim.Rack`
-    * Add optional foreign key field `group`
-* `dcim.RackReservation`
-    * Add read-only integer field `unit_count`
-* `extras.CustomField`
-    * Add JSON field `validation_schema`
-* `ipam.ASN`
-    * Add optional foreign key field `role`
-* `ipam.Role`
-    * Annotate count of assigned ASNs (`asn_count`)
-* `ipam.VLANGroup`
-    * Add read-only field `total_vlan_ids`
-* `virtualization.VirtualMachine`
-    * Add optional foreign key field `virtual_machine_type`
-    * The `primary_ip`, `primary_ip4`, and `primary_ip6` nested representations now include `nat_inside` and `nat_outside`
-    * The `cluster` field is now optional (nullable)

mkdocs.yml

@@ -1,4 +1,3 @@
-# Note: NetBox has migrated from MkDocs to Zensical
 site_name: NetBox Documentation
 site_dir: netbox/project-static/docs
 site_url: https://docs.netbox.dev/
@@ -152,7 +151,6 @@ nav:
             - Filters & Filter Sets: 'plugins/development/filtersets.md'
             - Search: 'plugins/development/search.md'
             - Event Types: 'plugins/development/event-types.md'
-            - Permissions: 'plugins/development/permissions.md'
             - Data Backends: 'plugins/development/data-backends.md'
             - Webhooks: 'plugins/development/webhooks.md'
             - User Interface: 'plugins/development/user-interface.md'
@@ -191,7 +189,6 @@ nav:
             - Job: 'models/core/job.md'
         - DCIM:
             - Cable: 'models/dcim/cable.md'
-            - CableBundle: 'models/dcim/cablebundle.md'
             - ConsolePort: 'models/dcim/consoleport.md'
             - ConsolePortTemplate: 'models/dcim/consoleporttemplate.md'
             - ConsoleServerPort: 'models/dcim/consoleserverport.md'
@@ -224,7 +221,6 @@ nav:
             - PowerPort: 'models/dcim/powerport.md'
             - PowerPortTemplate: 'models/dcim/powerporttemplate.md'
             - Rack: 'models/dcim/rack.md'
-            - RackGroup: 'models/dcim/rackgroup.md'
             - RackReservation: 'models/dcim/rackreservation.md'
             - RackRole: 'models/dcim/rackrole.md'
             - RackType: 'models/dcim/racktype.md'
@@ -289,7 +285,6 @@ nav:
             - VMInterface: 'models/virtualization/vminterface.md'
             - VirtualDisk: 'models/virtualization/virtualdisk.md'
             - VirtualMachine: 'models/virtualization/virtualmachine.md'
-            - VirtualMachineType: 'models/virtualization/virtualmachinetype.md'
         - VPN:
             - IKEPolicy: 'models/vpn/ikepolicy.md'
             - IKEProposal: 'models/vpn/ikeproposal.md'
@@ -327,7 +322,6 @@ nav:
         - git Cheat Sheet: 'development/git-cheat-sheet.md'
     - Release Notes:
         - Summary: 'release-notes/index.md'
-        - Version 4.6: 'release-notes/version-4.6.md'
         - Version 4.5: 'release-notes/version-4.5.md'
         - Version 4.4: 'release-notes/version-4.4.md'
         - Version 4.3: 'release-notes/version-4.3.md'

netbox/circuits/forms/model_forms.py

@@ -22,7 +22,7 @@ from utilities.forms.fields import (
     SlugField,
 )
 from utilities.forms.mixins import DistanceValidationMixin
-from utilities.forms.rendering import FieldSet, InlineFields, M2MAddRemoveFields
+from utilities.forms.rendering import FieldSet, InlineFields
 from utilities.forms.widgets import DatePicker, HTMXSelect, NumberWithOptions
 from utilities.templatetags.builtins.filters import bettertitle
 
@@ -48,42 +48,17 @@ class ProviderForm(PrimaryModelForm):
         label=_('ASNs'),
         required=False
     )
-    add_asns = DynamicModelMultipleChoiceField(
-        queryset=ASN.objects.all(),
-        label=_('Add ASNs'),
-        required=False
-    )
-    remove_asns = DynamicModelMultipleChoiceField(
-        queryset=ASN.objects.all(),
-        label=_('Remove ASNs'),
-        required=False
-    )
 
     fieldsets = (
-        FieldSet('name', 'slug', M2MAddRemoveFields('asns'), 'description', 'tags'),
+        FieldSet('name', 'slug', 'asns', 'description', 'tags'),
     )
 
     class Meta:
         model = Provider
         fields = [
-            'name', 'slug', 'description', 'owner', 'comments', 'tags',
+            'name', 'slug', 'asns', 'description', 'owner', 'comments', 'tags',
         ]
 
-    def __init__(self, *args, **kwargs):
-        super().__init__(*args, **kwargs)
-        if self.instance.pk and (count := self.instance.asns.count()) >= M2MAddRemoveFields.THRESHOLD:
-            # Add/remove mode for large M2M sets
-            self.fields.pop('asns')
-            self.fields['add_asns'].widget.add_query_param('provider_id__n', self.instance.pk)
-            self.fields['remove_asns'].widget.add_query_param('provider_id', self.instance.pk)
-            self.fields['remove_asns'].help_text = _("{count} ASNs currently assigned").format(count=count)
-        else:
-            # Simple mode for new objects or small M2M sets
-            self.fields.pop('add_asns')
-            self.fields.pop('remove_asns')
-            if self.instance.pk:
-                self.initial['asns'] = list(self.instance.asns.values_list('pk', flat=True))
-
 
 class ProviderAccountForm(PrimaryModelForm):
     provider = DynamicModelChoiceField(
@@ -93,14 +68,10 @@ class ProviderAccountForm(PrimaryModelForm):
         quick_add=True
     )
 
-    fieldsets = (
-        FieldSet('provider', 'account', 'name', 'description', 'tags'),
-    )
-
     class Meta:
         model = ProviderAccount
         fields = [
-            'provider', 'account', 'name', 'description', 'owner', 'comments', 'tags',
+            'provider', 'name', 'account', 'description', 'owner', 'comments', 'tags',
         ]
 
 

netbox/circuits/migrations/0057_default_ordering_indexes.py

@@ -1,35 +0,0 @@
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-    dependencies = [
-        ('circuits', '0056_gfk_indexes'),
-        ('contenttypes', '0002_remove_content_type_name'),
-        ('dcim', '0231_interface_rf_channel_frequency_precision'),
-        ('extras', '0136_customfield_validation_schema'),
-        ('tenancy', '0023_add_mptt_tree_indexes'),
-        ('users', '0015_owner'),
-    ]
-
-    operations = [
-        migrations.AddIndex(
-            model_name='circuit',
-            index=models.Index(fields=['provider', 'provider_account', 'cid'], name='circuits_ci_provide_a0c42c_idx'),
-        ),
-        migrations.AddIndex(
-            model_name='circuitgroupassignment',
-            index=models.Index(
-                fields=['group', 'member_type', 'member_id', 'priority', 'id'], name='circuits_ci_group_i_2f8327_idx'
-            ),
-        ),
-        migrations.AddIndex(
-            model_name='virtualcircuit',
-            index=models.Index(
-                fields=['provider_network', 'provider_account', 'cid'], name='circuits_vi_provide_989efa_idx'
-            ),
-        ),
-        migrations.AddIndex(
-            model_name='virtualcircuittermination',
-            index=models.Index(fields=['virtual_circuit', 'role', 'id'], name='circuits_vi_virtual_4b5c0c_idx'),
-        ),
-    ]

netbox/circuits/models/circuits.py

@@ -144,9 +144,6 @@ class Circuit(ContactsMixin, ImageAttachmentsMixin, DistanceMixin, PrimaryModel)
                 name='%(app_label)s_%(class)s_unique_provideraccount_cid'
             ),
         )
-        indexes = (
-            models.Index(fields=('provider', 'provider_account', 'cid')),  # Default ordering
-        )
         verbose_name = _('circuit')
         verbose_name_plural = _('circuits')
 
@@ -224,9 +221,6 @@ class CircuitGroupAssignment(CustomFieldsMixin, ExportTemplatesMixin, TagsMixin,
                 name='%(app_label)s_%(class)s_unique_member_group'
             ),
         )
-        indexes = (
-            models.Index(fields=('group', 'member_type', 'member_id', 'priority', 'id')),  # Default ordering
-        )
         verbose_name = _('Circuit group assignment')
         verbose_name_plural = _('Circuit group assignments')
 
@@ -353,13 +347,6 @@ class CircuitTermination(
         verbose_name = _('circuit termination')
         verbose_name_plural = _('circuit terminations')
 
-    def __init__(self, *args, **kwargs):
-        super().__init__(*args, **kwargs)
-
-        # Cache original values to detect changes
-        self._orig_circuit_id = self.__dict__.get('circuit_id')
-        self._orig_term_side = self.__dict__.get('term_side')
-
     def __str__(self):
         return f'{self.circuit}: Termination {self.term_side}'
 
@@ -373,39 +360,11 @@ class CircuitTermination(
             raise ValidationError(_("A circuit termination must attach to a terminating object."))
 
     def save(self, *args, **kwargs):
-        is_new = self._state.adding
-        update_fields = kwargs.get('update_fields')
-
-        # Only consider circuit/term_side changes if those fields
-        # are actually being persisted
-        if update_fields is not None:
-            tracking_relevant = 'circuit' in update_fields or 'term_side' in update_fields
-        else:
-            tracking_relevant = True
-
-        circuit_changed = tracking_relevant and self._orig_circuit_id and self._orig_circuit_id != self.circuit_id
-        term_side_changed = tracking_relevant and self._orig_term_side and self._orig_term_side != self.term_side
-
         # Cache objects associated with the terminating object (for filtering)
         self.cache_related_objects()
 
         super().save(*args, **kwargs)
 
-        # Clear the old termination reference if circuit or term_side changed
-        if circuit_changed or term_side_changed:
-            old_termination_name = f'termination_{self._orig_term_side.lower()}'
-            Circuit.objects.filter(pk=self._orig_circuit_id).update(**{old_termination_name: None})
-
-        # Update the cache if this is a new termination or circuit/term_side changed
-        if is_new or circuit_changed or term_side_changed:
-            # Update the new circuit's termination reference
-            termination_name = f'termination_{self.term_side.lower()}'
-            Circuit.objects.filter(pk=self.circuit_id).update(**{termination_name: self.pk})
-
-            # Update cached values for subsequent saves
-            self._orig_circuit_id = self.circuit_id
-            self._orig_term_side = self.term_side
-
     def cache_related_objects(self):
         self._provider_network = self._region = self._site_group = self._site = self._location = None
         if self.termination_type:

netbox/circuits/models/virtual_circuits.py

@@ -97,9 +97,6 @@ class VirtualCircuit(ContactsMixin, PrimaryModel):
                 name='%(app_label)s_%(class)s_unique_provideraccount_cid'
             ),
         )
-        indexes = (
-            models.Index(fields=('provider_network', 'provider_account', 'cid')),  # Default ordering
-        )
         verbose_name = _('virtual circuit')
         verbose_name_plural = _('virtual circuits')
 
@@ -153,9 +150,6 @@ class VirtualCircuitTermination(
 
     class Meta:
         ordering = ['virtual_circuit', 'role', 'pk']
-        indexes = (
-            models.Index(fields=('virtual_circuit', 'role', 'id')),  # Default ordering
-        )
         verbose_name = _('virtual circuit termination')
         verbose_name_plural = _('virtual circuit terminations')
 

netbox/circuits/signals.py

@@ -6,6 +6,17 @@ from dcim.signals import rebuild_paths
 from .models import CircuitTermination
 
 
+@receiver(post_save, sender=CircuitTermination)
+def update_circuit(instance, **kwargs):
+    """
+    When a CircuitTermination has been modified, update its parent Circuit.
+    """
+    termination_name = f'termination_{instance.term_side.lower()}'
+    instance.circuit.refresh_from_db()
+    setattr(instance.circuit, termination_name, instance)
+    instance.circuit.save()
+
+
 @receiver((post_save, post_delete), sender=CircuitTermination)
 def rebuild_cablepaths(instance, raw=False, **kwargs):
     """

netbox/circuits/tables/circuits.py

@@ -190,16 +190,14 @@ class CircuitGroupAssignmentTable(NetBoxTable):
     provider = tables.Column(
         accessor='member__provider',
         verbose_name=_('Provider'),
-        orderable=False,
-        linkify=True,
+        linkify=True
     )
     member_type = columns.ContentTypeColumn(
         verbose_name=_('Type')
     )
     member = tables.Column(
         verbose_name=_('Circuit'),
-        orderable=False,
-        linkify=True,
+        linkify=True
     )
     priority = tables.Column(
         verbose_name=_('Priority'),

netbox/circuits/tables/virtual_circuits.py

@@ -95,7 +95,6 @@ class VirtualCircuitTerminationTable(NetBoxTable):
         verbose_name=_('Provider network')
     )
     provider_account = tables.Column(
-        accessor=tables.A('virtual_circuit__provider_account'),
         linkify=True,
         verbose_name=_('Account')
     )
@@ -113,7 +112,7 @@ class VirtualCircuitTerminationTable(NetBoxTable):
     class Meta(NetBoxTable.Meta):
         model = VirtualCircuitTermination
         fields = (
-            'pk', 'id', 'virtual_circuit', 'provider', 'provider_network', 'provider_account', 'role', 'interface',
+            'pk', 'id', 'virtual_circuit', 'provider', 'provider_network', 'provider_account', 'role', 'interfaces',
             'description', 'created', 'last_updated', 'actions',
         )
         default_columns = (

netbox/circuits/tests/test_models.py

@@ -1,148 +0,0 @@
-from django.test import TestCase
-
-from circuits.models import Circuit, CircuitTermination, CircuitType, Provider, ProviderNetwork
-from dcim.models import Site
-
-
-class CircuitTerminationTestCase(TestCase):
-
-    @classmethod
-    def setUpTestData(cls):
-        provider = Provider.objects.create(name='Provider 1', slug='provider-1')
-        circuit_type = CircuitType.objects.create(name='Circuit Type 1', slug='circuit-type-1')
-
-        cls.sites = (
-            Site.objects.create(name='Site 1', slug='site-1'),
-            Site.objects.create(name='Site 2', slug='site-2'),
-        )
-
-        cls.circuits = (
-            Circuit.objects.create(cid='Circuit 1', provider=provider, type=circuit_type),
-            Circuit.objects.create(cid='Circuit 2', provider=provider, type=circuit_type),
-        )
-
-        cls.provider_network = ProviderNetwork.objects.create(name='Provider Network 1', provider=provider)
-
-    def test_circuit_termination_creation_populates_circuit_cache(self):
-        """
-        When a CircuitTermination is created, the parent Circuit's termination_a or termination_z
-        cache field should be populated.
-        """
-        # Create A termination
-        termination_a = CircuitTermination.objects.create(
-            circuit=self.circuits[0],
-            term_side='A',
-            termination=self.sites[0],
-        )
-        self.circuits[0].refresh_from_db()
-        self.assertEqual(self.circuits[0].termination_a, termination_a)
-        self.assertIsNone(self.circuits[0].termination_z)
-
-        # Create Z termination
-        termination_z = CircuitTermination.objects.create(
-            circuit=self.circuits[0],
-            term_side='Z',
-            termination=self.sites[1],
-        )
-        self.circuits[0].refresh_from_db()
-        self.assertEqual(self.circuits[0].termination_a, termination_a)
-        self.assertEqual(self.circuits[0].termination_z, termination_z)
-
-    def test_circuit_termination_circuit_change_clears_old_cache(self):
-        """
-        When a CircuitTermination's circuit is changed, the old Circuit's cache should be cleared
-        and the new Circuit's cache should be populated.
-        """
-        # Create termination on self.circuits[0]
-        termination = CircuitTermination.objects.create(
-            circuit=self.circuits[0],
-            term_side='A',
-            termination=self.sites[0],
-        )
-        self.circuits[0].refresh_from_db()
-        self.assertEqual(self.circuits[0].termination_a, termination)
-
-        # Move termination to self.circuits[1]
-        termination.circuit = self.circuits[1]
-        termination.save()
-
-        self.circuits[0].refresh_from_db()
-        self.circuits[1].refresh_from_db()
-
-        # Old circuit's cache should be cleared
-        self.assertIsNone(self.circuits[0].termination_a)
-        # New circuit's cache should be populated
-        self.assertEqual(self.circuits[1].termination_a, termination)
-
-    def test_circuit_termination_term_side_change_clears_old_cache(self):
-        """
-        When a CircuitTermination's term_side is changed, the old side's cache should be cleared
-        and the new side's cache should be populated.
-        """
-        # Create A termination
-        termination = CircuitTermination.objects.create(
-            circuit=self.circuits[0],
-            term_side='A',
-            termination=self.sites[0],
-        )
-        self.circuits[0].refresh_from_db()
-        self.assertEqual(self.circuits[0].termination_a, termination)
-        self.assertIsNone(self.circuits[0].termination_z)
-
-        # Change from A to Z
-        termination.term_side = 'Z'
-        termination.save()
-
-        self.circuits[0].refresh_from_db()
-
-        # A side should be cleared, Z side should be populated
-        self.assertIsNone(self.circuits[0].termination_a)
-        self.assertEqual(self.circuits[0].termination_z, termination)
-
-    def test_circuit_termination_circuit_and_term_side_change(self):
-        """
-        When both circuit and term_side are changed, the old Circuit's old side cache should be
-        cleared and the new Circuit's new side cache should be populated.
-        """
-        # Create A termination on self.circuits[0]
-        termination = CircuitTermination.objects.create(
-            circuit=self.circuits[0],
-            term_side='A',
-            termination=self.sites[0],
-        )
-        self.circuits[0].refresh_from_db()
-        self.assertEqual(self.circuits[0].termination_a, termination)
-
-        # Change to self.circuits[1] Z side
-        termination.circuit = self.circuits[1]
-        termination.term_side = 'Z'
-        termination.save()
-
-        self.circuits[0].refresh_from_db()
-        self.circuits[1].refresh_from_db()
-
-        # Old circuit's A side should be cleared
-        self.assertIsNone(self.circuits[0].termination_a)
-        self.assertIsNone(self.circuits[0].termination_z)
-        # New circuit's Z side should be populated
-        self.assertIsNone(self.circuits[1].termination_a)
-        self.assertEqual(self.circuits[1].termination_z, termination)
-
-    def test_circuit_termination_deletion_clears_cache(self):
-        """
-        When a CircuitTermination is deleted, the parent Circuit's cache should be cleared.
-        """
-        termination = CircuitTermination.objects.create(
-            circuit=self.circuits[0],
-            term_side='A',
-            termination=self.sites[0],
-        )
-        self.circuits[0].refresh_from_db()
-        self.assertEqual(self.circuits[0].termination_a, termination)
-
-        # Delete the termination
-        termination.delete()
-        self.circuits[0].refresh_from_db()
-
-        # Cache should be cleared (SET_NULL behavior)
-        self.assertIsNone(self.circuits[0].termination_a)

netbox/circuits/tests/test_tables.py

@@ -1,46 +1,23 @@
-from circuits.tables import *
-from utilities.testing import TableTestCases
+from django.test import RequestFactory, TestCase, tag
+
+from circuits.models import CircuitTermination
+from circuits.tables import CircuitTerminationTable
 
 
-class CircuitTypeTableTest(TableTestCases.StandardTableTestCase):
-    table = CircuitTypeTable
+@tag('regression')
+class CircuitTerminationTableTest(TestCase):
+    def test_every_orderable_field_does_not_throw_exception(self):
+        terminations = CircuitTermination.objects.all()
+        disallowed = {'actions', }
 
+        orderable_columns = [
+            column.name for column in CircuitTerminationTable(terminations).columns
+            if column.orderable and column.name not in disallowed
+        ]
+        fake_request = RequestFactory().get("/")
 
-class CircuitTableTest(TableTestCases.StandardTableTestCase):
-    table = CircuitTable
-
-
-class CircuitTerminationTableTest(TableTestCases.StandardTableTestCase):
-    table = CircuitTerminationTable
-
-
-class CircuitGroupTableTest(TableTestCases.StandardTableTestCase):
-    table = CircuitGroupTable
-
-
-class CircuitGroupAssignmentTableTest(TableTestCases.StandardTableTestCase):
-    table = CircuitGroupAssignmentTable
-
-
-class ProviderTableTest(TableTestCases.StandardTableTestCase):
-    table = ProviderTable
-
-
-class ProviderAccountTableTest(TableTestCases.StandardTableTestCase):
-    table = ProviderAccountTable
-
-
-class ProviderNetworkTableTest(TableTestCases.StandardTableTestCase):
-    table = ProviderNetworkTable
-
-
-class VirtualCircuitTypeTableTest(TableTestCases.StandardTableTestCase):
-    table = VirtualCircuitTypeTable
-
-
-class VirtualCircuitTableTest(TableTestCases.StandardTableTestCase):
-    table = VirtualCircuitTable
-
-
-class VirtualCircuitTerminationTableTest(TableTestCases.StandardTableTestCase):
-    table = VirtualCircuitTerminationTable
+        for col in orderable_columns:
+            for dir in ('-', ''):
+                table = CircuitTerminationTable(terminations)
+                table.order_by = f'{dir}{col}'
+                table.as_html(fake_request)

netbox/circuits/tests/test_views.py

@@ -196,20 +196,6 @@ class CircuitTestCase(ViewTestCases.PrimaryObjectViewTestCase):
             'comments': 'New comments',
         }
 
-    def test_circuit_type_display_colored(self):
-        circuit_type = CircuitType.objects.first()
-        circuit_type.color = '12ab34'
-        circuit_type.save()
-
-        circuit = Circuit.objects.first()
-
-        self.add_permissions('circuits.view_circuit')
-        response = self.client.get(circuit.get_absolute_url())
-
-        self.assertHttpStatus(response, 200)
-        self.assertContains(response, circuit_type.name)
-        self.assertContains(response, 'background-color: #12ab34')
-
     @override_settings(EXEMPT_VIEW_PERMISSIONS=['*'], EXEMPT_EXCLUDE_MODELS=[])
     def test_bulk_import_objects_with_terminations(self):
         site = Site.objects.first()

netbox/circuits/ui/panels.py

@@ -1,155 +0,0 @@
-from django.contrib.contenttypes.models import ContentType
-from django.utils.translation import gettext_lazy as _
-
-from netbox.ui import actions, attrs, panels
-from utilities.data import resolve_attr_path
-
-
-class CircuitCircuitTerminationPanel(panels.ObjectPanel):
-    """
-    A panel showing the CircuitTermination assigned to the object.
-    """
-
-    template_name = 'circuits/panels/circuit_circuit_termination.html'
-    title = _('Termination')
-
-    def __init__(self, side, accessor=None, **kwargs):
-        super().__init__(accessor=accessor, **kwargs)
-        self.side = side
-
-    def get_context(self, context):
-        return {
-            **super().get_context(context),
-            'side': self.side,
-            'termination': resolve_attr_path(context, f'{self.accessor}.termination_{self.side.lower()}'),
-        }
-
-
-class CircuitGroupAssignmentsPanel(panels.ObjectsTablePanel):
-    """
-    A panel showing all Circuit Groups attached to the object.
-    """
-
-    title = _('Group Assignments')
-    actions = [
-        actions.AddObject(
-            'circuits.CircuitGroupAssignment',
-            url_params={
-                'member_type': lambda ctx: ContentType.objects.get_for_model(ctx['object']).pk,
-                'member': lambda ctx: ctx['object'].pk,
-                'return_url': lambda ctx: ctx['object'].get_absolute_url(),
-            },
-            label=_('Assign Group'),
-        ),
-    ]
-
-    def __init__(self, **kwargs):
-        super().__init__(
-            'circuits.CircuitGroupAssignment',
-            filters={
-                'member_type_id': lambda ctx: ContentType.objects.get_for_model(ctx['object']).pk,
-                'member_id': lambda ctx: ctx['object'].pk,
-            },
-            **kwargs,
-        )
-
-
-class CircuitTerminationPanel(panels.ObjectAttributesPanel):
-    title = _('Circuit Termination')
-    circuit = attrs.RelatedObjectAttr('circuit', linkify=True)
-    provider = attrs.RelatedObjectAttr('circuit.provider', linkify=True)
-    termination = attrs.GenericForeignKeyAttr('termination', linkify=True, label=_('Termination point'))
-    connection = attrs.TemplatedAttr(
-        'pk',
-        template_name='circuits/circuit_termination/attrs/connection.html',
-        label=_('Connection'),
-    )
-    speed = attrs.TemplatedAttr(
-        'port_speed',
-        template_name='circuits/circuit_termination/attrs/speed.html',
-        label=_('Speed'),
-    )
-    xconnect_id = attrs.TextAttr('xconnect_id', label=_('Cross-Connect'), style='font-monospace')
-    pp_info = attrs.TextAttr('pp_info', label=_('Patch Panel/Port'))
-    description = attrs.TextAttr('description')
-
-
-class CircuitGroupPanel(panels.OrganizationalObjectPanel):
-    tenant = attrs.RelatedObjectAttr('tenant', linkify=True, grouped_by='group')
-
-
-class CircuitGroupAssignmentPanel(panels.ObjectAttributesPanel):
-    group = attrs.RelatedObjectAttr('group', linkify=True)
-    provider = attrs.RelatedObjectAttr('member.provider', linkify=True)
-    member = attrs.GenericForeignKeyAttr('member', linkify=True)
-    priority = attrs.ChoiceAttr('priority')
-
-
-class CircuitPanel(panels.ObjectAttributesPanel):
-    provider = attrs.RelatedObjectAttr('provider', linkify=True)
-    provider_account = attrs.RelatedObjectAttr('provider_account', linkify=True)
-    cid = attrs.TextAttr('cid', label=_('Circuit ID'), style='font-monospace', copy_button=True)
-    type = attrs.RelatedObjectAttr('type', linkify=True, colored=True)
-    status = attrs.ChoiceAttr('status')
-    distance = attrs.NumericAttr('distance', unit_accessor='get_distance_unit_display')
-    tenant = attrs.RelatedObjectAttr('tenant', linkify=True, grouped_by='group')
-    install_date = attrs.DateTimeAttr('install_date', spec='date')
-    termination_date = attrs.DateTimeAttr('termination_date', spec='date')
-    commit_rate = attrs.TemplatedAttr('commit_rate', template_name='circuits/circuit/attrs/commit_rate.html')
-    description = attrs.TextAttr('description')
-
-
-class CircuitTypePanel(panels.OrganizationalObjectPanel):
-    color = attrs.ColorAttr('color')
-
-
-class ProviderPanel(panels.ObjectAttributesPanel):
-    name = attrs.TextAttr('name')
-    asns = attrs.RelatedObjectListAttr('asns', linkify=True, label=_('ASNs'))
-    description = attrs.TextAttr('description')
-
-
-class ProviderAccountPanel(panels.ObjectAttributesPanel):
-    provider = attrs.RelatedObjectAttr('provider', linkify=True)
-    account = attrs.TextAttr('account', style='font-monospace', copy_button=True)
-    name = attrs.TextAttr('name')
-    description = attrs.TextAttr('description')
-
-
-class ProviderNetworkPanel(panels.ObjectAttributesPanel):
-    provider = attrs.RelatedObjectAttr('provider', linkify=True)
-    name = attrs.TextAttr('name')
-    service_id = attrs.TextAttr('service_id', label=_('Service ID'), style='font-monospace', copy_button=True)
-    description = attrs.TextAttr('description')
-
-
-class VirtualCircuitTypePanel(panels.OrganizationalObjectPanel):
-    color = attrs.ColorAttr('color')
-
-
-class VirtualCircuitPanel(panels.ObjectAttributesPanel):
-    provider = attrs.RelatedObjectAttr('provider', linkify=True)
-    provider_network = attrs.RelatedObjectAttr('provider_network', linkify=True)
-    provider_account = attrs.RelatedObjectAttr('provider_account', linkify=True)
-    cid = attrs.TextAttr('cid', label=_('Circuit ID'), style='font-monospace', copy_button=True)
-    type = attrs.RelatedObjectAttr('type', linkify=True, colored=True)
-    status = attrs.ChoiceAttr('status')
-    tenant = attrs.RelatedObjectAttr('tenant', linkify=True, grouped_by='group')
-    description = attrs.TextAttr('description')
-
-
-class VirtualCircuitTerminationPanel(panels.ObjectAttributesPanel):
-    provider = attrs.RelatedObjectAttr('virtual_circuit.provider', linkify=True)
-    provider_network = attrs.RelatedObjectAttr('virtual_circuit.provider_network', linkify=True)
-    provider_account = attrs.RelatedObjectAttr('virtual_circuit.provider_account', linkify=True)
-    virtual_circuit = attrs.RelatedObjectAttr('virtual_circuit', linkify=True)
-    role = attrs.ChoiceAttr('role')
-
-
-class VirtualCircuitTerminationInterfacePanel(panels.ObjectAttributesPanel):
-    title = _('Interface')
-
-    device = attrs.RelatedObjectAttr('interface.device', linkify=True)
-    interface = attrs.RelatedObjectAttr('interface', linkify=True)
-    type = attrs.ChoiceAttr('interface.type')
-    description = attrs.TextAttr('interface.description')

netbox/circuits/views.py

@@ -1,22 +1,13 @@
-from django.utils.translation import gettext_lazy as _
 
 from dcim.views import PathTraceView
-from extras.ui.panels import CustomFieldsPanel, ImageAttachmentsPanel, TagsPanel
 from ipam.models import ASN
 from netbox.object_actions import AddObject, BulkDelete, BulkEdit, BulkExport, BulkImport
-from netbox.ui import actions, layout
-from netbox.ui.panels import (
-    CommentsPanel,
-    ObjectsTablePanel,
-    RelatedObjectsPanel,
-)
 from netbox.views import generic
 from utilities.query import count_related
 from utilities.views import GetRelatedModelsMixin, register_model_view
 
 from . import filtersets, forms, tables
 from .models import *
-from .ui import panels
 
 #
 # Providers
@@ -38,37 +29,6 @@ class ProviderListView(generic.ObjectListView):
 @register_model_view(Provider)
 class ProviderView(GetRelatedModelsMixin, generic.ObjectView):
     queryset = Provider.objects.all()
-    layout = layout.SimpleLayout(
-        left_panels=[
-            panels.ProviderPanel(),
-            TagsPanel(),
-            CommentsPanel(),
-        ],
-        right_panels=[
-            RelatedObjectsPanel(),
-            CustomFieldsPanel(),
-        ],
-        bottom_panels=[
-            ObjectsTablePanel(
-                model='circuits.ProviderAccount',
-                filters={'provider_id': lambda ctx: ctx['object'].pk},
-                exclude_columns=['provider'],
-                actions=[
-                    actions.AddObject(
-                        'circuits.ProviderAccount', url_params={'provider': lambda ctx: ctx['object'].pk}
-                    ),
-                ],
-            ),
-            ObjectsTablePanel(
-                model='circuits.Circuit',
-                filters={'provider_id': lambda ctx: ctx['object'].pk},
-                exclude_columns=['provider'],
-                actions=[
-                    actions.AddObject('circuits.Circuit', url_params={'provider': lambda ctx: ctx['object'].pk}),
-                ],
-            ),
-        ],
-    )
 
     def get_extra_context(self, request, instance):
         return {
@@ -84,7 +44,7 @@ class ProviderView(GetRelatedModelsMixin, generic.ObjectView):
                         'provider_id',
                     ),
                 ),
-            ),
+                ),
         }
 
 
@@ -148,33 +108,6 @@ class ProviderAccountListView(generic.ObjectListView):
 @register_model_view(ProviderAccount)
 class ProviderAccountView(GetRelatedModelsMixin, generic.ObjectView):
     queryset = ProviderAccount.objects.all()
-    layout = layout.SimpleLayout(
-        left_panels=[
-            panels.ProviderAccountPanel(),
-            TagsPanel(),
-        ],
-        right_panels=[
-            RelatedObjectsPanel(),
-            CommentsPanel(),
-            CustomFieldsPanel(),
-        ],
-        bottom_panels=[
-            ObjectsTablePanel(
-                model='circuits.Circuit',
-                filters={'provider_account_id': lambda ctx: ctx['object'].pk},
-                exclude_columns=['provider_account'],
-                actions=[
-                    actions.AddObject(
-                        'circuits.Circuit',
-                        url_params={
-                            'provider': lambda ctx: ctx['object'].provider.pk,
-                            'provider_account': lambda ctx: ctx['object'].pk,
-                        },
-                    ),
-                ],
-            ),
-        ],
-    )
 
     def get_extra_context(self, request, instance):
         return {
@@ -241,33 +174,6 @@ class ProviderNetworkListView(generic.ObjectListView):
 @register_model_view(ProviderNetwork)
 class ProviderNetworkView(GetRelatedModelsMixin, generic.ObjectView):
     queryset = ProviderNetwork.objects.all()
-    layout = layout.SimpleLayout(
-        left_panels=[
-            panels.ProviderNetworkPanel(),
-            TagsPanel(),
-        ],
-        right_panels=[
-            RelatedObjectsPanel(),
-            CommentsPanel(),
-            CustomFieldsPanel(),
-        ],
-        bottom_panels=[
-            ObjectsTablePanel(
-                model='circuits.Circuit',
-                filters={'provider_network_id': lambda ctx: ctx['object'].pk},
-            ),
-            ObjectsTablePanel(
-                model='circuits.VirtualCircuit',
-                filters={'provider_network_id': lambda ctx: ctx['object'].pk},
-                exclude_columns=['provider_network'],
-                actions=[
-                    actions.AddObject(
-                        'circuits.VirtualCircuit', url_params={'provider_network': lambda ctx: ctx['object'].pk}
-                    ),
-                ],
-            ),
-        ],
-    )
 
     def get_extra_context(self, request, instance):
         return {
@@ -345,17 +251,6 @@ class CircuitTypeListView(generic.ObjectListView):
 @register_model_view(CircuitType)
 class CircuitTypeView(GetRelatedModelsMixin, generic.ObjectView):
     queryset = CircuitType.objects.all()
-    layout = layout.SimpleLayout(
-        left_panels=[
-            panels.CircuitTypePanel(),
-            TagsPanel(),
-        ],
-        right_panels=[
-            RelatedObjectsPanel(),
-            CommentsPanel(),
-            CustomFieldsPanel(),
-        ],
-    )
 
     def get_extra_context(self, request, instance):
         return {
@@ -423,20 +318,6 @@ class CircuitListView(generic.ObjectListView):
 @register_model_view(Circuit)
 class CircuitView(generic.ObjectView):
     queryset = Circuit.objects.all()
-    layout = layout.SimpleLayout(
-        left_panels=[
-            panels.CircuitPanel(),
-            panels.CircuitGroupAssignmentsPanel(),
-            CustomFieldsPanel(),
-            TagsPanel(),
-            CommentsPanel(),
-        ],
-        right_panels=[
-            panels.CircuitCircuitTerminationPanel(side='A'),
-            panels.CircuitCircuitTerminationPanel(side='Z'),
-            ImageAttachmentsPanel(),
-        ],
-    )
 
 
 @register_model_view(Circuit, 'add', detail=False)
@@ -509,15 +390,6 @@ class CircuitTerminationListView(generic.ObjectListView):
 @register_model_view(CircuitTermination)
 class CircuitTerminationView(generic.ObjectView):
     queryset = CircuitTermination.objects.all()
-    layout = layout.SimpleLayout(
-        left_panels=[
-            panels.CircuitTerminationPanel(),
-        ],
-        right_panels=[
-            CustomFieldsPanel(),
-            TagsPanel(),
-        ],
-    )
 
 
 @register_model_view(CircuitTermination, 'add', detail=False)
@@ -574,17 +446,6 @@ class CircuitGroupListView(generic.ObjectListView):
 @register_model_view(CircuitGroup)
 class CircuitGroupView(GetRelatedModelsMixin, generic.ObjectView):
     queryset = CircuitGroup.objects.all()
-    layout = layout.SimpleLayout(
-        left_panels=[
-            panels.CircuitGroupPanel(),
-            TagsPanel(),
-        ],
-        right_panels=[
-            RelatedObjectsPanel(),
-            CommentsPanel(),
-            CustomFieldsPanel(),
-        ],
-    )
 
     def get_extra_context(self, request, instance):
         return {
@@ -647,15 +508,6 @@ class CircuitGroupAssignmentListView(generic.ObjectListView):
 @register_model_view(CircuitGroupAssignment)
 class CircuitGroupAssignmentView(generic.ObjectView):
     queryset = CircuitGroupAssignment.objects.all()
-    layout = layout.SimpleLayout(
-        left_panels=[
-            panels.CircuitGroupAssignmentPanel(),
-            TagsPanel(),
-        ],
-        right_panels=[
-            CustomFieldsPanel(),
-        ],
-    )
 
 
 @register_model_view(CircuitGroupAssignment, 'add', detail=False)
@@ -708,17 +560,6 @@ class VirtualCircuitTypeListView(generic.ObjectListView):
 @register_model_view(VirtualCircuitType)
 class VirtualCircuitTypeView(GetRelatedModelsMixin, generic.ObjectView):
     queryset = VirtualCircuitType.objects.all()
-    layout = layout.SimpleLayout(
-        left_panels=[
-            panels.VirtualCircuitTypePanel(),
-            TagsPanel(),
-        ],
-        right_panels=[
-            RelatedObjectsPanel(),
-            CommentsPanel(),
-            CustomFieldsPanel(),
-        ],
-    )
 
     def get_extra_context(self, request, instance):
         return {
@@ -786,31 +627,6 @@ class VirtualCircuitListView(generic.ObjectListView):
 @register_model_view(VirtualCircuit)
 class VirtualCircuitView(generic.ObjectView):
     queryset = VirtualCircuit.objects.all()
-    layout = layout.SimpleLayout(
-        left_panels=[
-            panels.VirtualCircuitPanel(),
-            TagsPanel(),
-        ],
-        right_panels=[
-            CustomFieldsPanel(),
-            CommentsPanel(),
-            panels.CircuitGroupAssignmentsPanel(),
-        ],
-        bottom_panels=[
-            ObjectsTablePanel(
-                model='circuits.VirtualCircuitTermination',
-                title=_('Terminations'),
-                filters={'virtual_circuit_id': lambda ctx: ctx['object'].pk},
-                exclude_columns=['virtual_circuit'],
-                actions=[
-                    actions.AddObject(
-                        'circuits.VirtualCircuitTermination',
-                        url_params={'virtual_circuit': lambda ctx: ctx['object'].pk},
-                    ),
-                ],
-            ),
-        ],
-    )
 
 
 @register_model_view(VirtualCircuit, 'add', detail=False)
@@ -882,16 +698,6 @@ class VirtualCircuitTerminationListView(generic.ObjectListView):
 @register_model_view(VirtualCircuitTermination)
 class VirtualCircuitTerminationView(generic.ObjectView):
     queryset = VirtualCircuitTermination.objects.all()
-    layout = layout.SimpleLayout(
-        left_panels=[
-            panels.VirtualCircuitTerminationPanel(),
-            TagsPanel(),
-            CustomFieldsPanel(),
-        ],
-        right_panels=[
-            panels.VirtualCircuitTerminationInterfacePanel(),
-        ],
-    )
 
 
 @register_model_view(VirtualCircuitTermination, 'edit')

netbox/core/api/views.py

@@ -2,7 +2,7 @@ from django.http import Http404, HttpResponse
 from django.shortcuts import get_object_or_404
 from django.utils.translation import gettext_lazy as _
 from django_rq.queues import get_redis_connection
-from django_rq.settings import get_queues_list
+from django_rq.settings import QUEUES_LIST
 from django_rq.utils import get_statistics
 from drf_spectacular.types import OpenApiTypes
 from drf_spectacular.utils import OpenApiParameter, extend_schema
@@ -195,7 +195,7 @@ class BackgroundWorkerViewSet(BaseRQViewSet):
         return 'Background Workers'
 
     def get_data(self):
-        config = get_queues_list()[0]
+        config = QUEUES_LIST[0]
         return Worker.all(get_redis_connection(config['connection_config']))
 
     @extend_schema(
@@ -205,7 +205,7 @@ class BackgroundWorkerViewSet(BaseRQViewSet):
     )
     def retrieve(self, request, name):
         # all the RQ queues should use the same connection
-        config = get_queues_list()[0]
+        config = QUEUES_LIST[0]
         workers = Worker.all(get_redis_connection(config['connection_config']))
         worker = next((item for item in workers if item.name == name), None)
         if not worker:
@@ -229,7 +229,7 @@ class BackgroundTaskViewSet(BaseRQViewSet):
         return get_rq_jobs()
 
     def get_task_from_id(self, task_id):
-        config = get_queues_list()[0]
+        config = QUEUES_LIST[0]
         task = RQ_Job.fetch(task_id, connection=get_redis_connection(config['connection_config']))
         if not task:
             raise Http404

netbox/core/forms/model_forms.py

@@ -43,7 +43,7 @@ class DataSourceForm(PrimaryModelForm):
                 attrs={
                     'rows': 5,
                     'class': 'font-monospace',
-                    'placeholder': '.cache\n*.txt\nsubdir/*'
+                    'placeholder': '.cache\n*.txt'
                 }
             ),
         }
@@ -165,10 +165,9 @@ class ConfigRevisionForm(forms.ModelForm, metaclass=ConfigFormMetaclass):
         FieldSet('PAGINATE_COUNT', 'MAX_PAGE_SIZE', name=_('Pagination')),
         FieldSet('CUSTOM_VALIDATORS', 'PROTECTION_RULES', name=_('Validation')),
         FieldSet('DEFAULT_USER_PREFERENCES', name=_('User Preferences')),
-        FieldSet('CHANGELOG_RETENTION', 'CHANGELOG_RETAIN_CREATE_LAST_UPDATE', name=_('Change Log')),
         FieldSet(
-            'MAINTENANCE_MODE', 'COPILOT_ENABLED', 'GRAPHQL_ENABLED', 'JOB_RETENTION', 'MAPS_URL',
-            name=_('Miscellaneous'),
+            'MAINTENANCE_MODE', 'COPILOT_ENABLED', 'GRAPHQL_ENABLED', 'CHANGELOG_RETENTION', 'JOB_RETENTION',
+            'MAPS_URL', name=_('Miscellaneous'),
         ),
         FieldSet('comment', name=_('Config Revision'))
     )

netbox/core/jobs.py

@@ -5,7 +5,6 @@ from importlib import import_module
 import requests
 from django.conf import settings
 from django.core.cache import cache
-from django.db.models import Exists, OuterRef, Subquery
 from django.utils import timezone
 from packaging import version
 
@@ -15,7 +14,7 @@ from netbox.jobs import JobRunner, system_job
 from netbox.search.backends import search_backend
 from utilities.proxy import resolve_proxies
 
-from .choices import DataSourceStatusChoices, JobIntervalChoices, ObjectChangeActionChoices
+from .choices import DataSourceStatusChoices, JobIntervalChoices
 from .models import DataSource
 
 
@@ -127,51 +126,19 @@ class SystemHousekeepingJob(JobRunner):
         """
         Delete any ObjectChange records older than the configured changelog retention time (if any).
         """
-        self.logger.info('Pruning old changelog entries...')
+        self.logger.info("Pruning old changelog entries...")
         config = Config()
         if not config.CHANGELOG_RETENTION:
-            self.logger.info('No retention period specified; skipping.')
+            self.logger.info("No retention period specified; skipping.")
             return
 
         cutoff = timezone.now() - timedelta(days=config.CHANGELOG_RETENTION)
-        self.logger.debug(f'Changelog retention period: {config.CHANGELOG_RETENTION} days ({cutoff:%Y-%m-%d %H:%M:%S})')
+        self.logger.debug(
+            f"Changelog retention period: {config.CHANGELOG_RETENTION} days ({cutoff:%Y-%m-%d %H:%M:%S})"
+        )
 
-        expired_qs = ObjectChange.objects.filter(time__lt=cutoff)
-
-        # When enabled, retain each object's original create record and most recent update record while pruning expired
-        # changelog entries. This applies only to objects without a delete record.
-        if config.CHANGELOG_RETAIN_CREATE_LAST_UPDATE:
-            self.logger.debug('Retaining changelog create records and last update records (excluding deleted objects)')
-
-            deleted_exists = ObjectChange.objects.filter(
-                action=ObjectChangeActionChoices.ACTION_DELETE,
-                changed_object_type_id=OuterRef('changed_object_type_id'),
-                changed_object_id=OuterRef('changed_object_id'),
-            )
-
-            # Keep create records only where no delete exists for that object
-            create_pks_to_keep = (
-                ObjectChange.objects.filter(action=ObjectChangeActionChoices.ACTION_CREATE)
-                .annotate(has_delete=Exists(deleted_exists))
-                .filter(has_delete=False)
-                .values('pk')
-            )
-
-            # Keep the most recent update per object only where no delete exists for the object
-            latest_update_pks_to_keep = (
-                ObjectChange.objects.filter(action=ObjectChangeActionChoices.ACTION_UPDATE)
-                .annotate(has_delete=Exists(deleted_exists))
-                .filter(has_delete=False)
-                .order_by('changed_object_type_id', 'changed_object_id', '-time', '-pk')
-                .distinct('changed_object_type_id', 'changed_object_id')
-                .values('pk')
-            )
-
-            expired_qs = expired_qs.exclude(pk__in=Subquery(create_pks_to_keep))
-            expired_qs = expired_qs.exclude(pk__in=Subquery(latest_update_pks_to_keep))
-
-        count = expired_qs.delete()[0]
-        self.logger.info(f'Deleted {count} expired changelog records')
+        count = ObjectChange.objects.filter(time__lt=cutoff).delete()[0]
+        self.logger.info(f"Deleted {count} expired changelog records")
 
     def delete_expired_jobs(self):
         """

netbox/core/migrations/0022_default_ordering_indexes.py

@@ -1,21 +0,0 @@
-from django.conf import settings
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-    dependencies = [
-        ('contenttypes', '0002_remove_content_type_name'),
-        ('core', '0021_job_queue_name'),
-        migrations.swappable_dependency(settings.AUTH_USER_MODEL),
-    ]
-
-    operations = [
-        migrations.AddIndex(
-            model_name='configrevision',
-            index=models.Index(fields=['-created'], name='core_config_created_ef9552_idx'),
-        ),
-        migrations.AddIndex(
-            model_name='job',
-            index=models.Index(fields=['-created'], name='core_job_created_efa7cb_idx'),
-        ),
-    ]

netbox/core/migrations/0023_datasource_sync_permission.py

@@ -1,15 +0,0 @@
-from django.db import migrations
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('core', '0022_default_ordering_indexes'),
-    ]
-
-    operations = [
-        migrations.AlterModelOptions(
-            name='datasource',
-            options={'ordering': ('name',), 'permissions': [('sync', 'Synchronize data from remote source')]},
-        ),
-    ]

netbox/core/models/change_logging.py

@@ -11,7 +11,7 @@ from mptt.models import MPTTModel
 from core.choices import ObjectChangeActionChoices
 from core.querysets import ObjectChangeQuerySet
 from netbox.models.features import ChangeLoggingMixin, has_feature
-from utilities.data import deep_compare_dict
+from utilities.data import shallow_compare_dict
 
 __all__ = (
     'ObjectChange',
@@ -199,18 +199,18 @@ class ObjectChange(models.Model):
         # Determine which attributes have changed
         if self.action == ObjectChangeActionChoices.ACTION_CREATE:
             changed_attrs = sorted(postchange_data.keys())
-            return {
-                'pre': {k: prechange_data.get(k) for k in changed_attrs},
-                'post': {k: postchange_data.get(k) for k in changed_attrs},
-            }
-        if self.action == ObjectChangeActionChoices.ACTION_DELETE:
+        elif self.action == ObjectChangeActionChoices.ACTION_DELETE:
             changed_attrs = sorted(prechange_data.keys())
-            return {
-                'pre': {k: prechange_data.get(k) for k in changed_attrs},
-                'post': {k: postchange_data.get(k) for k in changed_attrs},
-            }
-        diff_added, diff_removed = deep_compare_dict(prechange_data, postchange_data)
+        else:
+            # TODO: Support deep (recursive) comparison
+            changed_data = shallow_compare_dict(prechange_data, postchange_data)
+            changed_attrs = sorted(changed_data.keys())
+
         return {
-            'pre': dict(sorted(diff_removed.items())),
-            'post': dict(sorted(diff_added.items())),
+            'pre': {
+                k: prechange_data.get(k) for k in changed_attrs
+            },
+            'post': {
+                k: postchange_data.get(k) for k in changed_attrs
+            },
         }

netbox/core/models/config.py

@@ -37,9 +37,6 @@ class ConfigRevision(models.Model):
 
     class Meta:
         ordering = ['-created']
-        indexes = (
-            models.Index(fields=('-created',)),  # Default ordering
-        )
         verbose_name = _('config revision')
         verbose_name_plural = _('config revisions')
         constraints = [

netbox/core/models/data.py

@@ -69,7 +69,7 @@ class DataSource(JobsMixin, PrimaryModel):
     ignore_rules = models.TextField(
         verbose_name=_('ignore rules'),
         blank=True,
-        help_text=_("Patterns (one per line) matching files or paths to ignore when syncing")
+        help_text=_("Patterns (one per line) matching files to ignore when syncing")
     )
     parameters = models.JSONField(
         verbose_name=_('parameters'),
@@ -87,9 +87,6 @@ class DataSource(JobsMixin, PrimaryModel):
         ordering = ('name',)
         verbose_name = _('data source')
         verbose_name_plural = _('data sources')
-        permissions = [
-            ('sync', 'Synchronize data from remote source'),
-        ]
 
     def __str__(self):
         return f'{self.name}'
@@ -261,22 +258,21 @@ class DataSource(JobsMixin, PrimaryModel):
             if path.startswith('.'):
                 continue
             for file_name in file_names:
-                file_path = os.path.join(path, file_name)
-                if not self._ignore(file_path):
-                    paths.add(file_path)
+                if not self._ignore(file_name):
+                    paths.add(os.path.join(path, file_name))
 
         logger.debug(f"Found {len(paths)} files")
         return paths
 
-    def _ignore(self, file_path):
+    def _ignore(self, filename):
         """
         Returns a boolean indicating whether the file should be ignored per the DataSource's configured
-        ignore rules. file_path is the full relative path (e.g. "subdir/file.txt").
+        ignore rules.
         """
-        if os.path.basename(file_path).startswith('.'):
+        if filename.startswith('.'):
             return True
         for rule in self.ignore_rules.splitlines():
-            if fnmatchcase(file_path, rule) or fnmatchcase(os.path.basename(file_path), rule):
+            if fnmatchcase(filename, rule):
                 return True
         return False
 

netbox/core/models/jobs.py

@@ -133,7 +133,6 @@ class Job(models.Model):
     class Meta:
         ordering = ['-created']
         indexes = (
-            models.Index(fields=('-created',)),  # Default ordering
             models.Index(fields=('object_type', 'object_id')),
         )
         verbose_name = _('job')

netbox/core/models/object_types.py

@@ -51,6 +51,7 @@ class ObjectTypeManager(models.Manager):
         """
         return self.get(app_label=app_label, model=model)
 
+    # TODO: Remove in NetBox v4.5
     def get_for_id(self, id):
         """
         Retrieve an ObjectType by its primary key (numeric ID).

netbox/core/tables/config.py

@@ -19,7 +19,6 @@ REVISION_BUTTONS = """
 class ConfigRevisionTable(NetBoxTable):
     is_active = columns.BooleanColumn(
         verbose_name=_('Is Active'),
-        accessor='active',
         false_mark=None
     )
     actions = columns.ActionsColumn(

netbox/core/tables/jobs.py

@@ -1,6 +1,4 @@
 import django_tables2 as tables
-from django.utils.html import conditional_escape
-from django.utils.safestring import mark_safe
 from django.utils.translation import gettext_lazy as _
 
 from core.constants import JOB_LOG_ENTRY_LEVELS
@@ -84,9 +82,3 @@ class JobLogEntryTable(BaseTable):
     class Meta(BaseTable.Meta):
         empty_text = _('No log entries')
         fields = ('timestamp', 'level', 'message')
-
-    def render_message(self, record, value):
-        if record.get('level') == 'error' and '\n' in value:
-            value = conditional_escape(value)
-            return mark_safe(f'<pre class="p-0">{value}</pre>')
-        return value

netbox/core/tests/test_changelog.py

@@ -1,16 +1,9 @@
-import logging
-import uuid
-from datetime import timedelta
-from unittest.mock import patch
-
 from django.contrib.contenttypes.models import ContentType
-from django.test import TestCase, override_settings
+from django.test import override_settings
 from django.urls import reverse
-from django.utils import timezone
 from rest_framework import status
 
 from core.choices import ObjectChangeActionChoices
-from core.jobs import SystemHousekeepingJob
 from core.models import ObjectChange, ObjectType
 from dcim.choices import InterfaceTypeChoices, ModuleStatusChoices, SiteStatusChoices
 from dcim.models import (
@@ -701,99 +694,3 @@ class ChangeLogAPITest(APITestCase):
         self.assertEqual(changes[3].changed_object_type, ContentType.objects.get_for_model(Module))
         self.assertEqual(changes[3].changed_object_id, module.pk)
         self.assertEqual(changes[3].action, ObjectChangeActionChoices.ACTION_DELETE)
-
-
-class ChangelogPruneRetentionTest(TestCase):
-    """Test suite for Changelog pruning retention settings."""
-
-    @staticmethod
-    def _make_oc(*, ct, obj_id, action, ts):
-        oc = ObjectChange.objects.create(
-            changed_object_type=ct,
-            changed_object_id=obj_id,
-            action=action,
-            user_name='test',
-            request_id=uuid.uuid4(),
-            object_repr=f'Object {obj_id}',
-        )
-        ObjectChange.objects.filter(pk=oc.pk).update(time=ts)
-        return oc.pk
-
-    @staticmethod
-    def _run_prune(*, retention_days, retain_create_last_update):
-        job = SystemHousekeepingJob.__new__(SystemHousekeepingJob)
-        job.logger = logging.getLogger('netbox.tests.changelog_prune')
-
-        with patch('core.jobs.Config') as MockConfig:
-            cfg = MockConfig.return_value
-            cfg.CHANGELOG_RETENTION = retention_days
-            cfg.CHANGELOG_RETAIN_CREATE_LAST_UPDATE = retain_create_last_update
-            job.prune_changelog()
-
-    def test_prune_retain_create_last_update_excludes_deleted_objects(self):
-        ct = ContentType.objects.get_for_model(Site)
-
-        retention_days = 90
-        now = timezone.now()
-        cutoff = now - timedelta(days=retention_days)
-
-        expired_old = cutoff - timedelta(days=10)
-        expired_newer = cutoff - timedelta(days=1)
-        not_expired = cutoff + timedelta(days=1)
-
-        # A) Not deleted: should keep CREATE + latest UPDATE, prune intermediate UPDATEs
-        a_create = self._make_oc(ct=ct, obj_id=1, action=ObjectChangeActionChoices.ACTION_CREATE, ts=expired_old)
-        a_update1 = self._make_oc(ct=ct, obj_id=1, action=ObjectChangeActionChoices.ACTION_UPDATE, ts=expired_old)
-        a_update2 = self._make_oc(ct=ct, obj_id=1, action=ObjectChangeActionChoices.ACTION_UPDATE, ts=expired_newer)
-
-        # B) Deleted (all expired): should keep NOTHING
-        b_create = self._make_oc(ct=ct, obj_id=2, action=ObjectChangeActionChoices.ACTION_CREATE, ts=expired_old)
-        b_update = self._make_oc(ct=ct, obj_id=2, action=ObjectChangeActionChoices.ACTION_UPDATE, ts=expired_newer)
-        b_delete = self._make_oc(ct=ct, obj_id=2, action=ObjectChangeActionChoices.ACTION_DELETE, ts=expired_newer)
-
-        # C) Deleted but delete is not expired: create/update expired should be pruned; delete remains
-        c_create = self._make_oc(ct=ct, obj_id=3, action=ObjectChangeActionChoices.ACTION_CREATE, ts=expired_old)
-        c_update = self._make_oc(ct=ct, obj_id=3, action=ObjectChangeActionChoices.ACTION_UPDATE, ts=expired_newer)
-        c_delete = self._make_oc(ct=ct, obj_id=3, action=ObjectChangeActionChoices.ACTION_DELETE, ts=not_expired)
-
-        self._run_prune(retention_days=retention_days, retain_create_last_update=True)
-
-        remaining = set(ObjectChange.objects.values_list('pk', flat=True))
-
-        # A) Not deleted -> create + latest update remain
-        self.assertIn(a_create, remaining)
-        self.assertIn(a_update2, remaining)
-        self.assertNotIn(a_update1, remaining)
-
-        # B) Deleted (all expired) -> nothing remains
-        self.assertNotIn(b_create, remaining)
-        self.assertNotIn(b_update, remaining)
-        self.assertNotIn(b_delete, remaining)
-
-        # C) Deleted, delete not expired -> delete remains, but create/update are pruned
-        self.assertNotIn(c_create, remaining)
-        self.assertNotIn(c_update, remaining)
-        self.assertIn(c_delete, remaining)
-
-    def test_prune_disabled_deletes_all_expired(self):
-        ct = ContentType.objects.get_for_model(Site)
-
-        retention_days = 90
-        now = timezone.now()
-        cutoff = now - timedelta(days=retention_days)
-        expired = cutoff - timedelta(days=1)
-        not_expired = cutoff + timedelta(days=1)
-
-        # expired create/update should be deleted when feature disabled
-        x_create = self._make_oc(ct=ct, obj_id=10, action=ObjectChangeActionChoices.ACTION_CREATE, ts=expired)
-        x_update = self._make_oc(ct=ct, obj_id=10, action=ObjectChangeActionChoices.ACTION_UPDATE, ts=expired)
-
-        # non-expired delete should remain regardless
-        y_delete = self._make_oc(ct=ct, obj_id=11, action=ObjectChangeActionChoices.ACTION_DELETE, ts=not_expired)
-
-        self._run_prune(retention_days=retention_days, retain_create_last_update=False)
-
-        remaining = set(ObjectChange.objects.values_list('pk', flat=True))
-        self.assertNotIn(x_create, remaining)
-        self.assertNotIn(x_update, remaining)
-        self.assertIn(y_delete, remaining)

netbox/core/tests/test_models.py

@@ -10,26 +10,6 @@ from dcim.models import Device, Location, Site
 from netbox.constants import CENSOR_TOKEN, CENSOR_TOKEN_CHANGED
 
 
-class DataSourceIgnoreRulesTestCase(TestCase):
-
-    def test_no_ignore_rules(self):
-        ds = DataSource(ignore_rules='')
-        self.assertFalse(ds._ignore('README.md'))
-        self.assertFalse(ds._ignore('subdir/file.py'))
-
-    def test_ignore_by_filename(self):
-        ds = DataSource(ignore_rules='*.txt')
-        self.assertTrue(ds._ignore('notes.txt'))
-        self.assertTrue(ds._ignore('subdir/notes.txt'))
-        self.assertFalse(ds._ignore('notes.py'))
-
-    def test_ignore_by_subdirectory(self):
-        ds = DataSource(ignore_rules='dev/*')
-        self.assertTrue(ds._ignore('dev/README.md'))
-        self.assertTrue(ds._ignore('dev/script.py'))
-        self.assertFalse(ds._ignore('prod/script.py'))
-
-
 class DataSourceChangeLoggingTestCase(TestCase):
 
     def test_password_added_on_create(self):

netbox/core/tests/test_tables.py

@@ -1,26 +0,0 @@
-from core.models import ObjectChange
-from core.tables import *
-from utilities.testing import TableTestCases
-
-
-class DataSourceTableTest(TableTestCases.StandardTableTestCase):
-    table = DataSourceTable
-
-
-class DataFileTableTest(TableTestCases.StandardTableTestCase):
-    table = DataFileTable
-
-
-class JobTableTest(TableTestCases.StandardTableTestCase):
-    table = JobTable
-
-
-class ObjectChangeTableTest(TableTestCases.StandardTableTestCase):
-    table = ObjectChangeTable
-    queryset_sources = [
-        ('ObjectChangeListView', ObjectChange.objects.all()),
-    ]
-
-
-class ConfigRevisionTableTest(TableTestCases.StandardTableTestCase):
-    table = ConfigRevisionTable

netbox/core/tests/test_views.py

@@ -6,7 +6,7 @@ from datetime import datetime
 from django.urls import reverse
 from django.utils import timezone
 from django_rq import get_queue
-from django_rq.settings import get_queues_map
+from django_rq.settings import QUEUES_MAP
 from django_rq.workers import get_worker
 from rq.job import Job as RQ_Job
 from rq.job import JobStatus
@@ -189,7 +189,7 @@ class BackgroundTaskTestCase(TestCase):
     def test_background_tasks_list_default(self):
         queue = get_queue('default')
         queue.enqueue(self.dummy_job_default)
-        queue_index = get_queues_map()['default']
+        queue_index = QUEUES_MAP['default']
 
         response = self.client.get(reverse('core:background_task_list', args=[queue_index, 'queued']))
         self.assertEqual(response.status_code, 200)
@@ -198,7 +198,7 @@ class BackgroundTaskTestCase(TestCase):
     def test_background_tasks_list_high(self):
         queue = get_queue('high')
         queue.enqueue(self.dummy_job_high)
-        queue_index = get_queues_map()['high']
+        queue_index = QUEUES_MAP['high']
 
         response = self.client.get(reverse('core:background_task_list', args=[queue_index, 'queued']))
         self.assertEqual(response.status_code, 200)
@@ -207,7 +207,7 @@ class BackgroundTaskTestCase(TestCase):
     def test_background_tasks_list_finished(self):
         queue = get_queue('default')
         job = queue.enqueue(self.dummy_job_default)
-        queue_index = get_queues_map()['default']
+        queue_index = QUEUES_MAP['default']
 
         registry = FinishedJobRegistry(queue.name, queue.connection)
         registry.add(job, 2)
@@ -218,7 +218,7 @@ class BackgroundTaskTestCase(TestCase):
     def test_background_tasks_list_failed(self):
         queue = get_queue('default')
         job = queue.enqueue(self.dummy_job_default)
-        queue_index = get_queues_map()['default']
+        queue_index = QUEUES_MAP['default']
 
         registry = FailedJobRegistry(queue.name, queue.connection)
         registry.add(job, 2)
@@ -229,7 +229,7 @@ class BackgroundTaskTestCase(TestCase):
     def test_background_tasks_scheduled(self):
         queue = get_queue('default')
         queue.enqueue_at(datetime.now(), self.dummy_job_default)
-        queue_index = get_queues_map()['default']
+        queue_index = QUEUES_MAP['default']
 
         response = self.client.get(reverse('core:background_task_list', args=[queue_index, 'scheduled']))
         self.assertEqual(response.status_code, 200)
@@ -238,7 +238,7 @@ class BackgroundTaskTestCase(TestCase):
     def test_background_tasks_list_deferred(self):
         queue = get_queue('default')
         job = queue.enqueue(self.dummy_job_default)
-        queue_index = get_queues_map()['default']
+        queue_index = QUEUES_MAP['default']
 
         registry = DeferredJobRegistry(queue.name, queue.connection)
         registry.add(job, 2)
@@ -335,7 +335,7 @@ class BackgroundTaskTestCase(TestCase):
         worker2 = get_worker('high')
         worker2.register_birth()
 
-        queue_index = get_queues_map()['default']
+        queue_index = QUEUES_MAP['default']
         response = self.client.get(reverse('core:worker_list', args=[queue_index]))
         self.assertEqual(response.status_code, 200)
         self.assertIn(str(worker1.name), str(response.content))

netbox/core/ui/panels.py

@@ -1,91 +0,0 @@
-from django.utils.translation import gettext_lazy as _
-
-from netbox.ui import attrs, panels
-
-
-class DataSourcePanel(panels.ObjectAttributesPanel):
-    title = _('Data Source')
-    name = attrs.TextAttr('name')
-    type = attrs.ChoiceAttr('type')
-    enabled = attrs.BooleanAttr('enabled')
-    status = attrs.ChoiceAttr('status')
-    sync_interval = attrs.ChoiceAttr('sync_interval', label=_('Sync interval'))
-    last_synced = attrs.DateTimeAttr('last_synced', label=_('Last synced'))
-    description = attrs.TextAttr('description')
-    source_url = attrs.TemplatedAttr(
-        'source_url',
-        label=_('URL'),
-        template_name='core/datasource/attrs/source_url.html',
-    )
-    ignore_rules = attrs.TemplatedAttr(
-        'ignore_rules',
-        label=_('Ignore rules'),
-        template_name='core/datasource/attrs/ignore_rules.html',
-    )
-
-
-class DataSourceBackendPanel(panels.ObjectPanel):
-    template_name = 'core/panels/datasource_backend.html'
-    title = _('Backend')
-
-
-class DataFilePanel(panels.ObjectAttributesPanel):
-    title = _('Data File')
-    source = attrs.RelatedObjectAttr('source', linkify=True)
-    path = attrs.TextAttr('path', style='font-monospace', copy_button=True)
-    last_updated = attrs.DateTimeAttr('last_updated')
-    size = attrs.TemplatedAttr('size', template_name='core/datafile/attrs/size.html')
-    hash = attrs.TextAttr('hash', label=_('SHA256 hash'), style='font-monospace', copy_button=True)
-
-
-class DataFileContentPanel(panels.ObjectPanel):
-    template_name = 'core/panels/datafile_content.html'
-    title = _('Content')
-
-
-class JobPanel(panels.ObjectAttributesPanel):
-    title = _('Job')
-    object_type = attrs.TemplatedAttr(
-        'object_type',
-        label=_('Object type'),
-        template_name='core/job/attrs/object_type.html',
-    )
-    name = attrs.TextAttr('name')
-    status = attrs.ChoiceAttr('status')
-    error = attrs.TextAttr('error')
-    user = attrs.TextAttr('user', label=_('Created by'))
-
-
-class JobSchedulingPanel(panels.ObjectAttributesPanel):
-    title = _('Scheduling')
-    created = attrs.DateTimeAttr('created')
-    scheduled = attrs.TemplatedAttr('scheduled', template_name='core/job/attrs/scheduled.html')
-    started = attrs.DateTimeAttr('started')
-    completed = attrs.DateTimeAttr('completed')
-    queue = attrs.TextAttr('queue_name', label=_('Queue'))
-
-
-class ObjectChangePanel(panels.ObjectAttributesPanel):
-    title = _('Change')
-    time = attrs.DateTimeAttr('time')
-    user = attrs.TemplatedAttr(
-        'user_name',
-        label=_('User'),
-        template_name='core/objectchange/attrs/user.html',
-    )
-    action = attrs.ChoiceAttr('action')
-    changed_object_type = attrs.TextAttr(
-        'changed_object_type',
-        label=_('Object type'),
-    )
-    changed_object = attrs.TemplatedAttr(
-        'object_repr',
-        label=_('Object'),
-        template_name='core/objectchange/attrs/changed_object.html',
-    )
-    message = attrs.TextAttr('message')
-    request_id = attrs.TemplatedAttr(
-        'request_id',
-        label=_('Request ID'),
-        template_name='core/objectchange/attrs/request_id.html',
-    )

netbox/core/utils.py

@@ -1,7 +1,7 @@
 from django.http import Http404
 from django.utils.translation import gettext_lazy as _
 from django_rq.queues import get_queue, get_queue_by_index, get_redis_connection
-from django_rq.settings import get_queues_list, get_queues_map
+from django_rq.settings import QUEUES_LIST, QUEUES_MAP
 from django_rq.utils import get_jobs, stop_jobs
 from rq import requeue_job
 from rq.exceptions import NoSuchJobError
@@ -31,7 +31,7 @@ def get_rq_jobs():
     """
     jobs = set()
 
-    for queue in get_queues_list():
+    for queue in QUEUES_LIST:
         queue = get_queue(queue['name'])
         jobs.update(queue.get_jobs())
 
@@ -78,13 +78,13 @@ def delete_rq_job(job_id):
     """
     Delete the specified RQ job.
     """
-    config = get_queues_list()[0]
+    config = QUEUES_LIST[0]
     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))
 
-    queue_index = get_queues_map()[job.origin]
+    queue_index = QUEUES_MAP[job.origin]
     queue = get_queue_by_index(queue_index)
 
     # Remove job id from queue and delete the actual job
@@ -96,13 +96,13 @@ def requeue_rq_job(job_id):
     """
     Requeue the specified RQ job.
     """
-    config = get_queues_list()[0]
+    config = QUEUES_LIST[0]
     try:
         job = RQ_Job.fetch(job_id, connection=get_redis_connection(config['connection_config']),)
     except NoSuchJobError:
         raise Http404(_("Job {id} not found.").format(id=job_id))
 
-    queue_index = get_queues_map()[job.origin]
+    queue_index = QUEUES_MAP[job.origin]
     queue = get_queue_by_index(queue_index)
 
     requeue_job(job_id, connection=queue.connection, serializer=queue.serializer)
@@ -112,13 +112,13 @@ def enqueue_rq_job(job_id):
     """
     Enqueue the specified RQ job.
     """
-    config = get_queues_list()[0]
+    config = QUEUES_LIST[0]
     try:
         job = RQ_Job.fetch(job_id, connection=get_redis_connection(config['connection_config']),)
     except NoSuchJobError:
         raise Http404(_("Job {id} not found.").format(id=job_id))
 
-    queue_index = get_queues_map()[job.origin]
+    queue_index = QUEUES_MAP[job.origin]
     queue = get_queue_by_index(queue_index)
 
     try:
@@ -144,13 +144,13 @@ def stop_rq_job(job_id):
     """
     Stop the specified RQ job.
     """
-    config = get_queues_list()[0]
+    config = QUEUES_LIST[0]
     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))
 
-    queue_index = get_queues_map()[job.origin]
+    queue_index = QUEUES_MAP[job.origin]
     queue = get_queue_by_index(queue_index)
 
     return stop_jobs(queue, job_id)[0]

netbox/core/views.py

@@ -14,7 +14,7 @@ from django.urls import reverse
 from django.utils.translation import gettext_lazy as _
 from django.views.generic import View
 from django_rq.queues import get_connection, get_queue_by_index, get_redis_connection
-from django_rq.settings import get_queues_list, get_queues_map
+from django_rq.settings import QUEUES_LIST, QUEUES_MAP
 from django_rq.utils import get_statistics
 from rq.exceptions import NoSuchJobError
 from rq.job import Job as RQ_Job
@@ -23,25 +23,14 @@ from rq.worker import Worker
 from rq.worker_registration import clean_worker_registry
 
 from core.utils import delete_rq_job, enqueue_rq_job, get_rq_jobs_from_status, requeue_rq_job, stop_rq_job
-from extras.ui.panels import CustomFieldsPanel, TagsPanel
 from netbox.config import PARAMS, get_config
 from netbox.object_actions import AddObject, BulkDelete, BulkExport, DeleteObject
 from netbox.plugins.utils import get_installed_plugins
-from netbox.ui import layout
-from netbox.ui.panels import (
-    CommentsPanel,
-    ContextTablePanel,
-    JSONPanel,
-    ObjectsTablePanel,
-    PluginContentPanel,
-    RelatedObjectsPanel,
-    TemplatePanel,
-)
 from netbox.views import generic
 from netbox.views.generic.base import BaseObjectView
 from netbox.views.generic.mixins import TableMixin
 from utilities.apps import get_installed_apps
-from utilities.data import deep_compare_dict
+from utilities.data import shallow_compare_dict
 from utilities.forms import ConfirmationForm
 from utilities.htmx import htmx_partial
 from utilities.json import ConfigJSONEncoder
@@ -59,7 +48,6 @@ from .jobs import SyncDataSourceJob
 from .models import *
 from .plugins import get_catalog_plugins, get_local_plugins
 from .tables import CatalogPluginTable, JobLogEntryTable, PluginVersionTable
-from .ui import panels
 
 #
 # Data sources
@@ -79,25 +67,6 @@ class DataSourceListView(generic.ObjectListView):
 @register_model_view(DataSource)
 class DataSourceView(GetRelatedModelsMixin, generic.ObjectView):
     queryset = DataSource.objects.all()
-    layout = layout.SimpleLayout(
-        left_panels=[
-            panels.DataSourcePanel(),
-            TagsPanel(),
-            CommentsPanel(),
-        ],
-        right_panels=[
-            panels.DataSourceBackendPanel(),
-            RelatedObjectsPanel(),
-            CustomFieldsPanel(),
-        ],
-        bottom_panels=[
-            ObjectsTablePanel(
-                model='core.DataFile',
-                filters={'source_id': lambda ctx: ctx['object'].pk},
-                exclude_columns=['source'],
-            ),
-        ],
-    )
 
     def get_extra_context(self, request, instance):
         return {
@@ -188,20 +157,6 @@ class DataFileListView(generic.ObjectListView):
 class DataFileView(generic.ObjectView):
     queryset = DataFile.objects.all()
     actions = (DeleteObject,)
-    layout = layout.Layout(
-        layout.Row(
-            layout.Column(
-                panels.DataFilePanel(),
-                panels.DataFileContentPanel(),
-                PluginContentPanel('left_page'),
-            ),
-        ),
-        layout.Row(
-            layout.Column(
-                PluginContentPanel('full_width_page'),
-            )
-        ),
-    )
 
 
 @register_model_view(DataFile, 'delete')
@@ -233,17 +188,6 @@ class JobListView(generic.ObjectListView):
 class JobView(generic.ObjectView):
     queryset = Job.objects.all()
     actions = (DeleteObject,)
-    layout = layout.SimpleLayout(
-        left_panels=[
-            panels.JobPanel(),
-        ],
-        right_panels=[
-            panels.JobSchedulingPanel(),
-        ],
-        bottom_panels=[
-            JSONPanel('data', title=_('Data')),
-        ],
-    )
 
 
 @register_model_view(Job, 'log')
@@ -256,13 +200,6 @@ class JobLogView(generic.ObjectView):
         badge=lambda obj: len(obj.log_entries),
         weight=500,
     )
-    layout = layout.Layout(
-        layout.Row(
-            layout.Column(
-                ContextTablePanel('table', title=_('Log Entries')),
-            ),
-        ),
-    )
 
     def get_extra_context(self, request, instance):
         table = JobLogEntryTable(instance.log_entries)
@@ -304,26 +241,6 @@ class ObjectChangeListView(generic.ObjectListView):
 @register_model_view(ObjectChange)
 class ObjectChangeView(generic.ObjectView):
     queryset = None
-    layout = layout.Layout(
-        layout.Row(
-            layout.Column(panels.ObjectChangePanel()),
-            layout.Column(TemplatePanel('core/panels/objectchange_difference.html')),
-        ),
-        layout.Row(
-            layout.Column(TemplatePanel('core/panels/objectchange_prechange.html')),
-            layout.Column(TemplatePanel('core/panels/objectchange_postchange.html')),
-        ),
-        layout.Row(
-            layout.Column(PluginContentPanel('left_page')),
-            layout.Column(PluginContentPanel('right_page')),
-        ),
-        layout.Row(
-            layout.Column(
-                TemplatePanel('core/panels/objectchange_related.html'),
-                PluginContentPanel('full_width_page'),
-            ),
-        ),
-    )
 
     def get_queryset(self, request):
         return ObjectChange.objects.valid_models()
@@ -356,14 +273,17 @@ class ObjectChangeView(generic.ObjectView):
             prechange_data = instance.prechange_data_clean
 
         if prechange_data and instance.postchange_data:
-            diff_added, diff_removed = deep_compare_dict(
-                prechange_data,
-                instance.postchange_data_clean,
+            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 = {}
-            diff_removed = {}
+            diff_added = None
+            diff_removed = None
 
         return {
             'diff_added': diff_added,
@@ -392,14 +312,6 @@ class ConfigRevisionListView(generic.ObjectListView):
 @register_model_view(ConfigRevision)
 class ConfigRevisionView(generic.ObjectView):
     queryset = ConfigRevision.objects.all()
-    layout = layout.Layout(
-        layout.Row(
-            layout.Column(
-                TemplatePanel('core/panels/configrevision_data.html'),
-                TemplatePanel('core/panels/configrevision_comment.html'),
-            ),
-        ),
-    )
 
     def get_extra_context(self, request, instance):
         """
@@ -528,13 +440,13 @@ class BackgroundTaskView(BaseRQView):
 
     def get(self, request, job_id):
         # all the RQ queues should use the same connection
-        config = get_queues_list()[0]
+        config = QUEUES_LIST[0]
         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))
 
-        queue_index = get_queues_map()[job.origin]
+        queue_index = QUEUES_MAP[job.origin]
         queue = get_queue_by_index(queue_index)
 
         try:
@@ -644,7 +556,7 @@ class WorkerView(BaseRQView):
 
     def get(self, request, key):
         # all the RQ queues should use the same connection
-        config = get_queues_list()[0]
+        config = QUEUES_LIST[0]
         worker = Worker.find_by_key('rq:worker:' + key, connection=get_redis_connection(config['connection_config']))
         # Convert microseconds to milliseconds
         worker.total_working_time = worker.total_working_time / 1000

netbox/dcim/api/serializers_/base.py

@@ -38,15 +38,7 @@ class ConnectedEndpointsSerializer(serializers.ModelSerializer):
 
     @extend_schema_field(serializers.BooleanField)
     def get_connected_endpoints_reachable(self, obj):
-        """
-        Return whether the connected endpoints are reachable via a complete, active cable path.
-        """
-        # Use the public `path` accessor rather than dereferencing `_path`
-        # directly. `path` already handles the stale in-memory relation case
-        # that can occur while CablePath rows are rebuilt during cable edits.
-        if path := obj.path:
-            return path.is_complete and path.is_active
-        return False
+        return obj._path and obj._path.is_complete and obj._path.is_active
 
 
 class PortSerializer(serializers.ModelSerializer):

netbox/dcim/api/serializers_/cables.py

@@ -3,7 +3,7 @@ from drf_spectacular.utils import extend_schema_field
 from rest_framework import serializers
 
 from dcim.choices import *
-from dcim.models import Cable, CableBundle, CablePath, CableTermination
+from dcim.models import Cable, CablePath, CableTermination
 from netbox.api.fields import ChoiceField, ContentTypeField
 from netbox.api.gfk_fields import GFKSerializerField
 from netbox.api.serializers import (
@@ -16,7 +16,6 @@ from tenancy.api.serializers_.tenants import TenantSerializer
 from utilities.api import get_serializer_for_model
 
 __all__ = (
-    'CableBundleSerializer',
     'CablePathSerializer',
     'CableSerializer',
     'CableTerminationSerializer',
@@ -25,18 +24,6 @@ __all__ = (
 )
 
 
-class CableBundleSerializer(PrimaryModelSerializer):
-    cable_count = serializers.IntegerField(read_only=True, default=0)
-
-    class Meta:
-        model = CableBundle
-        fields = [
-            'id', 'url', 'display_url', 'display', 'name', 'description', 'owner', 'comments', 'tags',
-            'custom_fields', 'created', 'last_updated', 'cable_count',
-        ]
-        brief_fields = ('id', 'url', 'display', 'name', 'description')
-
-
 class CableSerializer(PrimaryModelSerializer):
     a_terminations = GenericObjectSerializer(many=True, required=False)
     b_terminations = GenericObjectSerializer(many=True, required=False)
@@ -44,13 +31,12 @@ class CableSerializer(PrimaryModelSerializer):
     profile = ChoiceField(choices=CableProfileChoices, required=False)
     tenant = TenantSerializer(nested=True, required=False, allow_null=True)
     length_unit = ChoiceField(choices=CableLengthUnitChoices, allow_blank=True, required=False, allow_null=True)
-    bundle = CableBundleSerializer(nested=True, required=False, allow_null=True, default=None)
 
     class Meta:
         model = Cable
         fields = [
             'id', 'url', 'display_url', 'display', 'type', 'a_terminations', 'b_terminations', 'status', 'profile',
-            'tenant', 'bundle', 'label', 'color', 'length', 'length_unit', 'description', 'owner', 'comments', 'tags',
+            'tenant', 'label', 'color', 'length', 'length_unit', 'description', 'owner', 'comments', 'tags',
             'custom_fields', 'created', 'last_updated',
         ]
         brief_fields = ('id', 'url', 'display', 'label', 'description')

netbox/dcim/api/serializers_/device_components.py

@@ -423,29 +423,27 @@ class ModuleBaySerializer(OwnerMixin, NetBoxModelSerializer):
         required=False,
         allow_null=True
     )
-    _occupied = serializers.BooleanField(required=False, read_only=True)
 
     class Meta:
         model = ModuleBay
         fields = [
-            'id', 'url', 'display_url', 'display', 'device', 'module', 'name', 'label', 'position', 'enabled',
-            'description', 'installed_module', 'owner', 'tags', 'custom_fields', 'created', 'last_updated', '_occupied',
+            'id', 'url', 'display_url', 'display', 'device', 'module', 'name', 'installed_module', 'label', 'position',
+            'description', 'owner', 'tags', 'custom_fields', 'created', 'last_updated',
         ]
-        brief_fields = ('id', 'url', 'display', 'installed_module', 'name', 'enabled', 'description', '_occupied')
+        brief_fields = ('id', 'url', 'display', 'installed_module', 'name', 'description')
 
 
 class DeviceBaySerializer(OwnerMixin, NetBoxModelSerializer):
     device = DeviceSerializer(nested=True)
     installed_device = DeviceSerializer(nested=True, required=False, allow_null=True)
-    _occupied = serializers.BooleanField(required=False, read_only=True)
 
     class Meta:
         model = DeviceBay
         fields = [
-            'id', 'url', 'display_url', 'display', 'device', 'name', 'label', 'enabled', 'description',
-            'installed_device', 'owner', 'tags', 'custom_fields', 'created', 'last_updated', '_occupied',
+            'id', 'url', 'display_url', 'display', 'device', 'name', 'label', 'description', 'installed_device',
+            'owner', 'tags', 'custom_fields', 'created', 'last_updated',
         ]
-        brief_fields = ('id', 'url', 'display', 'device', 'name', 'enabled', 'description', '_occupied',)
+        brief_fields = ('id', 'url', 'display', 'device', 'name', 'description')
 
 
 class InventoryItemSerializer(OwnerMixin, NetBoxModelSerializer):

netbox/dcim/api/serializers_/devices.py

@@ -6,9 +6,8 @@ from drf_spectacular.utils import extend_schema_field
 from rest_framework import serializers
 
 from dcim.choices import *
-from dcim.constants import MACADDRESS_ASSIGNMENT_MODELS, MODULE_TOKEN
+from dcim.constants import MACADDRESS_ASSIGNMENT_MODELS
 from dcim.models import Device, DeviceBay, MACAddress, Module, VirtualDeviceContext
-from dcim.utils import get_module_bay_positions, resolve_module_placeholder
 from extras.api.serializers_.configtemplates import ConfigTemplateSerializer
 from ipam.api.serializers_.ip import IPAddressSerializer
 from netbox.api.fields import ChoiceField, ContentTypeField, RelatedObjectCountField
@@ -58,30 +57,10 @@ class DeviceSerializer(PrimaryModelSerializer):
     )
     status = ChoiceField(choices=DeviceStatusChoices, required=False)
     airflow = ChoiceField(choices=DeviceAirflowChoices, allow_blank=True, required=False)
-    primary_ip = IPAddressSerializer(
-        nested=True,
-        read_only=True,
-        allow_null=True,
-        fields=[*IPAddressSerializer.Meta.brief_fields, 'nat_inside', 'nat_outside'],
-    )
-    primary_ip4 = IPAddressSerializer(
-        nested=True,
-        required=False,
-        allow_null=True,
-        fields=[*IPAddressSerializer.Meta.brief_fields, 'nat_inside', 'nat_outside'],
-    )
-    primary_ip6 = IPAddressSerializer(
-        nested=True,
-        required=False,
-        allow_null=True,
-        fields=[*IPAddressSerializer.Meta.brief_fields, 'nat_inside', 'nat_outside'],
-    )
-    oob_ip = IPAddressSerializer(
-        nested=True,
-        required=False,
-        allow_null=True,
-        fields=[*IPAddressSerializer.Meta.brief_fields, 'nat_inside', 'nat_outside'],
-    )
+    primary_ip = IPAddressSerializer(nested=True, read_only=True, allow_null=True)
+    primary_ip4 = IPAddressSerializer(nested=True, required=False, allow_null=True)
+    primary_ip6 = IPAddressSerializer(nested=True, required=False, allow_null=True)
+    oob_ip = IPAddressSerializer(nested=True, required=False, allow_null=True)
     parent_device = serializers.SerializerMethodField()
     cluster = ClusterSerializer(nested=True, required=False, allow_null=True)
     virtual_chassis = VirtualChassisSerializer(nested=True, required=False, allow_null=True, default=None)
@@ -171,132 +150,15 @@ class ModuleSerializer(PrimaryModelSerializer):
     module_bay = NestedModuleBaySerializer()
     module_type = ModuleTypeSerializer(nested=True)
     status = ChoiceField(choices=ModuleStatusChoices, required=False)
-    replicate_components = serializers.BooleanField(
-        required=False,
-        default=True,
-        write_only=True,
-        label=_('Replicate components'),
-        help_text=_('Automatically populate components associated with this module type (default: true)')
-    )
-    adopt_components = serializers.BooleanField(
-        required=False,
-        default=False,
-        write_only=True,
-        label=_('Adopt components'),
-        help_text=_('Adopt already existing components')
-    )
 
     class Meta:
         model = Module
         fields = [
             'id', 'url', 'display_url', 'display', 'device', 'module_bay', 'module_type', 'status', 'serial',
             'asset_tag', 'description', 'owner', 'comments', 'tags', 'custom_fields', 'created', 'last_updated',
-            'replicate_components', 'adopt_components',
         ]
         brief_fields = ('id', 'url', 'display', 'device', 'module_bay', 'module_type', 'description')
 
-    def validate(self, data):
-        # When used as a nested serializer (e.g. as the `module` field on device component
-        # serializers), `data` is already a resolved Module instance — skip our custom logic.
-        if self.nested:
-            return super().validate(data)
-
-        # Pop write-only transient fields before ValidatedModelSerializer tries to
-        # construct a Module instance for full_clean(); restore them afterwards.
-        replicate_components = data.pop('replicate_components', True)
-        adopt_components = data.pop('adopt_components', False)
-        data = super().validate(data)
-
-        # For updates these fields are not meaningful; omit them from validated_data so that
-        # ModelSerializer.update() does not set unexpected attributes on the instance.
-        if self.instance:
-            return data
-
-        # Always pass the flags to create() so it can set the correct private attributes.
-        data['replicate_components'] = replicate_components
-        data['adopt_components'] = adopt_components
-
-        # Skip conflict checks when no component operations are requested.
-        if not replicate_components and not adopt_components:
-            return data
-
-        device = data.get('device')
-        module_type = data.get('module_type')
-        module_bay = data.get('module_bay')
-
-        # Required-field validation fires separately; skip here if any are missing.
-        if not all([device, module_type, module_bay]):
-            return data
-
-        positions = get_module_bay_positions(module_bay)
-
-        for templates_attr, component_attr in [
-            ('consoleporttemplates', 'consoleports'),
-            ('consoleserverporttemplates', 'consoleserverports'),
-            ('interfacetemplates', 'interfaces'),
-            ('powerporttemplates', 'powerports'),
-            ('poweroutlettemplates', 'poweroutlets'),
-            ('rearporttemplates', 'rearports'),
-            ('frontporttemplates', 'frontports'),
-        ]:
-            installed_components = {
-                component.name: component
-                for component in getattr(device, component_attr).all()
-            }
-
-            for template in getattr(module_type, templates_attr).all():
-                resolved_name = template.name
-                if MODULE_TOKEN in template.name:
-                    if not module_bay.position:
-                        raise serializers.ValidationError(
-                            _("Cannot install module with placeholder values in a module bay with no position defined.")
-                        )
-                    try:
-                        resolved_name = resolve_module_placeholder(template.name, positions)
-                    except ValueError as e:
-                        raise serializers.ValidationError(str(e))
-
-                existing_item = installed_components.get(resolved_name)
-
-                if adopt_components and existing_item and existing_item.module:
-                    raise serializers.ValidationError(
-                        _("Cannot adopt {model} {name} as it already belongs to a module").format(
-                            model=template.component_model.__name__,
-                            name=resolved_name
-                        )
-                    )
-
-                if not adopt_components and resolved_name in installed_components:
-                    raise serializers.ValidationError(
-                        _("A {model} named {name} already exists").format(
-                            model=template.component_model.__name__,
-                            name=resolved_name
-                        )
-                    )
-
-        return data
-
-    def create(self, validated_data):
-        replicate_components = validated_data.pop('replicate_components', True)
-        adopt_components = validated_data.pop('adopt_components', False)
-
-        # Tags are handled after save; pop them here to pass to _save_tags()
-        tags = validated_data.pop('tags', None)
-
-        # _adopt_components and _disable_replication must be set on the instance before
-        # save() is called, so we cannot delegate to super().create() here.
-        instance = self.Meta.model(**validated_data)
-        if adopt_components:
-            instance._adopt_components = True
-        if not replicate_components:
-            instance._disable_replication = True
-        instance.save()
-
-        if tags is not None:
-            self._save_tags(instance, tags)
-
-        return instance
-
 
 class MACAddressSerializer(PrimaryModelSerializer):
     assigned_object_type = ContentTypeField(

netbox/dcim/api/serializers_/devicetype_components.py

@@ -317,10 +317,10 @@ class ModuleBayTemplateSerializer(ComponentTemplateSerializer):
     class Meta:
         model = ModuleBayTemplate
         fields = [
-            'id', 'url', 'display', 'device_type', 'module_type', 'name', 'label', 'position', 'enabled', 'description',
+            'id', 'url', 'display', 'device_type', 'module_type', 'name', 'label', 'position', 'description',
             'created', 'last_updated',
         ]
-        brief_fields = ('id', 'url', 'display', 'name', 'enabled', 'description')
+        brief_fields = ('id', 'url', 'display', 'name', 'description')
 
 
 class DeviceBayTemplateSerializer(ComponentTemplateSerializer):
@@ -331,10 +331,10 @@ class DeviceBayTemplateSerializer(ComponentTemplateSerializer):
     class Meta:
         model = DeviceBayTemplate
         fields = [
-            'id', 'url', 'display', 'device_type', 'name', 'label', 'enabled', 'description',
+            'id', 'url', 'display', 'device_type', 'name', 'label', 'description',
             'created', 'last_updated'
         ]
-        brief_fields = ('id', 'url', 'display', 'name', 'enabled', 'description')
+        brief_fields = ('id', 'url', 'display', 'name', 'description')
 
 
 class InventoryItemTemplateSerializer(ComponentTemplateSerializer):

netbox/dcim/api/serializers_/racks.py

@@ -1,11 +1,9 @@
 from django.utils.translation import gettext as _
-from drf_spectacular.types import OpenApiTypes
-from drf_spectacular.utils import extend_schema_field
 from rest_framework import serializers
 
 from dcim.choices import *
 from dcim.constants import *
-from dcim.models import Rack, RackGroup, RackReservation, RackRole, RackType
+from dcim.models import Rack, RackReservation, RackRole, RackType
 from netbox.api.fields import ChoiceField, RelatedObjectCountField
 from netbox.api.serializers import OrganizationalModelSerializer, PrimaryModelSerializer
 from netbox.choices import *
@@ -18,7 +16,6 @@ from .sites import LocationSerializer, SiteSerializer
 
 __all__ = (
     'RackElevationDetailFilterSerializer',
-    'RackGroupSerializer',
     'RackReservationSerializer',
     'RackRoleSerializer',
     'RackSerializer',
@@ -26,20 +23,6 @@ __all__ = (
 )
 
 
-class RackGroupSerializer(OrganizationalModelSerializer):
-
-    # Related object counts
-    rack_count = RelatedObjectCountField('racks')
-
-    class Meta:
-        model = RackGroup
-        fields = [
-            'id', 'url', 'display_url', 'display', 'name', 'slug', 'description', 'owner', 'comments', 'tags',
-            'custom_fields', 'created', 'last_updated', 'rack_count',
-        ]
-        brief_fields = ('id', 'url', 'display', 'name', 'slug', 'description', 'rack_count')
-
-
 class RackRoleSerializer(OrganizationalModelSerializer):
 
     # Related object counts
@@ -104,11 +87,6 @@ class RackSerializer(RackBaseSerializer):
         allow_null=True,
         default=None
     )
-    group = RackGroupSerializer(
-        nested=True,
-        required=False,
-        allow_null=True
-    )
     tenant = TenantSerializer(
         nested=True,
         required=False,
@@ -149,11 +127,11 @@ class RackSerializer(RackBaseSerializer):
     class Meta:
         model = Rack
         fields = [
-            'id', 'url', 'display_url', 'display', 'name', 'facility_id', 'site', 'location', 'group', 'tenant',
-            'status', 'role', 'serial', 'asset_tag', 'rack_type', 'form_factor', 'width', 'u_height', 'starting_unit',
-            'weight', 'max_weight', 'weight_unit', 'desc_units', 'outer_width', 'outer_height', 'outer_depth',
-            'outer_unit', 'mounting_depth', 'airflow', 'description', 'owner', 'comments', 'tags', 'custom_fields',
-            'created', 'last_updated', 'device_count', 'powerfeed_count',
+            'id', 'url', 'display_url', 'display', 'name', 'facility_id', 'site', 'location', 'tenant', 'status',
+            'role', 'serial', 'asset_tag', 'rack_type', 'form_factor', 'width', 'u_height', 'starting_unit', 'weight',
+            'max_weight', 'weight_unit', 'desc_units', 'outer_width', 'outer_height', 'outer_depth', 'outer_unit',
+            'mounting_depth', 'airflow', 'description', 'owner', 'comments', 'tags', 'custom_fields', 'created',
+            'last_updated', 'device_count', 'powerfeed_count',
         ]
         brief_fields = ('id', 'url', 'display', 'name', 'description', 'device_count')
 
@@ -175,17 +153,11 @@ class RackReservationSerializer(PrimaryModelSerializer):
         allow_null=True,
     )
 
-    unit_count = serializers.SerializerMethodField()
-
-    @extend_schema_field(OpenApiTypes.INT32)
-    def get_unit_count(self, obj):
-        return len(obj.units)
-
     class Meta:
         model = RackReservation
         fields = [
-            'id', 'url', 'display_url', 'display', 'rack', 'units', 'unit_count', 'status', 'created', 'last_updated',
-            'user', 'tenant', 'description', 'owner', 'comments', 'tags', 'custom_fields',
+            'id', 'url', 'display_url', 'display', 'rack', 'units', 'status', 'created', 'last_updated', 'user',
+            'tenant', 'description', 'owner', 'comments', 'tags', 'custom_fields',
         ]
         brief_fields = ('id', 'url', 'display', 'status', 'user', 'description', 'units')
 

netbox/dcim/api/urls.py

@@ -12,7 +12,6 @@ router.register('sites', views.SiteViewSet)
 
 # Racks
 router.register('locations', views.LocationViewSet)
-router.register('rack-groups', views.RackGroupViewSet)
 router.register('rack-types', views.RackTypeViewSet)
 router.register('rack-roles', views.RackRoleViewSet)
 router.register('racks', views.RackViewSet)
@@ -64,7 +63,6 @@ router.register('mac-addresses', views.MACAddressViewSet)
 # Cables
 router.register('cables', views.CableViewSet)
 router.register('cable-terminations', views.CableTerminationViewSet)
-router.register('cable-bundles', views.CableBundleViewSet)
 
 # Virtual chassis
 router.register('virtual-chassis', views.VirtualChassisViewSet)

netbox/dcim/api/views.py

@@ -19,7 +19,6 @@ from netbox.api.pagination import StripCountAnnotationsPaginator
 from netbox.api.viewsets import MPTTLockedMixin, NetBoxModelViewSet, NetBoxReadOnlyModelViewSet
 from netbox.api.viewsets.mixins import SequentialBulkCreatesMixin
 from utilities.api import get_serializer_for_model
-from utilities.query import count_related
 from utilities.query_functions import CollateAsChar
 from virtualization.models import VirtualMachine
 
@@ -155,17 +154,6 @@ class LocationViewSet(MPTTLockedMixin, NetBoxModelViewSet):
     filterset_class = filtersets.LocationFilterSet
 
 
-#
-# Rack groups
-#
-
-
-class RackGroupViewSet(NetBoxModelViewSet):
-    queryset = RackGroup.objects.all()
-    serializer_class = serializers.RackGroupSerializer
-    filterset_class = filtersets.RackGroupFilterSet
-
-
 #
 # Rack roles
 #
@@ -586,14 +574,6 @@ class CableTerminationViewSet(NetBoxReadOnlyModelViewSet):
     filterset_class = filtersets.CableTerminationFilterSet
 
 
-class CableBundleViewSet(NetBoxModelViewSet):
-    queryset = CableBundle.objects.annotate(
-        cable_count=count_related(Cable, 'bundle')
-    )
-    serializer_class = serializers.CableBundleSerializer
-    filterset_class = filtersets.CableBundleFilterSet
-
-
 #
 # Virtual chassis
 #

netbox/dcim/cable_profiles.py

@@ -254,21 +254,6 @@ class Trunk8C4PCableProfile(BaseCableProfile):
     b_connectors = a_connectors
 
 
-class Breakout1C2Px2C1PCableProfile(BaseCableProfile):
-    a_connectors = {
-        1: 2,
-    }
-    b_connectors = {
-        1: 1,
-        2: 1,
-    }
-    _mapping = {
-        (1, 1): (1, 1),
-        (1, 2): (2, 1),
-        (2, 1): (1, 2),
-    }
-
-
 class Breakout1C4Px4C1PCableProfile(BaseCableProfile):
     a_connectors = {
         1: 4,

netbox/dcim/choices.py

@@ -1003,16 +1003,10 @@ class InterfaceTypeChoices(ChoiceSet):
     TYPE_800GE_SR8 = '800gbase-sr8'
     TYPE_800GE_VR8 = '800gbase-vr8'
 
-    # 1.6 Tbps Ethernet
-    TYPE_1TE_CR8 = '1.6tbase-cr8'
-    TYPE_1TE_DR8 = '1.6tbase-dr8'
-    TYPE_1TE_DR8_2 = '1.6tbase-dr8-2'
-
     # Ethernet (modular)
     TYPE_100ME_SFP = '100base-x-sfp'
     TYPE_1GE_GBIC = '1000base-x-gbic'
     TYPE_1GE_SFP = '1000base-x-sfp'
-    TYPE_2GE_SFP = '2.5gbase-x-sfp'
     TYPE_10GE_SFP_PLUS = '10gbase-x-sfpp'
     TYPE_10GE_XFP = '10gbase-x-xfp'
     TYPE_10GE_XENPAK = '10gbase-x-xenpak'
@@ -1040,11 +1034,8 @@ class InterfaceTypeChoices(ChoiceSet):
     TYPE_400GE_OSFP_RHS = '400gbase-x-osfp-rhs'
     TYPE_400GE_CDFP = '400gbase-x-cdfp'
     TYPE_400GE_CFP8 = '400gbase-x-cfp8'
-    TYPE_800GE_QSFP_DD = '800gbase-x-qsfpdd'  # TODO: Rename to _QSFP_DD800
-    TYPE_800GE_OSFP = '800gbase-x-osfp'  # TODO: Rename to _OSFP800
-    TYPE_1TE_OSFP1600 = '1.6tbase-x-osfp1600'
-    TYPE_1TE_OSFP1600_RHS = '1.6tbase-x-osfp1600-rhs'
-    TYPE_1TE_QSFP_DD1600 = '1.6tbase-x-qsfpdd1600'
+    TYPE_800GE_QSFP_DD = '800gbase-x-qsfpdd'
+    TYPE_800GE_OSFP = '800gbase-x-osfp'
 
     # Backplane Ethernet
     TYPE_1GE_KX = '1000base-kx'
@@ -1058,7 +1049,6 @@ class InterfaceTypeChoices(ChoiceSet):
     TYPE_100GE_KP4 = '100gbase-kp4'
     TYPE_100GE_KR2 = '100gbase-kr2'
     TYPE_100GE_KR4 = '100gbase-kr4'
-    TYPE_1TE_KR8 = '1.6tbase-kr8'
 
     # Wireless
     TYPE_80211A = 'ieee802.11a'
@@ -1308,21 +1298,12 @@ class InterfaceTypeChoices(ChoiceSet):
                 (TYPE_800GE_VR8, '800GBASE-VR8 (800GE)'),
             )
         ),
-        (
-            _('1.6 Tbps Ethernet'),
-            (
-                (TYPE_1TE_CR8, '1.6TBASE-CR8 (1.6TE)'),
-                (TYPE_1TE_DR8, '1.6TBASE-DR8 (1.6TE)'),
-                (TYPE_1TE_DR8_2, '1.6TBASE-DR8-2 (1.6TE)'),
-            )
-        ),
         (
             _('Pluggable transceivers'),
             (
                 (TYPE_100ME_SFP, 'SFP (100ME)'),
                 (TYPE_1GE_GBIC, 'GBIC (1GE)'),
                 (TYPE_1GE_SFP, 'SFP (1GE)'),
-                (TYPE_2GE_SFP, 'SFP (2.5GE)'),
                 (TYPE_10GE_SFP_PLUS, 'SFP+ (10GE)'),
                 (TYPE_10GE_XENPAK, 'XENPAK (10GE)'),
                 (TYPE_10GE_XFP, 'XFP (10GE)'),
@@ -1352,9 +1333,6 @@ class InterfaceTypeChoices(ChoiceSet):
                 (TYPE_400GE_OSFP_RHS, 'OSFP-RHS (400GE)'),
                 (TYPE_800GE_OSFP, 'OSFP (800GE)'),
                 (TYPE_800GE_QSFP_DD, 'QSFP-DD (800GE)'),
-                (TYPE_1TE_OSFP1600, 'OSFP1600 (1.6TE)'),
-                (TYPE_1TE_OSFP1600_RHS, 'OSFP1600-RHS (1.6TE)'),
-                (TYPE_1TE_QSFP_DD1600, 'QSFP-DD1600 (1.6TE)'),
             )
         ),
         (
@@ -1371,7 +1349,6 @@ class InterfaceTypeChoices(ChoiceSet):
                 (TYPE_100GE_KP4, '100GBASE-KP4 (100GE)'),
                 (TYPE_100GE_KR2, '100GBASE-KR2 (100GE)'),
                 (TYPE_100GE_KR4, '100GBASE-KR4 (100GE)'),
-                (TYPE_1TE_KR8, '1.6TBASE-KR8 (1.6TE)'),
             )
         ),
         (
@@ -1518,12 +1495,9 @@ class InterfaceSpeedChoices(ChoiceSet):
         (10000000, '10 Gbps'),
         (25000000, '25 Gbps'),
         (40000000, '40 Gbps'),
-        (50000000, '50 Gbps'),
         (100000000, '100 Gbps'),
         (200000000, '200 Gbps'),
         (400000000, '400 Gbps'),
-        (800000000, '800 Gbps'),
-        (1600000000, '1.6 Tbps'),
     ]
 
 
@@ -1776,7 +1750,6 @@ class CableProfileChoices(ChoiceSet):
     TRUNK_4C8P = 'trunk-4c8p'
     TRUNK_8C4P = 'trunk-8c4p'
     # Breakouts
-    BREAKOUT_1C2P_2C1P = 'breakout-1c2p-2c1p'
     BREAKOUT_1C4P_4C1P = 'breakout-1c4p-4c1p'
     BREAKOUT_1C6P_6C1P = 'breakout-1c6p-6c1p'
     BREAKOUT_2C4P_8C1P_SHUFFLE = 'breakout-2c4p-8c1p-shuffle'
@@ -1816,7 +1789,6 @@ class CableProfileChoices(ChoiceSet):
         (
             _('Breakout'),
             (
-                (BREAKOUT_1C2P_2C1P, _('1C2P:2C1P breakout')),
                 (BREAKOUT_1C4P_4C1P, _('1C4P:4C1P breakout')),
                 (BREAKOUT_1C6P_6C1P, _('1C6P:6C1P breakout')),
                 (BREAKOUT_2C4P_8C1P_SHUFFLE, _('2C4P:8C1P breakout (shuffle)')),

netbox/dcim/constants.py

@@ -1,5 +1,3 @@
-import re
-
 from django.db.models import Q
 
 from .choices import InterfaceTypeChoices
@@ -81,7 +79,6 @@ NONCONNECTABLE_IFACE_TYPES = VIRTUAL_IFACE_TYPES + WIRELESS_IFACE_TYPES
 #
 
 MODULE_TOKEN = '{module}'
-VC_POSITION_RE = re.compile(r'\{vc_position(?::([^}]*))?\}')
 
 MODULAR_COMPONENT_TEMPLATE_MODELS = Q(
     app_label='dcim',