Skip to content

Howie/sync3 - #48896

Open
Howie Leung (howieleung) wants to merge 47 commits into
mainfrom
howie/sync3
Open

Howie/sync3#48896
Howie Leung (howieleung) wants to merge 47 commits into
mainfrom
howie/sync3

Conversation

@howieleung

Copy link
Copy Markdown
Member

Description

Please add an informative description that covers that changes made by the pull request and link all relevant issues.

If an SDK is being regenerated based on a new API spec, a link to the pull request containing these API spec changes should be included above.

All SDK Contribution checklist:

  • The pull request does not introduce [breaking changes]
  • CHANGELOG is updated for new features, bug fixes or other significant changes.
  • I have read the contribution guidelines.

General Guidelines and Best Practices

  • Title of the pull request is clear and informative.
  • There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

  • Pull request includes test coverage for the included changes.

Darren Cohen (dargilco) and others added 30 commits July 1, 2026 15:27
…est, add new hosted agent samples for Teams message trigger and reminder preview (#48234)

* Remove sample_agent_toolbox_skill.py per bakcned folks and Linda request, add new hosted agent samples for Teams message trigger and reminder preview

* change log
Howie Leung (howieleung) and others added 17 commits July 28, 2026 15:24
* Refactor agent name retrieval to use a fallback mechanism

- Updated multiple sample scripts to change the way the agent name is retrieved from environment variables.
- Replaced the default value assignment using `os.environ.get("FOUNDRY_AGENT_NAME", "MyAgent")` with a more concise approach using `os.environ.get("FOUNDRY_AGENT_NAME") or "MyAgent"`.
- This change ensures that if the environment variable is not set, the fallback value "MyAgent" is still used, while improving code readability.
- The affected files include various agent tools and hosted agent samples across the project.

* rever dataset generation job polling and update assistant prompt
* Add A2A protocol support and update related models

- Introduced A2ATool and A2AToolboxTool classes for A2A protocol implementation.
- Added A2AProtocolVersion enum to define supported A2A protocol versions.
- Updated DataGenerationJobOptions to include SimulationSeedDataGenerationJobOptions and removed TaskGenerationDataGenerationJobOptions.
- Modified existing models and enums to accommodate new A2A features.
- Updated sample agent to utilize the new A2ATool class.
- Adjusted beta routines API to remove unsupported parameters and enhance pagination handling.
- Updated YAML configuration to reflect repository changes.

* Update tsp-location.yaml with latest commit and repository details

* Update sample_agent_to_agent.py skip reason in test_samples.py
#48795)

* Add BetaAgentInsightMonitorsOperations and related functionality

- Introduced `BetaAgentInsightMonitorsOperations` for managing agent insight monitors.
- Updated `BetaOperations` to include the new agent insight monitors operations.
- Removed outdated sample scripts for multi-agent workflows and MCP approval.
- Updated evaluation samples to reflect changes in Azure SDK and improve functionality.
- Adjusted role assignment parameters in evaluation samples for better clarity and structure.
- Updated `tsp-location.yaml` to include new directories for agent insights and voice agents.

* refactor: remove unnecessary RoleAssignmentProperties import in evaluation samples

* refactor: remove VoiceAgents from agent feature headers in tests and implementation

* refactor: remove VoiceAgents from non-beta optional test cases

* feat: add new features for agent insight monitors and Microsoft 365 publishing

* refactor: update RoleAssignment import and parameters format in evaluation samples

* refactor: update RoleAssignment parameters to use dictionary format in RBAC assignment

* feat: add multi-agent workflow samples with synchronous and asynchronous clients

* feat: update CHANGELOG and public methods documentation for AgentInsight monitors and Microsoft 365 publishing
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
10 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.


💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor
[Pilot] PR Pipeline Failure Analysis

What failed

Three failed Azure Pipeline builds were found:

  1. python - cosmos - ci (build 6787199) — two unrelated Cosmos test failures on Windows emulator jobs:

    • windows2022___EmulatorTestsPython310Standard: tests.test_shared_cache_integration_async.TestSharedCacheIntegrationAsync.test_clear_cache_triggers_repopulation_asyncAssertionError: 1 != 0 (cache not cleared as expected).
    • windows2022___EmulatorTestsPython313Standard: tests.test_timeout_and_failover_retry_policy.TestTimeoutRetryPolicy.test_cross_region_retry[502]AssertionError: assert '(127.0.0.1/redacted) == 'other-region' (retry didn't route to the fake failover region as expected).
  2. python - cosmos - native-test (build 6787212) — mirrors the same emulator jobs/checks as above (same underlying cosmos test suite).

  3. Build 6787203 — pure infrastructure breakage: git fetch failed with exit code: 128 (fatal: couldn't find remote ref refs/pull/48897/merge — wrong/stale PR ref, not 48896), plus cascading ENOENT/command not found errors (PIP_EXE, sdk_find_invalid_versions, dispatch_checks.py, Verify-Readme.ps1, Verify-ChangeLogs.ps1, TargetingString not recognized) — all downstream of the failed checkout, and a Publish Pipeline Artifacts failure (Path does not exist: /mnt/vss/_work/1/a/reports).

  4. Build 6787168 (azure-ai-projects whl check) — failed with error: Request failed after 3 retries in 6.0s while installing azure-sdk-tools wheel, caused by Caused by: stream error received: stream no longer needed fetching coverage-7.15.4...whl from the Azure DevOps package feed. The sdist check for the same package succeeded (792 passed, 105 skipped) — confirming this is a transient package-download/network issue, not a code problem.

Relevant pipeline output
git fetch failed with exit code: 128
fatal: couldn't find remote ref refs/pull/48897/merge

AssertionError: 1 != 0
tests\test_shared_cache_integration_async.py:118: AssertionError

AssertionError: assert '(127.0.0.1/redacted) == 'other-region'
tests\test_timeout_and_failover_retry_policy.py:188: AssertionError

error: Request failed after 3 retries in 6.0s
  Caused by: Failed to fetch: coverage-7.15.4-cp310-cp310-manylinux1...whl
  Caused by: client error (SendRequest)
  Caused by: http2 error
  Caused by: stream error received: stream no longer needed

Change under review

The PR (howie/sync3, head cee39c49) only renames sdk/ai/azure-ai-projects/tsp-location.yaml.savedtsp-location.yaml (1 file changed). None of the failing tests, infra checkout errors, or the package-feed timeout have any connection to this rename — the Cosmos test failures are in an unrelated package (azure-cosmos), the checkout failure references a different PR (refs/pull/48897/merge, not 48896), and the wheel install failure is a transient network stream error against the Azure DevOps package feed.

Recommended next steps

  • Retry the failed pipelines — build 6787203's checkout error referencing PR 48897 and the wheel-install network stream error in build 6787168 are both transient/infrastructure issues unrelated to this PR's diff.
  • For the Cosmos test failures (test_clear_cache_triggers_repopulation_async, test_cross_region_retry[502]), these look like pre-existing flaky/timing-sensitive assertions in the Cosmos SDK test suite (cache repopulation timing, mock retry routing) — unrelated to the tsp-location.yaml rename in this PR. Recommend re-running to confirm before further investigation; if they fail consistently across independent PRs, file a bug against azure-cosmos test suite maintainers.
  • See https://aka.ms/ci-fix

Copilot detected the failing pipeline and generated the analysis above. To have it attempt a fix automatically, reply with @copilot please fix the failing pipeline on this PR.

Generated by Pipeline Analysis Next Steps · auto · 61.2 AIC · ⌖ 2.67 AIC · ⊞ 9.6K ·

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.

3 participants