Helm v3 Chart does not convert input properties to output properties

[covik]

Hello!

• Vote on this issue by adding a 👍 reaction
• To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already)

Issue details

The documentation for helm.v3.Chart states all input properties are converted to output properties but that's not the case. At least in NodeJS version.

The helm.d.ts does not have any public members:

export declare class Chart extends yaml.CollectionComponentResource {
    /**
     * Create an instance of the specified Helm chart.
     * @param releaseName Name of the Chart (e.g., nginx-ingress).
     * @param config Configuration options for the Chart.
     * @param opts A bag of options that control this resource's behavior.
     */
    constructor(releaseName: string, config: ChartOpts | LocalChartOpts, opts?: pulumi.ComponentResourceOptions);
    parseChart(config: ChartOpts | LocalChartOpts, releaseName: string): pulumi.Output<{
        [key: string]: pulumi.CustomResource;
    }>;
}

I thought maybe inputs are converted at runtime but are lacking in the .d.ts file but that's not the case either.

Steps to reproduce

import * as k8s from '@pulumi/kubernetes'

const traefik = new k8s.helm.v3.Chart('ingress-controller', {
    chart: 'traefik',
    version: '10.6.2',
    fetchOpts: {
      repo: 'https://helm.traefik.io/traefik'
    }
})

console.assert(traefik.chart === 'traefik') // Assertion failed

Expected: traefik.chart to be 'traefik'
Actual: traefik.chart is undefined

[viveklak]

The wording comes from pulumi/pulumi's docs codegen template. It does make sense to add the parameters in the outputs though.

[EronWright]

I wonder whether the value of chart should be the literal input value (which could be a chart path, a chart name, or a URL), or should be the resolved chart name. What is the use-case? Can we close this as won't fix?