fix for rendering jaeger Deployment when specifying an image pull secret (#241)

* fix for rendering jaeger Deployment

* fixed typo in testing instructions

Author: loujar

TEST.md

@@ -52,7 +52,7 @@ sourcegraph:
 # More values to be added in order to test your change
 ```
 
-Make sure you test both enabled and disabled toggles. For example, if you added a new values to conditional render some templates, turn it on and off in the `overrid.yaml` to make sure they both work. You can also include your `override.yaml` in the `Test plan` during PR review to help others understand your testing strategy.
+Make sure you test both enabled and disabled toggles. For example, if you added a new values to conditional render some templates, turn it on and off in the `override.yaml` to make sure they both work. You can also include your `override.yaml` in the `Test plan` during PR review to help others understand your testing strategy.
 
 ### Inspect the entire rendered template
 

charts/sourcegraph/templates/jaeger/jaeger.Deployment.yaml

@@ -98,8 +98,8 @@ spec:
       {{- with .Values.sourcegraph.imagePullSecrets }}
       imagePullSecrets:
         {{- toYaml . | nindent 8 }}
-      {{- include "sourcegraph.renderServiceAccountName" (list . "jaeger") | trim | nindent 6 }}
       {{- end }}
+      {{- include "sourcegraph.renderServiceAccountName" (list . "jaeger") | trim | nindent 6 }}
       volumes:
       {{- if .Values.jaeger.extraVolumes }}
       {{- toYaml .Values.jaeger.extraVolumes | nindent 6 }}