From 1c31b5bc08a2f4bc55678c69eb67508122480906 Mon Sep 17 00:00:00 2001 From: Rob Crittenden Date: Fri, 8 May 2009 11:15:45 -0400 Subject: Add a reason to the NotFound exception so we can provide more robust errors --- ipalib/plugins/automount.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'ipalib/plugins/automount.py') diff --git a/ipalib/plugins/automount.py b/ipalib/plugins/automount.py index 6378a1385..705941874 100644 --- a/ipalib/plugins/automount.py +++ b/ipalib/plugins/automount.py @@ -291,7 +291,7 @@ class automount_delkey(crud.Del): keydn = k.get('dn') break if not keydn: - raise errors.NotFound(msg='Entry not found') + raise errors.NotFound(reason='Removing keys failed. key %s not found' % keyname) return ldap.delete(keydn) def output_for_cli(self, textui, result, *args, **options): """ @@ -369,7 +369,7 @@ class automount_modkey(crud.Mod): keydn = k.get('dn') break if not keydn: - raise errors.NotFound(msg='Entry not found') + raise errors.NotFound(reason='Update failed, unable to find key %s' % keyname) return ldap.update(keydn, **kw) def output_for_cli(self, textui, result, *args, **options): @@ -517,7 +517,7 @@ class automount_showkey(crud.Get): keydn = k.get('dn') break if not keydn: - raise errors.NotFound(msg='Entry not found') + raise errors.NotFound(reason='Unable to find key %s' % keyname) # FIXME: should kw contain the list of attributes to display? if kw.get('all', False): return ldap.retrieve(keydn) -- cgit