bug: when the Promise is invalid, installing it via Promise release gives you no feedback
kirederik opened this issue · comments
Steps to reproduce
- Create an invalid promise
- Add the following label to it:
kratix.io/promise-version: v1.0.0+beta.1
- Try to install this promise
- See that it fails
- Create a Promise Release pointing to this invalid promise
apiVersion: platform.kratix.io/v1alpha1
kind: PromiseRelease
metadata:
name: promise
spec:
version: v1.0.0+beta.1
sourceRef:
type: http
url: <url>
- Apply the Promise release
Expected
Some feedback
Actual
None
By feedback, I'm assuming it to be the PromiseRelease's status and maybe an event recorded for it. I wish to give it a try.
/assign
Any idea why we're skipping the error from CreateOrUpdate
here?
kratix/controllers/promiserelease_controller.go
Lines 148 to 166 in f680db0
@Bhargav-InfraCloud i'm not sure there's a specific reason behind skipping this error 🤔 my guess is that this is why the error is not bubbling up?
@kirederik Submitted PR #101 for this. Please review.
I'll take a look! We are in the process of putting a CLA together so getting it merged can get delayed for a few days but I'll check the code later today
Sure @kirederik