Skip to content

Commit

Permalink
Merge pull request #682 from gregsheremeta/DSPA-resource-was-not-foun…
Browse files Browse the repository at this point in the history
…d-Debug

Clarify the "DSPA not found" log message that happens when a DSPA is deleted
  • Loading branch information
openshift-merge-bot[bot] authored Aug 7, 2024
2 parents 455b92e + 8f7fabc commit 2fb7450
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 2fb7450

Please sign in to comment.