Skip to content

feat(ironic): add network device credentials config - #2408

Open
khackworth wants to merge 3 commits into
mainfrom
add-network-device-credentials-config
Open

khackworth wants to merge 3 commits into
mainfrom
add-network-device-credentials-config

Conversation

@khackworth

Copy link
Copy Markdown
Contributor

Adding volume mount for network-device-credentials secret
Adding oslo.config for netdev creds
includes get_credential and get_panos_credentials helpers

@khackworth
khackworth force-pushed the add-network-device-credentials-config branch from 45dfaf5 to 2016a8e Compare September 25, 2026 14:19
@github-actions

github-actions Bot commented Sep 25, 2026 •

Copy link
Copy Markdown

Python test results

80 tests  +80   80 ✅ +80   8s ⏱️ +8s
 1 suites + 1    0 💤 ± 0 
 1 files   ± 0    0 ❌ ± 0 

Results for commit d330bb4. ± Comparison against base commit eb7e2b5.

♻️ This comment has been updated with latest results.

@cardoe cardoe left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We really don't need to mess with mounting paths. We should use Oslo.config for anything we need to load.

- adds config and helper util to read creds
- integrates with puc-2053
- read in network-device-credentials-ini into etcSources as optional
- __init__.py to load panos, f5 and service_accounts
- helper to read from oslo.config
- enroll-fw to add network-device-credentials (future use?)
@khackworth
khackworth force-pushed the add-network-device-credentials-config branch from 70f5ff7 to 8122093 Compare September 28, 2026 17:04
@khackworth
khackworth requested a review from cardoe September 28, 2026 17:36

This branch has not been deployed

No deployments
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