-
Notifications
You must be signed in to change notification settings - Fork 26
Update Azure OpenAI integration docs with new model deployment names #293
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
… and GitLab documentation
…, GitHub, and GitLab documentation
… and GitLab documentation
…nd GitLab to clarify optional deployment names
WalkthroughThe documentation for self-hosted integration with Azure OpenAI deployments across Azure DevOps, Bitbucket, GitHub, and GitLab was updated. The recommended deployment names and corresponding environment variable names in the Changes
Poem
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (4)
docs/self-hosted/bitbucket.md (1)
59-59
: Nitpick: Clarify recommendation header
The Bash comment at line 59 can be more concise. Consider:- ## it is recommended to deploy gpt-4.1-mini, o4-mini, o3 deployments, gpt-4.1 (optionally). + # Recommended Azure OpenAI deployments: gpt-4.1-mini, o4-mini, and o3 (with optional gpt-4.1).docs/self-hosted/azure-devops.md (1)
67-67
: Nitpick: Clarify recommendation header
The comment prefix can be simplified to a single#
for consistency:- ## it is recommended to deploy gpt-4.1-mini, o4-mini, o3 deployments, gpt-4.1 (optionally). + # Recommended Azure OpenAI deployments: gpt-4.1-mini, o4-mini, and o3 (with optional gpt-4.1).docs/self-hosted/github.md (1)
70-70
: Nitpick: Clarify recommendation header
For readability and consistent comment styling, consider:- ## it is recommended to deploy gpt-4.1-mini, o4-mini, o3 deployments, gpt-4.1 (optionally). + # Recommended Azure OpenAI deployments: gpt-4.1-mini, o4-mini, and o3 (with optional gpt-4.1).docs/self-hosted/gitlab.md (1)
65-65
: Nitpick: Clarify recommendation header
Standardize comment syntax with a single hash:- ## it is recommended to deploy gpt-4.1-mini, o4-mini, o3 deployments, gpt-4.1 (optionally). + # Recommended Azure OpenAI deployments: gpt-4.1-mini, o4-mini, and o3 (with optional gpt-4.1).
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (4)
docs/self-hosted/azure-devops.md
(1 hunks)docs/self-hosted/bitbucket.md
(1 hunks)docs/self-hosted/github.md
(1 hunks)docs/self-hosted/gitlab.md
(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: Cloudflare Pages
🔇 Additional comments (4)
docs/self-hosted/bitbucket.md (1)
59-69
: Update Azure OpenAI deployment environment variables
The new environment variables (AZURE_GPT41MINI_DEPLOYMENT_NAME
,AZURE_O4MINI_DEPLOYMENT_NAME
,AZURE_O3_DEPLOYMENT_NAME
, optionalAZURE_GPT41_DEPLOYMENT_NAME
, and backward‑compatible fallbacks) correctly align with the updated naming conventions in other self‑hosted docs.docs/self-hosted/azure-devops.md (1)
67-77
: Update Azure OpenAI deployment environment variables
The.env
snippet is now in sync with the new standard: it recommendsgpt-4.1-mini
,o4-mini
,o3
(and optionalgpt-4.1
), plus fallback variables. This matches the updates in your Bitbucket, GitHub, and GitLab docs.docs/self-hosted/github.md (1)
70-80
: Update Azure OpenAI deployment environment variables
The.env
section now correctly listsAZURE_GPT41MINI_DEPLOYMENT_NAME
,AZURE_O4MINI_DEPLOYMENT_NAME
,AZURE_O3_DEPLOYMENT_NAME
, optionalAZURE_GPT41_DEPLOYMENT_NAME
, and the appropriate fallback variables. This harmonizes the GitHub guide with the other self‑hosted docs.docs/self-hosted/gitlab.md (1)
65-75
: Update Azure OpenAI deployment environment variables
The GitLab.env
example now mirrors the updated naming: requiredgpt-4.1-mini
,o4-mini
,o3
, optionalgpt-4.1
, plus legacy fallbacks. This ensures consistency across all platform docs.
No description provided.