get default variable value when following a terraform variable

[Adam-D-Lewis]

Reference Issues or PRs

This part of the release process was failing with error ValueError: Could not find loki:{'description': 'version to deploy for the loki helm chart', 'type': '${string}', 'default': '5.43.3'} directory in helm_charts.. chart_version was being set to {'description': 'version to deploy for the loki helm chart', 'type': '${string}', 'default': '5.43.3'} instead of '5.43.3'. This became an issue with the recent addition of loki.

I tested the change locally and it seems to run fine. I'm not sure if there are other use cases where we would not want to grab the default value, but it does seem to run successfully when I run it locally. Asking @viniciusdc to review since I believe he originally implemented this and may have more insight.

What does this implement/fix?

Put a x in the boxes that apply

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds a feature)
• Breaking change (fix or feature that would cause existing features not to work as expected)
• Documentation Update
• Code style update (formatting, renaming)
• Refactoring (no functional changes, no API changes)
• Build related changes
• Other (please describe):

Testing

• Did you test the pull request locally?
• Did you add new tests? No

Any other comments?

[viniciusdc]

LGTM, I don't see any problems with using the variable's default value. The primary purpose of this was to catch any issues with the default versions and avoid situations where the helm is no longer available.