Skip to content

Commit

Permalink
Clarify the "DSPA not found" log message that happens when a DSPA is …
Browse files Browse the repository at this point in the history
…deleted

Co-authored-by: Greg Sheremeta <[email protected]>
  • Loading branch information
Achyut Madhusudan and gregsheremeta committed Aug 7, 2024
1 parent f03a1a2 commit 8f7fabc
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion controllers/dspipeline_controller.go
Original file line number Diff line number Diff line change
Expand Up @@ -177,7 +177,8 @@ func (r *DSPAReconciler) Reconcile(ctx context.Context, req ctrl.Request) (ctrl.
dspa := &dspav1alpha1.DataSciencePipelinesApplication{}
err := r.Get(ctx, req.NamespacedName, dspa)
if err != nil && apierrs.IsNotFound(err) {
log.Info("DSPA resource was not found")
// TODO change this to a Debug message when doing https://issues.redhat.com/browse/RHOAIENG-1650
log.Info("DSPA resource was not found, assuming it was recently deleted, nothing to do here")
return ctrl.Result{}, nil
} else if err != nil {
log.Error(err, "Encountered error when fetching DSPA")
Expand Down

0 comments on commit 8f7fabc

Please sign in to comment.