[FIX] ai-pipeline non root 컨테이너 data 디렉터리 생성 권한 부여 (#102)

[juminzoomout]

관련 이슈

Closes #102

변경

apps/ai-pipeline/Dockerfile: /app/app/data를 app 유저 소유로 chown
→ non-root 유지하며 news_cache·images 쓰기 가능, CD 부팅 실패 해결.

Summary by CodeRabbit

• 작업
  • AI 파이프라인 컨테이너의 데이터 저장소 디렉터리 초기화 및 파일 소유권 설정을 개선했습니다.
  • 컨테이너 빌드 프로세스에서 필요한 시스템 디렉터리 구조가 올바르게 생성되도록 수정했습니다.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
safe-or-scam	Ready	Preview, Comment	Jun 19, 2026 2:15pm

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 766c99b7-e4bb-4f27-8807-3de9bdd6b752

📥 Commits

Reviewing files that changed from the base of the PR and between 7f78ca2 and 4bfb5b6.

📒 Files selected for processing (1)

• apps/ai-pipeline/Dockerfile

---

Walkthrough

apps/ai-pipeline/Dockerfile의 RUN 명령에 /app/app/data 디렉터리 생성(mkdir -p) 및 chown -R app:app 소유권 부여 단계가 추가되었습니다. 기존에는 app 시스템 사용자/그룹만 생성했습니다.

Changes

ai-pipeline Dockerfile 권한 수정

Layer / File(s)	Summary
데이터 디렉터리 생성 및 소유권 설정 apps/ai-pipeline/Dockerfile	app 사용자/그룹 생성 RUN 명령에 /app/app/data 디렉터리 생성(mkdir -p) 및 chown -R app:app 소유권 설정이 추가되어, non-root 컨테이너 기동 시 crawler.py import 시점의 news_cache 디렉터리 생성 권한 오류를 해소합니다.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목은 변경 내용의 핵심을 정확하게 설명하며, ai-pipeline 컨테이너의 비루트 사용자가 data 디렉터리에 접근할 수 있도록 권한을 부여하는 수정 사항을 명확하게 나타냅니다.
Linked Issues check	✅ Passed	PR의 변경 사항이 이슈 #102의 요구사항을 완벽하게 충족합니다. Dockerfile에서 /app/app/data 디렉터리의 소유자를 app 사용자로 변경하여 비루트 컨테이너가 권한 거부 없이 디렉터리를 생성할 수 있도록 수정했습니다.
Out of Scope Changes check	✅ Passed	모든 변경 사항이 이슈 #102의 범위 내에 있으며, Dockerfile의 사용자/그룹 생성 단계에서만 /app/app/data 디렉터리 생성 및 권한 설정이 추가되었습니다.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/#102-ai-pipeline-data-permission

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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