diff options
| author | Martin Basti <mbasti@redhat.com> | 2014-10-08 16:40:53 +0200 |
|---|---|---|
| committer | Martin Kosek <mkosek@redhat.com> | 2014-10-09 12:52:31 +0200 |
| commit | 56a146a66627e71fcd927ede7608ed3358cd904c (patch) | |
| tree | 3d55b51287be98922653d0d3f1d1af13e78190cf | |
| parent | f336bcbd9be139ff825433c055465e26f98b4f18 (diff) | |
| download | freeipa-56a146a66627e71fcd927ede7608ed3358cd904c.tar.gz freeipa-56a146a66627e71fcd927ede7608ed3358cd904c.tar.xz freeipa-56a146a66627e71fcd927ede7608ed3358cd904c.zip | |
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>
| -rwxr-xr-x | install/tools/ipactl | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/install/tools/ipactl b/install/tools/ipactl index 3f0e79867..7a1e41b01 100755 --- a/install/tools/ipactl +++ b/install/tools/ipactl @@ -159,7 +159,11 @@ def get_config(dirsrv): name = entry.single_value['cn'] for p in entry['ipaConfigString']: 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]) ordered_list = [] |