This message was deleted Pulumi Community #typescript

Join Slack

This message was deleted.

# typescript

sparse-intern-71089

11/16/2021, 4:15 PM

This message was deleted.

microscopic-florist-22719

11/16/2021, 5:09 PM

I think that I can answer this, though the answer for the

endpoint

bit isn’t particularly satisfying.

microscopic-florist-22719

11/16/2021, 5:11 PM

The constructor for the

pulumi.Resource

superclass needs to see all potential properties in the

props

bag, not just input properties. If the names of the output properties are not present in that bag, the

Resource

machinery can’t determine which output properties to expect.

microscopic-florist-22719

11/16/2021, 5:11 PM

So I think that if you change your code to this, things should work as you expect:

microscopic-florist-22719

11/16/2021, 5:12 PM

Copy code

class Cluster extends pulumi.dynamic.Resource {
    public readonly endpoint!: pulumi.Output<string>;

    constructor(
        name: string,
        props: IClusterInputs,
        opts?: pulumi.CustomResourceOptions,
    ) {
        super(new ClusterProvider(), name, { endpoint: undefined, ...props }, opts);
    }
}

microscopic-florist-22719

11/16/2021, 5:13 PM

The reason

pulumi.output(cluster.endpoint)

changed things is that it turned an

undefined

value into a resolved (but defined!)

pulumi.Output

value whose inner value is

undefined

microscopic-florist-22719

11/16/2021, 5:14 PM

w.r.t. `id`: you didn’t have to do anything special there because

id

is always defined by the SDK.

rough-hydrogen-27449

11/16/2021, 5:17 PM

Oh, yeah I think you're right. Why do we have to "initialize"

endpoint

like that in the dynamic resource constructor?

microscopic-florist-22719

11/16/2021, 6:09 PM

I think that it’s because without that initialization, it’s not possible for the constructor to know what the set of output properties is for the resource. IIRC there are problems with attempting to iterate over the properties defined on

this

(e.g. properties that are present on every object shouldn’t be considered output properties), but my JS/TS expertise may be failing me here (cc @microscopic-pilot-97530).

rough-hydrogen-27449

11/16/2021, 9:50 PM

Got it, that makes sense. Thanks.

12 Views

Open in Slack

Previous Next