Skip to content

Conversation

@pvaneck
Copy link
Member

@pvaneck pvaneck commented Nov 22, 2024

Some managed identity integration tests occasionally fail with gateway timeouts. This ensures that the requests to the app endpoints are retried in these cases by adding the RetryPolicy to the PipelineClient.

Also updated the az command to specify additional fields when creating a container instance in the post deploy script.

@pvaneck
Copy link
Member Author

pvaneck commented Nov 22, 2024

/azp run python - identity - tests-weekly

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@azure-sdk
Copy link
Collaborator

API change check

API changes are not detected in this pull request.

Some managed identity integration tests occasionally fail with gateway
timeouts. This ensures that the requests to the app endpoints are retried in
these cases by adding the RetryPolicy to the PipelineClient.

Signed-off-by: Paul Van Eck <paulvaneck@microsoft.com>
@pvaneck pvaneck force-pushed the identity-mi-tests-update branch from 1412f5f to db38657 Compare November 25, 2024 20:20
@pvaneck
Copy link
Member Author

pvaneck commented Nov 25, 2024

/azp run python - identity - tests-weekly

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@pvaneck pvaneck marked this pull request as ready for review November 25, 2024 21:21
@pvaneck pvaneck requested review from a team and xiangyan99 as code owners November 25, 2024 21:21
@pvaneck pvaneck merged commit 3750f4d into Azure:main Nov 25, 2024
50 checks passed
@pvaneck pvaneck deleted the identity-mi-tests-update branch November 25, 2024 22:15
l0lawrence pushed a commit to l0lawrence/azure-sdk-for-python that referenced this pull request Feb 19, 2025
Some managed identity integration tests occasionally fail with gateway
timeouts. This ensures that the requests to the app endpoints are retried in
these cases by adding the RetryPolicy to the PipelineClient.

Signed-off-by: Paul Van Eck <paulvaneck@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

3 participants