-
Notifications
You must be signed in to change notification settings - Fork 3
Ci/tag fix #40
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
Merged
Merged
Ci/tag fix #40
Conversation
This file contains hidden or 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
- Add Dockerfile with 2-stage build (builder and runtime) - Add Dockerfile.runtime for fast runtime-only rebuilds - Update Makefile with Docker build targets (docker_build_builder, docker_build_runtime, docker_build_all) - Install Rust via rustup in init.sh script instead of apt - Add RUST_VERSION to .env configuration - Update CI workflow to cache Rust via rustup - Configure vcpkg caching with BuildKit mount - Fix library and module paths in runtime stage - Set --modules-dir in ENTRYPOINT for convenience
# Conflicts: # .ci/scripts/init.sh # Dockerfile # Makefile
- Introduced a three-stage Docker build process: dependencies, builder, and runtime. - Created Dockerfiles for each stage: Dockerfile.dependencies, Dockerfile.builder, and Dockerfile.runtime. - Updated Makefile with new targets for building and managing Docker images. - Added .dockerignore to exclude unnecessary files from Docker context. - Enhanced README and added DOCKER_BUILD.md for detailed build instructions and CI/CD integration. This structure improves build efficiency and reduces image sizes for production.
- Renamed and reorganized Makefile targets for clarity, introducing `docker_build` for fast code rebuilds and `docker_push` for pushing all images. - Updated README and DOCKER_BUILD.md to reflect new commands and usage instructions, enhancing clarity for CI/CD integration. - Improved Docker image management by consolidating push commands and cleaning up unnecessary targets. - Adjusted build times in documentation to provide accurate estimates. These changes streamline the Docker workflow and improve user experience.
- Updated Dockerfile configurations to use 'latest' tags for dependencies and builder images, improving consistency and clarity. - Optimized runtime image by stripping binaries and reducing size, resulting in a production image of approximately 240 MB. - Enhanced README and DOCKER_BUILD.md with detailed instructions on image tagging and pushing, including examples for custom and latest tags. - Improved Makefile to support flexible tagging and streamlined push commands for Docker images. These changes improve the efficiency of the Docker workflow and provide clearer guidance for users.
- Added detailed instructions for managing Docker dependencies, including tagging and pushing images with custom and latest tags. - Updated Makefile to support a default dependencies tag and improved clarity in image tagging for builder and runtime stages. - Enhanced README and DOCKER_BUILD.md to reflect new tagging strategies and best practices for dependency management. These changes improve the usability and clarity of the Docker build process for developers.
remove num_validators, update genesis_time, and enhance genesis state generation
…k methods in ValidatorKeysManifestMock
Signed-off-by: turuslan <[email protected]>
…ig-rebased # Conflicts: # .github/workflows/docker-build.yml # src/app/configurator.cpp # src/executable/lean_node.cpp
Signed-off-by: turuslan <[email protected]>
Signed-off-by: turuslan <[email protected]>
# Conflicts: # src/blockchain/fork_choice.cpp # src/blockchain/fork_choice.hpp # src/blockchain/state_transition_function.cpp # src/blockchain/state_transition_function.hpp # src/crypto/xmss/types.hpp # src/crypto/xmss/xmss_provider_impl.cpp # src/crypto/xmss/xmss_util.cpp # src/crypto/xmss/xmss_util.hpp # src/modules/production/read_config_yaml.hpp # src/types/config.hpp # src/types/signed_attestation.hpp # tests/unit/blockchain/fork_choice_test.cpp # tests/unit/blockchain/state_transition_function_test.cpp # vcpkg-overlay/qdrvm-crates/portfile.cmake
# Conflicts: # .github/workflows/docker-build.yml # Makefile
kamilsa
approved these changes
Dec 2, 2025
turuslan
approved these changes
Dec 2, 2025
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.