From 06e4b4aea6d9bbc3401344c4f2e2096cc11b1f54 Mon Sep 17 00:00:00 2001
From: Rob Crittenden <rcritten@redhat.com>
Date: Apr 05 2012 20:18:36 +0000
Subject: Add missing comma to list of services that cannot be disabled.


https://fedorahosted.org/freeipa/ticket/2487

---

diff --git a/ipalib/plugins/service.py b/ipalib/plugins/service.py
index 7c563b3..333d5bb 100644
--- a/ipalib/plugins/service.py
+++ b/ipalib/plugins/service.py
@@ -208,7 +208,7 @@ def check_required_principal(ldap, hostname, service):
     try:
         host_is_master(ldap, hostname)
     except errors.ValidationError, e:
-        service_types = ['HTTP', 'ldap', 'DNS' 'dogtagldap']
+        service_types = ['HTTP', 'ldap', 'DNS', 'dogtagldap']
         if service in service_types:
             raise errors.ValidationError(name='principal', error=_('This principal is required by the IPA master'))