From 1dd677a50cedab1beb11d3ea2d0b964898c1e12c Mon Sep 17 00:00:00 2001 From: Clint Branham Date: Mon, 23 Mar 2026 16:18:32 -0500 Subject: [PATCH] fix(ci): stop masking nested PVE IPs so job outputs pass through ::add-mask:: on the VM IP addresses caused GitHub Actions to redact them in job outputs, making needs.provision.outputs.pve*_host empty in the test jobs. The IPs are internal lab addresses, not sensitive. Token masking is retained. Also removes terraform debug output from prior commit. Co-Authored-By: Claude Opus 4.6 (1M context) --- .github/workflows/integration-tests.yml | 7 ------- 1 file changed, 7 deletions(-) diff --git a/.github/workflows/integration-tests.yml b/.github/workflows/integration-tests.yml index 4d0ed90..a06a5ec 100644 --- a/.github/workflows/integration-tests.yml +++ b/.github/workflows/integration-tests.yml @@ -227,11 +227,6 @@ jobs: } } TFVARS - echo "=== tfvars ===" - cat /tmp/instances.tfvars.json - echo "=== RUNNER_TEMP=${RUNNER_TEMP} ===" - ls -la "${RUNNER_TEMP}/"*.iso 2>/dev/null || echo "No ISOs in RUNNER_TEMP" - ls -la "${CACHE_DIR}/"*.iso 2>/dev/null || echo "No ISOs in CACHE_DIR" terraform apply -auto-approve -input=false -var-file=/tmp/instances.tfvars.json # ── Wait for both PVE instances and create API tokens ──────────── @@ -247,7 +242,6 @@ jobs: 99909 "${PVE_PASSWORD}" 900) VM_IP=$(echo "$OUTPUT" | grep "^IP=" | cut -d= -f2) VM_TOKEN=$(echo "$OUTPUT" | grep "^TOKEN=" | cut -d= -f2-) - echo "::add-mask::${VM_IP}" echo "::add-mask::${VM_TOKEN}" echo "host=${VM_IP}" >> "$GITHUB_OUTPUT" echo "token=${VM_TOKEN}" >> "$GITHUB_OUTPUT" @@ -263,7 +257,6 @@ jobs: 99908 "${PVE_PASSWORD}" 900) VM_IP=$(echo "$OUTPUT" | grep "^IP=" | cut -d= -f2) VM_TOKEN=$(echo "$OUTPUT" | grep "^TOKEN=" | cut -d= -f2-) - echo "::add-mask::${VM_IP}" echo "::add-mask::${VM_TOKEN}" echo "host=${VM_IP}" >> "$GITHUB_OUTPUT" echo "token=${VM_TOKEN}" >> "$GITHUB_OUTPUT"