Michael Marshall

Results 208 comments of Michael Marshall

Great catch, @hpvd. I didn't realize I had missed advising _how_ to report a vulnerability. I agree with duplicating the information in the `SECURITY.md` file. This information won't change with...

Thanks for following up @tisonkun, yes, I believe #17039 completes this issue. Please reopen the issue if you see any issues @hpvd.

I missed a minor detail with how the feature is configured. I proposed a fix here https://github.com/apache/pulsar-helm-chart/pull/308.

@zymap @wolfstudy @nodece - are you able to review this? I am not familiar with why we have this tool in the chart, but since you're the top three contributors...

@rrva - thank you for your contribution!

@Anmol057 @dan-autonomic - I cannot make any guarantees about when this chart will be released, but I am hoping to get a release available within a couple weeks. Note that...

@lhotari - looks like the TLS tests are failing for 2.10.1, but not for 2.9.3. I see that https://github.com/apache/pulsar-helm-chart/pull/223 is related, but I haven't figured out the root issue yet....

The test might just be flaky. I added some extra logging here https://github.com/apache/pulsar-helm-chart/pull/304 (I meant to make the PR in my fork, but it's in the official project), and the...

Looks like it was just flakiness on the tests. I am going to merge this now. If the tests continue to be flaky, we can create an issue and look...

@mattisonchao - what do you think about making it configurable to skip the spawning of any database process in the docker image? Then, in the Pulsar Helm Chart, we could...