Merge pull request #242 from fluxcd/rev-change-event-panic

This commit is contained in:
Hidde Beydals 2020-12-16 15:43:13 +01:00 committed by GitHub
commit 72e00e82c0
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 2 deletions

View File

@ -188,8 +188,8 @@ func (r *HelmChartReconciler) Reconcile(req ctrl.Request) (ctrl.Result, error) {
}
// Emit an event if we did not have an artifact before, or the revision has changed
if (chart.Status.Artifact == nil && reconciledChart.Status.Artifact != nil) ||
reconciledChart.Status.Artifact.Revision != chart.Status.Artifact.Revision {
if (chart.GetArtifact() == nil && reconciledChart.GetArtifact() != nil) ||
(chart.GetArtifact() != nil && reconciledChart.GetArtifact() != nil && reconciledChart.GetArtifact().Revision != chart.GetArtifact().Revision) {
r.event(reconciledChart, events.EventSeverityInfo, sourcev1.HelmChartReadyMessage(reconciledChart))
}
r.recordReadiness(reconciledChart)