Skip to content

.Net: Revert RetryFacts for AzureOpenAI Integration Tests #12576

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

.Net: Revert RetryFacts for AzureOpenAI Integration Tests #12576

wants to merge 7 commits into from
+78 −25

Conversation

rogerbarreto
Copy link
Member

@rogerbarreto rogerbarreto commented Jun 24, 2025 •
edited
Loading

Motivation and Context

  • Added timeout to up to 100 seconds for potential failing long running tests
  • Removed recently added RetryFact due to a potential disruption in the current integration infrastructure when multiple integration tests are running in parallel from merge queue.

@rogerbarreto rogerbarreto requested a review from a team as a code owner June 24, 2025 16:43
@markwallace-microsoft markwallace-microsoft added .NET Issue or Pull requests regarding .NET code kernel Issues or pull requests impacting the core kernel labels Jun 24, 2025
@github-actions github-actions bot changed the title Revert RetryFacts as IT environment is overloading .Net: Revert RetryFacts as IT environment is overloading Jun 24, 2025
@rogerbarreto rogerbarreto added this pull request to the merge queue Jun 24, 2025
@rogerbarreto rogerbarreto removed this pull request from the merge queue due to a manual request Jun 24, 2025
@rogerbarreto rogerbarreto changed the title .Net: Revert RetryFacts as IT environment is overloading .Net: Revert RetryFacts for AzureOpenAI Integration Tests Jun 24, 2025
@rogerbarreto rogerbarreto added this pull request to the merge queue Jun 24, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jun 24, 2025
@rogerbarreto rogerbarreto added this pull request to the merge queue Jun 24, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jun 24, 2025
@rogerbarreto rogerbarreto added this pull request to the merge queue Jun 24, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jun 24, 2025
@rogerbarreto rogerbarreto added this pull request to the merge queue Jun 24, 2025
@markwallace-microsoft
Copy link
Member

One unrelated integration test failure

[xUnit.net 00:02:46.36]     SemanticKernel.IntegrationTests.Connectors.OpenAI.OpenAIChatCompletionNonStreamingTests.ChatCompletionWithAudioInputAndOutputAsync [FAIL]
[xUnit.net 00:02:46.36]       Microsoft.SemanticKernel.HttpOperationException : HTTP 500 (server_error: )
[xUnit.net 00:02:46.36]       
[xUnit.net 00:02:46.36]       The server had an error while processing your request. Sorry about that!
[xUnit.net 00:02:46.36]       ---- System.ClientModel.ClientResultException : HTTP 500 (server_error: )
[xUnit.net 00:02:46.36]       
[xUnit.net 00:02:46.36]       The server had an error while processing your request. Sorry about that!

@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jun 24, 2025
@markwallace-microsoft markwallace-microsoft added this pull request to the merge queue Jun 25, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jun 25, 2025
@rogerbarreto rogerbarreto added this pull request to the merge queue Jun 25, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jun 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shethaadit shethaadit shethaadit approved these changes

@markwallace-microsoft markwallace-microsoft markwallace-microsoft approved these changes

@westey-m westey-m westey-m approved these changes

Assignees
No one assigned
Labels
kernel Issues or pull requests impacting the core kernel .NET Issue or Pull requests regarding .NET code
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@rogerbarreto @markwallace-microsoft @shethaadit @westey-m