Skip to content

chore: Alloy v1.21.0 도입 및 filelog 정렬 통일 - #268

Open
geonusp wants to merge 5 commits into
mainfrom
chore/alloy-v1.21.0-adoption-direct
Open

geonusp wants to merge 5 commits into
mainfrom
chore/alloy-v1.21.0-adoption-direct

Conversation

@geonusp

@geonusp geonusp commented Oct 19, 2026 •

Copy link
Copy Markdown
Collaborator

🧰 Chore PR

🔗 GitHub Issue


🧩 한눈에 요약

무엇을 변경했나?

  • 인프라
  • 설정파일
  • 빌드 파일
  • CI
  • 의존성
  • 런타임/비즈니스 로직 리팩터링 없음 (있으면 Refactor PR 템플릿 사용)

왜 필요한가?

  • Alloy v1.21.0에 포함된 filelog 정렬 수정에 맞춰 기존 우회 설정을 제거하고, receiver의 정렬 설정을 직접 사용하도록 변경했다.

범위

  • Included: Alloy 이미지 v1.21.0 업그레이드, filelog 설정 이전, 회전 로그 순서 검증 추가, 로컬 민감 파일 ignore 규칙 추가, CI 컨테이너가 임시 Alloy 설정 파일을 읽도록 권한 수정
  • Not included: WSL Docker Desktop에서 로그 파일이 Alloy에 노출되지 않는 문제의 원인 확인([bug] WSL에서 Docker 로그 파일 마운트가 비어 Alloy 수집 검증 실패 #267), 운영 서버 적용

🧪 테스트 / 검증

로컬

  • Command: python deploy/observability/scripts/verify-log-pipeline.py
  • Result: PowerShell에서 전체 검증 PASS. 권한 수정 후 WSL에서는 Alloy가 설정을 읽고 기동하지만, Docker -> Alloy -> Loki 단계는 여전히 타임아웃되어 #267에서 조사 중이다.

CI

Smoke / 수동 검증

테스트 공백

  • WSL 실행에서만 첫 Docker 로그 수집 단계가 실패한다. 원인은 미확정이며 #267에서 조사한다.

🧭 영향 범위

  • 설정/환경변수 변경 없음
  • 설정/환경변수 변경 있음 (항목: Alloy 이미지 버전 및 filelog receiver 설정)
  • CI/빌드 파이프라인 영향 없음
  • CI/빌드 파이프라인 영향 있음
  • 의존성 변경 없음
  • 의존성 변경 있음 (Grafana Alloy v1.19.2 → v1.21.0)

⚠️ 리스크 & 롤백

리스크

롤백

  • 단순 revert로 가능
  • 버전 pin/되돌리기 필요 (대상: Alloy 이미지와 filelog 설정, 방법: 기존 v1.19.2 설정으로 revert)
  • 배포 롤백(이전 태그/이미지) 가능

💬 리뷰 가이드

- Alloy 앱 이미지 태그를 v1.21.0으로 고정

- 검증: v1.21.0 이미지 확인 및 Alloy 설정 검증 통과
- Alloy filelog 정렬을 receiver 수준 ordering_criteria로 적용

- 파일 수집 순서를 oldest-to-newest 기준으로 고정

- 검증: Alloy v1.21.0 설정 검증 및 파이프라인 테스트 통과
- 임시 file exporter로 Alloy가 내보낸 로그 순서를 직접 확인

- 110개 회전 로그가 오래된 순서대로 전달되는지 검증

- 검증: Alloy 로그 파이프라인 통합 테스트 통과
- PEM 키와 Firebase Admin SDK 자격증명 파일을 Git 제외 대상으로 지정
- Terraform plan/state 및 로컬 API 키 백업 파일 제외
- 검증: git check-ignore로 대상 백업 파일 제외 확인
@coderabbitai

coderabbitai Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration
  • Configuration used: Repository UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 77e43c89-18e3-45f8-8ccd-66257a643ec4

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@geonusp geonusp self-assigned this Oct 19, 2026
@geonusp
geonusp marked this pull request as draft October 19, 2026 20:31
- 임시 Alloy 설정 파일을 컨테이너에서도 읽을 수 있도록 권한을 0644로 설정

- compose 설정 파일의 0600 권한과 Alloy capability 제한은 유지

- Python 문법 및 diff 검사 통과; WSL E2E는 #267과 같은 Docker 로그 수집 타임아웃으로 종료
@geonusp
geonusp marked this pull request as ready for review October 20, 2026 07:23
@geonusp geonusp added the fix label Oct 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant