You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@polaris.apache.org by "Subham-KRLX (via GitHub)" <gi...@apache.org> on 2026/02/03 12:44:57 UTC
[PR] Feat/publish artifact hub [polaris]
Subham-KRLX opened a new pull request, #3649:
URL: https://github.com/apache/polaris/pull/3649
<!--
๐ Describe what changes you're proposing, especially breaking or user-facing changes.
๐ See https://github.com/apache/polaris/blob/main/CONTRIBUTING.md for more.
-->
Adds Artifact Hub metadata to [Chart.yaml](cci:7://file:///Users/subhamsangwan/airflow/chart/Chart.yaml:0:0-0:0) (links, license, screenshots) and creates [artifacthub-repo.yml](cci:7://file:///Users/subhamsangwan/polaris/artifacthub-repo.yml:0:0-0:0) to enable proper indexing and verification on Artifact Hub (fixes #3545).
## Checklist
- [ ] ๐ก๏ธ Don't disclose security issues! (contact security@apache.org)
- [x] ๐ Clearly explained why the changes are needed, or linked related issues: Fixes #3545
- [x] ๐งช Added/updated tests with good coverage, or manually tested (and explained how)
- Verified YAML syntax and schema compliance manually.
- [ ] ๐ก Added comments for complex logic
- [ ] ๐งพ Updated [CHANGELOG.md](cci:7://file:///Users/subhamsangwan/polaris/CHANGELOG.md:0:0-0:0) (if needed)
- [ ] ๐ Updated documentation in `site/content/in-dev/unreleased` (if needed).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Feat/publish artifact hub [polaris]
Posted by "dimas-b (via GitHub)" <gi...@apache.org>.
dimas-b commented on code in PR #3649:
URL: https://github.com/apache/polaris/pull/3649#discussion_r2790132232
##########
helm/polaris/artifacthub-repo.yml:
##########
@@ -17,5 +17,8 @@
# under the License.
#
-# See https://bugs.openjdk.org/browse/JDK-8349545
-org.jboss.weld.bootstrap.concurrentDeployment=false
+# Artifact Hub repository metadata file
+repositoryID: apache-polaris
Review Comment:
TIL - thx :+1:
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Feat/publish artifact hub [polaris]
Posted by "dimas-b (via GitHub)" <gi...@apache.org>.
dimas-b commented on code in PR #3649:
URL: https://github.com/apache/polaris/pull/3649#discussion_r2790139334
##########
helm/polaris/artifacthub-repo.yml:
##########
@@ -17,5 +17,8 @@
# under the License.
#
-# See https://bugs.openjdk.org/browse/JDK-8349545
-org.jboss.weld.bootstrap.concurrentDeployment=false
+# Artifact Hub repository metadata file
+repositoryID: apache-polaris
+owners:
+ - name: Apache Polaris PMC
Review Comment:
So, I think this is the only concern that is worth fixing before we merge. IMHO, it is good to fix it to avoid making the impression that the project has graduated.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Feat/publish artifact hub [polaris]
Posted by "adutra (via GitHub)" <gi...@apache.org>.
adutra commented on PR #3649:
URL: https://github.com/apache/polaris/pull/3649#issuecomment-3870890789
Dev ML discussion: https://lists.apache.org/thread/d19xs5ok7xmd43tffq4lblxgqymf7bn0
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Feat/publish artifact hub [polaris]
Posted by "Subham-KRLX (via GitHub)" <gi...@apache.org>.
Subham-KRLX commented on code in PR #3649:
URL: https://github.com/apache/polaris/pull/3649#discussion_r2766801848
##########
helm/polaris/artifacthub-repo.yml:
##########
Review 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.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Feat/publish artifact hub [polaris]
Posted by "adutra (via GitHub)" <gi...@apache.org>.
adutra commented on code in PR #3649:
URL: https://github.com/apache/polaris/pull/3649#discussion_r2781756590
##########
helm/polaris/artifacthub-repo.yml:
##########
Review Comment:
Doesn't this file need to be copied to the chart repository's root, at the same level as `index.yaml`?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Feat/publish artifact hub [polaris]
Posted by "dimas-b (via GitHub)" <gi...@apache.org>.
dimas-b commented on code in PR #3649:
URL: https://github.com/apache/polaris/pull/3649#discussion_r2799634375
##########
helm/polaris/Chart.yaml:
##########
@@ -37,3 +37,20 @@ maintainers: # Maintainers section is kept only to satisfy the Helm Chart Spec
- name: gerrit-k
- name: snazy
- name: flyrain
+annotations:
+ artifacthub.io/links: |
+ - name: Homepage
+ url: https://polaris.apache.org/
+ - name: Source Code
+ url: https://github.com/apache/polaris
+ - name: Documentation
+ url: https://polaris.apache.org/
Review Comment:
It looks like we went back to the general docs URL... @MonkeyCanCode : are you ok with that?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Feat/publish artifact hub [polaris]
Posted by "adutra (via GitHub)" <gi...@apache.org>.
adutra commented on code in PR #3649:
URL: https://github.com/apache/polaris/pull/3649#discussion_r2799704092
##########
helm/polaris/Chart.yaml:
##########
@@ -37,3 +37,20 @@ maintainers: # Maintainers section is kept only to satisfy the Helm Chart Spec
- name: gerrit-k
- name: snazy
- name: flyrain
+annotations:
+ artifacthub.io/links: |
+ - name: Homepage
+ url: https://polaris.apache.org/
+ - name: Source Code
+ url: https://github.com/apache/polaris
+ - name: Documentation
+ url: https://polaris.apache.org/
Review Comment:
I would say we can have the general URL by default, but overwrite it during release with a version-specific URL.
Let me ping @pingtimeout (no pun intended) โ do you think we could include this extra step in the release workflow?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Feat/publish artifact hub [polaris]
Posted by "dimas-b (via GitHub)" <gi...@apache.org>.
dimas-b commented on code in PR #3649:
URL: https://github.com/apache/polaris/pull/3649#discussion_r2789892921
##########
helm/polaris/Chart.yaml:
##########
@@ -37,3 +37,15 @@ maintainers: # Maintainers section is kept only to satisfy the Helm Chart Spec
- name: gerrit-k
- name: snazy
- name: flyrain
+annotations:
+ artifacthub.io/links: |
+ - name: Homepage
+ url: https://polaris.apache.org/
+ - name: Source Code
+ url: https://github.com/apache/polaris
+ - name: Documentation
+ url: https://polaris.apache.org/releases/1.3.0/
Review Comment:
Do we have to reference a specific version here?
Note: it's not aligned with `appVersion` above ๐ค
Why not just `https://polaris.apache.org`?
##########
helm/polaris/artifacthub-repo.yml:
##########
@@ -17,5 +17,8 @@
# under the License.
#
-# See https://bugs.openjdk.org/browse/JDK-8349545
-org.jboss.weld.bootstrap.concurrentDeployment=false
+# Artifact Hub repository metadata file
+repositoryID: apache-polaris
+owners:
+ - name: Apache Polaris PMC
Review Comment:
Technically, it's PPMC (still incubating)
##########
helm/polaris/artifacthub-repo.yml:
##########
@@ -17,5 +17,8 @@
# under the License.
#
-# See https://bugs.openjdk.org/browse/JDK-8349545
-org.jboss.weld.bootstrap.concurrentDeployment=false
+# Artifact Hub repository metadata file
+repositoryID: apache-polaris
Review Comment:
We have `https://artifacthub.io/packages/helm/apache-polaris/polaris` in Artifact Hub UI... TBH, I do not really know how it should align with the repository ID, but it looks a bit awkward ATM ๐ค ... WDYT?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Feat/publish artifact hub [polaris]
Posted by "MonkeyCanCode (via GitHub)" <gi...@apache.org>.
MonkeyCanCode commented on code in PR #3649:
URL: https://github.com/apache/polaris/pull/3649#discussion_r2785621731
##########
helm/polaris/Chart.yaml:
##########
@@ -37,3 +37,15 @@ maintainers: # Maintainers section is kept only to satisfy the Helm Chart Spec
- name: gerrit-k
- name: snazy
- name: flyrain
+annotations:
+ artifacthub.io/links: |
+ - name: Homepage
+ url: https://polaris.apache.org/
+ - name: Source Code
+ url: https://github.com/apache/polaris
+ - name: Documentation
+ url: https://polaris.apache.org/releases/1.3.0/
+ - name: Support
+ url: https://github.com/apache/polaris/issues
+ artifacthub.io/license: Apache-2.0
+ artifacthub.io/category: database
Review Comment:
Yes. We would either go with database or storage. Or we can leave empty and it will be auto determined on their end.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Feat/publish artifact hub [polaris]
Posted by "Subham-KRLX (via GitHub)" <gi...@apache.org>.
Subham-KRLX commented on code in PR #3649:
URL: https://github.com/apache/polaris/pull/3649#discussion_r2766801848
##########
helm/polaris/artifacthub-repo.yml:
##########
Review Comment:
Thanks for catching that! 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.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Feat/publish artifact hub [polaris]
Posted by "Subham-KRLX (via GitHub)" <gi...@apache.org>.
Subham-KRLX commented on PR #3649:
URL: https://github.com/apache/polaris/pull/3649#issuecomment-3855309379
Thanks @adutra for the legal insights. It is great to confirm this aligns with the release distribution policy.
Regarding the repository ID I believe `apache-polaris` is the right choice. It is consistent with other Apache projects on Artifact Hub (e.g., `apache-airflow`) which helps with consistency and discoverability.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Feat/publish artifact hub [polaris]
Posted by "adutra (via GitHub)" <gi...@apache.org>.
adutra commented on PR #3649:
URL: https://github.com/apache/polaris/pull/3649#issuecomment-3870815084
The changes in this PR look good to me, however this is just the very first step in a long path to ArtifactHub publication, afaict.
I think that before merging, it would be good to discuss on the ML about the next steps:
## Registration
According to the [Artifact Hub documentation](https://artifacthub.io/docs/topics/repositories/helm-charts/), for a Helm chart to appear on Artifact Hub: someone must manually register the repository in the Artifact Hub control panel (by signing in and adding the repository URL).
**This will require manual intervention**: @jbonofre could you handle that?
## Release Workflow
**The release scripts will need to be modified** as follows:
* The `Chart.yaml` file would need to be edited and its ArtifactHub annotations (e.g. documentation links) updated according to the version being released.
* The `artifacthub-repo.yml` file needs to be copied to the Helm chart repository root (same level as `index.yaml`) during the release process. (This file enables [Verified publisher](https://artifacthub.io/docs/topics/repositories/#verified-publisher) and [Ownership claim](https://artifacthub.io/docs/topics/repositories/#ownership-claim) features.)
FYI @pingtimeout @snazy
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Feat/publish artifact hub [polaris]
Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on PR #3649:
URL: https://github.com/apache/polaris/pull/3649#issuecomment-3871309007
I replied on the mailing list. It's totally fine to publish artifacts on non ASF resources (like this or DockerHub) as soon as the artifact is ASF compliant.
In terms of release process, I guess we have to ask infra to create account and provide GitHub secrets to us.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Feat/publish artifact hub [polaris]
Posted by "dimas-b (via GitHub)" <gi...@apache.org>.
dimas-b commented on code in PR #3649:
URL: https://github.com/apache/polaris/pull/3649#discussion_r2765554428
##########
artifacthub-repo.yml:
##########
Review Comment:
Does this file need to be in the source tree root?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Feat/publish artifact hub [polaris]
Posted by "adutra (via GitHub)" <gi...@apache.org>.
adutra commented on PR #3649:
URL: https://github.com/apache/polaris/pull/3649#issuecomment-3909533907
Thank you @Subham-KRLX for your contribution!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Feat/publish artifact hub [polaris]
Posted by "adutra (via GitHub)" <gi...@apache.org>.
adutra merged PR #3649:
URL: https://github.com/apache/polaris/pull/3649
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Feat/publish artifact hub [polaris]
Posted by "dimas-b (via GitHub)" <gi...@apache.org>.
dimas-b commented on code in PR #3649:
URL: https://github.com/apache/polaris/pull/3649#discussion_r2790139334
##########
helm/polaris/artifacthub-repo.yml:
##########
@@ -17,5 +17,8 @@
# under the License.
#
-# See https://bugs.openjdk.org/browse/JDK-8349545
-org.jboss.weld.bootstrap.concurrentDeployment=false
+# Artifact Hub repository metadata file
+repositoryID: apache-polaris
+owners:
+ - name: Apache Polaris PMC
Review Comment:
So, I think this is the only concern that is worth fixing before we merge. IMHO, it is good to fix it to avoid making the (erroneous) impression that the project has graduated.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Feat/publish artifact hub [polaris]
Posted by "adutra (via GitHub)" <gi...@apache.org>.
adutra commented on code in PR #3649:
URL: https://github.com/apache/polaris/pull/3649#discussion_r2790123425
##########
helm/polaris/artifacthub-repo.yml:
##########
@@ -17,5 +17,8 @@
# under the License.
#
-# See https://bugs.openjdk.org/browse/JDK-8349545
-org.jboss.weld.bootstrap.concurrentDeployment=false
+# Artifact Hub repository metadata file
+repositoryID: apache-polaris
Review Comment:
Heh it's fine ๐ The repository ID doesn't show up in the URL, it's an internal thing. The URL is (afaict) the concatenation of the org ID `apache-polaris` with the chart name `polaris` (as seen in Chart.yaml).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Feat/publish artifact hub [polaris]
Posted by "Subham-KRLX (via GitHub)" <gi...@apache.org>.
Subham-KRLX commented on code in PR #3649:
URL: https://github.com/apache/polaris/pull/3649#discussion_r2767371342
##########
helm/polaris/Chart.yaml:
##########
@@ -37,3 +37,20 @@ maintainers: # Maintainers section is kept only to satisfy the Helm Chart Spec
- name: gerrit-k
- name: snazy
- name: flyrain
+annotations:
+ artifacthub.io/links: |
+ - name: Homepage
+ url: https://polaris.apache.org/
+ - name: Source Code
+ url: https://github.com/apache/polaris
+ - name: Documentation
+ url: https://polaris.apache.org/
+ - name: Support
+ url: https://github.com/apache/polaris/issues
+ artifacthub.io/license: Apache-2.0
+ artifacthub.io/category: big-data
+ artifacthub.io/screenshots: |
+ - title: Polaris Architecture
Review 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.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Feat/publish artifact hub [polaris]
Posted by "Subham-KRLX (via GitHub)" <gi...@apache.org>.
Subham-KRLX commented on PR #3649:
URL: https://github.com/apache/polaris/pull/3649#issuecomment-3843085874
Hello @adutra all CI checks are now passing. I have added the Artifact Hub metadata and the required license header. This is ready for review.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Feat/publish artifact hub [polaris]
Posted by "MonkeyCanCode (via GitHub)" <gi...@apache.org>.
MonkeyCanCode commented on code in PR #3649:
URL: https://github.com/apache/polaris/pull/3649#discussion_r2785621731
##########
helm/polaris/Chart.yaml:
##########
@@ -37,3 +37,15 @@ maintainers: # Maintainers section is kept only to satisfy the Helm Chart Spec
- name: gerrit-k
- name: snazy
- name: flyrain
+annotations:
+ artifacthub.io/links: |
+ - name: Homepage
+ url: https://polaris.apache.org/
+ - name: Source Code
+ url: https://github.com/apache/polaris
+ - name: Documentation
+ url: https://polaris.apache.org/releases/1.3.0/
+ - name: Support
+ url: https://github.com/apache/polaris/issues
+ artifacthub.io/license: Apache-2.0
+ artifacthub.io/category: database
Review Comment:
Yes. We would either go with database or storage. Or we can lave empty and it will be auto determined on their end.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Feat/publish artifact hub [polaris]
Posted by "adutra (via GitHub)" <gi...@apache.org>.
adutra commented on code in PR #3649:
URL: https://github.com/apache/polaris/pull/3649#discussion_r2790125927
##########
helm/polaris/Chart.yaml:
##########
@@ -37,3 +37,15 @@ maintainers: # Maintainers section is kept only to satisfy the Helm Chart Spec
- name: gerrit-k
- name: snazy
- name: flyrain
+annotations:
+ artifacthub.io/links: |
+ - name: Homepage
+ url: https://polaris.apache.org/
+ - name: Source Code
+ url: https://github.com/apache/polaris
+ - name: Documentation
+ url: https://polaris.apache.org/releases/1.3.0/
Review Comment:
I think this will have to be updated at every release.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Feat/publish artifact hub [polaris]
Posted by "Subham-KRLX (via GitHub)" <gi...@apache.org>.
Subham-KRLX commented on code in PR #3649:
URL: https://github.com/apache/polaris/pull/3649#discussion_r2796847590
##########
helm/polaris/Chart.yaml:
##########
@@ -37,3 +37,15 @@ maintainers: # Maintainers section is kept only to satisfy the Helm Chart Spec
- name: gerrit-k
- name: snazy
- name: flyrain
+annotations:
+ artifacthub.io/links: |
+ - name: Homepage
+ url: https://polaris.apache.org/
+ - name: Source Code
+ url: https://github.com/apache/polaris
+ - name: Documentation
+ url: https://polaris.apache.org/releases/1.3.0/
Review Comment:
I have addressed the feedback by updating the nomenclature to "Apache Polaris PPMC" to correctly reflect the project's incubation status and simplified the documentation URL to https://polaris.apache.org/ to avoid needing version-specific updates with each release.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Feat/publish artifact hub [polaris]
Posted by "MonkeyCanCode (via GitHub)" <gi...@apache.org>.
MonkeyCanCode commented on code in PR #3649:
URL: https://github.com/apache/polaris/pull/3649#discussion_r2766864907
##########
helm/polaris/Chart.yaml:
##########
@@ -37,3 +37,20 @@ maintainers: # Maintainers section is kept only to satisfy the Helm Chart Spec
- name: gerrit-k
- name: snazy
- name: flyrain
+annotations:
+ artifacthub.io/links: |
+ - name: Homepage
+ url: https://polaris.apache.org/
+ - name: Source Code
+ url: https://github.com/apache/polaris
+ - name: Documentation
+ url: https://polaris.apache.org/
+ - name: Support
+ url: https://github.com/apache/polaris/issues
+ artifacthub.io/license: Apache-2.0
+ artifacthub.io/category: big-data
+ artifacthub.io/screenshots: |
+ - title: Polaris Architecture
Review Comment:
Curious, why these screenshots as they are blog specific?
##########
helm/polaris/Chart.yaml:
##########
@@ -37,3 +37,20 @@ maintainers: # Maintainers section is kept only to satisfy the Helm Chart Spec
- name: gerrit-k
- name: snazy
- name: flyrain
+annotations:
+ artifacthub.io/links: |
+ - name: Homepage
+ url: https://polaris.apache.org/
+ - name: Source Code
+ url: https://github.com/apache/polaris
+ - name: Documentation
+ url: https://polaris.apache.org/
+ - name: Support
+ url: https://github.com/apache/polaris/issues
+ artifacthub.io/license: Apache-2.0
+ artifacthub.io/category: big-data
Review 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.
```
##########
helm/polaris/Chart.yaml:
##########
@@ -37,3 +37,20 @@ maintainers: # Maintainers section is kept only to satisfy the Helm Chart Spec
- name: gerrit-k
- name: snazy
- name: flyrain
+annotations:
+ artifacthub.io/links: |
+ - name: Homepage
+ url: https://polaris.apache.org/
+ - name: Source Code
+ url: https://github.com/apache/polaris
+ - name: Documentation
+ url: https://polaris.apache.org/
Review 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.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Feat/publish artifact hub [polaris]
Posted by "adutra (via GitHub)" <gi...@apache.org>.
adutra commented on PR #3649:
URL: https://github.com/apache/polaris/pull/3649#issuecomment-3854954161
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 `apache-polaris`?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Feat/publish artifact hub [polaris]
Posted by "adutra (via GitHub)" <gi...@apache.org>.
adutra commented on code in PR #3649:
URL: https://github.com/apache/polaris/pull/3649#discussion_r2783961286
##########
helm/polaris/Chart.yaml:
##########
@@ -37,3 +37,15 @@ maintainers: # Maintainers section is kept only to satisfy the Helm Chart Spec
- name: gerrit-k
- name: snazy
- name: flyrain
+annotations:
+ artifacthub.io/links: |
+ - name: Homepage
+ url: https://polaris.apache.org/
+ - name: Source Code
+ url: https://github.com/apache/polaris
+ - name: Documentation
+ url: https://polaris.apache.org/releases/1.3.0/
+ - name: Support
+ url: https://github.com/apache/polaris/issues
+ artifacthub.io/license: Apache-2.0
+ artifacthub.io/category: database
Review Comment:
FYI I thought initially there could be a better category, but in fact we don't have better options:
https://artifacthub.io/docs/topics/annotations/helm/
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org