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.

Sign up for GitHub

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

Jump to bottom

Add RPCs necessary to register engine when debugging with PULUMI_DEBUG_LANGUAGES #1480

Open
brandonpollack23 opened this issue Nov 21, 2024 · 0 comments
Open

Add RPCs necessary to register engine when debugging with PULUMI_DEBUG_LANGUAGES #1480

brandonpollack23 opened this issue Nov 21, 2024 · 0 comments
Labels
kind/enhancement Improvements or new features

Comments

@brandonpollack23
Copy link

Blocked by pulumi#17821

See that PR for details and examples of how this is implemented in Go/Python/Nodejs
@pulumi-bot pulumi-bot added the needs-triage Needs attention from the triage team label Nov 21, 2024
@brandonpollack23 brandonpollack23 mentioned this issue Nov 21, 2024
Merged
@justinvp justinvp added kind/enhancement Improvements or new features and removed needs-triage Needs attention from the triage team labels Nov 21, 2024
github-merge-queue bot pushed a commit to pulumi/pulumi that referenced this issue Dec 11, 2024
@brandonpollack23
Add PULUMI_DEBUG_LANGUAGES support to go, python, and nodejs. (#17821)
4fee276 
Extending the idea for PULUMI_DEBUG_PROVIDERS but no into language
providers.

There is a missing piece that providers didn't need which is the ability
to inform the language of the engine address for sending messages the
other direction (ie for logging) (previously this was a parameter).

I added a basic RPC that will let the engine inject that (or anything
else needed in the future). IMO we should actually rename this at some
point to "Initialize Language Host" and pass everything that way and NOT
via CLI arguments, but for now just this.

Other languages will have to be done in their own repositories after
this lands.

dotnet issue: pulumi/pulumi-dotnet#398
java issue:  pulumi/pulumi-java#1480
yaml issue: pulumi/pulumi-yaml#694
@justinvp justinvp mentioned this issue Dec 12, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
kind/enhancement Improvements or new features
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@justinvp @brandonpollack23 @pulumi-bot