fix(cli): run terraform init in serial so no text file is busy in the…

… cache

Closes #2741

packages/@cdktf/cli-core/src/lib/cdktf-project.ts

@@ -361,6 +361,7 @@ export class CdktfProject {
     const stack = this.getStackExecutor(
       getSingleStack(stacks, opts?.stackName, "diff")
     );
+    await stack.initalizeTerraform(opts.noColor);
 
     try {
       await stack.diff(opts);
@@ -389,6 +390,7 @@ export class CdktfProject {
       !opts.parallelism || opts.parallelism < 0 ? Infinity : opts.parallelism;
     const allExecutions = [];
 
+    await this.initializeStacksToRunInSerial();
     while (this.stacksToRun.filter((stack) => stack.isPending).length > 0) {
       const runningStacks = this.stacksToRun.filter((stack) => stack.isRunning);
       if (runningStacks.length >= maxParallelRuns) {
@@ -545,6 +547,7 @@ export class CdktfProject {
       this.getStackExecutor(stack, {})
     );
 
+    await this.initializeStacksToRunInSerial();
     const outputs = await Promise.all(
       this.stacksToRun.map(async (s) => {
         const output = await s.fetchOutputs();
@@ -559,4 +562,11 @@ export class CdktfProject {
       {}
     ) as NestedTerraformOutputs;
   }
+
+  // Serially run terraform init to prohibit text file busy errors for the cache files
+  private async initializeStacksToRunInSerial(): Promise<void> {
+    for (const stack of this.stacksToRun) {
+      await stack.initalizeTerraform();
+    }
+  }
 }

packages/@cdktf/cli-core/src/lib/cdktf-stack.ts

@@ -226,13 +226,16 @@ export class CdktfStack {
     };
   }
 
-  private async initalizeTerraform(noColor?: boolean) {
-    const terraform = await getTerraformClient(
+  private async terraformClient() {
+    return await getTerraformClient(
       this.options.abortSignal,
       this.options.stack,
       this.createTerraformLogHandler.bind(this)
     );
+  }
 
+  public async initalizeTerraform(noColor?: boolean) {
+    const terraform = await this.terraformClient();
     const needsInit = await this.checkNeedsInit();
     if (!needsInit) {
       // Skip terraform init as everything's up to date
@@ -353,7 +356,7 @@ export class CdktfStack {
   }) {
     await this.run(async () => {
       this.updateState({ type: "planning", stackName: this.stack.name });
-      const terraform = await this.initalizeTerraform(noColor);
+      const terraform = await this.terraformClient();
 
       await terraform.plan({
         destroy: false,
@@ -377,7 +380,7 @@ export class CdktfStack {
     const { refreshOnly, terraformParallelism, noColor, vars, varFiles } = opts;
     await this.run(async () => {
       this.updateState({ type: "planning", stackName: this.stack.name });
-      const terraform = await this.initalizeTerraform(noColor);
+      const terraform = await this.terraformClient();
 
       const { cancelled } = await terraform.deploy(
         {
@@ -463,7 +466,7 @@ export class CdktfStack {
     const { terraformParallelism, noColor, vars, varFiles } = opts;
     await this.run(async () => {
       this.updateState({ type: "planning", stackName: this.stack.name });
-      const terraform = await this.initalizeTerraform(noColor);
+      const terraform = await this.terraformClient();
       const { cancelled } = await terraform.destroy(
         {
           autoApprove: this.options.autoApprove,
@@ -531,7 +534,7 @@ export class CdktfStack {
 
   public async fetchOutputs() {
     await this.run(async () => {
-      const terraform = await this.initalizeTerraform();
+      const terraform = await this.terraformClient();
 
       const outputs = await terraform.output();
       const outputsByConstructId = getConstructIdsForOutputs(