Drop all=True in IPACertRevocation cert_show call

This was causing a cache miss in the LDAPCache class. The
'*' + all default attributes was confusing the cache. We in fact
do not need all attributes so this is fine. This increases the
cache hits in cert.py from 7 to 24, reducing the number of
duplicate LDAP searches.

Related: freeipa#307

Signed-off-by: Rob Crittenden <rcritten@redhat.com>

src/ipahealthcheck/ipa/certs.py

@@ -1348,8 +1348,7 @@ def check(self):
 
             # Now we have the cert either way, check the recovation
             try:
-                result = api.Command.cert_show(cert.serial_number,
-                                               all=True)
+                result = api.Command.cert_show(cert.serial_number)
             except Exception as e:
                 yield Result(self, constants.ERROR,
                              key=id,