From ecaa75d96710bb1dce95bc08f4ebbe5ee047dee6 Mon Sep 17 00:00:00 2001 From: Markus Hilger Date: Tue, 30 Jun 2026 04:21:16 +0200 Subject: [PATCH 1/2] Fix validvalues typo valid_values is never checked and is a typo. Use validvalues instead. Note: This can break existing scripts if invalid values are used. --- confluent_server/confluent/config/attributes.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/confluent_server/confluent/config/attributes.py b/confluent_server/confluent/config/attributes.py index efbc4cb9..c45a87b4 100644 --- a/confluent_server/confluent/config/attributes.py +++ b/confluent_server/confluent/config/attributes.py @@ -610,7 +610,7 @@ node = { }, 'snmp.privacyprotocol': { 'description': 'The privacy protocol to use for SNMPv3', - 'valid_values': ('aes', 'des'), + 'validvalues': ('aes', 'des'), }, # 'secret.snmplocalizedkey': { # 'description': ("SNMPv3 key localized to this node's SNMP Engine id" @@ -657,7 +657,7 @@ node = { 'Note that if the trusted CA verifies the certificate,' ' that is accepted ignoring this policy. Default ' 'policy is "automatic"'), - 'valid_values': ('automatic', 'manual'), + 'validvalues': ('automatic', 'manual'), }, 'pubkeys.tls_hardwaremanager': { 'description': ('Fingerprint of the TLS certificate recognized as' From aed0bf0bea40bad92476c7e0e74f4d4b54df84f9 Mon Sep 17 00:00:00 2001 From: Markus Hilger Date: Tue, 30 Jun 2026 04:24:59 +0200 Subject: [PATCH 2/2] Add valid_values to hardwaremanagement.method --- confluent_server/confluent/config/attributes.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/confluent_server/confluent/config/attributes.py b/confluent_server/confluent/config/attributes.py index c45a87b4..d4c9a31e 100644 --- a/confluent_server/confluent/config/attributes.py +++ b/confluent_server/confluent/config/attributes.py @@ -417,7 +417,11 @@ node = { 'hardwaremanagement.method': { 'description': 'The method used to perform operations such as power ' 'control, get sensor data, get inventory, and so on. ' - 'ipmi is used if not specified.' + 'ipmi is used if not specified.', + 'validvalues': ('affluent', 'cnos', 'cooltera', 'deltapdu', + 'eatonpdu', 'enclosure', 'enlogic', 'enos', 'geist', + 'ipmi', 'null', 'nxos', 'pdu', 'proxmox', 'raritan', + 'redfish', 'srlinux', 'vcenter'), }, 'hardwaremanagement.port': { 'description': 'The port the BMC should be configured to connect to '