-
Notifications
You must be signed in to change notification settings - Fork 2.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Plugin events pipeline registration #15093
Comments
@arthanson does ordering matter in the |
This feature is essential for the evolution of plugins into more self-contained units, but it also requires a change to the plugin API that we cannot introduce at a micro-version boundary (3.7.2 -> 3.7.3). The soonest our semantic versioning contract would permit it is v4.0. In the meantime, we can consider alternative approaches that don't involve changes to the plugin API. |
Getting this issue out ot triage jail. We should revisit once the dust has settled from 4.0b1 and consider whether to take on the full implementation in v4.1, or look for an alternative approach that can be done within an existing release train without violating the plugin API's semver. |
NetBox version
v3.7.2
Feature type
New functionality
Proposed functionality
3.7.0 introduced the events pipeline described in #14132
This involves making an entry in
EVENTS_PIPELINE=[]
withinconfiguration.py
, it would be more ergonomic if plugins could register for events by declaring intent within their configuration.For example:
events_pipeline_registrations
would be optional to allow for backwards compatibility and default to[]
It would allow plugins which require access to the events pipeline to have entries added into
EVENTS_PIPELINE
automatically. This reduces complexity on those installing Netbox plugins which need events access.https://linear.app/netboxlabs/issue/BIZ-6/
Use case
Simplify access to the events pipeline for plugins, reduce configuration complexity.
Database changes
No response
External dependencies
No response
The text was updated successfully, but these errors were encountered: