PLUGINS_CONFIG appears twice in configuration.example.py #3561

Closed
opened 2025-12-29 18:29:53 +01:00 by adam · 0 comments
Owner

Originally created by @candlerb on GitHub (Apr 13, 2020).

Environment

  • Python version: N/A
  • NetBox version: 2.8.0

Steps to Reproduce

Read the PLUGINS_* settings from configuration.example.py

Expected Behavior

Consistent settings

Observed Behavior

PLUGINS_* settings appear in to different parts, and in particular PLUGINS_CONFIG appears in two places: here and here.

Most configuration settings are in alphabetical order, so I suggest:

  • Move PLUGINS_ENABLED up to where PLUGINS and PLUGINS_CONFIG are (although why is PLUGINS_ENABLED required as a separate setting, if you can just set PLUGINS to an empty list?)
  • Move the sample PLUGINS_CONFIG up to where PLUGINS_CONFIG is first defined
Originally created by @candlerb on GitHub (Apr 13, 2020). ### Environment * Python version: N/A * NetBox version: 2.8.0 ### Steps to Reproduce Read the `PLUGINS_*` settings from [configuration.example.py](https://github.com/netbox-community/netbox/blob/develop/netbox/netbox/configuration.example.py) ### Expected Behavior Consistent settings ### Observed Behavior `PLUGINS_*` settings appear in to different parts, and in particular PLUGINS_CONFIG appears in two places: [here](https://github.com/netbox-community/netbox/blob/develop/netbox/netbox/configuration.example.py#L182) and [here](https://github.com/netbox-community/netbox/blob/develop/netbox/netbox/configuration.example.py#L217). Most configuration settings are in alphabetical order, so I suggest: * Move PLUGINS_ENABLED up to where PLUGINS and PLUGINS_CONFIG are (although why is PLUGINS_ENABLED required as a separate setting, if you can just set PLUGINS to an empty list?) * Move the sample PLUGINS_CONFIG up to where PLUGINS_CONFIG is first defined
adam added the status: acceptedtype: documentation labels 2025-12-29 18:29:53 +01:00
adam closed this issue 2025-12-29 18:29:53 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#3561