#2287 Ignore unknown services in ipactl
Closed: wontfix 5 years ago Opened 12 years ago by rcritten.

If an unknown service is configure in cn=masters,cn=ipa,cn=etc,$SUFFIX then ipactl can blow up:

# ipactl restart
Restarting Directory Service
Shutting down dirsrv: 
    EXAMPLE-COM...                                         [  OK  ]
Starting dirsrv: 
    EXAMPLE-COM...                                         [  OK  ]
Restarting KDC Service
Redirecting to /bin/systemctl  restart  krb5kdc.service
Restarting KPASSWD Service
Redirecting to /bin/systemctl  restart  kadmin.service
Traceback (most recent call last):
  File "/usr/sbin/ipactl", line 391, in <module>
    sys.exit(main())
  File "/usr/sbin/ipactl", line 385, in main
    ipa_restart(options)
  File "/usr/sbin/ipactl", line 293, in ipa_restart
    svc_name = service.SERVICE_LIST[svc][0]
KeyError: 'MEMCACHE'

The fix is simple:

diff -u ipactl  /usr/sbin/ipactl 
--- ipactl      2012-01-20 11:01:38.000000000 -0500
+++ /usr/sbin/ipactl    2012-01-20 16:56:49.000000000 -0500
@@ -290,7 +290,11 @@
         return

     for (order, svc) in sorted(svc_list):
-        svc_name = service.SERVICE_LIST[svc][0]
+        try:
+            svc_name = service.SERVICE_LIST[svc][0]
+        except KeyError:
+            print "Unknown service %s" % svc
+            continue
         svchandle = ipaservices.service(svc_name)
         try:
             print "Restarting %s Service" % svc

I saw this only because I was testing the session code then re-installed rpms that lacked the patch without reinstalling the server. That a user would see this is unlikely.


Metadata Update from @rcritten:
- Issue assigned to rcritten
- Issue set to the milestone: Tickets Deferred

7 years ago

Thank you taking time to submit this request for FreeIPA. Unfortunately this bug was not given priority and the team lacks the capacity to work on it at this time.

Given that we are unable to fulfil this request I am closing the issue as wontfix. To request re-consideration of this decision please reopen this issue and provide additional technical details about its importance to you.

Metadata Update from @rcritten:
- Issue close_status updated to: wontfix
- Issue status updated to: Closed (was: Open)

5 years ago

Login to comment on this ticket.

Metadata