Skip to content

Conversation

@tinaem
Copy link
Contributor

@tinaem tinaem commented Dec 5, 2025

Added instructions for creating an account-level capability host and updated the hosted agent version section to reflect changes in replica parameters.

Added instructions for creating an account-level capability host and updated the hosted agent version section to reflect changes in replica parameters.
@prmerger-automator
Copy link
Contributor

@tinaem : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 7ea9050:

✅ Validation status: passed

File Status Preview URL Details
articles/ai-foundry/default/agents/concepts/hosted-agents.md ✅Succeeded

For more details, please refer to the build report.

@v-dirichards v-dirichards requested a review from Copilot December 5, 2025 16:14
@v-dirichards
Copy link
Contributor

Hi @tinaem - This pull request was opened in the public repo. PMs should work in the private repo, per the Microsoft Docs contributor guide. We can keep this PR open for review and merge, but would you make future content updates in the private repo? Thank you!


@aahill - Can you review the proposed changes?

Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator bot added the aq-pr-triaged C+L Pull Request Review Team label label Dec 5, 2025
Copilot finished reviewing on behalf of v-dirichards December 5, 2025 16:17
Copy link
Contributor

Copilot AI left a comment

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 adds documentation for creating an account-level capability host and updates the parameter requirements for the az cognitiveservices agent start command. The changes clarify that min/max replicas are now optional parameters when starting an agent deployment.

  • Adds new section documenting capability host creation with public hosting environment enabled
  • Updates replica parameters from required to optional in the agent start command documentation


```bash
curl --request PUT \
--url 'https://management.azure.com/subscriptions/96aede12-2f73-41cb-b983-6d11a904839b/resourceGroups/arthfa-ncus/providers/Microsoft.CognitiveServices/accounts/arthfa-ncus/capabilityHosts/accountcaphost?api-version=2025-10-01-preview' \
Copy link

Copilot AI Dec 5, 2025

Choose a reason for hiding this comment

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

This cURL example contains hardcoded subscription ID, resource group name, and account name that appear to be specific to a test environment ("arthfa-ncus"). These should be replaced with placeholders or generic example values to prevent confusion and potential security issues.

Consider using placeholders like:

  • {subscription-id} or your-subscription-id
  • {resource-group-name} or your-resource-group
  • {account-name} or your-foundry-account
Suggested change
--url 'https://management.azure.com/subscriptions/96aede12-2f73-41cb-b983-6d11a904839b/resourceGroups/arthfa-ncus/providers/Microsoft.CognitiveServices/accounts/arthfa-ncus/capabilityHosts/accountcaphost?api-version=2025-10-01-preview' \
--url 'https://management.azure.com/subscriptions/{subscription-id}/resourceGroups/{resource-group-name}/providers/Microsoft.CognitiveServices/accounts/{account-name}/capabilityHosts/accountcaphost?api-version=2025-10-01-preview' \

Copilot uses AI. Check for mistakes.
1. Grant pull permissions by assigning the Container Registry Repository Reader role to your project's managed identity on the container registry. For detailed steps, see [Azure Container Registry roles and permissions](/azure/container-registry/container-registry-roles).

### Create account-level Capability Host
Copy link

Copilot AI Dec 5, 2025

Choose a reason for hiding this comment

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

The heading uses sentence case ("Capability Host") instead of the expected lowercase style for multi-word terms. According to Microsoft Writing Style Guide, headings should be in sentence case. This should be "Create account-level capability host" (with "capability host" in lowercase).

Suggested change
### Create account-level Capability Host
### Create account-level capability host

Copilot uses AI. Check for mistakes.
### Create account-level Capability Host

Updating capability hosts is not supported. If you have existing capability host for your Foundry account, you must delete the existing one and recreate it with the property "enablePublicHostingEnvironment" set to true.

Copy link

Copilot AI Dec 5, 2025

Choose a reason for hiding this comment

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

The cURL command lacks introductory text explaining what the command does or when to use it. Consider adding a brief introduction before the code block, such as "To create the account-level capability host with public hosting enabled, use the following Azure REST API call:" This follows the pattern used elsewhere in the document (see lines 237-240, 293).

Suggested change
To create the account-level capability host with public hosting enabled, use the following Azure REST API call:

Copilot uses AI. Check for mistakes.

### Create account-level Capability Host

Updating capability hosts is not supported. If you have existing capability host for your Foundry account, you must delete the existing one and recreate it with the property "enablePublicHostingEnvironment" set to true.
Copy link

Copilot AI Dec 5, 2025

Choose a reason for hiding this comment

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

Extra space detected between "you" and "must" - should be a single space instead of two spaces.

Suggested change
Updating capability hosts is not supported. If you have existing capability host for your Foundry account, you must delete the existing one and recreate it with the property "enablePublicHostingEnvironment" set to true.
Updating capability hosts is not supported. If you have existing capability host for your Foundry account, you must delete the existing one and recreate it with the property "enablePublicHostingEnvironment" set to true.

Copilot uses AI. Check for mistakes.

### Create account-level Capability Host

Updating capability hosts is not supported. If you have existing capability host for your Foundry account, you must delete the existing one and recreate it with the property "enablePublicHostingEnvironment" set to true.
Copy link

Copilot AI Dec 5, 2025

Choose a reason for hiding this comment

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

Missing article "an" before "existing capability host". The sentence should read "If you have an existing capability host for your Foundry account..."

Suggested change
Updating capability hosts is not supported. If you have existing capability host for your Foundry account, you must delete the existing one and recreate it with the property "enablePublicHostingEnvironment" set to true.
Updating capability hosts is not supported. If you have an existing capability host for your Foundry account, you must delete the existing one and recreate it with the property "enablePublicHostingEnvironment" set to true.

Copilot uses AI. Check for mistakes.
@georgeollis
Copy link

Could you clarify what the enablePublicHostingEnvironment parameter is intended to do? The API docs don’t make its behaviour very clear

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.

4 participants