Lari Hotari comments

Results 996 comments of


                                            Lari Hotari

Tiered Storage config

@sijie @codelipenghui @eolivelli @addisonj This PR would be a valuable addition to Apache Pulsar Helm chart. Please review.

Unrecognized VM option 'PrintGCTimeStamps' - Unsupported Java 11 GC Flags causing init container fail

@samuelzimmermanphillips Thanks for reporting. This will be fixed as part of #190.

Pulsar manager always uses default user/password

Please review PR #212, it should fix this issue.

allow specifying the nodeSelector for the init jobs

> what do i do for the bookkeeper_cluster_initalize jobs? specify another nodeSelector value? or use the pulsar_metadata.nodeSelector value there as well? I think it's fine to use the pulsar_metadata.nodeSelector value...

Param pulsar_manager.admin.user/pulsar_manager.admin.password used in the wrong place

Please review PR #212, it should fix this issue.

Use appVersion as default tag for Pulsar images

> @lhotari just to confirm - for those of us who have overridden the `tag` that functionality will remain unchanged? Thanks! @frankjkelly yes, the intention is to be backwards compatible.

Use appVersion as default tag for Pulsar images

@sijie please review this PR since it's based [on your suggestion the dev mailing list](https://lists.apache.org/thread/bgkvcyt1qq6h67p2k8xwp89xlncbqn3d). I might have misunderstood your suggestion.

Bump to Pulsar 2.8.2

It seems that #188 should also be resolved before switching to Pulsar 2.8.x .

Bump to Pulsar 2.8.2

Failures in CI which points to #188. I'll push a fix to this PR. ``` Unrecognized VM option 'PrintGCTimeStamps' Error: Could not create the Java Virtual Machine. Error: A fatal...

Bump to Pulsar 2.8.2

> 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...