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
- 8 giờ 37 phút
- Pull request đã merge (30 ngày)
- 3
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
-
backend enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 63/100
-
Độ 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
Tất cả issue của Garvan-Data-Science-Platform/ctrl
Issue tương tự
-
e2e-failure ready-to-code
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
redhat-developer/rhdh-plugin-export-overlays#4460 · 1 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 77/100
Maintainer thường phản hồi trong vòng 1 ngày
-
docs(api): openapi.yaml still documents gif-tools optimize colors as 2-256 instead of 2, 4, 16, 256Đang mởarea: backend documentation priority: low
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
snapotter-hq/SnapOtter#2919 ·
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 77/100
obsidianmd/pele#39 ·
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 76/100
biomejs/biome#12475 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày