You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/07/18 05:25:51 UTC

[GitHub] [pulsar-helm-chart] mattisonchao opened a new pull request, #277: Bump Apache Pulsar version to 2.9.3

mattisonchao opened a new pull request, #277:
URL: https://github.com/apache/pulsar-helm-chart/pull/277

   ### Motivation
   
   Bump Apache Pulsar version to 2.9.3
   
   ### Modifications
   
   - Bump Apache Pulsar version to 2.9.3
   
   ### Verifying this change
   
   - [x] Make sure that the change passes the CI checks.
   


-- 
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-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar-helm-chart] lhotari commented on a diff in pull request #277: Bump Apache Pulsar version to 2.9.3

Posted by GitBox <gi...@apache.org>.
lhotari commented on code in PR #277:
URL: https://github.com/apache/pulsar-helm-chart/pull/277#discussion_r923370490


##########
charts/pulsar/Chart.yaml:
##########
@@ -18,7 +18,7 @@
 #
 
 apiVersion: v2
-appVersion: "2.9.2"
+appVersion: "2.9.3"
 description: Apache Pulsar Helm chart for Kubernetes
 name: pulsar
 version: 2.9.3

Review Comment:
   There's PR #200 which improves the situation, but that stalled since I didn't receive feedback from @sijie .



-- 
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-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar-helm-chart] mattisonchao commented on pull request #277: Bump Apache Pulsar version to 2.9.3

Posted by GitBox <gi...@apache.org>.
mattisonchao commented on PR #277:
URL: https://github.com/apache/pulsar-helm-chart/pull/277#issuecomment-1187452537

   > I guess tags referencing `2.9.2` should also be updated in `.ci/clusters/values-pulsar-image.yaml` file. It might be better to remove the tags from that file so that it's not necessary to update the file for each release.
   
   I'm not sure about it. maybe I can write the shell to help bump the version with one command.


-- 
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-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar-helm-chart] mattisonchao commented on pull request #277: Bump Apache Pulsar version to 2.9.3

Posted by GitBox <gi...@apache.org>.
mattisonchao commented on PR #277:
URL: https://github.com/apache/pulsar-helm-chart/pull/277#issuecomment-1187442014

   > pulsar_metadata.image.tag
   
   fixed, 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: dev-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar-helm-chart] mattisonchao commented on a diff in pull request #277: Bump Apache Pulsar version to 2.9.3

Posted by GitBox <gi...@apache.org>.
mattisonchao commented on code in PR #277:
URL: https://github.com/apache/pulsar-helm-chart/pull/277#discussion_r923376333


##########
charts/pulsar/Chart.yaml:
##########
@@ -18,7 +18,7 @@
 #
 
 apiVersion: v2
-appVersion: "2.9.2"
+appVersion: "2.9.3"
 description: Apache Pulsar Helm chart for Kubernetes
 name: pulsar
 version: 2.9.3

Review Comment:
   Thanks for your explanation.
   I changed the chart version to 2.9.4



-- 
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-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 #277: Bump Apache Pulsar version to 2.9.3

Posted by GitBox <gi...@apache.org>.
lhotari commented on PR #277:
URL: https://github.com/apache/pulsar-helm-chart/pull/277#issuecomment-1187443859

   I guess tags referencing `2.9.2` should also be updated in `.ci/clusters/values-pulsar-image.yaml` file.


-- 
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-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar-helm-chart] mattisonchao merged pull request #277: Bump Apache Pulsar version to 2.9.3

Posted by GitBox <gi...@apache.org>.
mattisonchao merged PR #277:
URL: https://github.com/apache/pulsar-helm-chart/pull/277


-- 
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-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [pulsar-helm-chart] lhotari commented on a diff in pull request #277: Bump Apache Pulsar version to 2.9.3

Posted by GitBox <gi...@apache.org>.
lhotari commented on code in PR #277:
URL: https://github.com/apache/pulsar-helm-chart/pull/277#discussion_r923368906


##########
charts/pulsar/Chart.yaml:
##########
@@ -18,7 +18,7 @@
 #
 
 apiVersion: v2
-appVersion: "2.9.2"
+appVersion: "2.9.3"
 description: Apache Pulsar Helm chart for Kubernetes
 name: pulsar
 version: 2.9.3

Review Comment:
   The chart version will also have to be bumped. The chart version is independent of Pulsar's version. The current version scheme for the chart version is confusing since it seems to match the Pulsar version, but that shouldn't be the goal.



-- 
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-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org