fix(helm): implement ordered cleanup for Helm releases based on dependency levels

[BabisK]

Fixes: #9837 Related: #7284 Merge before/after: Dependent or prerequisite PRs

Description

Respect the dependency graph between helm releases during clean up. Levels of the graph are cleaned on reverse order compared to installation. Releases in the same level are cleaned up in reverse order too.

User facing changes (remove if N/A)

Before Releases are undeployed in the order they are mentioned in Config.

^CCleaning up...
release "postgres-operator" uninstalled
release "postgres" uninstalled

After Releases are undeployed in the reverse order they were deployed, respecting the dependsOn values.

^CCleaning up...
release "postgres" uninstalled
release "postgres-operator" uninstalled

[plumpy]

@idsulik would you mind reviewing this since you added this dependency support?

[idsulik]

@plumpy hi! Everything looks fine now

[BabisK]

Fixed the UTs, the --client had been removed.

[idsulik]

@BabisK hi! What changes should I review if you rewrote the commit?

[BabisK]

@idsulik You are right. I'm still learning the processes of this repo.

The change was the removal of --client in helm version in the unit test of this change. The expectation of this command had changed across all this test file causing the test to fail.

[idsulik]

@idsulik You are right. I'm still learning the processes of this repo.

It's a bad idea to rewrite commits when contributing to open source projects or just working with someone on the same project.

If you just fix tests and they work, there is no need to request additional review.

Last but not least - keep going!)