Merge pull request #3377 from jasonvigil/fix-controller-status

fix: Update resource status even if no GCP fields are updated
GoogleCloudPlatform · Dec 13, 2024 · 9daca6b · 9daca6b
2 parents c3a37f7 + 2c1f836
commit 9daca6b
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/dev/tools/controllerbuilder/template/controller/controller.go b/dev/tools/controllerbuilder/template/controller/controller.go
@@ -227,7 +227,12 @@ func (a *{{.ProtoResource}}Adapter) Update(ctx context.Context, updateOp *direct
 
 	if len(paths) == 0 {
 		log.V(2).Info("no field needs update", "name", a.id.External)
-		return nil
+		status := &krm.{{.Kind}}Status{}
+		status.ObservedState = {{.Kind}}ObservedState_FromProto(mapCtx, a.actual)
+		if mapCtx.Err() != nil {
+			return mapCtx.Err()
+		}
+		return updateOp.UpdateStatus(ctx, status, nil)
 	}
 	updateMask := &fieldmaskpb.FieldMask{
 		Paths: sets.List(paths)}