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

Per-workload host services connections #265

Merged
merged 3 commits into from
Jun 10, 2024
Merged

Conversation

autodidaddict
Copy link
Contributor

Each workload is now given its own unique/private host services connection. These connections are created using the following order of configuration (1 being the highest priority)

  1. host services configuration on the deployment request
  2. host services configuration supplied for the entire node
  3. configuration copied from the node's control connection

@autodidaddict autodidaddict requested a review from a team as a code owner June 8, 2024 21:05
@autodidaddict
Copy link
Contributor Author

It looks like the tests are failing in and around the use of host services (no surprise there). However, when doing manual tests it all works, so I'm wondering if there's just something that needs to be updated in the proxy or fixtures?

@autodidaddict
Copy link
Contributor Author

autodidaddict commented Jun 8, 2024

The debug line showing the per-workload host services connection:

[INF] 2024-06-08 17:23:11 - Established host services connection for workload workload_name=echofunctionjs url=nats://127.0.0.1:4222
./nex devrun ../examples/v8/echofunction/src/echofunction.js --trigger_subject=foo.trigger --type=v8
Reusing existing issuer account key: /home/kevin/.nex/issuer.nk
Reusing existing publisher xkey: /home/kevin/.nex/publisher.xk
🚀 Workload 'echofunctionjs' accepted. You can now refer to this workload with ID: cpicmsn52omib90596g0 on node NC27GEVVHI6VTT6MOAH64VGZ5VMSYUKB5CWAELUY2SEC3QR4W5ZPJ4JE
kevin in wopr in nex/nex on  feat/perworkloadhsconfig via 🐹 v1.22.2 
❯ nats req 'foo.trigger' 'howdy'
17:23:49 Sending request on "foo.trigger"
17:23:49 Received with rtt 3.293876ms
{"triggered_on":"foo.trigger","payload":"howdy"}

@kthomas kthomas merged commit 175763d into main Jun 10, 2024
4 checks passed
@kthomas kthomas deleted the feat/perworkloadhsconfig branch June 10, 2024 02:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants