fix: handle reserved annotations for PVCs (#1103)

Closes #1102.

Details:
1. fix the `reconcileAfter` logic
2. handle reserved annotations for PVCs

Author: leafty

api/v1alpha1/zz_generated.deepcopy.go

@@ -166,7 +166,8 @@ func (r *AmaltheaSessionReconciler) Reconcile(ctx context.Context, req ctrl.Requ
 	}
 
 	newStatus := updates.Status(ctx, r, amaltheasession)
-	statusChanged := reflect.DeepEqual(amaltheasession.Status, newStatus)
+	statusChanged := !reflect.DeepEqual(amaltheasession.Status, newStatus)
+
 	amaltheasession.Status = newStatus
 	err = r.Status().Update(ctx, amaltheasession)
 	if err != nil {

internal/controller/amaltheasession_controller.go

@@ -186,8 +186,25 @@ func (c ChildResource[T]) Reconcile(ctx context.Context, clnt client.Client, cr
 					// NOTE: If the desired storage class is nil then the current spec contains the name for the default storage class
 					current.Spec.StorageClassName = desired.Spec.StorageClassName
 				}
+				// Do not touch reserved labels and annotations
+				// Reference: https://kubernetes.io/docs/reference/labels-annotations-taints/
+				// TODO: handle labels
 				current.Labels = desired.Labels
+				preservedAnnotations := map[string]string{}
+				for key := range current.Annotations {
+					domain, _, _ := strings.Cut(key, "/")
+					if domain == "kubernetes.io" || domain == "k8s.io" || strings.HasSuffix(domain, ".kubernetes.io") ||
+						strings.HasSuffix(domain, ".k8s.io") {
+						preservedAnnotations[key] = current.Annotations[key]
+					}
+				}
 				current.Annotations = desired.Annotations
+				if current.Annotations == nil {
+					current.Annotations = map[string]string{}
+				}
+				for key := range preservedAnnotations {
+					current.Annotations[key] = preservedAnnotations[key]
+				}
 			default:
 				return fmt.Errorf("attempting to reconcile PVC with unknown stategy %s", strategy)
 			}