refactor, update tests and UX for crp status

Signed-off-by: Britania Rodriguez Reyes <britaniar@microsoft.com>

Author: britaniar

pkg/controllers/clusterresourceplacement/placement_status.go

@@ -232,7 +232,18 @@ func setCRPConditions(
 			shouldSkipRestCondTypes = true
 		case rpsSetCondTypeCounter[i][condition.FalseConditionStatus] > 0:
 			// There is at least one False condition of the given type being set on the per cluster placement statuses.
-			crp.SetConditions(i.FalseClusterResourcePlacementCondition(crp.Generation, rpsSetCondTypeCounter[i][condition.FalseConditionStatus]))
+			if len(crp.Status.SelectedResources) == 0 && i == condition.AppliedCondition {
+				// If the CRP has not selected any resources
+				crp.SetConditions(metav1.Condition{
+					Status:             metav1.ConditionFalse,
+					Type:               string(fleetv1beta1.ClusterResourcePlacementAppliedConditionType),
+					Reason:             ApplyFailedReason,
+					Message:            fmt.Sprintf("Failed to apply resources to %d cluster(s). Please check placement status.", rpsSetCondTypeCounter[i][condition.FalseConditionStatus]),
+					ObservedGeneration: crp.Generation,
+				})
+			} else {
+				crp.SetConditions(i.FalseClusterResourcePlacementCondition(crp.Generation, rpsSetCondTypeCounter[i][condition.FalseConditionStatus]))
+			}
 			shouldSkipRestCondTypes = true
 		default:
 			// All the conditions of the given type are True.