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

2021-12-10 Thread GitBox
Carmezim commented on pull request #130: URL: https://github.com/apache/pulsar-helm-chart/pull/130#issuecomment-991129203 Hey guys, like @mkoertgen pointed out BK `v4.12.2` has been added to pulsar `v2.8.0` release, is there any other blockers to merge that PR and upgrade the chart?

[GitHub] [pulsar-helm-chart] lhotari opened a new pull request #187: Add LOG4J_FORMAT_MSG_NO_LOOKUPS=true as mitigation for Pulsar Functions process runtime

2021-12-11 Thread GitBox
lhotari opened a new pull request #187: URL: https://github.com/apache/pulsar-helm-chart/pull/187 ### Motivation - required for disabling the vulnerable feature in Pulsar Functions when using the process runtime ### Additional context - please notice that for mitigating

[GitHub] [pulsar-helm-chart] lhotari closed pull request #187: Add LOG4J_FORMAT_MSG_NO_LOOKUPS=true as mitigation for Pulsar Functions process runtime

2021-12-11 Thread GitBox
lhotari closed pull request #187: URL: https://github.com/apache/pulsar-helm-chart/pull/187 -- 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 #187: Add LOG4J_FORMAT_MSG_NO_LOOKUPS=true as mitigation for Pulsar Functions process runtime

2021-12-11 Thread GitBox
lhotari commented on pull request #187: URL: https://github.com/apache/pulsar-helm-chart/pull/187#issuecomment-991752452 Well, just realized that the Apache Pulsar Helm Chart doesn't provide an option for using the process runtime. I don't think this PR is needed. The previous workaround i

[GitHub] [pulsar-helm-chart] lhotari commented on pull request #187: Add LOG4J_FORMAT_MSG_NO_LOOKUPS=true as mitigation for Pulsar Functions process runtime

2021-12-11 Thread GitBox
lhotari commented on pull request #187: URL: https://github.com/apache/pulsar-helm-chart/pull/187#issuecomment-991752452 Well, just realized that the Apache Pulsar Helm Chart doesn't provide an option for using the process runtime. I don't think this PR is needed. The previous workaround i

[GitHub] [pulsar-helm-chart] lhotari closed pull request #187: Add LOG4J_FORMAT_MSG_NO_LOOKUPS=true as mitigation for Pulsar Functions process runtime

2021-12-11 Thread GitBox
lhotari closed pull request #187: URL: https://github.com/apache/pulsar-helm-chart/pull/187 -- 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] MarvinCai removed a comment on issue #154: Grafana data lost once pod restart.

2021-12-11 Thread GitBox
MarvinCai removed a comment on issue #154: URL: https://github.com/apache/pulsar-helm-chart/issues/154#issuecomment-979751473 I think you mean loki and prometheus, I'll work on adding option to enable persistent storage for loki and prometheus. -- This is an automated message from the A

[GitHub] [pulsar-helm-chart] MarvinCai commented on a change in pull request #155: Add grafana data persistence support.

2021-12-12 Thread GitBox
MarvinCai commented on a change in pull request #155: URL: https://github.com/apache/pulsar-helm-chart/pull/155#discussion_r767236120 ## File path: charts/pulsar/templates/grafana-deployment.yaml ## @@ -88,4 +91,20 @@ spec: name: "{{ template "pulsar.fullname" .

[GitHub] [pulsar-helm-chart] sijie commented on a change in pull request #181: [Istio] Kill istio proxy when container has completed

2021-12-13 Thread GitBox
sijie commented on a change in pull request #181: URL: https://github.com/apache/pulsar-helm-chart/pull/181#discussion_r768211745 ## File path: charts/pulsar/templates/bookkeeper-cluster-initialize.yaml ## @@ -69,6 +69,9 @@ spec: {{- end }} bin

[GitHub] [pulsar-helm-chart] wangshu3000 commented on pull request #182: Fixes #177 Fix indentation of component, as it should be under the la…

2021-12-13 Thread GitBox
wangshu3000 commented on pull request #182: URL: https://github.com/apache/pulsar-helm-chart/pull/182#issuecomment-993087126 Could anyone please merge this PR if no issue? Thanks. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitH

[GitHub] [pulsar-helm-chart] wangshu3000 commented on pull request #183: Update ingress api version, extension/v1beta1 will not be supported i…

2021-12-13 Thread GitBox
wangshu3000 commented on pull request #183: URL: https://github.com/apache/pulsar-helm-chart/pull/183#issuecomment-993089457 Could anybody in the team please review this PR? Thanks. -- This is an automated message from the Apache Git Service. To respond to the message, please log on t

[GitHub] [pulsar-client-node] hrsakai merged pull request #184: Add tarball files to .gitignore

2021-12-13 Thread GitBox
hrsakai merged pull request #184: URL: https://github.com/apache/pulsar-client-node/pull/184 -- 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-u

[GitHub] [pulsar-client-node] massakam opened a new pull request #187: Bump the master version to 1.6.0

2021-12-13 Thread GitBox
massakam opened a new pull request #187: URL: https://github.com/apache/pulsar-client-node/pull/187 I have already created branch-1.5 to perform the release, so bump the master version to 1.6.0-rc.0. -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [pulsar-client-node] hrsakai commented on issue #136: Support message encryption/decryption

2021-12-14 Thread GitBox
hrsakai commented on issue #136: URL: https://github.com/apache/pulsar-client-node/issues/136#issuecomment-994165495 Fixed by https://github.com/apache/pulsar-client-node/pull/138 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Git

[GitHub] [pulsar-client-node] hrsakai closed issue #136: Support message encryption/decryption

2021-12-14 Thread GitBox
hrsakai closed issue #136: URL: https://github.com/apache/pulsar-client-node/issues/136 -- 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-unsubs

[GitHub] [pulsar-client-node] hrsakai closed issue #98: Support getRedeliveryCount of Message

2021-12-14 Thread GitBox
hrsakai closed issue #98: URL: https://github.com/apache/pulsar-client-node/issues/98 -- 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-unsubscr

[GitHub] [pulsar-client-node] hrsakai commented on issue #98: Support getRedeliveryCount of Message

2021-12-14 Thread GitBox
hrsakai commented on issue #98: URL: https://github.com/apache/pulsar-client-node/issues/98#issuecomment-994170279 1.2.0` has already been released, so I closed this issue. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [pulsar-client-node] hrsakai edited a comment on issue #98: Support getRedeliveryCount of Message

2021-12-14 Thread GitBox
hrsakai edited a comment on issue #98: URL: https://github.com/apache/pulsar-client-node/issues/98#issuecomment-994170279 `1.2.0` has already been released, so I closed this issue. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

[GitHub] [pulsar-client-node] hrsakai closed issue #62: Listening for messages

2021-12-14 Thread GitBox
hrsakai closed issue #62: URL: https://github.com/apache/pulsar-client-node/issues/62 -- 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-unsubscr

[GitHub] [pulsar-client-node] hrsakai commented on issue #62: Listening for messages

2021-12-14 Thread GitBox
hrsakai commented on issue #62: URL: https://github.com/apache/pulsar-client-node/issues/62#issuecomment-994185872 listener has already been supported, so I closed this issue. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [pulsar-client-node] hrsakai merged pull request #187: Bump the master version to 1.6.0

2021-12-14 Thread GitBox
hrsakai merged pull request #187: URL: https://github.com/apache/pulsar-client-node/pull/187 -- 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-u

[GitHub] [pulsar-dotpulsar] usaguerrilla closed pull request #93: Feature/microbatching parallel consumer

2021-12-15 Thread GitBox
usaguerrilla closed pull request #93: URL: https://github.com/apache/pulsar-dotpulsar/pull/93 -- 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-

[GitHub] [pulsar-manager] sourabhaggrawal commented on a change in pull request #434: SAML Integration for Login

2021-12-17 Thread GitBox
sourabhaggrawal commented on a change in pull request #434: URL: https://github.com/apache/pulsar-manager/pull/434#discussion_r771301885 ## File path: src/main/java/org/apache/pulsar/manager/interceptor/WebAppConfigurer.java ## @@ -51,6 +51,7 @@ public void addInterceptors(Int

[GitHub] [pulsar-manager] sourabhaggrawal commented on a change in pull request #434: SAML Integration for Login

2021-12-17 Thread GitBox
sourabhaggrawal commented on a change in pull request #434: URL: https://github.com/apache/pulsar-manager/pull/434#discussion_r771312452 ## File path: src/main/java/org/apache/pulsar/manager/controller/SSOController.java ## @@ -0,0 +1,131 @@ +/** + * Licensed under the Apache L

[GitHub] [pulsar-helm-chart] sdwerwed edited a comment on pull request #127: Antiaffinity with zone topology for high availability

