kafka: the operator JVM test pins env by index, and MaxRAM truncates fractional limits
メンテナーはふだん 2 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 72/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- go, helm, kubernetes
調査の方向性
packages/system/kafka-operator/tests/jvm_sizing_test.yaml と packages/apps/kafka/templates/_jvm.tpl を読みます。まず、テストが3つの環境変数をどのように選択して比較するかを確認し、次にテンプレートがメモリ制限をどのように変換するかを調べます。テストでそれらの相対的な順序を確認し、小数のメモリ制限が指定された丸め動作でレンダリングされれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Two minor leftovers from the Kafka JVM sizing in #4865. Neither breaks anything today.
The cluster operator test in packages/system/kafka-operator/tests/jvm_sizing_test.yaml finds CONTAINER_MEMORY_LIMIT, CONTAINER_CPU_LIMIT and JAVA_OPTS at env[19], env[20] and env[21]. What has to hold is their relative order, because $(VAR) expands only from earlier entries. An env var added upstream shifts the indices, so a Strimzi bump turns the test red with no real defect. Asserting the order of the three names would survive that. #4665 tracks the same kind of position-based selection for whole documents.
kafka.javaToolOptions in packages/apps/kafka/templates/_jvm.tpl truncates the memory limit to an integer. For memory: 1.1Gi it renders -XX:MaxRAM=1181116006, while Kubernetes rounds the same quantity up to 1181116007. One byte under the limit is harmless. Rounding up would still match what the downward API hands the operator JVM.
- 主要言語
- Go
- スター
- 2.2k
- フォーク
- 209
- 平均マージ
- 3日 9時間
- マージ済み PR（30日）
- 242
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
cozystack/cozystack のほかの issue
-
docs(release): changelog check B looks for images/*/Dockerfile, which does not exist対応中かも @harshitrwt が 2 日前に担当しました。 オープンarea/release kind/bug kind/documentation triage/needs-triage
難易度 1/5 1時間未満 初心者へのやさしさ 82/100
cozystack/cozystack#4892 · コメント 1 件 · リアクション 1 件 ·
メンテナーはふだん 2 日以内に返信
-
triage/needs-triage
難易度 1/5 1時間未満 初心者へのやさしさ 64/100
cozystack/cozystack#4886 · リアクション 1 件 ·
メンテナーはふだん 2 日以内に返信
-
area/storage kind/bug priority/backlog triage/needs-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
cozystack/cozystack#4878 · リアクション 1 件 ·
メンテナーはふだん 2 日以内に返信
-
testing: helm-unittest's isNotNullOrEmpty passes when the path does not exist再び着手できるかも @Shxshxnkk1 が 9 日前に担当しましたが、オープン中のプルリクエストはありません。 オープンarea/testing kind/bug triage/needs-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
cozystack/cozystack#4841 · コメント 1 件 · リアクション 1 件 ·
メンテナーはふだん 2 日以内に返信
-
chore(cilium): drop externalIPs.enabled and nodePort.enabled, the Cilium chart does not read themオープンarea/cilium kind/cleanup triage/needs-triage
難易度 1/5 1時間未満 初心者へのやさしさ 82/100
cozystack/cozystack#4816 · リアクション 1 件 ·
メンテナーはふだん 2 日以内に返信
cozystack/cozystack の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
oasdiff/oasdiff#1337 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
dataiku/kiji-proxy#635 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 67/100
hyperledger-firefly/signer#126 ·
-
agent-review-finding chore
難易度 2/5 1〜3時間 初心者へのやさしさ 73/100
jordansmall/spindrift#5748 ·
メンテナーはふだん 1 日以内に返信