From ba80455e5a4aeb807fea8b40ba4503bde4f035b5 Mon Sep 17 00:00:00 2001 From: Michael DeHaan Date: Tue, 27 Feb 2007 14:46:54 -0500 Subject: sseago's patch to set_kernel_options, plus some additional error handling around kernel options processing. Plus some manpage/NEWS updates. --- cobbler/cobbler_msg.py | 1 + cobbler/item.py | 6 +++--- 2 files changed, 4 insertions(+), 3 deletions(-) (limited to 'cobbler') diff --git a/cobbler/cobbler_msg.py b/cobbler/cobbler_msg.py index 3e32e9d..6396706 100644 --- a/cobbler/cobbler_msg.py +++ b/cobbler/cobbler_msg.py @@ -125,6 +125,7 @@ _msg_table = { "exc_no_template" : "can't read /etc/cobbler/dhcp.template", "exc_dhcp_nomac" : "when cobbler is managing dhcpd.conf, all system names must be MAC addresses. Aborting.", "exc_ia64_noip" : "due to an IA64 bootloader limitation, system (%s) must define a pxe-address value (which is an IP address)", + "exc_kopts" : "Kernel options should be a space delimited list of values and key=value pairs", "lock" : "Locked. If cobbler is currently running, wait for termination, otherwise remove /var/lib/cobbler/lock", "check_ok" : """ No setup problems found. diff --git a/cobbler/item.py b/cobbler/item.py index 1249229..4263494 100644 --- a/cobbler/item.py +++ b/cobbler/item.py @@ -29,11 +29,11 @@ class Item(serializable.Serializable): def set_kernel_options(self,options): """ Kernel options are a space delimited list, - like 'a=b,c=d,e=f' or as a hash. + like 'a=b c=d e=f g h i=j' or a hash. """ - (success, value) = utils.input_string_or_hash(options," ") + (success, value) = utils.input_string_or_hash(options,None) if not success: - return False + raise cexceptions.CobblerException("exc_kopts") else: self.kernel_options = value return True -- cgit