Remove unnecessary TF_CACHE_KEY (!244) · Merge requests · GitLab.org / Terraform Images

João Alexandre Cunha requested to merge Alexand-master-patch-1ad6 into master Oct 25, 2023

I believe this was a copy/paste mistake.

I think this was going to be used because initially thought about implementing this feature as per the requirements in Add new CI template using the GitLab Terraform ... (gitlab#223131 - closed):

The user should be able to configure their terraform state and cache artifact name.

Although, mid-flight we found out that this was not possible at the time. As per gitlab!43802 (comment 422286984) discussion:

As we've discussed, we realized that we can't use Environment Variables for the artifacts name since the variables' values are not propagated expanded to the FE. I think this precludes us from specifying custom artifacts names for multiple terraform envs using a Parallel Matrix.

I believe nowadays we change the artifact path based on the TF_ROOT.

Edited Oct 25, 2023 by João Alexandre Cunha

Remove unnecessary TF_CACHE_KEY

Merge request reports