#7695 ipa service-del <principal name=""> should display principal name instead of Invalid 'principal'.
Opened 10 months ago by abbra. Modified 10 months ago

Ticket was cloned from Red Hat Bugzilla (product Red Hat Enterprise Linux 7): Bug 1623763

Description of problem: Deleting services from the UI/console should display
principal name instead of Invalid 'principal'

Version-Release number of selected component (if applicable):
ipa-server-4.6.4-6.el7.x86_64
sssd-1.16.2-12.el7.x86_64
package samba is not installed
pki-server-10.5.9-6.el7.noarch
selinux-policy-3.13.1-220.el7.noarch
389-ds-base-1.3.8.4-11.el7.x86_64


How reproducible: Always

Steps to Reproduce:
1. Install Master and 2 Replicas
2. Navigate to IPA UI --> Identity ---> Services.
3. Select all the services and click the 'Delete' button.


Actual results:
Services required by IPA are not deleted but the pop-up window displays Invalid
'principal' instead of the actual service deleted.

[root@master ~]# ipa service-del ldap/replica1.apollo.test@APOLLO.TEST
ipa: ERROR: invalid 'principal': This principal is required by the IPA master

[Thu Aug 30 11:50:39.407794 2018] [:error] [pid 11758] ipa: DEBUG: retrieving
schema for SchemaCache url=ldapi://%2fvar%2frun%2fslapd-APOLLO-TEST.socket
conn=<ldap.ldapobject.SimpleLDAPObject instance at 0x7f57f3199908>
[Thu Aug 30 11:50:39.697064 2018] [:error] [pid 11758] ipa: DEBUG: WSGI
wsgi_execute PublicError: Traceback (most recent call last):
[Thu Aug 30 11:50:39.697083 2018] [:error] [pid 11758]   File
"/usr/lib/python2.7/site-packages/ipaserver/rpcserver.py", line 369, in
wsgi_execute
[Thu Aug 30 11:50:39.697088 2018] [:error] [pid 11758]     result =
command(*args, **options)
[Thu Aug 30 11:50:39.697098 2018] [:error] [pid 11758]   File
"/usr/lib/python2.7/site-packages/ipalib/frontend.py", line 450, in __call__
[Thu Aug 30 11:50:39.697103 2018] [:error] [pid 11758]     return
self.__do_call(*args, **options)
[Thu Aug 30 11:50:39.697107 2018] [:error] [pid 11758]   File
"/usr/lib/python2.7/site-packages/ipalib/frontend.py", line 478, in __do_call
[Thu Aug 30 11:50:39.697112 2018] [:error] [pid 11758]     ret =
self.run(*args, **options)
[Thu Aug 30 11:50:39.697116 2018] [:error] [pid 11758]   File
"/usr/lib/python2.7/site-packages/ipalib/frontend.py", line 800, in run
[Thu Aug 30 11:50:39.697121 2018] [:error] [pid 11758]     return
self.execute(*args, **options)
[Thu Aug 30 11:50:39.697125 2018] [:error] [pid 11758]   File
"/usr/lib/python2.7/site-packages/ipaserver/plugins/baseldap.py", line 1587, in
execute
[Thu Aug 30 11:50:39.697129 2018] [:error] [pid 11758]     delete_entry(pkey)
[Thu Aug 30 11:50:39.697134 2018] [:error] [pid 11758]   File
"/usr/lib/python2.7/site-packages/ipaserver/plugins/baseldap.py", line 1538, in
delete_entry
[Thu Aug 30 11:50:39.697138 2018] [:error] [pid 11758]     dn = callback(self,
ldap, dn, *nkeys, **options)
[Thu Aug 30 11:50:39.697142 2018] [:error] [pid 11758]   File
"/usr/lib/python2.7/site-packages/ipaserver/plugins/service.py", line 669, in
pre_callback
[Thu Aug 30 11:50:39.697147 2018] [:error] [pid 11758]
check_required_principal(ldap, keys[-1])
[Thu Aug 30 11:50:39.697151 2018] [:error] [pid 11758]   File
"/usr/lib/python2.7/site-packages/ipaserver/plugins/service.py", line 287, in
check_required_principal
[Thu Aug 30 11:50:39.697156 2018] [:error] [pid 11758]     raise
errors.ValidationError(name='principal', error=_('This principal is required by
the IPA master'))
[Thu Aug 30 11:50:39.697160 2018] [:error] [pid 11758] ValidationError: invalid
'principal': This principal is required by the IPA master

Expected results: It's better to display the output in console and UI by
replacing the Invalid 'principal' with actual principal name to avoid confusion
as in that the principal name is invalid.

e.g
[root@master ~]# ipa service-del ldap/replica1.apollo.test@APOLLO.TEST
ipa: ERROR: ldap/replica1.apollo.test@APOLLO.TEST: This principal is required
by the IPA master

Additional info: Logging this as an bug to change the output being displayed
while deleting the service.

Metadata Update from @abbra:
- Custom field rhbz adjusted to https://bugzilla.redhat.com/show_bug.cgi?id=1623763

10 months ago

Login to comment on this ticket.

Metadata