Draft: Don't explicitly `cd` into `TF_ROOT`
What does this MR do and why?
This MR removes the explicit cd "${TF_ROOT}" from most of the jobs which don't require it anymore, because
of the -chdir "${TF_ROOT}" we do in the gitlab-terraform script.
There is a few things to consider here:
-
Baseneeds to be compatible withv0.43.0ofgitlab-terraform, which already supports-chdir - Before terraform-images!162 (merged)
plan.jsonwouldn't be stored withinTF_ROOTif there wasn't acd- once terraform-images!162 (merged) this MR can be merged, too - with (2) the
Base.latestdoesn't require any explicitcds at all - even with (2) the
Basestill requires acdfor thebuildjob so that it correctly stores theplan.jsonwithinTF_ROOT -
TF_PLAN_CACHEother thanTF_ROOT/plan.cacheandTF_PLAN_JSONother thanTF_ROOT/plan.jsonare NOT supported (see terraform-images!162 (comment 1205342947)) before this MR and after this MR. Let's handle it outside of this.
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.