2021-12-19 Thread GitBox
sdwerwed edited a comment on pull request #127: URL: https://github.com/apache/pulsar-helm-chart/pull/127#issuecomment-859123386 Hello @sijie, can you have a look at this PR, let me know if any other action is required from my side. -- This is an automated message from the Apache Git Ser

[GitHub] [pulsar-adapters] aditiwari01 commented on issue #29: Pulsar - Spark adapter for scala 2.11

2021-12-20 Thread GitBox
aditiwari01 commented on issue #29: URL: https://github.com/apache/pulsar-adapters/issues/29#issuecomment-997713940 The current available spark pulsar adapter seems to be unreliable. I've written a reliable connector with addition consideration for back-pressure and rate limit. Currently t

[GitHub] [pulsar-adapters] eolivelli commented on issue #29: Pulsar - Spark adapter for scala 2.11

2021-12-20 Thread GitBox
eolivelli commented on issue #29: URL: https://github.com/apache/pulsar-adapters/issues/29#issuecomment-998040594 > Can raise a PR if that's fine Sure! thanks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [pulsar-client-node] tschmidt64-SC commented on issue #134: Getting error while installing

2021-12-20 Thread GitBox
tschmidt64-SC commented on issue #134: URL: https://github.com/apache/pulsar-client-node/issues/134#issuecomment-998219865 I'm still having this issue to this day. Tried building in a docker container, same issue there. -- This is an automated message from the Apache Git Service. To re

[GitHub] [pulsar-client-node] Matt-Esch commented on issue #99: Authenticating using a Token

2021-12-20 Thread GitBox
Matt-Esch commented on issue #99: URL: https://github.com/apache/pulsar-client-node/issues/99#issuecomment-998386200 Can confirm this is an issue, occurs when upgrading to pulsar > 2.6.0 -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [pulsar-manager] sourabhaggrawal opened a new pull request #435: Fix for Sidebar items not visible to admin and superadmin

2021-12-22 Thread GitBox
sourabhaggrawal opened a new pull request #435: URL: https://github.com/apache/pulsar-manager/pull/435 ### Motivation *All newly added sidebar items (Roles,Users etc) are not visible to superadmin. Also when admin user login, only Token tab is visible to them.* ### M

[GitHub] [pulsar-helm-chart] sdwerwed edited a comment on pull request #127: Antiaffinity with zone topology for high availability

2021-12-22 Thread GitBox
sdwerwed edited a comment on pull request #127: URL: https://github.com/apache/pulsar-helm-chart/pull/127#issuecomment-859123386 Hello @wolfstudy, @sijie, can you have a look at this PR, let me know if any other action is required from my side. -- This is an automated message from the Ap

[GitHub] [pulsar-helm-chart] sdwerwed commented on pull request #127: Antiaffinity with zone topology for high availability

2021-12-22 Thread GitBox
sdwerwed commented on pull request #127: URL: https://github.com/apache/pulsar-helm-chart/pull/127#issuecomment-999727380 Hello @wolfstudy, @sijie, can you have a look at this PR, let me know if any other action is required from my side. -- This is an automated message from the

[GitHub] [pulsar-helm-chart] sdwerwed removed a comment on pull request #127: Antiaffinity with zone topology for high availability

2021-12-22 Thread GitBox
sdwerwed removed a comment on pull request #127: URL: https://github.com/apache/pulsar-helm-chart/pull/127#issuecomment-859123386 Hello @wolfstudy, @sijie, can you have a look at this PR, let me know if any other action is required from my side. -- This is an automated message from the A

[GitHub] [pulsar-client-node] izumo27 commented on issue #99: Authenticating using a Token

2021-12-22 Thread GitBox
izumo27 commented on issue #99: URL: https://github.com/apache/pulsar-client-node/issues/99#issuecomment-127066 @Matt-Esch What kind of environment did you use? Can you tell us the details of the issue? -- This is an automated message from the Apache Git Service. To respond to the m

[GitHub] [pulsar-manager] shiv4289 commented on a change in pull request #434: SAML Integration for Login

2021-12-22 Thread GitBox
shiv4289 commented on a change in pull request #434: URL: https://github.com/apache/pulsar-manager/pull/434#discussion_r774308333 ## File path: src/main/java/org/apache/pulsar/manager/controller/SSOController.java ## @@ -0,0 +1,131 @@ +/** + * Licensed under the Apache License,

[GitHub] [pulsar-manager] shiv4289 commented on a change in pull request #434: SAML Integration for Login

