Justin Van Patten

Results 93 issues of Justin Van Patten

Consider the following input [from the Kubernetes Python SDK](https://github.com/pulumi/pulumi-kubernetes/blob/60233379be4538a2444dabc2cd9efbb5df9b8151/sdk/python/pulumi_kubernetes/apiextensions/v1/_inputs.py#L901): ```python items: Optional[pulumi.Input[Union['JSONSchemaPropsArgs', Sequence[Any]]]] = None, ``` There are two issues: 1. Historically, the value could be passed as a `dict`....

kind/bug
area/sdks
language/python

With https://github.com/pulumi/pulumi-eks/pull/458, the `@pulumi/eks` package will require the `eks` plugin to be installed. The `package.json` has been annotated with `"pulumi": { "resource": true }` so the plugin will get installed...

kind/engineering

Checks like the following don't work correctly from multi-lang: https://github.com/pulumi/pulumi-eks/blob/e56aaeece8b55f4edf5e25b1086051d61a053bb6/nodejs/eks/nodegroup.ts#L339-L340 `isCoreData(args.cluster)` always returns true (incorrectly) due to its implementation: https://github.com/pulumi/pulumi-eks/blob/e56aaeece8b55f4edf5e25b1086051d61a053bb6/nodejs/eks/nodegroup.ts#L329-L331 In the multi-lang case, `args.cluster` can be an `Output` (which...

kind/bug

impact/no-changelog-required

In the TypeScript API, there are a few places that accept a `cluster` argument typed as `Cluster | CoreData` (`CoreData` is an interface representing the outputs of the cluster): https://github.com/pulumi/pulumi-eks/blob/26c027a9e6396a4e9346e720b80684871c94d215/nodejs/eks/nodegroup.ts#L260...

area/multi-language-components

While porting Python examples, I was running into cases where arguments aren't typed as `Input` in TypeScript and the implementation assumes the value is readily available (instead of accessing it...

area/multi-language-components
kind/bug

This test was failing with the following in the extra runtime validation. ``` --- FAIL: TestAccAwsGoAppSync (229.24s) panic: interface conversion: interface *** is map[string]interface ***, not string [recovered] panic: interface...

kind/engineering
impact/panic

We've seen at least one case where a nested output property is including the subsequent bullet as part of its docs. https://github.com/pulumi/pulumi-aws/blob/ed4659e42c6700e442c578be9f6fa1537f16a801/sdk/nodejs/types/output.ts#L600-L635 ```typescript export interface TargetGroupHealthCheck { /** * Indicates...

kind/bug

The styles for the API docs could use another round of refinement: fixing font sizes, indentation, etc., so it is more obvious where you on the page and the hierarchy...

area/docs
docs/api
kind/enhancement
kind/bug

The `build-pulumi-cli-docs` job ran successfully for the `3.40.2` release, but didn’t produce any changes to the Node or Python SDK docs. At the very least, I’d expect the Node SDK...

docs/core-sdk
language/python
language/javascript