Don't render artifacts if any of the phases are failed

[viveksinghggits]

Change Overview

We used to try to render artifacts for the succeeded phases
even if some of the phases are failed. That lead to a lot
confusion, even if the phases failed it looked like only
rendering has failed.
This is how things were being done earlier, but in the last PR where
we added support for eventual phase we tried to update that a bit.

This PR tries to fix that.

Pull request type

Please check the type of change your PR introduces:

• 🚧 Work in Progress
• 🌈 Refactoring (no functional changes, no api changes)
• 🐹 Trivial/Minor
• 🐛 Bugfix
• 🌻 Feature
• 🗺️ Documentation
• 🤖 Test

Issues

• #XXX

Test Plan

• 💪 Manual
• ⚡ Unit test
• 💚 E2E

» go test -check.f "ControllerSuite"
OK: 11 passed, 1 skipped
PASS
ok  	github.com/kanisterio/kanister/pkg/controller	110.194s

Full logs can be found here.