From 50243b67d579e7fcc4d7fb83fb8eb4f7caf02d43 Mon Sep 17 00:00:00 2001 From: Jarrod Johnson Date: Wed, 8 Oct 2025 14:20:44 -0400 Subject: [PATCH] Add a more helpful error when bmc not set When doing a configbmc, need to present a more reasonable message about lack of address. --- confluent_server/confluent/selfservice.py | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/confluent_server/confluent/selfservice.py b/confluent_server/confluent/selfservice.py index 6df8ff17..c0b185af 100644 --- a/confluent_server/confluent/selfservice.py +++ b/confluent_server/confluent/selfservice.py @@ -261,6 +261,10 @@ def handle_request(env, start_response): res['bmcvlan'] = vlan bmcaddr = hmattr.get('hardwaremanagement.manager', {}).get('value', None) + if not bmcaddr: + start_response('500 Internal Server Error', []) + yield 'Missing value in hardwaremanagement.manager' + return bmcaddr = bmcaddr.split('/', 1)[0] bmcaddr = socket.getaddrinfo(bmcaddr, 0)[0] bmcaddr = bmcaddr[-1][0]