Skip to content

fix(vector): add missing namespace to HPA, PDB, and PSP templates#537

Open
kobin23 wants to merge 1 commit intovectordotdev:developfrom
kobin23:fix/add-namespace-to-templates
Open

fix(vector): add missing namespace to HPA, PDB, and PSP templates#537
kobin23 wants to merge 1 commit intovectordotdev:developfrom
kobin23:fix/add-namespace-to-templates

Conversation

@kobin23
Copy link

@kobin23 kobin23 commented Mar 4, 2026

Description:

TL;DR

If you use Kustomize with Helm charts, ensure that your Helm templates explicitly set the namespace, for example:

metadata:
  namespace: {{ .Release.Namespace }}

If your charts already handle namespaces this way, this change should not introduce any breaking behavior.

Most of the resources in the Vector helm chart already include this namespace part, but not all of them, and rendering them using kustomize >= 5.8.0 leaves those resources w/o namespace

@hlhc
Copy link

hlhc commented Mar 5, 2026

Looks like the HAProxy manifests also missing namespace field. Should we add it for consistency?

…lates

Signed-off-by: Olimjon Kudratov <olimko98@gmail.com>
@kobin23 kobin23 force-pushed the fix/add-namespace-to-templates branch from a25ed53 to 7d201f3 Compare March 5, 2026 11:43
@kobin23
Copy link
Author

kobin23 commented Mar 5, 2026

Looks like the HAProxy manifests also missing namespace field. Should we add it for consistency?

@hlhc done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants