Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Sandbox docker image is ~700mb due to SRS being included. #1659

Closed
charlielye opened this issue Aug 18, 2023 · 0 comments · Fixed by #1704
Closed

Sandbox docker image is ~700mb due to SRS being included. #1659

charlielye opened this issue Aug 18, 2023 · 0 comments · Fixed by #1704
Assignees
@spypsy
Milestone
🚀 Initial F&F San...

Comments

@charlielye
Copy link
Contributor

Inside the container:

/usr/src/circuits/cpp/barretenberg/cpp/srs_db/ignition/monomial # ls -al
total 630016
drwxr-xr-x    2 root     root          4096 Aug 18 12:59 .
drwxr-xr-x    3 root     root          4096 Aug 18 12:58 ..
-rw-r--r--    1 root     root     322560412 Aug 18 12:59 transcript00.dat
-rw-r--r--    1 root     root     322560092 Aug 18 12:59 transcript01.dat

Removing should reduce container size to ~100mb, which will be appreciated for slower internet connections.
@github-project-automation github-project-automation bot moved this to Todo in A3 Aug 18, 2023
@iAmMichaelConnor iAmMichaelConnor mentioned this issue Aug 21, 2023
Closed
@spypsy spypsy self-assigned this Aug 21, 2023
@spypsy spypsy moved this from Todo to In Progress in A3 Aug 21, 2023
@spypsy spypsy mentioned this issue Aug 21, 2023
Merged
4 tasks
@spypsy spypsy moved this from In Progress to In Review in A3 Aug 21, 2023
PhilWindle pushed a commit that referenced this issue Aug 21, 2023
@spypsy
fix: Don't include SRS in sandbox docker img (#1704)
aa7f662 
Please provide a paragraph or two giving a summary of the change,
including relevant motivation and context.

Fixes #1659 

# Checklist:
Remove the checklist to signal you've completed it. Enable auto-merge if
the PR is ready to merge.
- [ ] If the pull request requires a cryptography review (e.g.
cryptographic algorithm implementations) I have added the 'crypto' tag.
- [ ] I have reviewed my diff in github, line by line and removed
unexpected formatting changes, testing logs, or commented-out code.
- [ ] Every change is related to the PR description.
- [ ] I have
[linked](https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue)
this pull request to relevant issues (if any exist).
@github-project-automation github-project-automation bot moved this from In Review to Done in A3 Aug 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@spypsy spypsy

Labels
None yet
Projects
A3
Archived in project
Milestone
🚀 Initial F&F Sandbox Release
Development

Successfully merging a pull request may close this issue.

fix: Don't include SRS in sandbox docker img AztecProtocol/aztec-packages
2 participants
@charlielye @spypsy