Skip to content

Commit 140092b

Browse files
committed
πŸ”§ Fix environment variables in GitHub Actions workflow
- Move all environment variables to job level for proper scope - Remove step-level environment variables that were causing issues - Use consistent naming: POSTGRES_RELEASE_NAME, APP_RELEASE_NAME, etc. - Fix all references to use job-level environment variables - Ensure all env variables are properly defined and accessible
1 parent 04448cd commit 140092b

File tree

1 file changed

+14
-14
lines changed

1 file changed

+14
-14
lines changed

β€Ž.github/workflows/deploy-kubernetes.ymlβ€Ž

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -32,8 +32,12 @@ jobs:
3232
HELM_NAMESPACE: studymate
3333
DOMAIN: ${{ github.event.inputs.domain || 'studymate.student.k8s.aet.cit.tum.de' }}
3434
IMAGE_TAG: k8s-latest
35-
POSTGRES_PERSISTENCE_SIZE: 8Gi
36-
WEAVIATE_PERSISTENCE_SIZE: 10Gi
35+
POSTGRES_PERSISTENCE_SIZE: 1Gi
36+
WEAVIATE_PERSISTENCE_SIZE: 1Gi
37+
POSTGRES_RELEASE_NAME: studymate-postgres
38+
POSTGRES_CHART_PATH: bitnami/postgresql
39+
APP_RELEASE_NAME: studymate
40+
APP_CHART_PATH: ./infra/helm
3741

3842
steps:
3943
- name: Checkout code
@@ -106,8 +110,6 @@ jobs:
106110
- name: Deploy PostgreSQL with Bitnami
107111
env:
108112
POSTGRES_PASSWORD: ${{ secrets.POSTGRES_PASSWORD }}
109-
HELM_RELEASE_NAME: studymate-postgres
110-
CHART_PATH: bitnami/postgresql
111113
run: |
112114
echo "πŸ—„οΈ Deploying PostgreSQL with Bitnami..."
113115
@@ -119,7 +121,7 @@ jobs:
119121
kubectl create namespace ${{ env.HELM_NAMESPACE }} --dry-run=client -o yaml | kubectl apply -f -
120122
121123
# Deploy Bitnami PostgreSQL
122-
helm upgrade --install ${{ env.HELM_RELEASE_NAME }} ${{ env.CHART_PATH }} \
124+
helm upgrade --install ${{ env.POSTGRES_RELEASE_NAME }} ${{ env.POSTGRES_CHART_PATH }} \
123125
--namespace ${{ env.HELM_NAMESPACE }} \
124126
--set auth.postgresPassword="${POSTGRES_PASSWORD}" \
125127
--set auth.database=studymate \
@@ -138,13 +140,11 @@ jobs:
138140
OPEN_WEBUI_API_KEY_CHAT: ${{ secrets.OPEN_WEBUI_API_KEY_CHAT }}
139141
OPEN_WEBUI_API_KEY_GEN: ${{ secrets.OPEN_WEBUI_API_KEY_GEN }}
140142
LANGSMITH_API_KEY: ${{ secrets.LANGSMITH_API_KEY }}
141-
HELM_RELEASE_NAME: studymate
142-
CHART_PATH: ./infra/helm
143143
run: |
144144
echo "πŸš€ Deploying StudyMate Application..."
145145
146146
# Use helm upgrade --install for better permission handling
147-
helm upgrade --install ${{ env.HELM_RELEASE_NAME }} ${{ env.CHART_PATH }} \
147+
helm upgrade --install ${{ env.APP_RELEASE_NAME }} ${{ env.APP_CHART_PATH }} \
148148
--namespace ${{ env.HELM_NAMESPACE }} \
149149
--create-namespace \
150150
--set ingress.host=${{ env.DOMAIN }} \
@@ -159,12 +159,12 @@ jobs:
159159
- name: Verify deployment
160160
run: |
161161
echo "πŸ” Checking deployment status..."
162-
kubectl get pods -n ${{ env.HELM_NAMESPACE }} -l app.kubernetes.io/instance=${{ env.HELM_RELEASE_NAME }}
163-
kubectl get services -n ${{ env.HELM_NAMESPACE }} -l app.kubernetes.io/instance=${{ env.HELM_RELEASE_NAME }}
162+
kubectl get pods -n ${{ env.HELM_NAMESPACE }} -l app.kubernetes.io/instance=${{ env.APP_RELEASE_NAME }}
163+
kubectl get services -n ${{ env.HELM_NAMESPACE }} -l app.kubernetes.io/instance=${{ env.APP_RELEASE_NAME }}
164164
kubectl get ingress -n ${{ env.HELM_NAMESPACE }}
165165
166166
echo "πŸ₯ Waiting for pods to be ready..."
167-
kubectl wait --for=condition=ready pod -l app.kubernetes.io/instance=${{ env.HELM_RELEASE_NAME }} -n ${{ env.HELM_NAMESPACE }} --timeout=300s || echo "Some pods may still be starting"
167+
kubectl wait --for=condition=ready pod -l app.kubernetes.io/instance=${{ env.APP_RELEASE_NAME }} -n ${{ env.HELM_NAMESPACE }} --timeout=300s || echo "Some pods may still be starting"
168168
169169
- name: Get access information
170170
run: |
@@ -173,6 +173,6 @@ jobs:
173173
echo ""
174174
echo "πŸ“Š Useful commands:"
175175
echo "kubectl get pods -n ${{ env.HELM_NAMESPACE }}"
176-
echo "kubectl logs -f deployment/${{ env.HELM_RELEASE_NAME }}-client -n ${{ env.HELM_NAMESPACE }}"
177-
echo "kubectl logs -f deployment/${{ env.HELM_RELEASE_NAME }}-auth-service -n ${{ env.HELM_NAMESPACE }}"
178-
echo "kubectl port-forward svc/${{ env.HELM_RELEASE_NAME }}-client 8080:80 -n ${{ env.HELM_NAMESPACE }}"
176+
echo "kubectl logs -f deployment/${{ env.APP_RELEASE_NAME }}-client -n ${{ env.HELM_NAMESPACE }}"
177+
echo "kubectl logs -f deployment/${{ env.APP_RELEASE_NAME }}-auth-service -n ${{ env.HELM_NAMESPACE }}"
178+
echo "kubectl port-forward svc/${{ env.APP_RELEASE_NAME }}-client 8080:80 -n ${{ env.HELM_NAMESPACE }}"

0 commit comments

Comments
Β (0)