Rapha/dockerization #101
Closed
Rapha/dockerization #101
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Create runtime gRPC proto definition - gRPC implementation for tendermint runtime - Implementation of calling runtime methods from core - Add separate interface for gRPC runtime Signed-off-by: Kostas Christopoulos <[email protected]>
- Add CLI commands for calling the runtime directly. - Change dist path.
- Migrate tendrmint functionality to grpc runtime - Update docker proxy calling interface - Fix Dockerfile to allow runtime.proto. - Reduce Dockerfile stages for optimization. - Update documentation to include instructions on how to run the image. Signed-off-by: Kostas Christopoulos <[email protected]>
- Removes second interface implementd for docker integrations - Removes refactoring to core to call the new interface Signed-off-by: Kostas Christopoulos <[email protected]>
- cleanup various stuff - apply docker and code fixes to comply with new file structur Signed-off-by: Kostas Christopoulos <[email protected]>
- Adapt proto file to needs, remove unneeed stuff, fix types - Updated changelog and readme - Add scripts exporting grpc typescript code Signed-off-by: Kostas Christopoulos <[email protected]>
* Place the 'runtime.proto' file in 'common/types/src/protos' and use it as the source of truth. * Change the 'export-grpc-stubs' script in the root of the repo to loop through all dirs under 'docker-integrations' and compile the TS files from the 'runtime.proto' which is defined in the first bullet. * In the index.ts of the 'tendermint-grpc' docker integration, remove the dependency of importing the runtime.proto and only use the compiled TS files. Signed-off-by: Harry Touloupas <[email protected]>
Signed-off-by: Kostas Christopoulos <[email protected]>
The linter can be run from the root of the repo with 'yarn lint' Signed-off-by: Harry Touloupas <[email protected]>
- remove not needed files Signed-off-by: Kostas Christopoulos <[email protected]>
Attach the 'shutdown' function to be run when the process receives SIGINT or SIGTERM. Also add some ESLint configuration files and change the scripts to use them. Signed-off-by: Harry Touloupas <[email protected]>
Signed-off-by: Kostas Christopoulos <[email protected]>
- move proto from types to protocal - updated proto to work with new voting functionality - updated client/server to work with new proto Signed-off-by: Kostas Christopoulos <[email protected]>
Signed-off-by: Kostas Christopoulos <[email protected]>
Signed-off-by: Kostas Christopoulos <[email protected]>
Signed-off-by: Kostas Christopoulos <[email protected]>
The TARGETARCH build argument for Docker is available by default and can be 'amd64' or 'arm64' (between others). Use it to extract the correct binary architecture we need to build and copy that to the runtime stage of both core and the tendermint runtime. Also add .dockerignore to all 'common/*' in order to minimize the build context. Signed-off-by: Harry Touloupas <[email protected]>
Signed-off-by: Kostas Christopoulos <[email protected]>
Signed-off-by: Kostas Christopoulos <[email protected]>
shifty11
requested review from
troykessler,
christopherbrumm and
mbreithecker
as code owners
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.