do-not-merge: Test kata-deploy helm changes

confidential-containers / operator

Operator to deploy confidential containers runtime

Apache License 2.0

113 stars 60 forks source link

do-not-merge: Test kata-deploy helm changes #422

Closed fidencio closed 2 months ago

wainersm commented 3 months ago

Hi @fidencio !

Your change is correct here so it's indeed passing the tests (unless the image you built has wrong code, which I doubt).

I ran the operator tests 4x on same environment using quay.io/fidencio/kata-deploy:debug-uninstall-operator, they passed 100% of time. It's working beautifully \o/

ldoktor commented 3 months ago

I don't want to be overly optimistic here but previously I never got over 50 deploy/cleanup iterations with the upstream latest but with your image it's on the iteration 51 and still running...

ldoktor commented 2 months ago

Actually it's 193 and still going so @fidencio either it's just different timeouts or the kata-containers change indeed improved the stability and I'm looking forward for it to be merged.