Skip to content

fix(ci): build azure.ai.inspector from source in test workflow#8915

Merged
v1212 merged 2 commits into
mainfrom
fix/ci-inspector-from-source
Jul 6, 2026
Merged

fix(ci): build azure.ai.inspector from source in test workflow#8915
v1212 merged 2 commits into
mainfrom
fix/ci-inspector-from-source

Conversation

@v1212

@v1212 v1212 commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Problem

All runs of test-ext-azure-ai-agents.yml are failing since the 1.0.0-beta.1 version bump (#8896):

ERROR: failed to install extension: installed dependency azure.ai.inspector version 0.0.1-preview does not satisfy constraint "~1.0.0-beta.1"

Root cause: The workflow installs azure.ai.inspector from the registry, but only 0.0.1-preview has been published there. The source was bumped to 1.0.0-beta.1 but never released to the registry.

This blocks all PRs touching the agents extension (8 different branches affected today).

Fixes #8916

Fix

Build and install azure.ai.inspector from source using ci-build.ps1 + azd x pack/publish -- the same approach already used for azure.ai.agents itself.

Also adds cli/azd/extensions/azure.ai.inspector/** to the workflow path trigger so inspector source changes trigger the test.

Verification

  • Confirmed all 8 failed runs from Jun 30-Jul 1 show the identical error
  • The fix mirrors the existing pattern for building azure.ai.agents from source
  • CI test job passes on this PR

The azure.ai.inspector extension was bumped to 1.0.0-beta.1 in source
(#8896) but not yet published to the registry. The CI workflow was
installing it from the registry, getting the old 0.0.1-preview version
which doesn't satisfy azure.ai.agents' dependency constraint ~1.0.0-beta.1.

Fix by building and installing azure.ai.inspector from source (same
approach used for azure.ai.agents itself).

Also add cli/azd/extensions/azure.ai.inspector/** to the workflow path
trigger so inspector changes are tested.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings July 1, 2026 07:48

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.

Pull request overview

This PR fixes the test-ext-azure-ai-agents.yml CI workflow, which has been failing for all PRs touching the agents extension since the 1.0.0-beta.1 version bump (#8896). The agents extension depends on azure.ai.inspector ~1.0.0-beta.1, but only 0.0.1-preview of the inspector is published to the registry. The fix builds and installs the inspector from source (matching the existing approach used for azure.ai.agents itself) so the dependency constraint resolves locally.

Changes:

  • Replaced the registry-based azd extension install azure.ai.inspector with a dedicated step that builds the inspector from source via ci-build.ps1 and installs it through azd x pack/azd x publish from the local source.
  • Added cli/azd/extensions/azure.ai.inspector/** to the workflow's pull_request path trigger so inspector source changes also exercise this test.

@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown

📋 Prioritization Note

Thanks for the contribution! The linked issue isn't in the current milestone yet.
Thank you for logging this issue; our team is reviewing it. If you need urgent prioritization, tag @RickWinter and @kristenwomack to let us know.

@RickWinter RickWinter left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This unblocks the agents extension test workflow by building azure.ai.inspector from source instead of pulling it from the registry, where only 0.0.1-preview exists against the new ~1.0.0-beta.1 constraint. The approach is the right shape: it mirrors the existing from-source build/pack/publish/install pattern used for azure.ai.agents in the same job, and the inspector version.txt (1.0.0-beta.1) satisfies the dependency constraint, so the resolution error goes away.

Nothing here blocks merge. One thing worth deciding explicitly rather than by accident: pairing the new azure.ai.inspector/** path trigger with a from-source build means inspector-only PRs now build and test their own inspector code, which is the behavior we want. If we ever revert to installing the published inspector, we reintroduce exactly this failure on the next version bump and stop testing PR inspector changes, so I'd keep the from-source build as the intended long-term state and adjust the comment to say so.

The redundant -Version argument and the --force on install are both consistent with the agents step directly below, so leaving them as-is is correct for readability parity. No security or correctness concerns in the diff.

Comment thread .github/workflows/test-ext-azure-ai-agents.yml Outdated

@jongio jongio left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

+1 on @RickWinter's inline: the comment on line 49 should frame building from source as the intended approach (it tests PR inspector changes and avoids future version-bump breakage), not a stopgap until the registry catches up. Minor wording fix, doesn't block merge.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

@jongio jongio left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Comment wording updated per my earlier feedback. The framing now correctly presents building from source as the intended CI approach rather than a temporary workaround. No other changes.

@v1212
v1212 enabled auto-merge (squash) July 3, 2026 07:44
@v1212
v1212 requested review from huimiu and removed request for huimiu July 3, 2026 07:46
@JeffreyCA

Copy link
Copy Markdown
Contributor

/check-enforcer override

@v1212
v1212 merged commit 008d57d into main Jul 6, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI: test-ext-azure-ai-agents workflow broken — inspector dependency version mismatch

7 participants