Repository navigation
fix(foundry): prepare toolbox access before deployment - #10736
Closed
Travis Angevine (trangevi) wants to merge 1 commit into
Closed
Travis Angevine (trangevi) wants to merge 1 commit into
Travis Angevine (trangevi) wants to merge 1 commit into
Conversation
Share Foundry developer access preparation between agents and toolboxes. Probe effective access before conditional project-scoped grants, wait for propagation, and reuse environment, tenant, and credential helpers without granting roles during provisioning or read/delete operations. Fixes: #10718 Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
|
Azure Pipelines: Successfully started running 4 pipeline(s). 23 pipeline(s) were filtered out due to trigger conditions. There may be pipelines that require an authorized user to comment /azp run to run. |
This was referenced Oct 27, 2026
Member
Author
|
Superseded by the requested three-PR draft stack: #10738 (core Foundry helpers), #10739 (targeted toolbox deploy/create fix for #10718), and #10740 (broader write coverage and agent consolidation). Native stack #10741 preserves that bottom-to-top order. The replacement layers include injected access-propagation feedback and retain the probe-first, endpoint-bound access policy. Core release and real extension SDK dependency bumps remain required before publishing consumers. Original branch retained for reference. |
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
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.
Why
Toolbox deployment can run before an agent's predeploy hook grants the developer Foundry data-plane access. Subscription Owner or Contributor alone is not sufficient, and standalone toolbox writes never reach that hook.
Fixes: #10718
Approach
pkg/foundry, used by both agents and toolboxes. Keep only service-specific probes and output/error mapping in the extensions.Release sequencing
This draft includes both the core helpers and extension consumers for review. The new core SDK APIs must land and be released first, then the independently published extensions must update their SDK dependencies. Their current v1.37.0 pins do not include these APIs; cross-module validation used an external Go workspace, with no repository-local replacements.
Validation
2.00-setup-deploy-shared-agent.yamlscenario. No live Azure scenario was run; live verification requires thefoundry-extension-scenario-orchestratorand incurs Azure cost.