Update _helpers.tpl

[shahid430]

Changed namespace from default to monitoring

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[illrill]

This helper template is designed to default the installation into {{ .Release.Name }}-system instead of Helm's default choice of default, in the scenario that the user doesn't specify any namespace using helm install --namespace.

This change would mean that if a user does specify monitoring as the desired namespace, the installation instead goes into {{ .Release.Name }}-system. In contrast, if a user doesn't specify any namespace, the installation goes into default (which is what this helper template meant to prevent). This chart would basically become installable in any namespace except for monitoring.

[yorugac]

@shahid430, thanks for the PR but it's indeed not clear what this fix is for: unless specified, namespace will be %v-system as is defined here:
https://github.com/grafana/k6-operator/blob/main/charts/k6-operator/templates/_helpers.tpl#L93

@illrill Thanks for helping out 😄

[yorugac]

@shahid430 I'm closing this PR due to reasons stated above. Please feel free to re-open / open a new one if there's something else!