https://pulumi.com logo
Join Slack
Powered by
This message was deleted.
# typescript
s
This message was deleted.
b
before removing node_modules locally and reinstalling with 
yarn
I wasn't seeing this issue locally. So it leads me to believe something changed with dependency.
and even stranger.
Copy code
/home/p/Code/ss/infrastructure/node_modules/@pulumi/helm/v3/helm.ts:125:33
doesn't exist on my system
@gorgeous-egg-16927 🙏
g
Yeah, https://github.com/pulumi/pulumi-kubernetes/pull/1151 was a big change, so it’s possible something broke. Would you mind opening an issue with more details?
🙏 1
b
absolutely! What type of extra detail would be helpful do you think 😅.
g
Ideally the code snippet with the Helm constructor and the error messages
👍 1
b
sadly I'm not sure which helm is causing this
from error we use many
i'll see if I can repro on a blank stack for minimal example
g
Thanks!
Just noticed something odd with the import path — The helm package is part of the 
kubernetes
package, so I would have expected 
@pulumi/kubernetes/helm/v3
rather than 
.../@pulumi/helm/v3/helm.ts:125:33
FWIW, I just confirmed that 
helm.v3.Chart
is working for me on a fresh project. It seems like the change may have messed up node_modules somehow
b
yeah It worked for me too with a traefik example. trying to find which part of our stack might demonstrate it.
The main reason i'm bamboozled is because in CI the only thing that could have changed is the docker image cause we use 
:latest
so running a working flow from friday now fails.
and now experiencing it locally on that same stack / commit
@gorgeous-egg-16927 i've discovered the problem code that generates the error above, but no idea why. Has to do with 1 place I de-structure 
Chart
on my import. Seems to happen even on blank stack. here is working / borkted code.
Copy code
import * as infra from '@myOrg/infrastructure'
import { Chart } from '@pulumi/kubernetes/helm/v2'
import * as kube from '@pulumi/kubernetes'

// get metadata for our Kubernetes cluster
const skunkworksCluster = infra.kube.getClusterData('skunkworks', 'stage')

new Chart( // <--- ERROR
    `influxdb-helmchart`,
    {
        chart: 'stable/influxdb',
        version: '1.3.4',
        values: {
            persistence: {
                storageClass: 'gp2',
                size: '250Gi'
            },
            ingress: {
                enabled: true,
                tls: false,
                hostname: 'influxdb'
            }
        }
    },
    {
        provider: skunkworksCluster.provider
    }
)

new kube.helm.v2.Chart( // <-- DOES not error
    `influxdb-helmchart`,
    {
        chart: 'stable/influxdb',
        version: '1.3.4',
        values: {
            persistence: {
                storageClass: 'gp2',
                size: '250Gi'
            },
            ingress: {
                enabled: true,
                tls: false,
                hostname: 'influxdb'
            }
        }
    },
    {
        provider: skunkworksCluster.provider
    }
)
very weird
g
That is odd. Previously, the v3 import was just an alias to v2, so perhaps that change is what’s causing the error.
b
lmk if it repros on your end. It makes me feel like i'm overlooking something, because those should be identical
g
Did you try deleting 
node_modules
and reinstalling?
b
oh yeah, and yarn-upgrade-all
the above problem works on fresh stack
when I 
go to definition
both 
Chart
and 
kube.helm.v2.Chart
take me to same place too 🤷
g
Weird. I was able to repro locally as well
👍 1
b
Oh well it's EZ fix for my needs. Thanks for helping. I'm glad I was able to repro, but I have no clue beyond it.
👍 1
c
I have the same issue
Trying to create a log group. Instantiating a group causes the same issue for me.
What's weird that I have 2 project on one I get that same error and the other I get no error and they have the same dependencies.
Open in Slack
PreviousNext
Powered by