Fix ipactl service ordering

Ipactl sorted service start order as string, which causes service with start order
100 starts before service with start order 30.

Patch fixes ipactl to use integers for ordering.

Reviewed-By: David Kupka <dkupka@redhat.com>
This commit is contained in:
Martin Basti 2014-10-08 16:40:53 +02:00 committed by Martin Kosek
parent 7b7567aabf
commit 57c510dcc7

View File

@ -159,7 +159,11 @@ def get_config(dirsrv):
name = entry.single_value['cn'] name = entry.single_value['cn']
for p in entry['ipaConfigString']: for p in entry['ipaConfigString']:
if p.startswith('startOrder '): if p.startswith('startOrder '):
order = p.split()[1] try:
order = int(p.split()[1])
except ValueError:
raise IpactlError("Expected order as integer in: %s:%s" % (
name, p))
svc_list.append([order, name]) svc_list.append([order, name])
ordered_list = [] ordered_list = []