has anyone got any pointers on installing Helm cha...
# generald
damp-room-71337
08/27/2019, 2:05 PMhas anyone got any pointers on installing Helm charts into non-default/current namespaces? I’ve tried passing a custom provider to Helm as mentioned here, but it gets ignored https://github.com/pulumi/pulumi-kubernetes/issues/217#issuecomment-504160101
c
cool-egg-852
08/27/2019, 2:22 PMYes, we use a transformation to handle it.
cool-egg-852
08/27/2019, 2:22 PM Copy code
new k8s.helm.v2.Chart(config.project, {
chart: 'stable/jenkins',
version: '1.3.3',
namespace: namespace.metadata.name,
values: {
master: {
serviceType: 'ClusterIP',
}
},
transformations: [
(resource: any) => {
resource.metadata.namespace = namespace.metadata.name;
// This fixes the test failing
if (resource.kind == 'Pod') {
resource.metadata.annotations = {
'<http://sidecar.istio.io/inject|sidecar.istio.io/inject>': 'false'
};
}
}
]
});d
damp-room-71337
08/27/2019, 2:23 PMI’m a bit wary of that, in case a chart uses non-cluster-scoped resources
c
cool-egg-852
08/27/2019, 2:23 PMIt hasn’t seemed to have broken anything for us, but I get your worry. I think there are some other more complicated transformations that do the same thing, but ignore those types of resources
cool-egg-852
08/27/2019, 2:23 PMIt’s the only solution that I am aware of.
cool-egg-852
08/27/2019, 2:24 PMThe proper solution though is to submit a PR against the chart and have them add the namespace explicitly.
cool-egg-852
08/27/2019, 2:24 PMTiller is going away, so they will need it anyways I believe. I’ve seen several charts accept these changes.
d
damp-room-71337
08/27/2019, 2:24 PMok, cheers - I don’t get why setting the namespace on the k8s provider doesn’t work though 🤔
g
gorgeous-egg-16927
08/27/2019, 2:25 PM@damp-room-71337 Would you mind filing an issue with details on the provider namespace not working for you? https://github.com/pulumi/pulumi-kubernetes/issues/new
d
damp-room-71337
08/27/2019, 2:25 PMsure
🙏 1
damp-room-71337
08/27/2019, 2:34 PMg
gorgeous-egg-16927
08/27/2019, 3:00 PMUpdated the ticket with a resolution. This chart was already fixed upstream, so you just have to set the
namespaced
damp-room-71337
08/27/2019, 3:01 PMYep just saw - that’s a fix for this specific chart, but I’d guess that there’s a general issue with Helm not respecting the k8s provider’s namespace?
g
gorgeous-egg-16927
08/27/2019, 3:01 PMRight. We don’t use Tiller, so some charts don’t work out of the box. Best thing is to fix the chart upstream, but the other ways can be used to work around in the meantime
gorgeous-egg-16927
08/27/2019, 3:03 PMTo clarify, the provider namespace flag is a default, not an override, so the chart’s value was taking precedence here
d
damp-room-71337
08/27/2019, 3:03 PMfair enough - I know it’s not Pulumi’s doing, but this is pretty hard to work with at the moment. The workarounds detailed here don’t always work for all Helm charts https://github.com/pulumi/pulumi-kubernetes/issues/217#issuecomment-504160101
g
gorgeous-egg-16927
08/27/2019, 3:05 PMIf you find specific cases that are confusing, please open an issue. We’re trying to make it as easy as possible, but it’s hard given the variety of approaches taken by helm charts
gorgeous-egg-16927
08/27/2019, 3:05 PMSince Tiller is going away with Helm 3, I’d expect the namespace issue to get better over time
2 Views