Improved error message when no agent connection is specified.

Author: Silvenga

src/Contrast.K8s.AgentOperator/Core/State/Appliers/AgentInjectorApplier.cs

@@ -48,13 +48,15 @@ public override async ValueTask<AgentInjectorResource> CreateFrom(V1Beta1AgentIn
             );
             var selector = GetSelector(spec, @namespace);
 
-            var connectionName = spec.Connection?.Name ?? _clusterDefaults.GetDefaultAgentConnectionName(@namespace);
-            var connectionReference = new AgentInjectorConnectionReference(@namespace, connectionName);
+            var namespaceDefaultConnectionName = _clusterDefaults.GetDefaultAgentConnectionName(@namespace);
+            var connectionName = spec.Connection?.Name ?? namespaceDefaultConnectionName;
+            var isConnectionNamespaceDefault = connectionName == namespaceDefaultConnectionName;
+            var connectionReference = new AgentInjectorConnectionReference(@namespace, connectionName, isConnectionNamespaceDefault);
 
             var namespaceDefaultConfigurationName = _clusterDefaults.GetDefaultAgentConfigurationName(@namespace);
             var configurationName = spec.Configuration?.Name ?? namespaceDefaultConfigurationName;
-            var isNamespaceDefault = namespaceDefaultConfigurationName == configurationName;
-            var configurationReference = new AgentConfigurationReference(@namespace, configurationName, isNamespaceDefault);
+            var isConfigurationNamespaceDefault = namespaceDefaultConfigurationName == configurationName;
+            var configurationReference = new AgentConfigurationReference(@namespace, configurationName, isConfigurationNamespaceDefault);
 
             var pullSecretName = spec.Image.PullSecretName != null ? new SecretReference(@namespace, spec.Image.PullSecretName, ".dockerconfigjson") : null;
             var pullPolicy = spec.Image.PullPolicy ?? "Always";

src/Contrast.K8s.AgentOperator/Core/State/Resources/Primitives/AgentInjectorConnectionReference.cs

@@ -3,5 +3,5 @@
 
 namespace Contrast.K8s.AgentOperator.Core.State.Resources.Primitives
 {
-    public record AgentInjectorConnectionReference(string Namespace, string Name);
+    public record AgentInjectorConnectionReference(string Namespace, string Name, bool IsNamespaceDefault);
 }

src/Contrast.K8s.AgentOperator/Core/State/StateContainerExtensions.cs

@@ -35,6 +35,7 @@ public static async ValueTask<ReadyResult<AgentInjectorResource>> GetReadyAgentI
                                               && await state.GetReadyAgentConnectionById(
                                                   connectionRef.Name,
                                                   connectionRef.Namespace,
+                                                  connectionRef.IsNamespaceDefault,
                                                   context,
                                                   cancellationToken
                                               ) != null;
@@ -76,6 +77,7 @@ public static async ValueTask<ReadyResult<AgentInjectorResource>> GetReadyAgentI
         private static async ValueTask<AgentConnectionResource?> GetReadyAgentConnectionById(this IStateContainer state,
                                                                                              string name,
                                                                                              string @namespace,
+                                                                                             bool isNamespaceDefault,
                                                                                              ReadyContext readyContext,
                                                                                              CancellationToken cancellationToken = default)
         {
@@ -112,7 +114,9 @@ public static async ValueTask<ReadyResult<AgentInjectorResource>> GetReadyAgentI
             }
             else
             {
-                readyContext.AddFailureReason($"AgentConnection '{@namespace}/{name}' could not be found.");
+                readyContext.AddFailureReason(isNamespaceDefault
+                    ? "AgentConnection  was not specified, and no cluster default was found."
+                    : $"AgentConnection '{@namespace}/{name}' could not be found.");
             }
 
             return null;