Merge pull request #812 from fluxcd/comments-update

Minor comment updates

controllers/bucket_controller.go

@@ -417,8 +417,9 @@ func (r *BucketReconciler) notify(ctx context.Context, oldObj, newObj *sourcev1.
 // reconcileStorage ensures the current state of the storage matches the
 // desired and previously observed state.
 //
-// All Artifacts for the object except for the current one in the Status are
-// garbage collected from the Storage.
+// The garbage collection is executed based on the flag configured settings and
+// may remove files that are beyond their TTL or the maximum number of files
+// to survive a collection cycle.
 // If the Artifact in the Status of the object disappeared from the Storage,
 // it is removed from the object.
 // If the object does not have an Artifact in its Status, a Reconciling

controllers/gitrepository_controller.go

@@ -359,7 +359,7 @@ func (r *GitRepositoryReconciler) shouldNotify(oldObj, newObj *sourcev1.GitRepos
 // reconcileStorage ensures the current state of the storage matches the
 // desired and previously observed state.
 //
-// The garbage collection is executed based on the flag based settings and
+// The garbage collection is executed based on the flag configured settings and
 // may remove files that are beyond their TTL or the maximum number of files
 // to survive a collection cycle.
 // If the Artifact in the Status of the object disappeared from the Storage,

controllers/helmchart_controller.go

@@ -332,8 +332,9 @@ func (r *HelmChartReconciler) notify(ctx context.Context, oldObj, newObj *source
 // reconcileStorage ensures the current state of the storage matches the
 // desired and previously observed state.
 //
-// All Artifacts for the object except for the current one in the Status are
-// garbage collected from the Storage.
+// The garbage collection is executed based on the flag configured settings and
+// may remove files that are beyond their TTL or the maximum number of files
+// to survive a collection cycle.
 // If the Artifact in the Status of the object disappeared from the Storage,
 // it is removed from the object.
 // If the object does not have an Artifact in its Status, a Reconciling

controllers/helmrepository_controller.go

@@ -302,8 +302,9 @@ func (r *HelmRepositoryReconciler) notify(ctx context.Context, oldObj, newObj *s
 // reconcileStorage ensures the current state of the storage matches the
 // desired and previously observed state.
 //
-// All Artifacts for the object except for the current one in the Status are
-// garbage collected from the Storage.
+// The garbage collection is executed based on the flag configured settings and
+// may remove files that are beyond their TTL or the maximum number of files
+// to survive a collection cycle.
 // If the Artifact in the Status of the object disappeared from the Storage,
 // it is removed from the object.
 // If the object does not have an Artifact in its Status, a Reconciling

controllers/storage.go

@@ -54,12 +54,12 @@ type Storage struct {
  // Hostname is the file server host name used to compose the artifacts URIs.
  Hostname string `json:"hostname"`
 
- // ArtifactRetentionTTL is the maximum number of artifacts to be kept in storage
- // after a garbage collection.
+ // ArtifactRetentionTTL is the duration of time that artifacts will be kept
+ // in storage before being garbage collected.
  ArtifactRetentionTTL time.Duration `json:"artifactRetentionTTL"`
 
- // ArtifactRetentionRecords is the duration of time that artifacts will be kept in
- // storage before being garbage collected.
+ // ArtifactRetentionRecords is the maximum number of artifacts to be kept in
+ // storage after a garbage collection.
  ArtifactRetentionRecords int `json:"artifactRetentionRecords"`
 }