[Bug]: Bitnami Helm chart helpers bundled in charts/ are not attributed in LICENSE

オープン 初心者向け
#4,476 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 2 日以内に返信

まだ誰も着手していません。

評価

難易度
1/5
見積もり時間
1時間未満
初心者へのやさしさ
82/100
issue の種類
ドキュメント
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
helm
領域
documentation

調査の方向性

ルートの LICENSE にある「This product bundles various third-party components also under the Apache Software License 2.0」というタイトルのセクションを読み、charts/amoro/templates/common/ 配下の該当する5つのファイルを調べてください。VMware の著作権表示と Apache License 2.0 を含め、Bitnami charts common 2.11.1 の帰属表示をそこに追加してください。ルートの LICENSE に、同梱された5つのヘルパーすべてとその出典が記載されれば完了です。

索引モデルが issue の本文から書いたものです。

説明

What happened?

The source release bundles five Helm template helpers copied from the Bitnami charts common library chart (Copyright VMware, Inc., Apache-2.0, not an ASF project), but the root LICENSE does not mention them.

Found while reviewing apache-amoro-0.9.0-incubating-src.tar.gz (RC14, tag v0.9.0-rc14, commit 18d12b99d5297dbd0bb4eb72517c4855d75210f6). The same files were also present, unattributed, in 0.8.1-incubating.

Affected files:

  • charts/amoro/templates/common/_capabilities.tpl
  • charts/amoro/templates/common/_ingress.tpl
  • charts/amoro/templates/common/_labels.tpl
  • charts/amoro/templates/common/_names.tpl
  • charts/amoro/templates/common/_tplvalues.tpl

Each file correctly keeps its upstream header:

Copyright VMware, Inc.
SPDX-License-Identifier: APACHE-2.0

// This file under the http://www.apache.org/licenses/LICENSE-2.0, copied from
// https://github.com/bitnami/charts/blob/common/2.11.1/bitnami/common/templates/<file>.tpl

However, grep -n -i 'bitnami\|vmware\|charts/' LICENSE returns nothing. Per the ASF third-party licensing guidance, bundled third-party code should be listed in LICENSE.

Suggested fix

Add an entry to the root LICENSE, in the existing "This product bundles various third-party components also under the Apache Software License 2.0" section, listing the five files above as from Bitnami charts common 2.11.1 (https://github.com/bitnami/charts), Copyright VMware, Inc., Apache License 2.0.

If the Helm chart is ever packaged and distributed on its own, it should also carry this attribution.

Affects Versions

0.9.0-incubating (RC14), 0.8.1-incubating

How to reproduce
tar xzf apache-amoro-0.9.0-incubating-src.tar.gz && cd amoro-0.9.0-incubating
head -7 charts/amoro/templates/common/_names.tpl
grep -n -i 'bitnami\|vmware' LICENSE   # no output
主要言語
Java
スター
1.2k
フォーク
398
平均マージ
3日 7時間
マージ済み PR(30日)
15

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

apache/amoro のほかの issue

apache/amoro の issue をすべて見る

似ている issue

Java の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。