Skip to content

fix(tf-checks): skip azure/login when AZURE_CREDENTIALS is not set#317

Merged
clouddrove-ci merged 1 commit intomasterfrom
fix/tf-checks-azure-login-guard
Feb 14, 2026
Merged

fix(tf-checks): skip azure/login when AZURE_CREDENTIALS is not set#317
clouddrove-ci merged 1 commit intomasterfrom
fix/tf-checks-azure-login-guard

Conversation

@anmolnagpal
Copy link
Collaborator

Why\nTerraform module repos that don't pass provider inherit the azurerm default and currently fail early at azure/login@v2 when AZURE_CREDENTIALS is not configured.\n\n## What\n- Add a credentials guard to both Azure login steps in .github/workflows/tf-checks.yml:\n - inputs.provider == 'azurerm' && secrets.AZURE_CREDENTIALS != ''\n\n## Impact\n- Prevents false CI failures in non-Azure module repos using this reusable workflow\n- Keeps Azure auth enabled when credentials are actually provided\n

@anmolnagpal anmolnagpal requested review from a team and clouddrove-ci as code owners February 14, 2026 09:02
@anmolnagpal
Copy link
Collaborator Author

Why

Terraform module repos that don't pass provider inherit the azurerm default and can fail early at azure/login@v2 when AZURE_CREDENTIALS is not configured.

What

  • Add a credentials guard to both Azure login steps in .github/workflows/tf-checks.yml:
  • inputs.provider == 'azurerm' && secrets.AZURE_CREDENTIALS != ''

Impact

  • Prevents false CI failures in non-Azure module repos using this reusable workflow.
  • Keeps Azure auth enabled when credentials are actually provided.

@clouddrove-ci clouddrove-ci merged commit 3343efe into master Feb 14, 2026
12 checks passed
@clouddrove-ci clouddrove-ci deleted the fix/tf-checks-azure-login-guard branch February 14, 2026 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants