fix: properly report missing CRDs from informers (#1540)

Fixes #1539

Author: metacosm

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/ReconcilerUtils.java

@@ -133,7 +133,7 @@ public static void handleKubernetesClientException(Exception e, String resourceT
         // only throw MissingCRDException if the 404 error occurs on the target CRD
         if (resourceTypeName.equals(ke.getFullResourceName())
             || matchesResourceType(resourceTypeName, ke)) {
-          throw new MissingCRDException(resourceTypeName, null, e.getMessage(), e);
+          throw new MissingCRDException(resourceTypeName, ke.getVersion(), e.getMessage(), e);
         }
       }
     }

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/Controller.java

@@ -299,7 +299,7 @@ public synchronized void start(boolean startEventProcessor) throws OperatorExcep
       log.info("'{}' controller started, pending event sources initialization", controllerName);
     } catch (MissingCRDException e) {
       stop();
-      throwMissingCRDException(crdName, specVersion, controllerName);
+      throwMissingCRDException(e.getCrdName(), e.getSpecVersion(), controllerName);
     }
   }
 

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/event/source/informer/InformerWrapper.java

@@ -7,6 +7,9 @@
 import java.util.function.Predicate;
 import java.util.stream.Stream;
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
 import io.fabric8.kubernetes.api.model.HasMetadata;
 import io.fabric8.kubernetes.client.informers.ResourceEventHandler;
 import io.fabric8.kubernetes.client.informers.SharedIndexInformer;
@@ -21,6 +24,8 @@
 class InformerWrapper<T extends HasMetadata>
     implements LifecycleAware, IndexerResourceCache<T> {
 
+  private static final Logger log = LoggerFactory.getLogger(InformerWrapper.class);
+
   private final SharedIndexInformer<T> informer;
   private final Cache<T> cache;
 
@@ -45,8 +50,7 @@ public void start() throws OperatorException {
               });
             } else {
               final var apiTypeClass = informer.getApiTypeClass();
-              final var fullResourceName =
-                  HasMetadata.getFullResourceName(apiTypeClass);
+              final var fullResourceName = HasMetadata.getFullResourceName(apiTypeClass);
               final var version = HasMetadata.getVersion(apiTypeClass);
               throw new IllegalStateException(
                   "Cannot retrieve 'stopped' callback to listen to informer stopping for informer for "
@@ -55,8 +59,12 @@ public void start() throws OperatorException {
           });
 
     } catch (Exception e) {
-      ReconcilerUtils.handleKubernetesClientException(e,
-          HasMetadata.getFullResourceName(informer.getApiTypeClass()));
+      final var apiTypeClass = informer.getApiTypeClass();
+      final var fullResourceName = HasMetadata.getFullResourceName(apiTypeClass);
+      final var version = HasMetadata.getVersion(apiTypeClass);
+      log.error("Couldn't start informer for " + fullResourceName + "/" + version + " resources",
+          e);
+      ReconcilerUtils.handleKubernetesClientException(e, fullResourceName);
       throw e;
     }
   }