Skip to content

fix(helm): use correct templating for mcpClientConfig (#2673) #7

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.

Sign up for GitHub

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

Merged
dominictayloruk merged 1 commit into from
Apr 13, 2026
Merged

fix(helm): use correct templating for mcpClientConfig (#2673) #7

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion helm-charts/bifrost/Chart.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: v2
name: bifrost
description: A Helm chart for deploying Bifrost - AI Gateway with unified interface for multiple providers
type: application
version: 2.0.17
version: 2.0.18
appVersion: "1.4.11"
keywords:
- ai
Expand Down
10 changes: 9 additions & 1 deletion helm-charts/bifrost/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,18 @@

Official Helm charts for deploying [Bifrost](https://github.com/maximhq/bifrost) - a high-performance AI gateway with unified interface for multiple providers.

**Latest Version:** 2.0.17
**Latest Version:** 2.0.18

## Changelog

### v2.0.18

- Fixed MCP client config template to correctly map camelCase keys in Helm values:
- `toolsToExecute` → `tools_to_execute`
- `toolsToAutoExecute` → `tools_to_auto_execute`
- `authType` → `auth_type`
- `oauthConfigId` → `oauth_config_id`

### v2.0.16

- Fixed disabled custom plugins being completely removed from rendered config.json instead of being kept with `enabled: false`
Expand Down
16 changes: 8 additions & 8 deletions helm-charts/bifrost/templates/_helpers.tpl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -697,17 +697,17 @@ false
{{- if $client.headers }}
{{- $_ := set $cc "headers" $client.headers }}
{{- end }}
{{- if $client.tools_to_execute }}
{{- $_ := set $cc "tools_to_execute" $client.tools_to_execute }}
{{- if $client.toolsToExecute }}
{{- $_ := set $cc "tools_to_execute" $client.toolsToExecute }}
{{- end }}
{{- if $client.tools_to_auto_execute }}
{{- $_ := set $cc "tools_to_auto_execute" $client.tools_to_auto_execute }}
{{- if $client.toolsToAutoExecute }}
{{- $_ := set $cc "tools_to_auto_execute" $client.toolsToAutoExecute }}
{{- end }}
{{- if $client.auth_type }}
{{- $_ := set $cc "auth_type" $client.auth_type }}
{{- if $client.authType }}
{{- $_ := set $cc "auth_type" $client.authType }}
{{- end }}
{{- if $client.oauth_config_id }}
{{- $_ := set $cc "oauth_config_id" $client.oauth_config_id }}
{{- if $client.oauthConfigId }}
{{- $_ := set $cc "oauth_config_id" $client.oauthConfigId }}
{{- end }}
{{- if hasKey $client "isPingAvailable" }}
{{- $_ := set $cc "is_ping_available" $client.isPingAvailable }}
Expand Down
21 changes: 21 additions & 0 deletions helm-charts/index.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,27 @@
apiVersion: v1
entries:
bifrost:
- apiVersion: v2
appVersion: 1.4.11
created: "2026-04-13T12:00:00.000000+00:00"
description: A Helm chart for deploying Bifrost - AI Gateway with unified interface for multiple providers
digest: ""
home: https://www.getmaxim.ai/bifrost
icon: https://www.getmaxim.ai/bifrost/bifrost-logo-only.png
keywords:
- ai
- gateway
- llm
maintainers:
- email: akshay@getmaxim.ai
name: Bifrost Team
name: bifrost
sources:
- https://github.com/maximhq/bifrost
type: application
urls:
- https://maximhq.github.io/bifrost/helm-charts/bifrost-2.0.18.tgz
version: 2.0.18
Comment on lines +4 to +24
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot Apr 13, 2026

Choose a reason for hiding this comment

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

medium

The metadata for the new entry in index.yaml (version 2.0.18) is inconsistent with the metadata defined in helm-charts/bifrost/Chart.yaml. Specifically:

  • Maintainers: index.yaml uses akshay@getmaxim.ai while Chart.yaml uses support@getbifrost.ai.
  • Icon: index.yaml uses https://www.getmaxim.ai/bifrost/bifrost-logo-only.png while Chart.yaml uses https://www.getbifrost.ai/favicon.png.
  • Keywords: index.yaml is missing openai and anthropic which are present in Chart.yaml.

It is recommended to regenerate the index.yaml file using helm repo index or manually synchronize these fields to ensure consistency across the repository.

- apiVersion: v2
appVersion: 1.4.11
created: "2026-04-08T12:00:00.000000+00:00"
Expand Down