Fix #24

[twavv]

The use of {{- with ... }} seems useless here, and it breaks everything else. I think(?) it doesn't cause an issue in the default configuration because .Values.networkPolicy.ingress.grpc is an empty array (and I guess that means it doesn't actually switch into that scope?).

Would be nice™ to add a test too (I made this PR from the GitHub web app).

[EladDolev]

doesn't seem useless at all...
think the right fix here would be placing a $ sign before .Release in places they are inside a {{- with...}} scope
.Release.Name -> $.Release.Name

[twavv]

Maybe I'm missing something, but it looks to me like the with is only being used to make toYaml . work - everything else in the block doesn't seem to depend on accessing anything inside .Values.networkPolicy.ingress.grpc.

But yeah, just using $ to access the root object would also work here (it would just require changing more lines of code).

[DuskEagle]

Good catch @travigd. I agree that the {{- with ... }} doesn't seem necessary, and that this is an easier change than replacing every templated call within the with scope to start with$.

If you increment the version field of chart.yaml as described in CONTRIBUTING.md, I can merge this change.