Sanketh I

Results 9 comments of Sanketh I

Flowing the tserver tolerations to this job makes sense (because it is essentially talking to the tservers) @bhavin192 @baba230896 @amannijhawan - any opinions?

Looks good, can you add a test plan? Might be worth testing an upgrade scenario?

Thanks for the report @zs-ko ! cc @bhavin192 @baba230896

@gris-gris @Brice187 thanks for raising this issue. We'll try to fit this into the future roadmap and prioritize. If there is an existing PR, let us know.

@miclip : Thanks for filing this, this is useful feedback. 1. What do you suggest having in the CN instead? Given that the certificate is common to all pods, I...

@miclip : ok, got it. We are planning to iterate on the cert-manager functionality soon and will take this feedback into account.

@carhartl : Thanks for letting us know. Please feel free to send over a PR, otherwise, we will include this in our bug prioritization.