You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by "geomagilles (via GitHub)" <gi...@apache.org> on 2023/07/10 13:28:55 UTC
[GitHub] [pulsar-helm-chart] geomagilles opened a new pull request, #367: Update README.md
geomagilles opened a new pull request, #367:
URL: https://github.com/apache/pulsar-helm-chart/pull/367
### Motivation
this repo is not actively maintained and has many open issues - it gives a bad impression to a newcomer regarding the Pulsar community.
### Modifications
I propose to remove the "Official" term from the description of this repository and added a warning section to manage expectations.
### Verifying this change
- not 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: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar-helm-chart] tisonkun commented on pull request #367: Update README.md
Posted by "tisonkun (via GitHub)" <gi...@apache.org>.
tisonkun commented on PR #367:
URL: https://github.com/apache/pulsar-helm-chart/pull/367#issuecomment-1677764259
I'm going to send a simila PR according to the lazy consensus on https://lists.apache.org/thread/zrqs973cbvm7szs8kktgbgfohs8gdrzp.
As mentioned in the thread:
> I totally appreciate anyone who is willing to maintain the pulsar-helm-chart, but let's do not block the description updates by such an argument. Instead, update the description, and change it back when the development and maintenance really happen. It also reflects the low traffic during the past few years.
--
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: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar-helm-chart] merlimat commented on pull request #367: Update README.md
Posted by "merlimat (via GitHub)" <gi...@apache.org>.
merlimat commented on PR #367:
URL: https://github.com/apache/pulsar-helm-chart/pull/367#issuecomment-1630409999
@eolivelli @lhotari The current problems with the Helm chart are not something that can be easily fixed, unless significant investment is committed to the effort.
In my view:
* The Helm chart is lagging behind Pulsar releases with no ETAs on whether the "current" release will be available
* Many of the feature combinations are not working out of the box, because of changes in K8S APIS, compatibility with different cloud providers, etc..
* The result is that, currently, it's very difficult to get anything other than a trivial basic clusters working out of the box.
As per the source of these issues, I think it comes that mostly everyone uses different means to deploy Pulsar in Kubernetes (eg: operators or forks of the original Helm chart), therefore this Helm Chart is practically untested by most of the people working on the project.
For a user, though, we are promising that this Helm chart is the best preferred way to deploy Pulsar in Kubernetes, while in fact it does not work out of the box for the majority of the users (as you can see from the number of people getting stuck and asking questions in Slack).
So, again in my view, there are 2 options:
1. Set the expectations on the "real" status of this Helm chart
2. Invest significant resources to make it work well, in the way users would expect from an "officially supported" component.
Is there anyone who has the bandwidth to commit for making (2) happen?
--
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: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar-helm-chart] tisonkun commented on a diff in pull request #367: Update README.md
Posted by "tisonkun (via GitHub)" <gi...@apache.org>.
tisonkun commented on code in PR #367:
URL: https://github.com/apache/pulsar-helm-chart/pull/367#discussion_r1259639469
##########
README.md:
##########
@@ -18,12 +18,14 @@
under the License.
-->
-# Official Apache Pulsar Helm Chart
+# Apache Pulsar Helm Chart
-This is the officially supported Helm Chart for installing Apache Pulsar on Kubernetes.
+This is a Helm Chart for installing Apache Pulsar on Kubernetes.
Read [Deploying Pulsar on Kubernetes](http://pulsar.apache.org/docs/deploy-kubernetes/) for more details.
+> :warning: This helm chart is updated outside of the regular Pulsar release cycle and might lag behind a bit. It only supports basic Kubernetes features. Ultimately, it's meant as a template and starting point for a Kubernetes deployment. In many cases, it would require some customizations.
Review Comment:
```suggestion
> :warning: This helm chart is updated outside of the regular Pulsar release cycle and might lag behind a bit. It only supports basic Kubernetes features now. Currently, it can be used as no more than a template and starting point for a Kubernetes deployment. In many cases, it would require some customizations.
```
--
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: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar-helm-chart] lhotari commented on pull request #367: Update README.md
Posted by "lhotari (via GitHub)" <gi...@apache.org>.
lhotari commented on PR #367:
URL: https://github.com/apache/pulsar-helm-chart/pull/367#issuecomment-1632727363
I'm closing this PR since Apache projects make project decisions on the mailing list.
--
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: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar-helm-chart] lhotari commented on pull request #367: Update README.md
Posted by "lhotari (via GitHub)" <gi...@apache.org>.
lhotari commented on PR #367:
URL: https://github.com/apache/pulsar-helm-chart/pull/367#issuecomment-1631328957
@merlimat @tisonkun Could either of you bring this issue to the dev mailing list for discussion so that the community is aware of this situation and we collectively decide on the resolution. Thanks!
--
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: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar-helm-chart] lhotari closed pull request #367: Update README.md
Posted by "lhotari (via GitHub)" <gi...@apache.org>.
lhotari closed pull request #367: Update README.md
URL: https://github.com/apache/pulsar-helm-chart/pull/367
--
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: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org