aeskey and jwt Secrets are missing helm.sh/resource-policy: keep
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 1/5
- Thời gian dự kiến
- Dưới một giờ
- Mức phù hợp với người mới
- 88/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- helm, typescript
- Lĩnh vực
- devops
Hướng nghiên cứu
So sánh .helm/ctrl/templates/secrets/aeskey.yaml và jwt.yaml với postgres.yaml, tệp đã có chú thích keep. Thêm cùng chú thích siêu dữ liệu đó vào cả hai Secrets; sau đó kiểm tra các template Helm đã render để xác nhận chú thích có mặt. Hoàn thành nghĩa là khi gỡ cài đặt release, các Secrets khóa AES và JWT vẫn được giữ lại.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
The ctrl-postgres Secret in .helm/ctrl/templates/secrets/postgres.yaml carries "helm.sh/resource-policy": keep. The aeskey and jwt Secrets don't, and never have, both were created in f453dec without it.
Helm removes the resources it created on helm uninstall unless they carry that annotation. The Postgres data volume isn't one of those, it comes from the postgresql subchart's StatefulSet volumeClaimTemplate, so Kubernetes keeps it.
So an uninstall takes the AES encryption key away and leaves the database behind. A reinstall generates a fresh key, and everything written under the old one becomes unreadable. Names, email, DOB, mobile, address, next of kin, plus Organisation.elsaToken and Study.redcapToken. The app would start normally, so this surfaces as ciphertext appearing in the UI rather than as an error.
Ordinary deploys aren't affected. The lookup in _helpers.tpl reuses the existing Secret on helm upgrade, so the key survives a normal release. This is specifically about uninstall and reinstall, which is what happens when a deployment gets cleared down or moved between clusters.
Same shape as #701 (Redeploy loses database password), which PR #709 addressed for the postgres password by adding the annotation. The other two secrets were never brought in line.
Fix: add "helm.sh/resource-policy": keep to the metadata annotations in aeskey.yaml and jwt.yaml, matching postgres.yaml.
- Ngôn ngữ chính
- TypeScript
- Star
- 4
- Fork
- 0
- Merge trung bình
- 14 giờ 50 phút
- Pull request đã merge (30 ngày)
- 5
Chuẩn bị môi trường
- Có Dockerfile hoặc tệp Docker Compose
- Có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của Garvan-Data-Science-Platform/ctrl
-
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 82/100
-
admin-client backend bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
-
backend bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
Garvan-Data-Science-Platform/ctrl#970 · 1 bình luận ·
-
backend bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
-
backend bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Tất cả issue của Garvan-Data-Science-Platform/ctrl
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 74/100
AvaProtocol/ava-sdk-js#276 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
useFragment does not require `variables` when the fragment's variables type has required fieldsĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
apollographql/apollo-client#13493 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Custom hosts written as trailing-dot FQDNs (`host.local.`) aren't detected as local backendsCó thể đã có người làm @roboomp đã nhận hôm nay. Đang mởbug prio:p2 providers triaged
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
can1357/oh-my-pi#15798 · 3 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
code-yeongyu/oh-my-openagent#10172 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
PDFCraftTool/pdfcraft#189 ·