Micah Nagel
Micah Nagel
> There should be a way to disable creating the certificates and to provider the names of the relevant secrets created before running the `jaeger-operator` Helm chart. @avishefi This is...
Issues when overriding the image are the same as reported in https://github.com/jaegertracing/helm-charts/issues/351 . There seems to be a lack of quality reviewing of updates...CRDs are rarely updated and it seems...
@norwoodj would you also conditionally support escaping `|` in the actual values? Example: ```yaml # -- This is an example value example: "{{ .Values.other | base64encode }}" ``` When this...
@Szymongib leaving this in draft right now because I have two questions regarding it... 1. Clusterinstallation uses the same functions for the update job. Since these changes require adding a...
@Szymongib appreciate the feedback, I believe I've implemented a disable value in a simple way that should work (and added some tests around all of this new functionality). Let me...
@gabrieljackson / @Szymongib should be good to go? Rebased and fixed those comments - appreciate the feedback and assistance 😄
I'm happy to attempt work on these changes, wanted to open the issue to get the conversation started and see if this would be supported. The primary motivation for this...
@Szymongib definitely bring up some good points here...it's a balance between providing customization to the end user and making sure that things won't break with that customization. Could potentially go...
This will be a long comment...I've been poking around through each of the areas I identified and seeing what the most feasible path forward would be (in my mind). I...
To narrow the focus of this issue to just the update-check I opened #298 to handle the labels/annotations support for the main app server pods.