Move kubernetes config check out of providerConfigure

[jrhouston]

Description

This PR moves the logic to check that the kubernetes config for the provider is not empty from providerConfigure() to GetHelmConfiguration() which gets called later. Doing this early was triggering the progressive apply issue.

Fixes #647

Acceptance tests

• Have you added an acceptance test for the functionality being added?
• Have you run the acceptance tests on this branch?

See GitHub Actions output.

Release Note

Release note for CHANGELOG:

Fix error when using outputs from another resource in the provider block

References

Community Note

• Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
• If you are interested in working on this issue or have submitted a pull request, please leave a comment

[aareet]

Not sure how yet but maybe later we can add some sort of regression test with the kind provider to test this regularly. Not required for this to merge.

[jrhouston]

Yeah, I managed to craft a config using modules that reproduces the issue locally without provisioning a cluster on a cloud provider so we should be able to write a test to trigger this case. I agree that we should hotfix this first then add the test in another PR – I'll open an issue to capture.

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 hashibot-feedback@hashicorp.com. Thanks!