Skip to content

feat: Add Support for Secrets defined at other objects#120

Open
psuet wants to merge 1 commit into
miaow2:mainfrom
psuet:secrets-precedence
Open

feat: Add Support for Secrets defined at other objects#120
psuet wants to merge 1 commit into
miaow2:mainfrom
psuet:secrets-precedence

Conversation

@psuet

@psuet psuet commented May 24, 2026

Copy link
Copy Markdown

Right now this plugin only supports configuring secrets for each device seperately. This is not ideal in setups where the same credentials are used for a class of devices.

This pull request introduces a configurable and extensible mechanism for fetching secrets from related objects, allowing users to define the precedence order for where credentials are sourced (device, role, platform, etc.).

The plugin supports a SECRETS_PRECEDENCE configuration option, letting users specify the order in which the device secrets and related object secrets are checked for credentials. The default configuration for SECRETS_PRECEDENCE is set to ["device", "role", "platform"]

@psuet psuet changed the title feat: Add Support for Secrets defined at other levels feat: Add Support for Secrets defined at other objects May 24, 2026
@miaow2

miaow2 commented Jun 13, 2026

Copy link
Copy Markdown
Owner

Hi @psuet, thanks for your PR.

I have added two comments on the code. Please check them.

Comment thread docs/secrets.md Outdated
Comment thread netbox_config_diff/compliance/secrets.py Outdated
@psuet psuet force-pushed the secrets-precedence branch from a494153 to 6f50f12 Compare June 13, 2026 22:14
@psuet psuet requested a review from miaow2 June 16, 2026 11:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants