Skip to content

Extend Azure CLI timeout for tests on arm32v7#7505

Merged
damonbarry merged 4 commits into
Azure:release/1.5from
damonbarry:azcli-timeout
May 21, 2026
Merged

Extend Azure CLI timeout for tests on arm32v7#7505
damonbarry merged 4 commits into
Azure:release/1.5from
damonbarry:azcli-timeout

Conversation

@damonbarry
Copy link
Copy Markdown
Member

@damonbarry damonbarry commented May 15, 2026

The end-to-end tests were intermittently (but frequently) failing on Debian 11 arm32v7, which is running on an old Raspberry Pi 3B agent (i.e., slow). We're only running the end-to-end tests on this platform in "minimal" mode, meaning we're just running one basic test--TempSensor--but it was still failing. The problem seems to be that getting a token from the Azure CLI (as with az account get-access-token) can be pretty slow on the old Pi, and can timeout.

I did two things to reduce the chances of failure:

  1. In the Microsoft.Azure.Devices.Edge.Test.Common.IotHub class, where the Azure CLI credential is required in four different places (three in the constructor alone), cache and use a single instance of the credential rather than doing new AzureCliCredential() each time. This translates to just a single az account get-access-token call.
  2. If we're requesting a new AzureCliCredential on an arm32v7 platform, then increase the timeout for the az account get-access-token call from the default 13 seconds to 60 seconds.

I ran the end-to-end tests a couple times against the change and they pass on all platforms.

Azure IoT Edge PR checklist:

This checklist is used to make sure that common guidelines for a pull request are followed.

General Guidelines and Best Practices

  • I have read the contribution guidelines.
  • Title of the pull request is clear and informative.
  • Description of the pull request includes a concise summary of the enhancement or bug fix.

Testing Guidelines

  • Pull request includes test coverage for the included changes.
  • Description of the pull request includes
    • concise summary of tests added/modified
    • local testing done.

@damonbarry damonbarry marked this pull request as ready for review May 18, 2026 17:45
@damonbarry damonbarry changed the base branch from main to release/1.5 May 18, 2026 18:48
@damonbarry damonbarry merged commit 5317cde into Azure:release/1.5 May 21, 2026
11 checks passed
@damonbarry damonbarry deleted the azcli-timeout branch May 21, 2026 22:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants