Merge pull request #48 from fluxcd/status-message

status: add revision to the ready condition message

Author: stefanprodan

controllers/gitrepository_controller.go

@@ -358,7 +358,7 @@ func (r *GitRepositoryReconciler) sync(ctx context.Context, repository sourcev1.
 		return sourcev1.GitRepositoryNotReady(repository, sourcev1.StorageOperationFailedReason, err.Error()), err
 	}
 
-	message := fmt.Sprintf("Git repoistory artifacts are available at: %s", artifact.Path)
+	message := fmt.Sprintf("Fetched revision: %s", artifact.Revision)
 	return sourcev1.GitRepositoryReady(repository, artifact, url, sourcev1.GitOperationSucceedReason, message), nil
 }
 

controllers/helmchart_controller.go

@@ -248,7 +248,7 @@ func (r *HelmChartReconciler) sync(ctx context.Context, repository sourcev1.Helm
 		return sourcev1.HelmChartNotReady(chart, sourcev1.StorageOperationFailedReason, err.Error()), err
 	}
 
-	message := fmt.Sprintf("Helm chart is available at: %s", artifact.Path)
+	message := fmt.Sprintf("Fetched revision: %s", artifact.Revision)
 	return sourcev1.HelmChartReady(chart, artifact, chartUrl, sourcev1.ChartPullSucceededReason, message), nil
 }
 

controllers/helmrepository_controller.go

@@ -216,7 +216,7 @@ func (r *HelmRepositoryReconciler) sync(ctx context.Context, repository sourcev1
 		return sourcev1.HelmRepositoryNotReady(repository, sourcev1.StorageOperationFailedReason, err.Error()), err
 	}
 
-	message := fmt.Sprintf("Helm repository index is available at: %s", artifact.Path)
+	message := fmt.Sprintf("Fetched revision: %s", artifact.Revision)
 	return sourcev1.HelmRepositoryReady(repository, artifact, indexURL, sourcev1.IndexationSucceededReason, message), nil
 }