-
Notifications
You must be signed in to change notification settings - Fork 76
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
Deployment Image for Soroban RPC #382
Comments
@satyamz , when the new docker image is published from this effort, |
You can open a PR for that whenever docker image is ready. Also you would need to update the image
Captive core settings needs to be updated on the futurenet horizon which is not included in the soroban-rpc helm chart. |
@sreuland we maintain official core docker images, it could be nicer to reuse them instead. In k8s soroban-rpc Deployment could use 2 containers inside the same pod which is a common pattern. For local development we could recommend minikube or similar |
the proposed |
I am proposing for |
ok, yes, the scope of this effort can follow that suggestion for the soroban rpc image, and scope includes updating the helm chart to use it correctly. The latest rpc version runs captive core, which the helm chart doesn't reflect yet, the rpc image needs to have the core bin in it now and that can be copied into the image during build as multi-stage from existing stellar/stellar-core image. Part of the design here on rpc image will also need to account for how to provide the captive core config file to rpc service. Could mount a volume having the captive core config file specific to the network desired for connection or we just include a copy of each .cfg for each network available in the rpc image. Quickstart as one reference, it follows the latter approach of maintaining a canned version of the .cfg's for each network in the rpc image it builds |
Currently "release" Dockerfile and Makefile arehere For example we can build it like this:
|
@tsachiherman , I re-scoped the description on ticket as most of it was obsoleted by recent work already merged by @jacekn - #710 I think the |
The remaining criteria on this ticket is resolved:
closing the ticket. |
thanks @sreuland! |
What problem does your feature solve?
Add multi-arch(linux/amd64 and linux/arm64) support for building the image from cmd/soroban-rpc/Dockerfile.
Document the image in cmd/soroban-rpc/docker/README:
The text was updated successfully, but these errors were encountered: