Skip to content

Commit d5209ea

Browse files
asithadeasithade
authored
feat(helm): add missing environment variables to chart (#137)
Add comprehensive environment variable configuration including: - PCC_AUTH0_SECRET for session management - Snowflake analytics configuration (account, credentials, connection pool) - Optional configuration with sensible defaults (LOG_LEVEL, SUPABASE_STORAGE_BUCKET) - Complete documentation in README.md LFXV2-691 Generated with [Claude Code](https://claude.ai/code) Signed-off-by: Asitha de Silva <asithade@gmail.com>
1 parent edcd722 commit d5209ea

File tree

2 files changed

+82
-6
lines changed

2 files changed

+82
-6
lines changed

‎charts/lfx-v2-ui/README.md‎

Lines changed: 42 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,24 @@ environment:
4545
value: 'https://api.openai.com/v1/chat/completions'
4646
AI_API_KEY:
4747
value: 'your-openai-api-key'
48+
49+
# Required: Snowflake Analytics configuration
50+
SNOWFLAKE_ACCOUNT:
51+
value: 'your-org-account'
52+
SNOWFLAKE_USERNAME:
53+
value: 'your-username'
54+
SNOWFLAKE_ROLE:
55+
value: 'your-read-role'
56+
SNOWFLAKE_DATABASE:
57+
value: 'your-database'
58+
SNOWFLAKE_WAREHOUSE:
59+
value: 'your-warehouse'
60+
SNOWFLAKE_API_KEY:
61+
value: 'your-private-key'
62+
63+
# Required: Auth0 session secret
64+
PCC_AUTH0_SECRET:
65+
value: 'sufficiently-long-random-string'
4866
```
4967
5068
#### Using Kubernetes Secrets
@@ -116,12 +134,13 @@ environment:
116134

117135
#### Auth0 Configuration (User Authentication)
118136

119-
| Parameter | Description | Required | Default |
120-
| --------------------------------------- | ----------------------------- | -------- | -------------------------------------------------- |
121-
| `environment.PCC_AUTH0_ISSUER_BASE_URL` | Auth0 issuer base URL | No | `https://linuxfoundation-dev.auth0.com/` |
122-
| `environment.PCC_AUTH0_AUDIENCE` | Auth0 API audience identifier | No | `https://api-gw.dev.platform.linuxfoundation.org/` |
123-
| `environment.PCC_AUTH0_CLIENT_ID` | Auth0 client ID (secret) | **Yes** | - |
124-
| `environment.PCC_AUTH0_CLIENT_SECRET` | Auth0 client secret (secret) | **Yes** | - |
137+
| Parameter | Description | Required | Default |
138+
| --------------------------------------- | ---------------------------------------- | -------- | -------------------------------------------------- |
139+
| `environment.PCC_AUTH0_ISSUER_BASE_URL` | Auth0 issuer base URL | No | `https://linuxfoundation-dev.auth0.com/` |
140+
| `environment.PCC_AUTH0_AUDIENCE` | Auth0 API audience identifier | No | `https://api-gw.dev.platform.linuxfoundation.org/` |
141+
| `environment.PCC_AUTH0_CLIENT_ID` | Auth0 client ID (secret) | **Yes** | - |
142+
| `environment.PCC_AUTH0_CLIENT_SECRET` | Auth0 client secret (secret) | **Yes** | - |
143+
| `environment.PCC_AUTH0_SECRET` | Auth0 session secret (sufficiently long) | **Yes** | - |
125144

126145
#### LFX Auth Configuration (Service-to-Service)
127146

@@ -155,6 +174,23 @@ environment:
155174
| `environment.AI_PROXY_URL` | AI service proxy URL (OpenAI compatible) | **Yes** | - |
156175
| `environment.AI_API_KEY` | API key for AI service | **Yes** | - |
157176

177+
#### Snowflake Analytics Configuration
178+
179+
Required for analytics endpoints (active-weeks-streak, pull-requests-merged, code-commits):
180+
181+
| Parameter | Description | Required | Default |
182+
| --------------------------------------- | ------------------------------------------------ | -------- | -------- |
183+
| `environment.SNOWFLAKE_ACCOUNT` | Snowflake account identifier (org-account) | **Yes** | - |
184+
| `environment.SNOWFLAKE_USERNAME` | Snowflake service user for read-only queries | **Yes** | - |
185+
| `environment.SNOWFLAKE_ROLE` | Snowflake user role with SELECT-only permissions | **Yes** | - |
186+
| `environment.SNOWFLAKE_DATABASE` | Snowflake analytics database name | **Yes** | - |
187+
| `environment.SNOWFLAKE_WAREHOUSE` | Snowflake warehouse for query execution | **Yes** | - |
188+
| `environment.SNOWFLAKE_API_KEY` | Snowflake private key for authentication | **Yes** | - |
189+
| `environment.SNOWFLAKE_LOG_LEVEL` | Snowflake SDK log level | No | `ERROR` |
190+
| `environment.SNOWFLAKE_LOCK_STRATEGY` | Lock strategy for query deduplication | No | `memory` |
191+
| `environment.SNOWFLAKE_MIN_CONNECTIONS` | Minimum connection pool size | No | `2` |
192+
| `environment.SNOWFLAKE_MAX_CONNECTIONS` | Maximum connection pool size | No | `10` |
193+
158194
#### Logging Configuration
159195

160196
| Parameter | Description | Required | Default |

‎charts/lfx-v2-ui/values.yaml‎

Lines changed: 40 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -168,6 +168,10 @@ environment:
168168
QUERY_SERVICE_URL:
169169
value: "http://query-service.default.svc.cluster.local/query/resources"
170170

171+
# Optional configuration with defaults
172+
LOG_LEVEL:
173+
value: "info"
174+
171175
# required environment variable secrets
172176

173177
PCC_AUTH0_CLIENT_ID:
@@ -176,6 +180,9 @@ environment:
176180
PCC_AUTH0_CLIENT_SECRET:
177181
value:
178182

183+
PCC_AUTH0_SECRET:
184+
value:
185+
179186
SUPABASE_URL:
180187
value:
181188

@@ -199,3 +206,36 @@ environment:
199206

200207
AI_API_KEY:
201208
value:
209+
210+
# Snowflake Analytics Configuration
211+
# Required for analytics endpoints: active-weeks-streak, pull-requests-merged, code-commits
212+
SNOWFLAKE_ACCOUNT:
213+
value:
214+
215+
SNOWFLAKE_USERNAME:
216+
value:
217+
218+
SNOWFLAKE_ROLE:
219+
value:
220+
221+
SNOWFLAKE_DATABASE:
222+
value:
223+
224+
SNOWFLAKE_WAREHOUSE:
225+
value:
226+
227+
SNOWFLAKE_API_KEY:
228+
value:
229+
230+
# Optional Snowflake configuration
231+
SNOWFLAKE_LOG_LEVEL:
232+
value: "ERROR"
233+
234+
SNOWFLAKE_LOCK_STRATEGY:
235+
value: "memory"
236+
237+
SNOWFLAKE_MIN_CONNECTIONS:
238+
value: "2"
239+
240+
SNOWFLAKE_MAX_CONNECTIONS:
241+
value: "10"

0 commit comments

Comments
 (0)