Skip to content

Orc 50 - Simplified the way the processor address is supplied - #29

Merged
solvicode merged 2 commits into
mainfrom
ORC-50
Sep 27, 2025
Merged

Orc 50 - Simplified the way the processor address is supplied#29
solvicode merged 2 commits into
mainfrom
ORC-50

Conversation

@solvicode

Copy link
Copy Markdown
Contributor

For some background, there are two addresses (or IPs) at play:

  • The address of the gRPC server that the processor get's exposed on
  • The address that the orca core service queries in order to reach the processor

When orca core and the processor are on the same local network these two addresses are the same. But, when using the orca cli (for example), these are different as orca core is sitting inside a docker network. It has to reach the host network via a static IP.

In any case, the port in both these scenarios is the same, and the processor is always exposed on [::] - i.e. all network interfaces. So we can collapse the two env vars into 1 and just split out the port and address.

@solvicode solvicode self-assigned this Sep 27, 2025
@solvicode solvicode added bug Something isn't working enhancement New feature or request and removed bug Something isn't working labels Sep 27, 2025
@solvicode

Copy link
Copy Markdown
Contributor Author

FYI @christywastaken. These changes will mean that only one env var is require for the processor:

PROCESSOR_ADDRESS

And it will take the form of <host>:<ip>.

@solvicode
solvicode merged commit 2550bcb into main Sep 27, 2025
5 checks passed
@solvicode
solvicode deleted the ORC-50 branch September 27, 2025 20:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant