Draft: Leave environment up for debugging instead of dumping logs and uninstalling

These changes disable dumping the logs from the controller in the event that test.sh fails. These logs are often quite long, making it difficult to read the CI job log.

Instead, CI will print out a command to check the logs of the controller in the correct namespace.

To facilitate this, when: always was removed from the cleanup job. This means it defaults to when: success, meaning the environment will be cleaned up only if test.sh and QA are successful.

In the future, we may want to consider automatically shutting down these environments after a given number of days (as we do in the Charts CI).