Skip to content

Conversation

@olehnikolaiev
Copy link
Collaborator

fixed #478

reworked build process for release containers

Copilot AI review requested due to automatic review settings October 2, 2025 14:45
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR reworks the build process for release containers to address issue #478 related to container size optimization. The changes move the heavy build process out of Docker containers and into GitHub Actions workflows, resulting in smaller runtime containers that only contain pre-built binaries.

Key changes include:

  • Added parallel build support using CPU core detection across build scripts
  • Moved SGX SDK/PSW installation and dependency building from Dockerfiles to GitHub Actions workflows
  • Updated container architecture to copy pre-built binaries rather than building from source
  • Added cleanup procedures to reduce build artifacts and container size

Reviewed Changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
scripts/build_deps.py Added CPU core detection and parallel build flags, plus cleanup procedures
jsonrpc/build.sh Simplified build script with parallel make and updated git checkout
DockerfileSimulation Converted to runtime-only container copying pre-built binaries
DockerfileRelease Converted to runtime-only container copying pre-built binaries
DockerfileIntelSubmission Converted to runtime-only container copying pre-built binaries
DockerfileBase Optimized build layers and added cleanup procedures
Dockerfile Converted to runtime-only container copying pre-built binaries
.github/workflows/*.yml Added host-based build steps before Docker container creation

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

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.

investigate sgxwallet container's size

2 participants