atomic: true by default?

[mumoshu]

Creating this dedicated issue for visibility. See #471 for more context.

[mumoshu]

For backward-compatibility reason, this unlikely to happen in shorter term.

Instead, we'd better add helmfile sync --atomic that sets atomic: true to every release.

Or helmfile sync --atomic-by-default to set atomic: true only when the release explicitly set it to atomic: false for more granular control.

[mumoshu]

Context: #256 (comment)

[bozaro]

I'm removed --atomic from my CI test environments deployments, because it's make too difficult diagnostics: I should redeploy manually and hope for bug reproduce.

Also I add extra logic to workaround Error: UPGRADE FAILED: "xxx" has no deployed releases, because upgrade --install --force don't solve this error with helm3.

[RobertoUa]

I would be super awesome if when I send SIGTERM (or cmd-c in terminal) during upgrade/install of a chart, it would revert it, instead of leaving it hanging out.

[Xtigyro]

Hey @bozaro - could you please shed some more light on what extra logic you've started using to work around the issue?