Skip to content

Revise Cosmos DB container requirements for Standard setup#729

Open
georgeollis wants to merge 1 commit into
MicrosoftDocs:mainfrom
georgeollis:patch-2
Open

Revise Cosmos DB container requirements for Standard setup#729
georgeollis wants to merge 1 commit into
MicrosoftDocs:mainfrom
georgeollis:patch-2

Conversation

@georgeollis
Copy link
Copy Markdown
Contributor

Updated Cosmos DB container requirements for Standard setup to include two additional containers and clarified the distinction between Classic and New Foundry Agent Service runtimes.

Updated Cosmos DB container requirements for Standard setup to include two additional containers and clarified the distinction between Classic and New Foundry Agent Service runtimes.
@prmerger-automator
Copy link
Copy Markdown
Contributor

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

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 0b9bf7a:

✅ Validation status: passed

File Status Preview URL Details
articles/foundry/agents/includes/concepts-standard-agent-setup-content.md ✅Succeeded

For more details, please refer to the build report.

@v-dirichards v-dirichards requested a review from Copilot April 21, 2026 20:37
@v-dirichards
Copy link
Copy Markdown
Contributor

@fosteramanda

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 Apr 21, 2026
Copy link
Copy Markdown
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

Updates the Standard agent setup documentation to reflect new Cosmos DB container requirements and to clarify differences between the Classic and New Foundry Agent Service runtimes.

Changes:

  • Updates the Cosmos DB container list for Standard setup to include agent-definitions-v1 and run-state-v1.
  • Adds text clarifying which containers are used by Classic vs New runtimes.
  • Adds an inline warning-style callout (currently implemented as a heading) about the runtime/container differences.
Comments suppressed due to low confidence (1)

articles/foundry/agents/includes/concepts-standard-agent-setup-content.md:72

  • The Project-level data isolation section below still states that “Three containers are provisioned in your Cosmos DB account…”, which contradicts the updated container requirements above and the Classic/New split. Please update that paragraph to reflect the correct container(s) for each runtime (and avoid reintroducing the outdated 3-container description).
The **Classic** and **New** Foundry Agent Service runtimes use **different Cosmos DB containers**.  

## Project-level data isolation

Standard setup enforces project-level data isolation by default. Two blob storage containers are automatically provisioned in your storage account: one for files and one for intermediate system data (chunks, embeddings). Three containers are provisioned in your Cosmos DB account: one for user threads, one for system messages, and one for agent configuration data such as instructions, tools, and names. This default behavior reduces setup complexity while still enforcing strict data boundaries between projects.

Comment on lines +49 to 52
The **Standard setup** provisions **five containers**, each requiring **1000 RU/s**:

| Container | Purpose |
|-----------|---------|
Copy link

Copilot AI Apr 21, 2026

Choose a reason for hiding this comment

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

The statement that the Standard setup provisions five containers is hard to reconcile with the later Classic/New split (New runtime uses only agent-definitions-v1 and run-state-v1). Please clarify whether users must provision all five containers in all cases, or whether the required containers (and RU/s totals) depend on which runtime they use.

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The service automatically provisions these containers for you

Copy link
Copy Markdown
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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 6, 2026

This pull request has been inactive for at least 14 days. If you are finished with your changes, don't forget to sign off. See the contributor guide for instructions.
Get Help
Docs Support Teams Channel
Resolve Merge Conflict

@github-actions github-actions Bot added the inactive This PR is inactive for more than 14 days label May 6, 2026
@tkudo-ms
Copy link
Copy Markdown
Contributor

Hi team,

We have an active support case where a customer is confused by the Cosmos DB container naming discrepancy between documentation and their actual environment.

The existing page "Use your own resources" already documents the newer container names (agent-definitions-v1, run-state-v1, agent-entity-store):
https://learn.microsoft.com/en-us/azure/foundry/agents/how-to/use-your-own-resources#use-an-existing-azure-cosmos-db-for-nosql-account-for-conversation-storage

However, the "Standard agent setup" concepts page (which this PR targets) still only references the classic containers (thread-message-store, system-thread-message-store). This inconsistency between the two pages is causing customer confusion — they expect to see containers listed in the concepts page but find different ones in their Cosmos DB account.

Could we prioritize getting this PR reviewed and merged? It would resolve the documentation gap and help us close the support case with a clear public reference.

Additionally, the Cosmos DB-side page (https://learn.microsoft.com/en-us/azure/cosmos-db/gen-ai/azure-agent-service) also still references only the classic containers. I've filed a separate issue for that: MicrosoftDocs/nosql-docs#19

Thank you!

@v-regandowner
Copy link
Copy Markdown
Contributor

@sdgilley - are you able to take a look at this urgent review request?

@github-actions github-actions Bot removed the inactive This PR is inactive for more than 14 days label May 27, 2026
@prmerger-automator
Copy link
Copy Markdown
Contributor

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

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.

7 participants