Conversation
Subham-KRLX
force-pushed
the
feat/publish-artifact-hub
branch
from
efbaab3 to
2ca0b12
Compare
Subham-KRLX
force-pushed
the
feat/publish-artifact-hub
branch
from
2ca0b12 to
e8742f5
Compare
dimas-b
left a comment
dimas-b
left a comment
There was a problem hiding this comment.
Thanks for your contribution, @Subham-KRLX !
There was a problem hiding this comment.
Does this file need to be in the source tree root?
There was a problem hiding this comment.
Thanks for catching I have moved artifacthub-repo.yml to helm/polaris/ where it logically belongs with the chart.The file is still discoverable by Artifact Hub in that location.
helm/polaris/Chart.yaml
Outdated
| artifacthub.io/license: Apache-2.0 | ||
| artifacthub.io/category: big-data | ||
| artifacthub.io/screenshots: | | ||
| - title: Polaris Architecture |
There was a problem hiding this comment.
Curious, why these screenshots as they are blog specific?
There was a problem hiding this comment.
I included them initially for visual context, but you're right that they are blog-specific. I have removed the artifacthub.io/screenshots section in the latest commit to keep the metadata clean.
helm/polaris/Chart.yaml
Outdated
| - name: Source Code | ||
| url: https://github.com/apache/polaris | ||
| - name: Documentation | ||
| url: https://polaris.apache.org/ |
There was a problem hiding this comment.
I think the documentation page is https://polaris.apache.org/releases/1.3.0/ and we will likely need to bump this version as we move to the next release.
helm/polaris/Chart.yaml
Outdated
| - name: Support | ||
| url: https://github.com/apache/polaris/issues | ||
| artifacthub.io/license: Apache-2.0 | ||
| artifacthub.io/category: big-data |
There was a problem hiding this comment.
I am not sure if big-data is valid for artifacthub.io/category:
artifacthub.io/category (string, see example below)
This annotation allows publishers to provide the package's category. Please use only one category from the following list: ai-machine-learning, database, integration-delivery, monitoring-logging, networking, security, storage or streaming-messaging.
|
A few insights from a legal perspective: ASF's Release Distribution Policy explicitly allows distribution through "downstream channels (rpm, deb, homebrew, etc.)": https://infra.apache.org/release-distribution.html I think ArtifactHub falls into this category as a package discovery/distribution platform. We also have some other Apache projects doing this: Airflow, Pulsar, etc. @jbonofre do you agree? Also, what do you think about the repository ID |
|
Thanks @adutra for the legal insights. It is great to confirm this aligns with the release distribution policy. |
4 participants
Adds Artifact Hub metadata to Chart.yaml (links, license, screenshots) and creates artifacthub-repo.yml to enable proper indexing and verification on Artifact Hub (fixes #3545).
Checklist
site/content/in-dev/unreleased(if needed).