Skip to content

Docs: keep the inode when editing the running IAM nginx.conf for the 504 timeout fix - #243

Merged
MZC-CSC merged 2 commits into
m-cmp:mainfrom
MZC-CSC:develop
Nov 9, 2026
Merged

MZC-CSC merged 2 commits into
m-cmp:mainfrom
MZC-CSC:develop

Conversation

@MZC-CSC

@MZC-CSC MZC-CSC commented Nov 9, 2026

Copy link
Copy Markdown
Member

What's Changed

README: keep the inode when editing the running IAM nginx.conf (504 Known Issues)

The Known Issues step for raising proxy_read_timeout on a running installation used sed -i followed by nginx -s reload. The generated nginx.conf is bind-mounted into mc-iam-manager-nginx as a single file (./container-volume/mc-iam-manager/nginx/nginx.conf:/etc/nginx/nginx.conf:ro). sed -i replaces the file with a new inode, so the container keeps reading the old content and the reload has no effect.

  • README.md / README_kr.md (Known Issues, IAM nginx 504):
    • Write the edited copy to a temporary file and copy it back with cat > so the file keeps its inode, then run nginx -t && nginx -s reload
    • Add a step to check the value nginx is actually using: nginx -T | grep proxy_read_timeout
    • Add a note explaining the single-file bind mount and that if the file was already edited with sed -i (or an editor that replaces the file), docker restart mc-iam-manager-nginx is needed instead of reload
  • Documentation only; no changes to templates, compose files, or images

Verified locally with nginx:1.25-alpine and a single-file bind mount: sed -i + reload kept the old value, docker restart applied the new value, and cat > + reload applied the new value.

yh-noh and others added 2 commits November 9, 2026 16:02
실행 중 nginx.conf 는 nginx 컨테이너에 단일 파일 bind mount 되어 있어
sed -i 로 수정하면 inode 가 바뀌어 reload 해도 반영되지 않는다.
수정본을 cat > 로 덮어써 inode 를 유지한 뒤 reload 하도록 절차를 바꾸고,
nginx -T 로 적용값을 확인하는 단계와 sed -i 를 이미 쓴 경우 restart 안내를 추가한다.
docs: readme 설명 보완 nginx.conf 수정 시 inode 유지
@MZC-CSC
MZC-CSC merged commit 3458ed3 into m-cmp:main Nov 9, 2026
1 check passed
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.

2 participants