Skip to content

fix: upgrade azure/login to v2.2.0 to prevent cleanup warnings#2584

Open
AyhanSetirekli wants to merge 1 commit intodiggerhq:developfrom
AyhanSetirekli:fix/disable-azure-cleanup-when-not-used
Open

fix: upgrade azure/login to v2.2.0 to prevent cleanup warnings#2584
AyhanSetirekli wants to merge 1 commit intodiggerhq:developfrom
AyhanSetirekli:fix/disable-azure-cleanup-when-not-used

Conversation

@AyhanSetirekli
Copy link

@AyhanSetirekli AyhanSetirekli commented Feb 27, 2026

Summary

Fixes Azure CLI cleanup warning when setup-azure is false.

Problem

Current version causes a warning:
image

Root Cause & Solution

Digger uses azure/login@v2.1.1 which lacks support for AZURE_LOGIN_POST_CLEANUP env var.

This PR:

  1. Upgrades to azure/login@v2.2.0 (adds post-if support via move pre cleanup to main and add pre-if and post-if Azure/login#484)
  2. Sets AZURE_LOGIN_POST_CLEANUP: false when Azure is not being used

Testing

Before/after comparison: https://github.com/AyhanSetirekli/digger-azure-cleanup-test/actions/runs/22491196175

  • v2.1.1: Warning annotation appears
  • v2.2.0: No warning ✓

Impact

Eliminates warnings for non-Azure workflows. No change when setup-azure: true.

Fixes #2540


AI Assistance: This PR was written primarily by Claude Code.

Upgrades azure/login from v2.1.1 to v2.2.0 which includes support for
AZURE_LOGIN_POST_CLEANUP environment variable via post-if condition.

The env var is set to false when setup-azure is not true, preventing
the cleanup step from running and eliminating the warning about missing
az CLI when Azure is not being used.

Fixes diggerhq#2540
@AyhanSetirekli AyhanSetirekli force-pushed the fix/disable-azure-cleanup-when-not-used branch from 6ee4e4e to 2044d82 Compare February 27, 2026 14:23
@AyhanSetirekli AyhanSetirekli changed the title fix: disable Azure login cleanup when Azure is not used fix: upgrade azure/login to v2.2.0 to prevent cleanup warnings Feb 27, 2026
@AyhanSetirekli AyhanSetirekli marked this pull request as ready for review February 27, 2026 14:28
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.

Login pre-cleanup warns about az on custom runners

1 participant