Helm chart: allow setting ingressClassName on the Ingress
メンテナーはふだん 1 日以内に返信
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 85/100
- issue の種類
- 機能追加
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- helm, kubernetes, yaml
- 領域
- devops
調査の方向性
pkg/helm/values.yaml と pkg/helm/templates/ingress.yaml を読み、ingress の設定がどのように定義され、レンダリングされるかを確認してください。任意の ingress.className 値を追加し、設定されている場合は spec.ingressClassName としてレンダリングしてください。chart の README を更新してこの値を記載してください。空のデフォルト値では現在のマニフェストが維持され、値を設定すると Ingress に反映される状態になれば完了です。
索引モデルが issue の本文から書いたものです。
説明
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.classannotation throughingress.annotations. Kubernetes deprecated it in favor ofspec.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.
- 主要言語
- Python
- スター
- 3.9k
- フォーク
- 910
- 平均マージ
- 1日 16時間
- マージ済み PR(30日)
- 41
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
pgadmin-org/pgadmin4 のほかの issue
-
Feature
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
pgadmin-org/pgadmin4#10426 · コメント 3 件 · リアクション 1 件 ·
メンテナーはふだん 1 日以内に返信
-
Feature
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
pgadmin-org/pgadmin4#10272 · コメント 5 件 ·
メンテナーはふだん 1 日以内に返信
-
Housekeeping
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
pgadmin-org/pgadmin4#10157 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
Removing a parent from "Inherited from" hides its columns in the grid, but Save doesn't drop them対応中かも @VIBVEL47 が今日担当しました。 オープンBug
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
pgadmin-org/pgadmin4#10657 ·
メンテナーはふだん 1 日以内に返信
-
難易度 5/5 1週間以上 初心者へのやさしさ 1/100
pgadmin-org/pgadmin4#10656 ·
メンテナーはふだん 1 日以内に返信
pgadmin-org/pgadmin4 の issue をすべて見る
似ている issue
-
[Bug]: Revit selector reads process ids 2000-2099 as years (flaky AgentClientTests.AYearOrAProcessIdSelectsTheHost)対応中かも @Wurschdhaud が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 83/100
pyrevitlabs/pyRevit#3817 ·
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
521xueweihan/HelloGitHub#4074 ·
-
follow-up
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
-
matplotlibオープン
難易度 1/5 1時間未満 初心者へのやさしさ 73/100
nansencenter/xhycom#29 ·