Skip to content

Add Bedrock Mantle GovCloud region#23860

Open
CHARLESPALEN-OAI wants to merge 1 commit into
mainfrom
codex/bedrock-govcloud-region
Open

Add Bedrock Mantle GovCloud region#23860
CHARLESPALEN-OAI wants to merge 1 commit into
mainfrom
codex/bedrock-govcloud-region

Conversation

@CHARLESPALEN-OAI
Copy link
Copy Markdown

Summary

  • Add us-gov-west-1 to the Bedrock Mantle supported region list
  • Cover the GovCloud endpoint URL in the existing base_url unit test

Test

  • cargo test -p codex-model-provider

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 21, 2026

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@CHARLESPALEN-OAI
Copy link
Copy Markdown
Author

I have read the CLA Document and I hereby sign the CLA

Copy link
Copy Markdown
Contributor

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0851846a5d

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread codex-rs/model-provider/src/amazon_bedrock/mantle.rs
@CHARLESPALEN-OAI
Copy link
Copy Markdown
Author

recheck

github-actions Bot added a commit that referenced this pull request May 21, 2026
Copy link
Copy Markdown

@usgenes39-dotcom usgenes39-dotcom left a comment

Choose a reason for hiding this comment

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

I found one blocking provider-routing issue on the current head.

The GovCloud region itself is valid, but the base URL path this PR codifies is not. AWS documents the Mantle OpenAI-compatible base URL as https://bedrock-mantle.<region>.api.aws/v1, and a no-auth probe returns 401 for /v1/models but 404 for /openai/v1/models in both us-gov-west-1 and us-east-1. With this PR, users who select the newly supported GovCloud region will route Responses requests to /openai/v1/responses, so the region unlock still fails at request time. CI is green, but this should be fixed before merge.

Comment thread codex-rs/model-provider/src/amazon_bedrock/mantle.rs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants