-
-
Notifications
You must be signed in to change notification settings - Fork 39
Modernize fan control scripts to use shared utilities #49
Copy link
Copy link
Open
Labels
choreRefactor, dependency update, CI, docsRefactor, dependency update, CI, docsinfraProxmox, Ceph, SDN, networking, host scriptsProxmox, Ceph, SDN, networking, host scripts
Description
Description
The fan control scripts have not been updated to use the current shared utility framework:
Host/FanControl/DellIPMIFanControl.sh- Dell IPMI fan control, hardware dependentHost/FanControl/EnablePWMFanControl.sh- PWM fan control, hardware dependent
These scripts need to be modernized to use ArgumentParser.sh for argument handling and follow current script conventions established by recently updated scripts. Being hardware-dependent, they need careful testing considerations noted in the scripts.
Acceptance Criteria
-
DellIPMIFanControl.shusesArgumentParser.shfor argument handling -
EnablePWMFanControl.shusesArgumentParser.shfor argument handling - Both scripts source shared utilities consistently
- Hardware dependency requirements are documented in script headers
- Existing functionality preserved
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
choreRefactor, dependency update, CI, docsRefactor, dependency update, CI, docsinfraProxmox, Ceph, SDN, networking, host scriptsProxmox, Ceph, SDN, networking, host scripts
Projects
Status
Backlog