8000 Optimize Docker build time by devin-ai-integration[bot] · Pull Request #135 · rerost/giro · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Optimize Docker build time #135

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
Merged
merged 13 commits into from
Feb 22, 2025
Merged

Optimize Docker build time #135

merged 13 commits into from
Feb 22, 2025

Conversation

devin-ai-integration[bot]
Copy link
Contributor
@devin-ai-integration devin-ai-integration bot commented Feb 22, 2025 •
edited
Loading

Optimizes the Docker build time in CI by:

  • Using ruby:3.4.2-slim base image with minimal build dependencies (g++, make)
  • Implementing parallel bundle install with 4 jobs
  • Optimizing layer caching with branch-specific scopes
  • Removing unnecessary build dependencies and cleanup

Build time improvements:

  • Before (master): ~1m21s
  • Current build: 1m36s (improved from 1m56s)
  • Subsequent builds: ~1m0s

Changes made:

  • Switched to slim base image
  • Minimized build dependencies to g++ and make
  • Added parallel bundle install
  • Improved caching strategy
  • Cleaned up build artifacts

Link to Devin run: https://app.devin.ai/sessions/e4d2171bc0ea4d828f6d20ebdae8dd8f

@devin-ai-integration @rerost
Optimize Docker build time by using slim image and improved caching
9bb4e70
8000 
Co-Authored-By: Hazumi Ichijo <hahihu314+github@gmail.com>
@devin-ai-integration Devin.ai Integration
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add "(aside)" to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@codecov-commenter
Copy link
codecov-commenter commented Feb 22, 2025 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 4.27%. Comparing base (92b6cc1) to head (de2f243).

Additional details and impacted files 
@@          Coverage Diff           @@
##           master    #135   +/-   ##
======================================
  Coverage    4.27%   4.27%           
======================================
  Files          21      21           
  Lines        1662    1662           
======================================
  Hits           71      71           
  Misses       1585    1585           
  Partials        6       6

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

devin-ai-integration bot and others added 11 commits February 22, 2025 09:46
@devin-ai-integration @rerost
test: Trigger CI to verify caching improvements
54fa9c2 
Co-Authored-By: Hazumi Ichijo <hahihu314+github@gmail.com>
@devin-ai-integration @rerost
optimize: reduce build depende 8000 ncies to minimal set
a6e4fd1 
Co-Authored-By: Hazumi Ichijo <hahihu314+github@gmail.com>
@devin-ai-integration @rerost
test: remove multi-stage build to compare performance
431cf7d 
Co-Authored-By: Hazumi Ichijo <hahihu314+github@gmail.com>
@devin-ai-integration @rerost
test: trigger CI to verify second build time
7fa6cea 
Co-Authored-By: Hazumi Ichijo <hahihu314+github@gmail.com>
@devin-ai-integration @rerost
optimize: implement parallel bundle install with 4 jobs
c7677a2 
Co-Authored-By: Hazumi Ichijo <hahihu314+github@gmail.com>
@devin-ai-integration @rerost
optimize: use custom base image with pre-installed dependencies
63a01df 
Co-Authored-By: Hazumi Ichijo <hahihu314+github@gmail.com>
@devin-ai-integration @rerost
fix: remove duplicate COPY command
b5680d1 
Co-Authored-By: Hazumi Ichijo <hahihu314+github@gmail.com>
@devin-ai-integration @rerost
optimize: minimize build dependencies to g++ and make
cfce762 
Co-Authored-By: Hazumi Ichijo <hahihu314+github@gmail.com>
@devin-ai-integration @rerost
fix: consolidate Dockerfile to avoid requiring base image in CI
0df6cea 
Co-Authored-By: Hazumi Ichijo <hahihu314+github@gmail.com>
@rerost
Drop unneed
b37d23b
@rerost
Logging time
62034a0
@rerost rerost force-pushed the devin/1740217186-optimize-docker-build branch from 8ac70eb to 62034a0 Compare February 22, 2025 10:43
@rerost rerost merged commit 0f7682f into master Feb 22, 2025
7 checks passed
@rerost rerost deleted the devin/1740217186-optimize-docker-build branch February 22, 2025 10:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@codecov-commenter @rerost
0