Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

Helm chart: allow setting ingressClassName on the Ingress

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#10,658 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

@commarla đang làm issue này rồi.

Từ ngày 22/10/2026.

  • #10659 của @commarla — đang mở

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
85/100
Loại issue
Tính năng
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
helm, kubernetes, yaml
Lĩnh vực
devops

Hướng nghiên cứu

Đọc pkg/helm/values.yaml và pkg/helm/templates/ingress.yaml để xem các thiết lập ingress được định nghĩa và render như thế nào. Thêm giá trị tùy chọn ingress.className và render nó thành spec.ingressClassName khi được thiết lập; cập nhật README của chart để ghi giá trị này. Hoàn tất khi giá trị mặc định rỗng giữ nguyên manifest hiện tại và giá trị được cấu hình xuất hiện trên Ingress.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

Describe the solution you'd like

Add an optional ingress.className value to the Helm chart (pkg/helm) and render it as spec.ingressClassName on the Ingress when set.

Today templates/ingress.yaml never sets ingressClassName. On a cluster with no default IngressClass, or with several ingress controllers, no controller picks up the Ingress. We deploy the chart behind Traefik and currently patch the rendered manifest with a post-renderer to add the field.

With an empty default (className: ""), the chart renders exactly what it renders today, so the change is backward compatible. Many common charts expose the same value under the same name.

Describe alternatives you've considered

  • The kubernetes.io/ingress.class annotation through ingress.annotations. Kubernetes deprecated it in favor of spec.ingressClassName, and some controllers no longer honor it.
  • A Helm post-renderer or Kustomize patch that adds spec.ingressClassName. It works, but every user has to maintain it outside the chart.

Additional context

I can open a pull request for this: one value in values.yaml, three lines in templates/ingress.yaml, one row in the chart README.

Ngôn ngữ chính
Python
Star
3.9k
Fork
910
Merge trung bình
1 ngày 16 giờ
Pull request đã merge (30 ngày)
41

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của pgadmin-org/pgadmin4

Tất cả issue của pgadmin-org/pgadmin4

Issue tương tự

Thêm issue về Python

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.