Skip to content
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

Update environment variables passed to the prefect server chart #259

Merged
merged 2 commits into from
Oct 17, 2023
+105 −27
Merged

Update environment variables passed to the prefect server chart #259

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
d105e06
add ui url
jamiezieziula Oct 16, 2023
f451d22
overhaul of server env vars
jamiezieziula Oct 16, 2023
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
1 change: 1 addition & 0 deletions charts/prefect-server/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -162,6 +162,7 @@ postgresql+asyncpg://{username}:{password}@{hostname}/{database}?ssl=verify-ca
| server.podSecurityContext.runAsUser | int | `1001` | set server pod's security context runAsUser |
| server.priorityClassName | string | `""` | priority class name to use for the server pods; if the priority class is empty or doesn't exist, the server pods are scheduled without a priority class |
| server.publicApiUrl | string | `"http://localhost:4200/api"` | sets PREFECT_UI_API_URL; should be publicly accessible API URL |
| server.publicUiUrl | string | `"http://localhost:4200"` | sets PREFECT_UI_URL |
| server.readinessProbe.config.failureThreshold | int | `3` | The number of consecutive failures allowed before considering the probe as failed. |
| server.readinessProbe.config.initialDelaySeconds | int | `10` | The number of seconds to wait before starting the first probe. |
| server.readinessProbe.config.periodSeconds | int | `10` | The number of seconds to wait between consecutive probes. |
Expand Down
4 changes: 2 additions & 2 deletions charts/prefect-server/templates/deployment.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -78,10 +78,10 @@ spec:
value: /home/prefect
- name: PREFECT_DEBUG_MODE
value: {{ .Values.server.image.debug | quote }}
{{- if .Values.server.publicApiUrl }}
- name: PREFECT_UI_API_URL
value: {{ .Values.server.publicApiUrl | quote }}
{{- end }}
- name: PREFECT_UI_URL
value: {{ .Values.server.publicUiUrl | quote }}
{{- if .Values.postgresql.enabled }}
- name: PREFECT_API_DATABASE_CONNECTION_URL
valueFrom:
Expand Down
6 changes: 6 additions & 0 deletions charts/prefect-server/values.schema.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -93,6 +93,12 @@
"description": "sets PREFECT_UI_API_URL; should be publicly accessible API URL",
"form": true
},
"publicUiUrl": {
"type": "string",
"title": "Public UI URL",
"description": "sets PREFECT_UI_URL",
"form": true
},
"env": {
"type": "array",
"title": "Environment Variables",
Expand Down
3 changes: 3 additions & 0 deletions charts/prefect-server/values.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,9 @@ server:
# -- sets PREFECT_UI_API_URL; should be publicly accessible API URL
publicApiUrl: http://localhost:4200/api

# -- sets PREFECT_UI_URL
publicUiUrl: "http://localhost:4200"
jamiezieziula marked this conversation as resolved.
Show resolved Hide resolved

# -- array with environment variables to add to server nodes
env: []
## env:
Expand Down
Loading