Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Format method: runtime error rendered into docs #3125

Closed
t0yv0 opened this issue Dec 12, 2023 · 6 comments
Closed

Format method: runtime error rendered into docs #3125

t0yv0 opened this issue Dec 12, 2023 · 6 comments
Assignees
Labels
area/docs Improvements or additions to documentation kind/bug Some behavior is incorrect or out of spec resolution/duplicate This issue is a duplicate of another issue

Comments

@t0yv0
Copy link
Member

t0yv0 commented Dec 12, 2023

What happened?

https://www.pulumi.com/registry/packages/aws/api-docs/iot/getendpoint/ see the Java pane.

The code says:


public class App {
    public static void main(String[] args) {
        Pulumi.run(App::stack);
    }

    public static void stack(Context ctx) {
        final var example = IotFunctions.getEndpoint();

        var agent = new Kubernetes_pod("agent", Kubernetes_podArgs.builder()        
            .metadata(%!v(PANIC=Format method: runtime error: invalid memory address or nil pointer dereference))
            .spec(%!v(PANIC=Format method: runtime error: invalid memory address or nil pointer dereference))
            .build());

    }
}

Looks like something is rendering a PANIC into the example.

Example

See above.

Output of pulumi about

N/A

Additional context

N/A

Contributing

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).

@t0yv0 t0yv0 added kind/bug Some behavior is incorrect or out of spec needs-triage Needs attention from the triage team labels Dec 12, 2023
@iwahbe iwahbe added area/docs Improvements or additions to documentation and removed needs-triage Needs attention from the triage team labels Dec 12, 2023
@iwahbe
Copy link
Member

iwahbe commented Dec 12, 2023

@t0yv0 My understanding is that this is a program-gen issue. Should we transfer it to pulumi/pulumi?

@t0yv0
Copy link
Member Author

t0yv0 commented Dec 12, 2023

Ah yes, let me run one more check here. I can see if this persists under PULUMI_CONVERT=1 and if so extract a minimal repro in terms of pulumi convert.

@t0yv0
Copy link
Member Author

t0yv0 commented Apr 22, 2024

Duplicate of pulumi/pulumi-java#1341

@t0yv0 t0yv0 marked this as a duplicate of pulumi/pulumi-java#1341 Apr 22, 2024
@t0yv0 t0yv0 added the resolution/duplicate This issue is a duplicate of another issue label Apr 22, 2024
@t0yv0 t0yv0 closed this as completed Apr 22, 2024
@t0yv0 t0yv0 reopened this May 1, 2024
@t0yv0
Copy link
Member Author

t0yv0 commented May 1, 2024

Per investigation in pulumi-java#1341 this may just mean that we're missing a pinned k8s dependency here.

@t0yv0
Copy link
Member Author

t0yv0 commented May 1, 2024

But we do seem to have it: https://github.com/pulumi/pulumi-aws/blob/upstream-v5.47.0/.ci-mgmt.yaml#L24

Curious. Investigating.

@t0yv0
Copy link
Member Author

t0yv0 commented May 6, 2024

This was finally fixed in #3891

@t0yv0 t0yv0 closed this as completed May 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/docs Improvements or additions to documentation kind/bug Some behavior is incorrect or out of spec resolution/duplicate This issue is a duplicate of another issue
Projects
None yet
Development

No branches or pull requests

2 participants