address comments

Signed-off-by: Zhiying Lin <[email protected]>

Author: zhiying-lin

charts/hub-agent/README.md

@@ -40,4 +40,4 @@ _See [helm install](https://helm.sh/docs/helm/helm_install/) for command documen
 | ConcurrentResourceChangeSyncs     | The number of resourceChange reconcilers that are allowed to run concurrently.                                                                               | `20`                                             |
 | logFileMaxSize                    | Max size of log file before rotation                                                                                                                         | `1000000`                                        |
 | MaxFleetSizeSupported             | The max number of member clusters this fleet supports.                                                                                                       | `100`                                            |
-| resourceSnapshotCreationInterval  | The interval (in seconds) at which resource snapshots are created.                                                                                           | `1m`                                            |
+| resourceSnapshotCreationInterval  | The interval at which resource snapshots are created.                                                                                           | `1m`                                            |

pkg/controllers/clusterresourceplacement/controller.go

@@ -497,9 +497,9 @@ func (r *Reconciler) getOrCreateClusterResourceSnapshot(ctx context.Context, crp
 		// When the latest resource snapshot without the isLastest label, it means it fails to create the new
 		// resource snapshot in the last reconcile and we don't need to check and delay the request.
 		if since := time.Since(latestResourceSnapshot.CreationTimestamp.Time); since < r.ResourceSnapshotCreationInterval {
-			// If the latest resource snapshot is created less than a minute ago, requeue the request to avoid
-			// too frequent update.
-			klog.V(2).InfoS("The latest resource snapshot is created less than a minute ago, skipping the update", "clusterResourcePlacement", crpKObj,
+			// If the latest resource snapshot is created less than configured the resourceSnapshotCreationInterval,
+			//  requeue the request to avoid too frequent update.
+			klog.V(2).InfoS("The latest resource snapshot is just created, skipping the update", "clusterResourcePlacement", crpKObj,
 				"clusterResourceSnapshot", klog.KObj(latestResourceSnapshot), "creationTime", latestResourceSnapshot.CreationTimestamp, "configuredResourceSnapshotCreationInterval", r.ResourceSnapshotCreationInterval, "afterDuration", r.ResourceSnapshotCreationInterval-since)
 			return ctrl.Result{Requeue: true, RequeueAfter: r.ResourceSnapshotCreationInterval - since}, nil, nil
 		}