Skip to content

Record tests and disable Azure functions #40852

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
Merged
nick863 merged 1 commit into from
May 1, 2025
Merged

Record tests and disable Azure functions #40852

nick863 merged 1 commit into from
May 1, 2025

Conversation

@nick863
Copy link
Member

@nick863 nick863 commented May 1, 2025

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 swagger spec, a link to the pull request containing these swagger spec changes has been 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.

@github-actions github-actions bot added the AI label May 1, 2025
@nick863 nick863 merged commit 8c40265 into jhakulin/agents-naming-changes May 1, 2025
15 of 16 checks passed
@nick863 nick863 deleted the nirovins/record_tests branch May 1, 2025 22:48
jhakulin pushed a commit that referenced this pull request May 2, 2025
@nick863
Jhakulin/agents naming changes (#40845)
c63892d 
* naming changes

* update tests

* Record the tests and disable azue functions for now (#40852)

* Nirovins/record tests (#40854)

* Record the tests and disable azue functions for now

* Add recordings for azure functions

* Record instrumentation tests

---------

Co-authored-by: Jarno Hakulinen <[email protected]>

---------

Co-authored-by: Nikolay Rovinskiy <[email protected]>
nick863 added a commit that referenced this pull request May 2, 2025
@nick863
Uncomment function tests (#40855)
1d76e38 
* naming changes

* update names

* update

* update tests

* update

* Record the tests and disable azue functions for now (#40852)

* Nirovins/record tests (#40854)

* Record the tests and disable azue functions for now

* Add recordings for azure functions

* Record instrumentation tests

---------

Co-authored-by: Jarno Hakulinen <[email protected]>

* Uncomment function tests

* Fix

---------

Co-authored-by: jhakulin <[email protected]>
howieleung added a commit that referenced this pull request May 8, 2025
@howieleung @M-Hietala
@nick863 @glharper @scbedd
Feature/azure ai agents v1 (#40818)
b64185c 
* init agents v1 with new operations

* update

* add samples

* aio

* update

* update

* updates

* update

* update

* update

* update

* update

* add create_thread_and_run

* create thread and process run

* add tests and update README (#40760)

* update README

* update tests

* fix sample

* adding tracing (#40769)

* Fix unit tests (#40783)

* Fix unit tests

* Record instrumentor tests

* modifying multiagent sample (#40799)

* Jhakulin/azure sdk review fixes (#40795)

Fix static analysis, updates from azure sdk reviews.

---------

Co-authored-by: nick863 <[email protected]>

* run black, fix some readme update snip issue (#40815)

* Howie/ffix toolcall (#40750)

* max retry and tests

* fix test

* Update samples csv data (#40821)

* update samples csv data

* add cspell

* Add ew artifact to list (#40833)

* [AI] [Agents] tool schema updates (#40841)

* [AI] [Agents] tool changes for Bing tools and Azure AI Search

* add implementation for abstract tool methods

* Remove packages not handled in the branch from the CI (#40842)

* Jhakulin/agents naming changes (#40845)

* naming changes

* update tests

* Record the tests and disable azue functions for now (#40852)

* Nirovins/record tests (#40854)

* Record the tests and disable azue functions for now

* Add recordings for azure functions

* Record instrumentation tests

---------

Co-authored-by: Jarno Hakulinen <[email protected]>

---------

Co-authored-by: Nikolay Rovinskiy <[email protected]>

* bring tests README back (#40858)

* Uncomment function tests (#40855)

* naming changes

* update names

* update

* update tests

* update

* Record the tests and disable azue functions for now (#40852)

* Nirovins/record tests (#40854)

* Record the tests and disable azue functions for now

* Add recordings for azure functions

* Record instrumentation tests

---------

Co-authored-by: Jarno Hakulinen <[email protected]>

* Uncomment function tests

* Fix

---------

Co-authored-by: jhakulin <[email protected]>

* Jhakulin/list op update (#40871)

* update list operations

* update tests

* update README

* update

* Fix the instrumentation tests (#40877)

* Fix the instrumentation tests

* Better instrumentation handling

* pylint

---------

Co-authored-by: Nikolay Rovinskiy <[email protected]>

* Howie/poll timeout (#40825)

* add timeout for polling

* fix merge

* fix merge

* fix merge

* fixed merge

* enable_auto_function_calls param changes (#40820)

* get CI green (#40884)

* fix logic app sample (#40883)

* agent azure monitor tracing sample updates (#40865)

* agent azure monitor tracing sample updates

* disable pylint for global statement

* agent tracing runid to feature branch (#40888)

* omit azure-ai-agents from pypy compatibility map

* [AI] [Agents] write initial CHANGELOG and add missing samples (#40891)

* [AI] [Agents] write initial CHANGELOG

* add sample for multiple connected agents

* add sample for connected agents

* update snippets

* try get green pipeline (#40914)

* resolved comments (#40922)

* update code owner (#40933)

* Howie/toolcall fix (#40939)

* update code owner

* fix tool call for async to be same as sync

* fix test

* Resolved Johan comments (#40943)

* resolved comments

* resolved comments

* Resolved comments

* change project label

* Hide the hack to use connection string. (#40945)

* FIx for function name changes (#40951)

* Remove Key Authentication (#40954)

* Remove Key Authentication

* Exclude samples

* Fix linter

* Fixed logger (#40957)

* clean up py (#40959)

---------

Co-authored-by: jhakulin <[email protected]>
Co-authored-by: M-Hietala <[email protected]>
Co-authored-by: Nikolay Rovinskiy <[email protected]>
Co-authored-by: Glenn Harper <[email protected]>
Co-authored-by: Scott Beddall <[email protected]>
M-Hietala added a commit that referenced this pull request May 12, 2025
@M-Hietala @nick863
@howieleung @glharper @scbedd
M hietala/merge remove enable telemetry function (#41009)
f32244d 
* init agents v1 with new operations

* update

* add samples

* aio

* update

* update

* updates

* update

* update

* update

* update

* update

* add create_thread_and_run

* create thread and process run

* add tests and update README (#40760)

* update README

* update tests

* fix sample

* adding tracing (#40769)

* Fix unit tests (#40783)

* Fix unit tests

* Record instrumentor tests

* modifying multiagent sample (#40799)

* Jhakulin/azure sdk review fixes (#40795)

Fix static analysis, updates from azure sdk reviews.

---------

Co-authored-by: nick863 <[email protected]>

* run black, fix some readme update snip issue (#40815)

* Howie/ffix toolcall (#40750)

* max retry and tests

* fix test

* Update samples csv data (#40821)

* update samples csv data

* add cspell

* Add ew artifact to list (#40833)

* [AI] [Agents] tool schema updates (#40841)

* [AI] [Agents] tool changes for Bing tools and Azure AI Search

* add implementation for abstract tool methods

* Remove packages not handled in the branch from the CI (#40842)

* Jhakulin/agents naming changes (#40845)

* naming changes

* update tests

* Record the tests and disable azue functions for now (#40852)

* Nirovins/record tests (#40854)

* Record the tests and disable azue functions for now

* Add recordings for azure functions

* Record instrumentation tests

---------

Co-authored-by: Jarno Hakulinen <[email protected]>

---------

Co-authored-by: Nikolay Rovinskiy <[email protected]>

* bring tests README back (#40858)

* Uncomment function tests (#40855)

* naming changes

* update names

* update

* update tests

* update

* Record the tests and disable azue functions for now (#40852)

* Nirovins/record tests (#40854)

* Record the tests and disable azue functions for now

* Add recordings for azure functions

* Record instrumentation tests

---------

Co-authored-by: Jarno Hakulinen <[email protected]>

* Uncomment function tests

* Fix

---------

Co-authored-by: jhakulin <[email protected]>

* Jhakulin/list op update (#40871)

* update list operations

* update tests

* update README

* update

* Fix the instrumentation tests (#40877)

* Fix the instrumentation tests

* Better instrumentation handling

* pylint

---------

Co-authored-by: Nikolay Rovinskiy <[email protected]>

* Howie/poll timeout (#40825)

* add timeout for polling

* fix merge

* fix merge

* fix merge

* fixed merge

* enable_auto_function_calls param changes (#40820)

* get CI green (#40884)

* fix logic app sample (#40883)

* agent azure monitor tracing sample updates (#40865)

* agent azure monitor tracing sample updates

* disable pylint for global statement

* agent tracing runid to feature branch (#40888)

* omit azure-ai-agents from pypy compatibility map

* [AI] [Agents] write initial CHANGELOG and add missing samples (#40891)

* [AI] [Agents] write initial CHANGELOG

* add sample for multiple connected agents

* add sample for connected agents

* update snippets

* try get green pipeline (#40914)

* resolved comments (#40922)

* update code owner (#40933)

* Howie/toolcall fix (#40939)

* update code owner

* fix tool call for async to be same as sync

* fix test

* Resolved Johan comments (#40943)

* resolved comments

* resolved comments

* Resolved comments

* change project label

* Hide the hack to use connection string. (#40945)

* FIx for function name changes (#40951)

* Remove Key Authentication (#40954)

* Remove Key Authentication

* Exclude samples

* Fix linter

* Fixed logger (#40957)

* clean up py (#40959)

* removing enable_trace function from agents telemetry

* removing an import of the removed function

* redoing change that was incorrectly merged

* removing one remaining enable_telemetry import

* adding instrumentor call to azure monitor tracing samples

* fixing multiagent sample trace configurator

* updating tracing snippet in readme

---------

Co-authored-by: jhakulin <[email protected]>
Co-authored-by: Nikolay Rovinskiy <[email protected]>
Co-authored-by: Howie Leung <[email protected]>
Co-authored-by: Glenn Harper <[email protected]>
Co-authored-by: Scott Beddall <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@luigiw luigiw Awaiting requested review from luigiw

@needuv needuv Awaiting requested review from needuv

@paulshealy1 paulshealy1 Awaiting requested review from paulshealy1

@singankit singankit Awaiting requested review from singankit

@dargilco dargilco Awaiting requested review from dargilco

1 more reviewer

@jhakulin jhakulin jhakulin approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

AI

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@nick863 @jhakulin