Skip to content

Conversation

@jakehwll
Copy link
Contributor

@jakehwll jakehwll commented Nov 17, 2025

Caution

In whichever release this lands, we've removed the ability to provide keys via a YAML file (specifically on openai_key, anthropic_key, bedrock_access_key and finally bedrock_access_key_secret). This will need to be described in the release notes as to not break peoples AI Bridge integrations upgrading from older versions.

This pull-request ensures that we can see the overview of the settings of the AI Bridge feature within the /deployment/observability route. This set of options only render when the aibridge feature flag is enabled.

Preview

preview-ai-bridge-observability

@matifali
Copy link
Member

@jakehwll, can you share a screenshot of what this will look like?

@jakehwll jakehwll force-pushed the jakehwll/ai-bridge-observability branch from ff41882 to 51cb6b9 Compare November 18, 2025 01:21
Copy link
Member

@matifali matifali left a comment

Choose a reason for hiding this comment

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

Looks good 🔥

Copy link
Member

@mtojek mtojek left a comment

Choose a reason for hiding this comment

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

only nit-picks

@matifali
Copy link
Member

@jakehwll make sure we hide the API keys in UI. I see red boxes but they should not be visible.

Copy link
Member

@mtojek mtojek left a comment

Choose a reason for hiding this comment

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

Thanks!

@dannykopping dannykopping changed the title feat: implement AI Bridge heading to /deployment/observability feat!: implement AI Bridge heading to /deployment/observability Nov 28, 2025
@github-actions github-actions bot added the release/breaking This label is applied to PRs to detect breaking changes as part of the release process label Nov 28, 2025
@dannykopping
Copy link
Contributor

@jakehwll I added a ! to the PR title to indicate a breaking change 👍

Copy link
Contributor

@dannykopping dannykopping left a comment

Choose a reason for hiding this comment

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

LGTM, please ensure description screenshot is updated prior to merge

@jakehwll jakehwll enabled auto-merge (squash) December 1, 2025 21:11
@jakehwll jakehwll merged commit ab4366f into main Dec 1, 2025
36 checks passed
@jakehwll jakehwll deleted the jakehwll/ai-bridge-observability branch December 1, 2025 21:23
@github-actions github-actions bot locked and limited conversation to collaborators Dec 1, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

release/breaking This label is applied to PRs to detect breaking changes as part of the release process

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants