[GitHub] [pulsar-helm-chart] lhotari commented on a change in pull request #130: Bump pulsar 2.8.0

lhotari commented on a change in pull request #130: URL: https://github.com/apache/pulsar-helm-chart/pull/130#discussion_r778610093 ## File path: charts/pulsar/values.yaml ## @@ -278,7 +278,7 @@ zookeeper: replicaCount: 3 updateStrategy: type: RollingUpdate - podMan

[OUTREACH] Jan '22 Edition of 'Happenings in the Neighborhood' is out now

Hello Apache Pulsar Neighbors, Did you know that the ASF just released its "Apache by the digits" blog post and Apache Pulsar is the fifth most popular project by commits? In this issue of Happen

[GitHub] [pulsar-helm-chart] lhotari closed pull request #130: Bump pulsar 2.8.0

lhotari closed pull request #130: URL: https://github.com/apache/pulsar-helm-chart/pull/130 -- 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: dev-un

[GitHub] [pulsar-helm-chart] lhotari commented on pull request #130: Bump pulsar 2.8.0

lhotari commented on pull request #130: URL: https://github.com/apache/pulsar-helm-chart/pull/130#issuecomment-1005454379 superseded by #190 -- 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

[GitHub] [pulsar-helm-chart] lhotari commented on issue #114: Use hooks instead of initialize=true

lhotari commented on issue #114: URL: https://github.com/apache/pulsar-helm-chart/issues/114#issuecomment-1005452447 There's already a PR #138 for resolving this. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the U

[GitHub] [pulsar-helm-chart] lhotari commented on a change in pull request #138: automate initialize

lhotari commented on a change in pull request #138: URL: https://github.com/apache/pulsar-helm-chart/pull/138#discussion_r778603221 ## File path: charts/pulsar/templates/bookkeeper-cluster-initialize.yaml ## @@ -16,7 +16,7 @@ # specific language governing permissions and limit

Re: [DISCUSSION] PIP-132: Include message header size when check maxMessageSize of non-batch message on the client side.

Hi mattison, Yes, this is an alternative way to solve the case of properties is too large. But I think this approach is more complex in coding and introduces new concept and configs, and I don't see the benefit of limiting the header size and payload size separately. Thanks, Haiting Jiang

[GitHub] [pulsar-helm-chart] lhotari closed pull request #161: Bump Pulsar version to 2.8.1

lhotari closed pull request #161: URL: https://github.com/apache/pulsar-helm-chart/pull/161 -- 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: dev-un

[GitHub] [pulsar-helm-chart] lhotari commented on pull request #161: Bump Pulsar version to 2.8.1

lhotari commented on pull request #161: URL: https://github.com/apache/pulsar-helm-chart/pull/161#issuecomment-1005446823 superseded by #190 -- 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

Re: [DISCUSSION] PIP-124: Create init subscription before sending message to DLQ

Thanks for the great comments, Michael. Let me try to clarify some context about the issue that users encountered and the improvement that the proposal wants to Introduce. > Before we get further into the implementation, I'd like to discuss whether the current behavior is the expected behavior, a

退订

退订

Re: [DISCUSSION] PIP-121: Pulsar cluster level auto failover

+1 Penghui On Tue, Jan 4, 2022 at 4:51 PM Hang Chen wrote: > https://github.com/apache/pulsar/issues/13315 > > Pasted below for quoting convenience. > > > ### Motivation > We have geo-replication to support Pulsar cluster level failover. We > can setup Pulsar cluster A as a primary cluster

Re: [VOTE] PIP-130: Apply redelivery backoff policy for ack timeout

+1 (binding) Best, Hang Enrico Olivelli 于2022年1月5日周三 06:52写道： > > +1 (binding) > > Enrico > > Il Mar 4 Gen 2022, 16:19 刘德志 ha scritto: > > > > > > > +1 > > > > Thanks, > > Dezhi Liu

Re: [DISCUSSION] PIP-122: Change loadBalancer default loadSheddingStrategy to ThresholdShedder

Hi Michael, Thanks for you review, it will push a PR to add a test for `ThresholdShedder `. Best, Hang Michael Marshall 于2022年1月5日周三 00:55写道： > > Hi Hang Chen, > > I support changing the default for 2.10. > > However, as far as I can tell, there are unit tests for the > `OverloadShedder` cla

Re: [VOTE] PIP-130: Apply redelivery backoff policy for ack timeout

+1 (binding) Enrico Il Mar 4 Gen 2022, 16:19 刘德志 ha scritto: > > > +1 > > Thanks, > Dezhi Liu

[OUTREACH] Apache Pulsar is #5 in commits for all ASF Projects!

Hello Apache Pulsar Neighbors, In our email this morning we had some exciting news, the ASF has released their "Apache by the digits" blog post and Apache Pulsar is the fifth most popular project by commits! Pretty amazing, eh? Check out our blog post

[GitHub] [pulsar-helm-chart] ckdarby commented on pull request #190: Bump to Pulsar 2.8.2

ckdarby commented on pull request #190: URL: https://github.com/apache/pulsar-helm-chart/pull/190#issuecomment-1005227662 > > Also, why should we ship 2.8 as default instead of 2.9.1? > > I guess it could be about doing one step at a time (2.7.x -> 2.8.x, then 2.8.x->2.9.x) and also

[GitHub] [pulsar-helm-chart] frankjkelly commented on pull request #181: [Istio] Provide ability to kill istio proxy in sidecar when Init Job container has completed

frankjkelly commented on pull request #181: URL: https://github.com/apache/pulsar-helm-chart/pull/181#issuecomment-1005168159 Helm chart 2.7.7 was already released. See https://github.com/apache/pulsar-helm-chart/releases/tag/pulsar-2.7.7 and https://pulsar.apache.org/charts/index.

[GitHub] [pulsar-helm-chart] lhotari commented on issue #114: Use hooks instead of initialize=true

lhotari commented on issue #114: URL: https://github.com/apache/pulsar-helm-chart/issues/114#issuecomment-1005052105 Another solution proposed by @mkoertgen https://github.com/apache/pulsar-helm-chart/issues/124#issuecomment-1004724440 -- This is an automated message from the Apache Gi

[GitHub] [pulsar-helm-chart] lhotari commented on issue #124: pulsar waiting to start: PodInitializing

lhotari commented on issue #124: URL: https://github.com/apache/pulsar-helm-chart/issues/124#issuecomment-1005049161 > To me it looks like the problem has to do with helm-templates expanding empty lines without breaks (`\`) causing the shell call to fail. > Here is what the helm render

[GitHub] [pulsar-helm-chart] lhotari commented on issue #134: Change to PULSAR_GC in values.yaml -XX args for Pulsar 2.8.0 needed

lhotari commented on issue #134: URL: https://github.com/apache/pulsar-helm-chart/issues/134#issuecomment-1005048226 This is handled as part of #190 -- 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

[GitHub] [pulsar-helm-chart] michaeljmarshall commented on pull request #194: [CI] Improve change detection by reusing solution from apache/pulsar

michaeljmarshall commented on pull request #194: URL: https://github.com/apache/pulsar-helm-chart/pull/194#issuecomment-1005035759 +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 specifi

[GitHub] [pulsar-helm-chart] michaeljmarshall merged pull request #166: Workaround kustomize bug in pulsar cluster init

michaeljmarshall merged pull request #166: URL: https://github.com/apache/pulsar-helm-chart/pull/166 -- 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-mai

Re: [DISCUSSION] PIP-122: Change loadBalancer default loadSheddingStrategy to ThresholdShedder

Hi Hang Chen, I support changing the default for 2.10. However, as far as I can tell, there are unit tests for the `OverloadShedder` class but not for the `ThresholdShedder` class. I think we should add unit tests before we change the default. Regarding integration tests, I assume that we implic

[GitHub] [pulsar-helm-chart] lhotari commented on pull request #178: [Performance] Remove -XX:-ResizePLAB JVM option which degrades performance on JDK11

lhotari commented on pull request #178: URL: https://github.com/apache/pulsar-helm-chart/pull/178#issuecomment-1004907742 this change is already combined in #190 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the U

[GitHub] [pulsar-helm-chart] lhotari closed pull request #178: [Performance] Remove -XX:-ResizePLAB JVM option which degrades performance on JDK11

lhotari closed pull request #178: URL: https://github.com/apache/pulsar-helm-chart/pull/178 -- 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: dev-un

[GitHub] [pulsar-helm-chart] lhotari merged pull request #195: [CI] Improve logging in CI scripts and add timeouts

lhotari merged pull request #195: URL: https://github.com/apache/pulsar-helm-chart/pull/195 -- 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: dev-un

RE: [VOTE] PIP-130: Apply redelivery backoff policy for ack timeout

+1 Thanks, Dezhi Liu

[DISCUSSION] PIP-122: Change loadBalancer default loadSheddingStrategy to ThresholdShedder

https://github.com/apache/pulsar/issues/13340 Pasted below for quoting convenience. ### Motivation The ThresholdShedder load balance policy since Pulsar 2.6.0 by https://github.com/apache/pulsar/pull/6772. It can resolve many load balance issues of `OverloadShedder` and works well in many Pu

[GitHub] [pulsar-helm-chart] lhotari commented on pull request #190: Bump to Pulsar 2.8.2

lhotari commented on pull request #190: URL: https://github.com/apache/pulsar-helm-chart/pull/190#issuecomment-1004765915 I created a PR #195 which adds better logging to CI which would help investigating CI failures. I have observed the "ZK TLS Only" CI job failing, presumably with the k

[GitHub] [pulsar-helm-chart] lhotari opened a new pull request #195: [CI] Improve logging in CI scripts and add timeouts

lhotari opened a new pull request #195: URL: https://github.com/apache/pulsar-helm-chart/pull/195 ### Motivation Currently it's hard to investigate CI failures since there isn't sufficient logging. ### Modifications - show Kubernetes events every 15 seconds - dump lo

[GitHub] [pulsar-helm-chart] mkoertgen commented on issue #124: pulsar waiting to start: PodInitializing

mkoertgen commented on issue #124: URL: https://github.com/apache/pulsar-helm-chart/issues/124#issuecomment-1004724440 Probably, a suggestion for the future: Would it not be possible to default the `initialize=true`-value from [Helm's builtin-objects](https://helm.sh/docs/chart_template_g

[GitHub] [pulsar-helm-chart] mkoertgen edited a comment on issue #124: pulsar waiting to start: PodInitializing

mkoertgen edited a comment on issue #124: URL: https://github.com/apache/pulsar-helm-chart/issues/124#issuecomment-1004721707 Strange, because the helm-template looks fine to me. I have no idea where this extra line is coming from - https://github.com/apache/pulsar-helm-chart/blob/mast

[GitHub] [pulsar-helm-chart] mkoertgen commented on issue #124: pulsar waiting to start: PodInitializing

mkoertgen commented on issue #124: URL: https://github.com/apache/pulsar-helm-chart/issues/124#issuecomment-1004721707 Strange, because the helm-template looks fine to me. I have no idea where this extra line is coming from - https://github.com/apache/pulsar-helm-chart/blob/master/char

Re: Pulsar Summit Asia 2021 will be online only

Hi Apache Pulsar Community, Hope that you have all enjoyed a wonderful start of 2022! We are sending this email to update the status of Pulsar Summit Asia 2021. Considering the current situation, Pulsar Summit Asia 2021 will be *online only* instead of hybrid (both in-person and online). The even

[GitHub] [pulsar-helm-chart] mkoertgen commented on issue #124: pulsar waiting to start: PodInitializing

mkoertgen commented on issue #124: URL: https://github.com/apache/pulsar-helm-chart/issues/124#issuecomment-1004718311 To me it looks like the problem has to do with helm-templates expanding empty lines without breaks (`\`) causing the shell call to fail. Here is what the helm renders

[GitHub] [pulsar-helm-chart] mkoertgen edited a comment on issue #124: pulsar waiting to start: PodInitializing

mkoertgen edited a comment on issue #124: URL: https://github.com/apache/pulsar-helm-chart/issues/124#issuecomment-1004706854 Kind of similar problem for me. Reinstalling a pulsar cluster chart=2.7.7 with initialize=true results in brokers / proxies getting stuck waiting for zookeeper.

[GitHub] [pulsar-helm-chart] mkoertgen commented on issue #124: pulsar waiting to start: PodInitializing

mkoertgen commented on issue #124: URL: https://github.com/apache/pulsar-helm-chart/issues/124#issuecomment-1004706854 Kind of similar problem for me. Reinstalling a pulsar cluster chart=2.7.7 with initialize=true results in brokers / proxies getting stuck waiting for zookeeper. I

[GitHub] [pulsar-helm-chart] lhotari commented on pull request #192: [CI] Upgrade k8s to 1.18 and also upgrade helm, kind & chart releaser versions

lhotari commented on pull request #192: URL: https://github.com/apache/pulsar-helm-chart/pull/192#issuecomment-1004658284 Mailing list discussion: https://lists.apache.org/thread/jwhb980svfm8rfbd7grswzb1dzf964ny -- This is an automated message from the Apache Git Service. To respond to

Lifting Kubernetes minimum version requirement for Apache Pulsar Helm Charts from k8s 1.14 to 1.18

Hi all, Currently k8s 1.14 version is used in CI to verify the Helm chart changes. k8s 1.14 became end-of-life 2019-12-11 , over 2 years ago [1]. The oldest maintained version for Kubernetes is 1.20 and it will become end-of-life on 2022-02-28, in less than 2 months from now [2]. There's a PR to

[GitHub] [pulsar-helm-chart] lhotari merged pull request #194: [CI] Improve change detection by reusing solution from apache/pulsar

lhotari merged pull request #194: URL: https://github.com/apache/pulsar-helm-chart/pull/194 -- 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: dev-un

[DISCUSSION] PIP-121: Pulsar cluster level auto failover

https://github.com/apache/pulsar/issues/13315 Pasted below for quoting convenience. ### Motivation We have geo-replication to support Pulsar cluster level failover. We can setup Pulsar cluster A as a primary cluster in data center A, and setup Pulsar cluster B as backup cluster in data cente

[GitHub] [pulsar-helm-chart] lhotari commented on a change in pull request #192: [CI] Upgrade k8s to 1.18 and also upgrade helm, kind & chart releaser versions

lhotari commented on a change in pull request #192: URL: https://github.com/apache/pulsar-helm-chart/pull/192#discussion_r777888049 ## File path: .gitignore ## @@ -16,3 +16,6 @@ charts/**/*.lock PRIVATEKEY PUBLICKEY +.vagrant/ +pulsarctl-amd64-linux.tar.gz +pulsarctl-amd64-

[GitHub] [pulsar-helm-chart] lhotari opened a new pull request #194: [CI] Improve change detection by reusing solution from apache/pulsar

lhotari opened a new pull request #194: URL: https://github.com/apache/pulsar-helm-chart/pull/194 ### Motivation - The built-in GitHub "paths" solution will only consider that latest commit to the branch. This can cause checks to be skipped for some pull requests. ### Modifica