fix(infra-apps): bump nginx-ingress to 0.34.*

[hairmare]

Description

bump nginx-ingress chart to 0.34.*:

• Add namespace field in the namespace scoped resource templates (added namespace field in the namespace scoped resource templates of helm chart kubernetes/ingress-nginx#7256)

Issues

Checklist

• I updated the version in Chart.yaml
• I updated applicable README.md files using pre-commit run -a
• I documented any high-level concepts I'm introducing in docs/
• If I updated a dependency tool, or app, this PR contains a short summary of the changes I'm pulling
• CI is currently green and this is ready for review
• I am ready to test changes after they are applied and released

[eyenx]

Why only a patchfix bump?

[hairmare]

Why only a patchfix bump?

because the upstream change is a patch and not a feature

by adding namespace fields to all the resources upstream is really only fixing a extremely specific issue with gitlab managed deployments based on running helm template

[eyenx]

Well the upstream version change is from 0.33.x to 0.34.x tho... and not onlly a patchfix. Did they do it wrong?

[hairmare]

Well the upstream version change is from 0.33.x to 0.34.x tho... and not onlly a patchfix. Did they do it wrong?

They are still on 0.y.z for initial development which is described as follows in the SemVer spec:

Major version zero (0.y.z) is for initial development. Anything MAY change at any time. The public API SHOULD NOT be considered stable.