Chart namespace is applied to non-namespaced objects

[maybedino]

Currently, if you set a namespace for a chart, it will set the namespace on every object in the chart that does not have one:

cdk8s-core/src/api-object.ts

Line 133 in ab3fbf3

namespace: props.metadata?.namespace ?? this.chart.namespace,

While every Kubernetes object can have a metadata.namespace, some are cluster objects that are not namespaced. Applying a namespace to a cluster object breaks tools that check for this, like Terraform (I was trying to deploy with cdktf-cdk8s)

[iliapolo]

@maybedino Oooh nice! Thanks for this, we'll look into it.

[github-actions]

This issue has not received any attention in 1 year and will be closed soon. If you want to keep it open, please leave a comment below @mentioning a maintainer.

[maybedino]

This does not really cause any issues unless you have to validate your YAML files. The Kubernetes API does not complain.

@iliapolo Maybe it would still be good to keep this open, since technically it creates invalid YAML files which can cause problems with some deployment tools.

Our current workaround is to do this:

const ns = new k8s.KubeNamespace(this, "FooBar", {
	metadata: { name: "FooBar" },
});
ns.addJsonPatch(JsonPatch.remove("/metadata/namespace"));