search

AzBuilder / terrakube

Open source IaC Automation and Collaboration Software.
https://docs.terrakube.io
Apache License 2.0
529 stars 47 forks source link

fix: always exec plan after error #1549

Closed BenjaminDecreusefond closed 1 week ago

BenjaminDecreusefond commented 1 week ago

Hey @alfespa17 !

So I'm pushing as a first idea. The logic is that, for read-only operations like plan command we might want to run what's next to for instance display a slack message in case of error as mentionned in #1548. I don't think it is breaking any current logic and allows to achieve more complex patterns with templates ?

Idk what do you think ? Lemme know if I missed something !

alfespa17 commented 1 week ago

I am not going to merge this, you only hardcoded the value XD, it wont break any logic but at least try to change the code to validate if the execution needs to continue in some way not just hardode the value to true always XD