Currently, we swallowed the error and directly update the phase in the status when a pipeline with the same name already exist without a pipeline Rollout manage it. We should instead return the error as reconciliation failure so the error is recorded by the metrics via the common error handler.
Message from the maintainers:
If you wish to see this enhancement implemented please add a 👍 reaction to this issue! We often sort issues this way to know what to prioritize.
Summary
Currently, we swallowed the error and directly update the phase in the status when a pipeline with the same name already exist without a pipeline Rollout manage it. We should instead return the error as reconciliation failure so the error is recorded by the metrics via the common error handler.
Message from the maintainers:
If you wish to see this enhancement implemented please add a 👍 reaction to this issue! We often sort issues this way to know what to prioritize.