Skip to content

fix(docker): 서비스 restart 정책 보강 및 mc-observability-front nginx 프록시 추가#47

Open
dogfootman wants to merge 4 commits into
developfrom
worktree-fix+compose-restart-depends-managers
Open

fix(docker): 서비스 restart 정책 보강 및 mc-observability-front nginx 프록시 추가#47
dogfootman wants to merge 4 commits into
developfrom
worktree-fix+compose-restart-depends-managers

Conversation

@dogfootman

Copy link
Copy Markdown
Collaborator

Summary

  • docker-compose.yaml: mc-workflow-manager, mc-application-manager, mc-data-manager에 restart: unless-stopped + depends_on 추가 — 초기 기동 순서 문제 및 서버 재시작 후 미기동 현상 수정
  • docker-compose.yaml: mc-observability-rabbitmq, mc-observability-influx, mc-observability-influx-2, mc-observability-loki의 restart: on-failurerestart: unless-stopped 변경 — clean exit(0) 후에도 재시작 보장
  • nginx.template.conf / nginx.template.local.conf: mc-observability-front용 nginx 프록시 서버 블록 추가 (포트 MC_OBSERVABILITY_FRONT_PORT) — iframe이 HTTPS 컨텍스트에서 접근 가능하도록 SSL 종료
  • api.yaml: mc-observability-fe 서비스 URL 등록 — mc-web-console Monitorings 화면에서 iframe 표시에 필요

Test plan

  • ./mcc infra run 실행 후 mc-workflow-manager, mc-application-manager, mc-data-manager가 정상 기동되는지 확인
  • 서버 재시작 후 위 서비스들과 observability 관련 서비스들(rabbitmq, influx, loki)이 자동 복구되는지 확인
  • https://<IP>:<MC_OBSERVABILITY_FRONT_PORT>/embed/monitoring/default HTTPS 접근 확인
  • mc-web-console 로그인 → Workspace/Project 선택 → Monitorings 화면에서 iframe 콘텐츠 표시 확인

🤖 Generated with Claude Code

MZC-CSC and others added 4 commits June 16, 2026 10:44
feat: sync api.yaml and fix api endpoints
mc-workflow-manager, mc-application-manager, mc-data-manager 등이
초기 기동 및 서버 재부팅 시 일부 올라오지 않는 문제 수정.

근본 원인
- 일부 서비스에 restart 정책이 없어, 무거운 의존 서비스(Jenkins/Nexus/
  RabbitMQ/infra-manager/MariaDB)보다 먼저 떠서 크래시한 뒤 자력 복구 불가.
- 재부팅 시 depends_on 순서는 적용되지 않고 restart 정책이 있는 컨테이너만
  데몬이 자동 기동하므로, 정책이 없는 매니저들이 올라오지 않음.

변경
- mc-workflow-manager: restart: unless-stopped 추가, jenkins(healthy) 의존 추가
- mc-application-manager: restart: unless-stopped 추가, nexus/infra-manager
  (healthy)/rabbitmq(healthy) 의존 추가
- mc-application-manager-sonatype-nexus: restart: unless-stopped 추가
- mc-data-manager-db: restart: always 추가 (재부팅 시 DB 미기동 → data-manager
  재시작 반복 문제 해결)

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
- nginx.template.conf/local.conf에 mc-observability-front HTTPS/HTTP 프록시 서버 블록 추가 (포트 MC_OBSERVABILITY_FRONT_PORT)
- docker-compose.yaml: mc-observability-front ports 제거(nginx가 처리), mc-iam-manager-nginx에 MC_OBSERVABILITY_FRONT_PORT 포트 바인딩 추가
- docker-compose.yaml: mc-observability-rabbitmq·influx·influx-2·loki restart 정책을 unless-stopped으로 변경 (서버 재시작 후 자동 복구)
- api.yaml: mc-observability-fe 서비스 URL 등록 (iframe 표시에 필요)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

AI Direct Commit Attribution Blocked

This pull request cannot be merged because it contains commits with AI tool attribution (Claude, Anthropic, GitHub Copilot, etc.) in commit message trailers (for example Co-Authored-By, Signed-off-by, Generated-By) or in the author/committer email.

Why this is blocked

In open-source contributions, the human author is expected to create commits and open pull requests. Under current open-source license frameworks (DCO, Sign-off, CLA, etc.) AI-attributed commits may have unclear implications. Following the community proposal from 2026-05-08, until the project policy is finalized, AI direct commit signoff and co-authored-by records are prohibited in this repository.

Matched commits


### 227d207067822e0919898b0e2b970425dadf8aaf
  subject: fix(docker): mc-observability-front nginx 프록시 및 서비스 URL 등록
  - body/trailer match:
      Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

### 029dbb5dab3d2bc08e777ea53a780cea0d64c452
  subject: fix(docker): 매니저 서비스 restart 정책·depends_on 보강
  - body/trailer match:
      Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

How to fix

Most recent commit only - strip the attribution trailers:

git commit --amend --message="$(git log -1 --pretty='%s%n%n%b' | grep -vE 'Co-Authored-By:|Generated-By:|Assisted-By:|Co-Developed-By:')"
git push --force-with-lease

Multiple commits - use interactive rebase:

git rebase -i HEAD~5
# Mark each commit as `reword` and remove the trailer lines in the editor
git push --force-with-lease

The check will re-run automatically after the force-push, and the merge can proceed once the AI direct commit signoff and co-authored-by records are removed. Repository administrators may also enforce this check via Branch Protection rules.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants