Micah Nagel

Results 25 comments of 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.