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

Support calling inflight methods from different processes #4560

Closed
Chriscbr opened this issue Oct 16, 2023 · 1 comment
Closed

Support calling inflight methods from different processes #4560

Chriscbr opened this issue Oct 16, 2023 · 1 comment
Labels
🐶 dogfood Discovered while dogfooding Winglang ✨ enhancement New feature or request 🎨 sdk SDK 🕹️ simulator Related to the `sim` compilation target

Comments

@Chriscbr
Copy link
Contributor

Chriscbr commented Oct 16, 2023

Feature Spec

Allow compute resources (like cloud.Service, cloud.Function, etc.) in separate processes to interact with simulated resources

Use Cases

Today, the resources in the Wing Cloud Library are constrained to being part of the same Node.js process as the simulator. Each resource (like an a cloud.Bucket instance) maps to a single object in Node.js, and references to these objects are shared and passed around by inflight code. But this architecture won't work if the inflight code is running in a separate process, such as a Docker container. Instead, information about the resource and the method that needs to be called would have to be communicated through some form of IPC, like sockets, pipes, etc.

Implementation Notes

An initial version of this was implemented in #4105 using an express server, but it was reverted in #4557 since it resulted in many flakey tests for contributors and in CI.

Component

SDK

Community Notes

  • Please vote by adding a 👍 reaction to the issue to help us prioritize.
  • If you are interested to work on this issue, please leave a comment.
  • If this issue is labeled needs-discussion, it means the spec has not been finalized yet. Please reach out on the #dev channel in the Wing Slack.
@Chriscbr Chriscbr added ✨ enhancement New feature or request 🎨 sdk SDK 🕹️ simulator Related to the `sim` compilation target labels Oct 16, 2023
@Chriscbr Chriscbr changed the title Allow simulator inflight clients to call methods from different processes Support calling inflight methods from different processes Oct 16, 2023
@staycoolcall911 staycoolcall911 added this to the KubeCon23 milestone Oct 19, 2023
@staycoolcall911 staycoolcall911 added the 🐶 dogfood Discovered while dogfooding Winglang label Oct 30, 2023
@staycoolcall911
Copy link
Contributor

@Chriscbr - is there anything in this issue not done in #4697? Can we close?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐶 dogfood Discovered while dogfooding Winglang ✨ enhancement New feature or request 🎨 sdk SDK 🕹️ simulator Related to the `sim` compilation target
Projects
Archived in project
Development

No branches or pull requests

2 participants