@firestartr/cli 2.1.0-snapshot-1 → 2.1.0

package/build/index.js

@@ -356187,39 +356187,33 @@ function createCRrefFrom(claimRef, needsSecret) {
  * @returns string
  */
 function createCodeOwnersData(claim, additionalRules) {
-    const paddedAsterisk = '*'.padEnd(25);
-    // we avoid generating the same line twice (idempotency)
-    const redactedLines = [];
-    const concatLine = function (messageFinal, line) {
-        if (redactedLines.includes(line)) {
-            return messageFinal;
+    const rulesMap = new Map();
+    const addOwnerToPath = (path, owner) => {
+        const owners = rulesMap.get(path) ?? [];
+        if (!owners.includes(owner)) {
+            owners.push(owner);
         }
-        else {
-            redactedLines.push(line);
-            messageFinal += line;
-        }
-        return messageFinal;
+        rulesMap.set(path, owners);
     };
     let message = `# This file was generated by firestartr.
 # WARNING: Please don't edit this file directly in the repository
 # Go to gitops repository to modify it!`;
     if (claim.owner) {
-        message = concatLine(message, `\n${paddedAsterisk}${resolveCodeownersRef(claim.owner, claim.providers.github.org)}`);
+        addOwnerToPath('*', resolveCodeownersRef(claim.owner, claim.providers.github.org));
     }
     if (claim.platformOwner) {
-        message = concatLine(message, `\n${'/.github/'.padEnd(25)}${resolveCodeownersRef(claim.platformOwner, claim.providers.github.org)}`);
+        addOwnerToPath('/.github/', resolveCodeownersRef(claim.platformOwner, claim.providers.github.org));
     }
     if (additionalRules) {
         for (const rule of additionalRules) {
-            let line = `\n${rule.path.padEnd(25)}`;
             for (const owner of rule.owners) {
-                line += `${resolveCodeownersRef(owner, claim.providers.github.org)} `;
+                addOwnerToPath(rule.path, resolveCodeownersRef(owner, claim.providers.github.org));
             }
-            message = concatLine(message, line);
-            // Remove trailing <space> from the previous loop
-            message = message.substring(0, message.length - 1);
         }
     }
+    for (const [path, owners] of rulesMap.entries()) {
+        message += `\n${path.padEnd(25)} ${owners.join(' ')}`;
+    }
     return message;
 }
 function resolveCodeownersRef(ref, org) {
@@ -371115,6 +371109,7 @@ function processHandler(processToHandle, ctl, onTimedOut) {
 
 
 
+const TOFU_LOCK_TIMEOUT = '-lock-timeout=60s';
 async function utils_validate(path, secrets) {
     return await tfExec(path, ['validate'], secrets);
 }
@@ -371135,7 +371130,7 @@ async function plan(path, secrets, format, args = ['plan'], stream, ctl) {
 }
 async function apply(path, secrets, stream, ctl) {
     terraform_provisioner_src_logger.debug(`Running terraform apply in path ${path}`);
-    return await tfExec(path, ['apply', '-auto-approve'], secrets, ['-input=false'], stream, ctl);
+    return await tfExec(path, ['apply', '-auto-approve', TOFU_LOCK_TIMEOUT], secrets, ['-input=false'], stream, ctl);
 }
 async function customCommand(path, secrets, args, stream) {
     terraform_provisioner_src_logger.debug(`Running terraform customCommand in path ${path} ${args.join(',')}`);
@@ -371143,7 +371138,7 @@ async function customCommand(path, secrets, args, stream) {
 }
 async function destroy(path, secrets, stream, ctl) {
     terraform_provisioner_src_logger.debug(`Running terraform destroy in path ${path}`);
-    return await tfExec(path, ['destroy', '-auto-approve'], secrets, ['-input=false'], stream, ctl);
+    return await tfExec(path, ['destroy', '-auto-approve', TOFU_LOCK_TIMEOUT], secrets, ['-input=false'], stream, ctl);
 }
 async function output(path, secrets) {
     terraform_provisioner_src_logger.debug(`Running terraform output in path ${path}`);
@@ -375297,47 +375292,23 @@ async function endDebug(entity) {
 
 
 const tp_bridge_TF_PROJECTS_PATH = '/tmp/gh-workspaces';
-const STATE_LOCK_RETRY_ATTEMPTS = 3;
-const STATE_LOCK_RETRY_DELAY_MS = 5000;
-function isStateLockError(error) {
-    const message = String(error);
-    return (message.includes('Error acquiring the state lock') ||
-        message.includes('already locked by another tofu client'));
-}
-async function tp_bridge_sleep(ms) {
-    await new Promise((resolve) => setTimeout(resolve, ms));
-}
-async function runTerraformProvisionerWithLockRetry(entity, command, streaming, customArgs, ctl) {
-    for (let attempt = 1;; attempt++) {
-        try {
-            return await runTerraformProvisioner(tp_bridge_buildContext(entity), command, streaming, customArgs, ctl);
-        }
-        catch (error) {
-            if (!isStateLockError(error) || attempt >= STATE_LOCK_RETRY_ATTEMPTS) {
-                throw error;
-            }
-            gh_provisioner_src_logger.warn(`State lock detected for ${entity.k8sId} while running '${command}'. Retrying in ${STATE_LOCK_RETRY_DELAY_MS}ms (attempt ${attempt + 1}/${STATE_LOCK_RETRY_ATTEMPTS}).`);
-            await tp_bridge_sleep(STATE_LOCK_RETRY_DELAY_MS);
-        }
-    }
-}
 async function runOnTerraform(entity, command, customArgs, opts = {}) {
     gh_provisioner_src_logger.info(`Running on terraform entity '${entity.k8sId}' with command '${command}' customArgs = ${customArgs ? customArgs.join(',') : 'null'} `);
     const streaming = entity.streamTFProvisioner;
     if (command === 'import-with-reimport') {
         // we nuke the tf state
-        await runTerraformProvisionerWithLockRetry(entity, 'destroy-state-only', streaming, customArgs, opts.ctl);
+        await runTerraformProvisioner(tp_bridge_buildContext(entity), 'destroy-state-only', streaming, customArgs, opts.ctl);
         // we get the elements we need to import
         await entity.loadAddressesToImport();
-        return await runTerraformProvisionerWithLockRetry(entity, 'custom-import', streaming, entity.importDocument, opts.ctl);
+        return await runTerraformProvisioner(tp_bridge_buildContext(entity), 'custom-import', streaming, entity.importDocument, opts.ctl);
     }
     else if (command === 'import') {
         // we get the elements we need to import
         await entity.loadAddressesToImport();
-        return await runTerraformProvisionerWithLockRetry(entity, 'custom-import', streaming, entity.importDocument, opts.ctl);
+        return await runTerraformProvisioner(tp_bridge_buildContext(entity), 'custom-import', streaming, entity.importDocument, opts.ctl);
     }
     else {
-        return await runTerraformProvisionerWithLockRetry(entity, command, streaming, customArgs, opts.ctl);
+        return await runTerraformProvisioner(tp_bridge_buildContext(entity), command, streaming, customArgs, opts.ctl);
     }
 }
 function tp_bridge_buildContext(entity) {
@@ -379486,9 +379457,9 @@ const crs_analyzerSubcommand = {
 };
 
 ;// CONCATENATED MODULE: ./package.json
-const package_namespaceObject = JSON.parse('{"i8":"2.1.0-snapshot-1"}');
+const package_namespaceObject = {"i8":"2.1.0"};
 ;// CONCATENATED MODULE: ../../package.json
-const package_namespaceObject_1 = {"i8":"2.0.0"};
+const package_namespaceObject_1 = {"i8":"2.1.0"};
 ;// CONCATENATED MODULE: ./src/subcommands/index.ts
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@firestartr/cli",
-  "version": "2.1.0-snapshot-1",
+  "version": "2.1.0",
   "private": false,
   "description": "Commandline tool",
   "main": "build/main.js",