From 2ebba36824cb01012ed43690070864873de96fc6 Mon Sep 17 00:00:00 2001 From: Spike Lu Date: Wed, 12 Jun 2024 14:41:09 -0700 Subject: [PATCH] fix update provider settings error --- README.md | 21 +++++++++---------- .../storage/postgresql/provider_settings.go | 1 + 2 files changed, 11 insertions(+), 11 deletions(-) diff --git a/README.md b/README.md index b1565c8..45f1bff 100644 --- a/README.md +++ b/README.md @@ -141,16 +141,16 @@ docker pull luyuanxin1995/bricksllm:1.4.0 > | `POSTGRESQL_DB_NAME` | optional | Name for Postgresql DB. | > | `POSTGRESQL_USERNAME` | required | Postgresql DB username | > | `POSTGRESQL_PASSWORD` | required | Postgresql DB password | -> | `POSTGRESQL_SSL_MODE` | optional | Postgresql SSL mode| `disable` -> | `POSTGRESQL_PORT` | optional | The port that Postgresql DB runs on| `5432` -> | `POSTGRESQL_READ_TIME_OUT` | optional | Timeout for Postgresql read operations | `2m` -> | `POSTGRESQL_WRITE_TIME_OUT` | optional | Timeout for Postgresql write operations | `5s` -> | `REDIS_HOSTS` | required | Host for Redis. Separated by , | `localhost` +> | `POSTGRESQL_SSL_MODE` | optional | Postgresql SSL mode| `disable` | +> | `POSTGRESQL_PORT` | optional | The port that Postgresql DB runs on| `5432` | +> | `POSTGRESQL_READ_TIME_OUT` | optional | Timeout for Postgresql read operations | `2m` | +> | `POSTGRESQL_WRITE_TIME_OUT` | optional | Timeout for Postgresql write operations | `5s` | +> | `REDIS_HOSTS` | required | Host for Redis. Separated by , | `localhost` | > | `REDIS_PASSWORD` | optional | Redis Password | -> | `REDIS_PORT` | optional | The port that Redis DB runs on | `6379` -> | `REDIS_READ_TIME_OUT` | optional | Timeout for Redis read operations | `1s` -> | `REDIS_WRITE_TIME_OUT` | optional | Timeout for Redis write operations | `500ms` -> | `IN_MEMORY_DB_UPDATE_INTERVAL` | optional | The interval BricksLLM API gateway polls Postgresql DB for latest key configurations | `1s` +> | `REDIS_PORT` | optional | The port that Redis DB runs on | `6379` | +> | `REDIS_READ_TIME_OUT` | optional | Timeout for Redis read operations | `1s` | +> | `REDIS_WRITE_TIME_OUT` | optional | Timeout for Redis write operations | `500ms` | +> | `IN_MEMORY_DB_UPDATE_INTERVAL` | optional | The interval BricksLLM API gateway polls Postgresql DB for latest key configurations | `1s` | > | `STATS_PROVIDER` | optional | This value can only be datadog. Required for integration with Datadog. | > | `PROXY_TIMEOUT` | optional | Timeout for proxy HTTP requests. | `600s` | > | `NUMBER_OF_EVENT_MESSAGE_CONSUMERS` | optional | Number of event message consumers that help handle counting tokens and inserting event into db. | `3` | @@ -159,8 +159,7 @@ docker pull luyuanxin1995/bricksllm:1.4.0 > | `AMAZON_REGION` | optional | Region for AWS. | `us-west-2` | > | `AMAZON_REQUEST_TIMEOUT` | optional | Timeout for amazon requests. | `5s` | > | `AMAZON_CONNECTION_TIMEOUT` | optional | Timeout for amazon connection. | `10s` | -> | `AWS_SECRET_ACCESS_KEY` | optional | Required for PII detection features. | -> | `AWS_ACCESS_KEY_ID` | optional | Required for PII detection features. | +> | `ADMIN_PASS` | optional | Simple password for the admin server. | ## Configuration Endpoints The configuration server runs on Port `8001`. diff --git a/internal/storage/postgresql/provider_settings.go b/internal/storage/postgresql/provider_settings.go index a8131ab..dffcb0e 100644 --- a/internal/storage/postgresql/provider_settings.go +++ b/internal/storage/postgresql/provider_settings.go @@ -175,6 +175,7 @@ func (s *Store) UpdateProviderSetting(id string, setting *provider.UpdateSetting if setting.AllowedModels != nil { values = append(values, sliceToSqlStringArray(*setting.AllowedModels)) fields = append(fields, fmt.Sprintf("allowed_models = $%d", d)) + d++ } if setting.CostMap != nil {