Skip to content

Commit

Permalink
Fix DependsOn readiness check
Browse files Browse the repository at this point in the history
Signed-off-by: Sean Eagan <seaneagan@microsoft.com>
  • Loading branch information
seaneagan committed Apr 18, 2024
1 parent 5e760db commit 806947c
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion internal/controller/helmrelease_controller.go
Original file line number Diff line number Diff line change
Expand Up @@ -585,7 +585,7 @@ func (r *HelmReleaseReconciler) checkDependencies(ctx context.Context, obj *v2.H
return fmt.Errorf("unable to get '%s' dependency: %w", ref, err)
}

if dHr.Generation != dHr.Status.ObservedGeneration || !conditions.IsTrue(dHr, meta.ReadyCondition) {
if dHr.Generation != conditions.GetObservedGeneration(dHr, meta.ReadyCondition) || !conditions.IsTrue(dHr, meta.ReadyCondition) {
return fmt.Errorf("dependency '%s' is not ready", ref)
}
}
Expand Down

0 comments on commit 806947c

Please sign in to comment.