Skip to content

Commit

Permalink
[proxy/cplane-communication] refactor: support switching auth type (#92)
Browse files Browse the repository at this point in the history
This adds additional parameter, `--redis-auth-type` to proxy startup CLI arguments. The default is "irsa", and it should be a backward compatible default.

neondatabase/cloud#14462
  • Loading branch information
chaporgin authored Jul 22, 2024
1 parent 3db6825 commit 64c9eab
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 2 deletions.
2 changes: 1 addition & 1 deletion charts/neon-proxy/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: v2
name: neon-proxy
description: Neon Proxy
type: application
version: 1.7.25
version: 1.7.26
appVersion: "0.1.0"
kubeVersion: "^1.18.x-x"
home: https://neon.tech
Expand Down
3 changes: 2 additions & 1 deletion charts/neon-proxy/README.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# neon-proxy

![Version: 1.7.25](https://img.shields.io/badge/Version-1.7.25-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) [![Lint and Test Charts](https://github.com/neondatabase/helm-charts/actions/workflows/lint-test.yaml/badge.svg)](https://github.com/neondatabase/helm-charts/actions/workflows/lint-test.yaml)
![Version: 1.7.26](https://img.shields.io/badge/Version-1.7.26-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) [![Lint and Test Charts](https://github.com/neondatabase/helm-charts/actions/workflows/lint-test.yaml/badge.svg)](https://github.com/neondatabase/helm-charts/actions/workflows/lint-test.yaml)

Neon Proxy

Expand Down Expand Up @@ -92,6 +92,7 @@ Kubernetes: `^1.18.x-x`
| settings.parquetUploadRemoteStorage | string | `""` | (string) Storage location to upload the parquet files to. |
| settings.parquetUploadRowGroupSize | string | `"8192"` | (string) How many rows to include in a row group |
| settings.parquetUploadSize | string | `"100000000"` | (string) How large the total parquet file should be in bytes |
| settings.redisAuthType | string | `"irsa"` | (string) What auth type to use for regional Redis client. "irsa" and "plain" are supported. "plain" means use URI from settings.redisNotifications. "irsa" means AWS IRSA. |
| settings.redisClusterName | string | `"regional-control-plane-redis"` | (string) Redis cluster name, used in aws elasticache |
| settings.redisHost | string | `""` | (string) Redis host for streaming connections (might be different from the notifications host) |
| settings.redisNotifications | string | `""` | (url) Configures redis client |
Expand Down
4 changes: 4 additions & 0 deletions charts/neon-proxy/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -156,6 +156,10 @@ spec:
- --redis-notifications
- {{ . }}
{{- end }}
{{- with .Values.settings.redisAuthType}}
- --redis-auth-type
- {{ . }}
{{- end }}
{{- with .Values.settings.metricBackupCollectionInterval }}
- --metric-backup-collection-interval={{ . }}
{{ end }}
Expand Down
2 changes: 2 additions & 0 deletions charts/neon-proxy/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,8 @@ settings:
connectComputeLock: ""
# settings.redisNotifications -- (url) Configures redis client
redisNotifications: ""
# settings.redisAuthType -- (string) What auth type to use for regional Redis client. "irsa" and "plain" are supported. "plain" means use URI from settings.redisNotifications. "irsa" means AWS IRSA.
redisAuthType: "irsa"
# settings.sqlOverHttpTimeout -- (string) timeout for http connection requests
sqlOverHttpTimeout: "15s"
# settings.httpPoolOptIn -- (bool) Sets the SQL over HTTP Pool to opt-in-only mode if true. Set false to enable it always
Expand Down

0 comments on commit 64c9eab

Please sign in to comment.