Rancher just released the new version 2.4.0 but I ...
# kubernetesf
fast-dinner-32080
03/31/2020, 4:20 PMRancher just released the new version 2.4.0 but I am unable to upgrade to it due to the helm fecth extracting two directories. Seems like the logic to get the templates/values expects the untar to only have a single directory extracted and not two. I logged an issue earlier here https://github.com/pulumi/pulumi-kubernetes/issues/1046 but now I cannot upgrade rancher which is something we need to start testing. I am using dotnet but I am guessing the other sdks also have the same issue.
g
gorgeous-egg-16927
03/31/2020, 4:37 PMAre you running
helm fetchf
fast-dinner-32080
03/31/2020, 5:00 PMI am letting the class do it. I just tested the fetch command myself and am able to recreate the issue by seeing it extra two directories.
fast-dinner-32080
03/31/2020, 5:01 PMThe code is setup to expect a single extracted directory which doesn't work if more than one is exacted. Due to file ordering it picks the wrong directory which doesn't hold the chart contents.
g
gorgeous-egg-16927
03/31/2020, 5:03 PMGot it; thanks. Looks like this issue is specific to the .NET provider. I’ve updated the issue with that info
gorgeous-egg-16927
03/31/2020, 5:04 PM Copy code
new k8s.helm.v3.Chart(
"rancher",
{
repo: "rancher-latest",
version: "2.4.0",
chart: "rancher",
},
);f
fast-dinner-32080
03/31/2020, 5:05 PMYea @tall-librarian-49374 wasn't sure if it was just .net specific since the code was ported over from the node sdk.
fast-dinner-32080
03/31/2020, 5:08 PMLooking at the nodejs code it also expects a single directory based on the index 0
https://github.com/pulumi/pulumi-kubernetes/blob/master/sdk/nodejs/helm/v2/helm.ts#L172
fast-dinner-32080
03/31/2020, 5:10 PMSo if you checked out the temp dir it may just be different ordering which makes it work unless yours isn't extracting two directories.
fast-dinner-32080
03/31/2020, 5:24 PMSo figured it out. When you provide the fetchOps and repo url directly it extracts it into two directories.
Copy code
new k8s.helm.v3.Chart(
"rancher",
{
version: "2.4.0",
chart: "rancher",
fetchOpts: {
repo: "<https://releases.rancher.com/server-charts/latest>"
}
},
);fast-dinner-32080
03/31/2020, 5:24 PMso the work around is to add the helm repo before hand
g
gorgeous-egg-16927
03/31/2020, 5:24 PMAha, thanks for tracking that down.
f
fast-dinner-32080
03/31/2020, 5:29 PMAnytime! I updated the issue.
fast-dinner-32080
03/31/2020, 5:38 PMHey @gorgeous-egg-16927 is there a way to add the helm repo using pulumi or is that going to be a manual step before hand?
g
gorgeous-egg-16927
03/31/2020, 5:40 PMIt’s not built in currently. You could either handle it out of band (e.g., in CI), or could also shell out from your pulumi program
f
fast-dinner-32080
03/31/2020, 5:40 PMOkay thought so! ill figure something out
4 Views