2021-12-22 Thread GitBox
shiv4289 commented on a change in pull request #434: URL: https://github.com/apache/pulsar-manager/pull/434#discussion_r774308333 ## File path: src/main/java/org/apache/pulsar/manager/controller/SSOController.java ## @@ -0,0 +1,131 @@ +/** + * Licensed under the Apache License,

[GitHub] [pulsar-manager] sourabhaggrawal opened a new pull request #436: Allow user to assign tenant as resource to role

2021-12-23 Thread GitBox
sourabhaggrawal opened a new pull request #436: URL: https://github.com/apache/pulsar-manager/pull/436 ### Motivation Improve Tenant/Namespace resource assignment workflow. * When assigning the resource to a role, it only let you choose from Namespace,Tenants,SCHEMA,Functions but

[GitHub] [pulsar-client-node] Matt-Esch commented on issue #99: Authenticating using a Token

2021-12-23 Thread GitBox
Matt-Esch commented on issue #99: URL: https://github.com/apache/pulsar-client-node/issues/99#issuecomment-1000215683 I am attempting an upgrade to node 16 which means fast-forwarding the version of pulsar-client-node we are using from 1.2.0 to 1.4.1, which comes with the minimum pulsar v

[GitHub] [pulsar-helm-chart] samuelzimmermanphillips opened a new issue #188: Unrecognized VM option 'PrintGCTimeStamps' - Unsupported Java 11 GC Flags causing init container fail

2021-12-23 Thread GitBox
samuelzimmermanphillips opened a new issue #188: URL: https://github.com/apache/pulsar-helm-chart/issues/188 Need to update the PULSAR_GC options in the default values to remove flags or add `-XX:+IgnoreUnrecognizedVMOptions`, otherwise init containers fail. See here: https://confl

[GitHub] [pulsar-helm-chart] Technoboy- opened a new pull request #189: Bump to Pulsar 2.7.4

2021-12-23 Thread GitBox
Technoboy- opened a new pull request #189: URL: https://github.com/apache/pulsar-helm-chart/pull/189 Update the Helm Chart according to the release process https://github.com/apache/pulsar/wiki/Release-process#11-release-helm-chart -- This is an automated message from the Apache Git Se

[GitHub] [pulsar-helm-chart] eolivelli commented on a change in pull request #189: Bump to Pulsar 2.7.4

2021-12-23 Thread GitBox
eolivelli commented on a change in pull request #189: URL: https://github.com/apache/pulsar-helm-chart/pull/189#discussion_r774908390 ## File path: charts/pulsar/Chart.yaml ## @@ -18,10 +18,10 @@ # apiVersion: v1 -appVersion: "2.7.2" +appVersion: "2.7.4" description: Apach

[GitHub] [pulsar-client-node] Sraleik commented on issue #99: Authenticating using a Token

2021-12-24 Thread GitBox
Sraleik commented on issue #99: URL: https://github.com/apache/pulsar-client-node/issues/99#issuecomment-1000809752 It is now working for me with this setup: node version: 16.13.0 pulsar-client-node: 1.5.0 apache-pulsar-client(library C++): 2.9.1 with apache-pulsar-cli

[GitHub] [pulsar-client-node] Matt-Esch commented on issue #99: Authenticating using a Token

2021-12-24 Thread GitBox
Matt-Esch commented on issue #99: URL: https://github.com/apache/pulsar-client-node/issues/99#issuecomment-1000845729 Trying with v2.9.1 as per @Sraleik suggestion ``` [2021-12-24T13:39:43.164Z] 2021-12-24 13:39:42.898 INFO [139633915127552] ClientImpl:491 | Closing Pulsar clien

[GitHub] [pulsar-adapters] aditiwari01 opened a new pull request #31: [Issue #29] [pulsar-spark] Adding SparkPulsarReliableReceiver

2021-12-26 Thread GitBox
aditiwari01 opened a new pull request #31: URL: https://github.com/apache/pulsar-adapters/pull/31 Fixes #29 ### Motivation Current pulsar-spark adapter uses spark_streaming_2.10 while scala dependency is 2.11. Apart from this current receiver does not take care about reliabil

[GitHub] [pulsar-adapters] aditiwari01 commented on issue #29: Pulsar - Spark adapter for scala 2.11

2021-12-26 Thread GitBox
aditiwari01 commented on issue #29: URL: https://github.com/apache/pulsar-adapters/issues/29#issuecomment-1001188765 @eolivelli I have raised a PR with a new receiver. I'm yet to add examples/documentation for the same, but would appreciate if you can go through it and provide the reev

[GitHub] [pulsar-adapters] aditiwari01 edited a comment on issue #29: Pulsar - Spark adapter for scala 2.11

2021-12-26 Thread GitBox
aditiwari01 edited a comment on issue #29: URL: https://github.com/apache/pulsar-adapters/issues/29#issuecomment-1001188765 @eolivelli I have raised a PR with a new receiver. I'm yet to add examples/documentation for the same, but would appreciate if you can go through it and provide t

[GitHub] [pulsar-adapters] aditiwari01 commented on issue #29: Pulsar - Spark adapter for scala 2.11

2021-12-26 Thread GitBox
aditiwari01 commented on issue #29: URL: https://github.com/apache/pulsar-adapters/issues/29#issuecomment-1001189132 Also can you approve running workflows so I can check if there are any issues. -- This is an automated message from the Apache Git Service. To respond to the message, plea

[GitHub] [pulsar-adapters] aditiwari01 commented on issue #29: Pulsar - Spark adapter for scala 2.11

2021-12-26 Thread GitBox
aditiwari01 commented on issue #29: URL: https://github.com/apache/pulsar-adapters/issues/29#issuecomment-1001223370 @eolivelli Need a small help. In local I've been building with skip License. Flow is failing because of `Some files do not have the expected license header` How ca

[GitHub] [pulsar-adapters] michaeljmarshall commented on issue #29: Pulsar - Spark adapter for scala 2.11

2021-12-28 Thread GitBox
michaeljmarshall commented on issue #29: URL: https://github.com/apache/pulsar-adapters/issues/29#issuecomment-1002236797 @aditiwari01 - try running `mvn initialize license:format` from the base directory of the project. Let me know if that doesn't work. -- This is an automated message f

[GitHub] [pulsar-adapters] eolivelli commented on a change in pull request #31: [Issue #29] [pulsar-spark] Adding SparkPulsarReliableReceiver

2021-12-28 Thread GitBox
eolivelli commented on a change in pull request #31: URL: https://github.com/apache/pulsar-adapters/pull/31#discussion_r776054579 ## File path: pulsar-spark/src/main/scala/org/apache/pulsar/spark/SparkStreamingReliablePulsarReceiver.scala ## @@ -0,0 +1,158 @@ +package org.apac

[GitHub] [pulsar-manager] sourabhaggrawal opened a new pull request #437: Set read/write access to resource for each role

2021-12-28 Thread GitBox
sourabhaggrawal opened a new pull request #437: URL: https://github.com/apache/pulsar-manager/pull/437 ### Motivation Assign resource level read/write access to each role of a user. This way we can assign multiple resources to a user with read/write permission. * A user may exist

[GitHub] [pulsar-helm-chart] lhotari commented on a change in pull request #189: Bump to Pulsar 2.7.4

2021-12-29 Thread GitBox
lhotari commented on a change in pull request #189: URL: https://github.com/apache/pulsar-helm-chart/pull/189#discussion_r776227429 ## File path: charts/pulsar/Chart.yaml ## @@ -18,7 +18,7 @@ # apiVersion: v1 -appVersion: "2.7.2" +appVersion: "2.7.4" description: Apache Pu

[GitHub] [pulsar-helm-chart] qq516249940 commented on issue #23: Missing arguments in prepare_helm_release.sh script

2021-12-29 Thread GitBox
qq516249940 commented on issue #23: URL: https://github.com/apache/pulsar-helm-chart/issues/23#issuecomment-1002485188 in china how to use prepare_helm_release.sh ??? ```bash curl: (7) Failed to connect to raw.githubusercontent.com port 443: 拒绝连接 generate the token for admin Ge

[GitHub] [pulsar-adapters] aditiwari01 commented on a change in pull request #31: [Issue #29] [pulsar-spark] Adding SparkPulsarReliableReceiver

2021-12-29 Thread GitBox
aditiwari01 commented on a change in pull request #31: URL: https://github.com/apache/pulsar-adapters/pull/31#discussion_r776248456 ## File path: pulsar-spark/src/main/scala/org/apache/pulsar/spark/SparkStreamingReliablePulsarReceiver.scala ## @@ -0,0 +1,158 @@ +package org.ap

[GitHub] [pulsar-helm-chart] Technoboy- commented on a change in pull request #189: Bump to Pulsar 2.7.4

2021-12-29 Thread GitBox
Technoboy- commented on a change in pull request #189: URL: https://github.com/apache/pulsar-helm-chart/pull/189#discussion_r776559018 ## File path: charts/pulsar/Chart.yaml ## @@ -18,7 +18,7 @@ # apiVersion: v1 -appVersion: "2.7.2" +appVersion: "2.7.4" description: Apache

[GitHub] [pulsar-helm-chart] Technoboy- commented on a change in pull request #189: Bump to Pulsar 2.7.4

2021-12-29 Thread GitBox
Technoboy- commented on a change in pull request #189: URL: https://github.com/apache/pulsar-helm-chart/pull/189#discussion_r776559787 ## File path: charts/pulsar/Chart.yaml ## @@ -18,7 +18,7 @@ # apiVersion: v1 -appVersion: "2.7.2" +appVersion: "2.7.4" description: Apache

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

2021-12-29 Thread GitBox
thomasechen commented on pull request #161: URL: https://github.com/apache/pulsar-helm-chart/pull/161#issuecomment-1002887755 Pulasr 2.9.1 was released, could we have the helm that support these two releases? -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [pulsar-helm-chart] lhotari merged pull request #189: Bump to Pulsar 2.7.4

2021-12-29 Thread GitBox
lhotari merged pull request #189: URL: https://github.com/apache/pulsar-helm-chart/pull/189 -- 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-client-node] zhaoyajun2009 opened a new issue #188: When installing the Pulsar Node.js client, the operation steps need to be optimized

2021-12-31 Thread GitBox
zhaoyajun2009 opened a new issue #188: URL: https://github.com/apache/pulsar-client-node/issues/188 **In step “Prerequisites”, in addition to the c++ library, you need to install gcc and g++ dependency, otherwise the following problems will occur in step “Install pulsar-client in your

[GitHub] [pulsar-manager] rsrini7 commented on issue #429: Bug: default user pulsar/pulsar does not have permission to create a new environment

2021-12-31 Thread GitBox
rsrini7 commented on issue #429: URL: https://github.com/apache/pulsar-manager/issues/429#issuecomment-1003401420 Tried something like root@pulsar-server:/pulsar/bin# ./pulsar-admin namespaces grant-permission public/default --actions produce,consume --role '*' but no success.

[GitHub] [pulsar-helm-chart] 315157973 opened a new pull request #190: Bump to Pulsar 2.8.2

2022-01-01 Thread GitBox
315157973 opened a new pull request #190: URL: https://github.com/apache/pulsar-helm-chart/pull/190 ### Motivation Bump to Pulsar 2.8.2 @codelipenghui Please help me push the docker images to the docker hub, and then review this PR. Thanks. -- This is an automated message from

[GitHub] [pulsar-helm-chart] yuweisung opened a new pull request #191: added arm64 in common_auth.sh

2022-01-01 Thread GitBox
yuweisung opened a new pull request #191: URL: https://github.com/apache/pulsar-helm-chart/pull/191 Fixes # ### Motivation Helm chart failed to detect mac m1/arm64 ARCH and failed to install pulsarctl from streamnative. pulsarctl version 0.4.0 (The default in the script) h

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

2022-01-02 Thread GitBox
eolivelli commented on pull request #161: URL: https://github.com/apache/pulsar-helm-chart/pull/161#issuecomment-1003724823 @thomasechen you can override the docker image in your values.yaml file -- This is an automated message from the Apache Git Service. To respond to the message, plea

[GitHub] [pulsar-adapters] aditiwari01 commented on pull request #31: [Issue #29] [pulsar-spark] Adding SparkPulsarReliableReceiver

2022-01-02 Thread GitBox
aditiwari01 commented on pull request #31: URL: https://github.com/apache/pulsar-adapters/pull/31#issuecomment-1003896760 Did anyone get a chance to review this? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

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

2022-01-02 Thread GitBox
lhotari opened a new pull request #192: URL: https://github.com/apache/pulsar-helm-chart/pull/192 ### Motivation 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](https://kubernetes.io/releas

[GitHub] [pulsar-adapters] eolivelli commented on a change in pull request #31: [Issue #29] [pulsar-spark] Adding SparkPulsarReliableReceiver

2022-01-02 Thread GitBox
eolivelli commented on a change in pull request #31: URL: https://github.com/apache/pulsar-adapters/pull/31#discussion_r777321048 ## File path: pulsar-spark/src/main/scala/org/apache/spark/streaming/pulsar/SparkStreamingReliablePulsarReceiver.scala ## @@ -0,0 +1,181 @@ +/** +

[GitHub] [pulsar-adapters] aditiwari01 commented on a change in pull request #31: [Issue #29] [pulsar-spark] Adding SparkPulsarReliableReceiver

2022-01-02 Thread GitBox
aditiwari01 commented on a change in pull request #31: URL: https://github.com/apache/pulsar-adapters/pull/31#discussion_r777326350 ## File path: pulsar-spark/src/main/scala/org/apache/spark/streaming/pulsar/SparkStreamingReliablePulsarReceiver.scala ## @@ -0,0 +1,181 @@ +/**

[GitHub] [pulsar-helm-chart] lhotari opened a new pull request #193: [CI] Add job 45min timeout and cancel duplicate jobs

2022-01-02 Thread GitBox
lhotari opened a new pull request #193: URL: https://github.com/apache/pulsar-helm-chart/pull/193 ### Motivation - CI jobs timeout after several hours and this wastes CI resources - When making multiple changes to a PR job, the previous jobs keep on running. This wastes CI resourc

[GitHub] [pulsar-helm-chart] eolivelli merged pull request #193: [CI] Add job 45min timeout and cancel duplicate jobs

2022-01-03 Thread GitBox
eolivelli merged pull request #193: URL: https://github.com/apache/pulsar-helm-chart/pull/193 -- 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-

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

2022-01-03 Thread GitBox
lhotari commented on pull request #192: URL: https://github.com/apache/pulsar-helm-chart/pull/192#issuecomment-1003979521 > Build failed: > > : INSTALLATION FAILED: unable to build kubernetes objects from release manifest: unable to recognize "": no matches for kind "DaemonSet" in v

[GitHub] [pulsar-helm-chart] eolivelli commented on pull request #180: Use NIOServerCnxnFactory for Zookeeper to fix NPE issues with Pulsar 2.8.0+

2022-01-03 Thread GitBox
eolivelli commented on pull request #180: URL: https://github.com/apache/pulsar-helm-chart/pull/180#issuecomment-1004013791 I am following up with the issue on Zookeeper and I will help in cutting a release as soon as possible. It would be probably 3.7.1 -- This is an automated messa

[GitHub] [pulsar-helm-chart] eolivelli merged pull request #180: Use NIOServerCnxnFactory for Zookeeper to fix NPE issues with Pulsar 2.8.0+

2022-01-03 Thread GitBox
eolivelli merged pull request #180: URL: https://github.com/apache/pulsar-helm-chart/pull/180 -- 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-

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

2022-01-03 Thread GitBox
lhotari commented on pull request #190: URL: https://github.com/apache/pulsar-helm-chart/pull/190#issuecomment-1004035982 It seems that #188 should also be resolved before switching to Pulsar 2.8.x . -- This is an automated message from the Apache Git Service. To respond to the message,

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

2022-01-03 Thread GitBox
lhotari commented on pull request #190: URL: https://github.com/apache/pulsar-helm-chart/pull/190#issuecomment-1004101125 Failures in CI which points to #188. I'll push a fix to this PR. ``` Unrecognized VM option 'PrintGCTimeStamps' Error: Could not create the

[GitHub] [pulsar-helm-chart] lhotari commented on issue #188: Unrecognized VM option 'PrintGCTimeStamps' - Unsupported Java 11 GC Flags causing init container fail

2022-01-03 Thread GitBox
lhotari commented on issue #188: URL: https://github.com/apache/pulsar-helm-chart/issues/188#issuecomment-1004129237 @samuelzimmermanphillips Thanks for reporting. This will be fixed as part of #190. -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [pulsar-helm-chart] lhotari merged pull request #182: Fixes #177 Fix indentation of component, as it should be under the la…

2022-01-03 Thread GitBox
lhotari merged pull request #182: URL: https://github.com/apache/pulsar-helm-chart/pull/182 -- 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 closed issue #177: The indentation of dashboard-ingress.yaml is wrong, when deploying shows component not found.

2022-01-03 Thread GitBox
lhotari closed issue #177: URL: https://github.com/apache/pulsar-helm-chart/issues/177 -- 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-unsubsc

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

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

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

2022-01-03 Thread GitBox
eolivelli commented on pull request #190: URL: https://github.com/apache/pulsar-helm-chart/pull/190#issuecomment-1004370311 Also, why should we ship 2.8 as default instead of 2.9.1? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to G

[GitHub] [pulsar-helm-chart] michaeljmarshall commented on a change in pull request #183: Update ingress api version, extension/v1beta1 will not be supported i…

2022-01-03 Thread GitBox
michaeljmarshall commented on a change in pull request #183: URL: https://github.com/apache/pulsar-helm-chart/pull/183#discussion_r25211 ## File path: charts/pulsar/templates/dashboard-ingress.yaml ## @@ -19,7 +19,11 @@ {{- if .Values.extra.dashboard }} {{- if .Values.d

[GitHub] [pulsar-helm-chart] michaeljmarshall commented on pull request #183: Update ingress api version, extension/v1beta1 will not be supported i…

2022-01-03 Thread GitBox
michaeljmarshall commented on pull request #183: URL: https://github.com/apache/pulsar-helm-chart/pull/183#issuecomment-1004388405 @lhotari - do you know why the CI checks wouldn't have run for this PR? I haven't seen that happen before. -- This is an automated message from the Apache G

[GitHub] [pulsar-helm-chart] wangshu3000 commented on a change in pull request #183: Update ingress api version, extension/v1beta1 will not be supported i…

2022-01-03 Thread GitBox
wangshu3000 commented on a change in pull request #183: URL: https://github.com/apache/pulsar-helm-chart/pull/183#discussion_r70205 ## File path: charts/pulsar/templates/dashboard-ingress.yaml ## @@ -19,7 +19,11 @@ {{- if .Values.extra.dashboard }} {{- if .Values.dashbo

[GitHub] [pulsar-helm-chart] wangshu3000 commented on a change in pull request #183: Update ingress api version, extension/v1beta1 will not be supported i…

2022-01-03 Thread GitBox
wangshu3000 commented on a change in pull request #183: URL: https://github.com/apache/pulsar-helm-chart/pull/183#discussion_r71254 ## File path: charts/pulsar/templates/dashboard-ingress.yaml ## @@ -19,7 +19,11 @@ {{- if .Values.extra.dashboard }} {{- if .Values.dashbo

[GitHub] [pulsar-helm-chart] michaeljmarshall commented on a change in pull request #183: Update ingress api version, extension/v1beta1 will not be supported i…

2022-01-03 Thread GitBox
michaeljmarshall commented on a change in pull request #183: URL: https://github.com/apache/pulsar-helm-chart/pull/183#discussion_r777832255 ## File path: charts/pulsar/templates/dashboard-ingress.yaml ## @@ -19,7 +19,11 @@ {{- if .Values.extra.dashboard }} {{- if .Values.d

[GitHub] [pulsar-helm-chart] wangshu3000 commented on a change in pull request #183: Update ingress api version, extension/v1beta1 will not be supported i…

2022-01-03 Thread GitBox
wangshu3000 commented on a change in pull request #183: URL: https://github.com/apache/pulsar-helm-chart/pull/183#discussion_r777833010 ## File path: charts/pulsar/templates/dashboard-ingress.yaml ## @@ -19,7 +19,11 @@ {{- if .Values.extra.dashboard }} {{- if .Values.dashbo

[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

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

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

2022-01-03 Thread GitBox
lhotari commented on pull request #190: URL: https://github.com/apache/pulsar-helm-chart/pull/190#issuecomment-1004554921 > 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 about

[GitHub] [pulsar-helm-chart] michaeljmarshall closed issue #157: Ingresses still use the extensions/v1beta1 versioned resource

2022-01-03 Thread GitBox
michaeljmarshall closed issue #157: URL: https://github.com/apache/pulsar-helm-chart/issues/157 -- 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: de

[GitHub] [pulsar-helm-chart] michaeljmarshall merged pull request #183: Update ingress api version, extension/v1beta1 will not be supported i…

2022-01-03 Thread GitBox
michaeljmarshall merged pull request #183: URL: https://github.com/apache/pulsar-helm-chart/pull/183 -- 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

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

2022-01-04 Thread GitBox
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

[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

2022-01-04 Thread GitBox
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 merged pull request #194: [CI] Improve change detection by reusing solution from apache/pulsar

2022-01-04 Thread GitBox
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

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

2022-01-04 Thread GitBox
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

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

2022-01-04 Thread GitBox
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] mkoertgen edited a comment on issue #124: pulsar waiting to start: PodInitializing

2022-01-04 Thread GitBox
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

2022-01-04 Thread GitBox
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 commented on issue #124: pulsar waiting to start: PodInitializing

2022-01-04 Thread GitBox
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

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

2022-01-04 Thread GitBox
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

2022-01-04 Thread GitBox
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] lhotari opened a new pull request #195: [CI] Improve logging in CI scripts and add timeouts

2022-01-04 Thread GitBox
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

  1   2   3   4   5   6   7   8   9   10   >