Skip to content

chore(docker): merge consecutive RUN instructions#3473

Closed
zarfishan20 wants to merge 7 commits intoOWASP:mainfrom
zarfishan20:merge-docker-run-instructions-fix
Closed

chore(docker): merge consecutive RUN instructions#3473
zarfishan20 wants to merge 7 commits intoOWASP:mainfrom
zarfishan20:merge-docker-run-instructions-fix

Conversation

@zarfishan20
Copy link

Fix Dockerfile syntax by ensuring BuildKit
Immediately after RUN instructions.

Note: The previous PR was closed unintentionally.

Resolves #3465

Checklist

  • [x ] Required: I followed the contributing workflow
  • [ x] Required: I verified that my code works as intended and resolves the issue as described
  • Required: I ran make check-test locally: all warnings addressed, tests passed
  • [ x] I used AI for code, documentation, tests, or communication related to this PR

@github-actions github-actions bot added the docker Pull requests that update Docker code label Jan 22, 2026
@github-actions
Copy link

PR validation failed: No linked issue and no valid closing issue reference in PR description

@github-actions github-actions bot closed this Jan 22, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 22, 2026

Caution

Review failed

The pull request is closed.

Summary by CodeRabbit

  • Chores
    • Updated Docker build configurations for backend service images across multiple container variants to improve build efficiency.

✏️ Tip: You can customize this high-level summary in your review settings.

Walkthrough

The PR consolidates multiple consecutive RUN instructions across five backend Dockerfiles into fewer layers by merging APK setup, cache mounting, user creation, and chown operations, reducing image layers while maintaining functional behavior.

Changes

Cohort / File(s) Change Summary
RUN instruction consolidation
docker/backend/Dockerfile, docker/backend/Dockerfile.fuzz, docker/backend/Dockerfile.local, docker/backend/Dockerfile.test, docker/backend/Dockerfile.video
Merges separate RUN steps (APK setup, cache mounting, user creation, chown) into single RUN instructions; adjusts PIP_CACHE_DIR and APK_CACHE_DIR cache mount placement; removes redundant intermediate layers while preserving functionality.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~18 minutes

Possibly related PRs

Suggested labels

backend, docker

Suggested reviewers

  • arkid15r
  • kasya
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sonarqubecloud
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docker Pull requests that update Docker code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments