Skip to content
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

Refactor plugins configuration #206

Open
3 of 7 tasks
damyan opened this issue Oct 29, 2024 · 1 comment · Fixed by #207, #213, #219 or #222
Open
3 of 7 tasks

Refactor plugins configuration #206

damyan opened this issue Oct 29, 2024 · 1 comment · Fixed by #207, #213, #219 or #222
Assignees
Labels
enhancement New feature or request

Comments

@damyan
Copy link
Member

damyan commented Oct 29, 2024

Summary

Currently, each plugin (except metal) is configured through a string list of parameters. It is a limitation coming from coredhcp. With the configuration becoming more and more complicated, a dedicated knowledge about parameter order and meaning is required. Better would be a configuration through a plugin-specific yaml file (as for the metal plugin), where the parameter meaning can be deduced from the configuration structure.

Basic example

IPAM configuration, old:

- plugins
  - ipam: ipam-ns ipam-subnet1,ipam-subnet2
[…]

IPAM configuration, new:

- plugins
  - ipam: ipam_config.yaml

ipam_config.yaml:

namespace: ipam-ns
subnets:
  - ipam-subnet1
  - ipam-subnet2

Motivation

This wil improve greatly the configuration readability.

Implement for:

  • blufield
  • httpboot
  • ipam
  • metal
  • onmetal
  • oob
  • pxeboot
@damyan damyan added the enhancement New feature or request label Oct 29, 2024
@damyan damyan self-assigned this Oct 29, 2024
@damyan
Copy link
Member Author

damyan commented Oct 29, 2024

/cc @afritzler

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment