Install Microsoft.AITools.BinlogMcp from dotnet-public#1682
Merged
Conversation
![@5803ac22924bfa9e33fff9b0c6d45a3ced[api]](https://avatars.githubusercontent.com/u/174604979?s=40&v=4){kind=small}
The Microsoft.AITools.BinlogMcp package is now published on nuget.org as stable 1.0.0, so the binlog-mcp container no longer needs the dnceng AzDO dotnet-tools feed. - Dockerfile: pin BINLOG_MCP_VERSION to stable 1.0.0 and remove the DOTNET_TOOLS_FEED ARG / --add-source (nuget.org is the default source) - renovate.json: point the BinlogMcp registry to nuget.org and drop the now-obsolete pre-release allowance (stable versions are now published) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
mthalman
approved these changes
Jun 16, 2026
Member
|
FYI @dkurepa is trying to fix this in another PR |
Member
|
I didn't open a PR with this yet so it'll be good to see if it works here :D |
Member
|
FYI the build pipeline is currently not in the best shape because of Network Isolation, so even if this PR gets in it won't get built until we make it green. |
Member
Author
I took sources from dkurepa's PR to see if it resolves my problem :) |
YuliiaKovalova
force-pushed
the
binlogmcp-nuget-feed
branch
from
0d2d647 to
3bc5eed
Compare
mmitche
reviewed
Jun 17, 2026
Address review feedback: route the package through the dnceng dotnet-public AzDO feed instead of nuget.org directly, matching the existing pattern used for the python manager.
Per PR review feedback, the package feed must be explicitly set in the Dockerfile (matching the dotnet-public feed used in eng/renovate.json) rather than relying on the implicit nuget.org default source.
premun
changed the title
mthalman
approved these changes
Jun 18, 2026
Member
|
looks like this is failing on network isolation because it's still trying to reach nuget.org |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
![@5803ac22924bfa9e33fff9b0c6d45a3ced[api]](https://avatars.githubusercontent.com/u/174604979?s=52&v=4){kind=small}
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The
Microsoft.AITools.BinlogMcppackage is now published on nuget.org as stable1.0.0, so thebinlog-mcpcontainer image no longer needs the dnceng AzDOdotnet-toolsfeed.Changes
BINLOG_MCP_VERSIONto stable1.0.0and remove theDOTNET_TOOLS_FEEDARG and--add-source(nuget.org is the default source). The previously pinned1.0.0-preview.*version only exists on the AzDO feed.Microsoft.AITools.BinlogMcpregistryUrlTemplatetohttps://api.nuget.org/v3/index.jsonand drop the now-obsolete pre-release allowance packageRule (stable versions are now published).