Skip to content

docs(decisions)+rig: #708 이 되돌린 문서 4개·README·bootstrap 토큰 주석 복구 (#813) - #814

Merged
Khyojae merged 1 commit into
mainfrom
fix/restore-708-reverted-docs
Sep 23, 2026
Merged

Khyojae merged 1 commit into
mainfrom
fix/restore-708-reverted-docs

Conversation

@Khyojae

@Khyojae Khyojae commented Sep 23, 2026

Copy link
Copy Markdown
Member

무엇

#708(풀 사이징 10~20, main e4e66334)이 main 에 있던 내용을 되돌린 자리를 복구한다 — 문서 4개 + README 한 단어 + bootstrap.sh 토큰 주석.

왜 지워졌나

#708 브랜치의 되돌리기 커밋 3258ff0a(«P4/R10-b/bootstrap 6개 커밋 — 다른 경로로 이미 반영됨»)가 #635·#649·#663 계열과 겹치는 커밋을 되돌렸다. 그런데 브랜치 기준(f560dd4c)이 이미 그 내용을 품고 있어서, 되돌리기가 브랜치 사본이 아니라 main 원본을 지웠다. #708 의 삭제 줄은 전부 그 출처다. 경위 상세는 #813.

복구 방식

파일을 #708 직전으로 통째 되돌리지 않았다 — experiment-inventory.md·ai-receive-path-scaling.md 는 #708 뒤에 #742(축 B)·#751 도 고쳤기 때문이다(통째로 되돌리면 같은 사고를 반복한다). 대신 #708 의 diff 만 역적용(git apply -R --3way, 전부 clean) — 뒤 커밋 내용은 그대로 남는 것을 확인했다.

파일 복구한 것
docs/decisions/replication-lag-and-semisync.md #663 의 다른 AZ 대조 라운드 §0-4 설계·§0-5 회수. main 은 결과(replication-aws-cross-az-2026-09-02/)가 있는데 «다른 AZ 는 안 쟀다» 로 적혀 있었다
docs/decisions/r10-loadgen-topology.md #649 의 §9 R10-b 회수·상태 줄(«R10-b 는 미룬다» 로 남아 있었다)
docs/decisions/ai-receive-path-scaling.md 머리말 R10-b 반증 문장
docs/decisions/experiment-inventory.md 09-01 갱신(max_workers=10 행·오진 정정)·#663 의 P4 행
loadtest/aws/README.md 문구 한 단어
loadtest/aws/bootstrap.sh #578 토큰 주석·.env 블록. 동작은 안 깨져 있었다 — heredoc 위의 랜덤 생성이 먼저 돌아 unused-in-this-role 로 떨어질 일이 없었다. 주석이 거꾸로 말하던 것만

안 한 것

검증

Refs #813

🤖 Generated with Claude Code

#708(풀 사이징 10~20, e4e6633) 브랜치의 되돌리기 커밋 3258ff0 가 «main 에 이미 있다»며
#635·#649 계열 커밋을 되돌렸는데, 브랜치 기준(f560dd4)이 이미 그 내용을 품고 있어서
스쿼시 결과가 main 의 원본까지 지웠다. 그 뒤 이 자리를 다시 채운 커밋은 없다.

복구 방식: 파일 통째 되돌리기가 아니라 #708 의 diff 만 역적용(git apply -R --3way).
그래서 #708 이후 같은 파일을 고친 #742·#751 내용은 그대로 남는다(검증함).

- replication-lag-and-semisync.md — #663 의 다른 AZ 대조 라운드(§0-4 설계·§0-5 회수) 복구.
  지금 main 은 결과 문서가 있는데도 «다른 AZ 는 안 쟀다» 로 적혀 있었다
- r10-loadgen-topology.md — #649 의 §9 R10-b 회수·상태 줄 복구(«R10-b 는 미룬다» 로 남아 있었다)
- ai-receive-path-scaling.md — 머리말의 R10-b 반증 문장 복구
- experiment-inventory.md — 09-01 갱신(max_workers=10 행·오진 정정 기록)과 #663 의 P4 행 복구
- loadtest/aws/README.md — 문구 한 단어
- bootstrap.sh — #578 토큰 주석·.env 블록 복구. 동작은 안 깨져 있었다(heredoc 위에서 랜덤
  생성이 먼저 돌아 `unused-in-this-role` 로 떨어질 일이 없었다) — 주석이 거꾸로 말하던 것만

run_all.sh 의 R10-b 원격 배선은 방향 결정이 필요해 여기 안 넣었다(#813).

Refs #813

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Sep 23, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 4 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Repository: Shadowfit/init/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 36bab052-26a6-4dde-ab2e-a8965f425e3f

📥 Commits

Reviewing files that changed from the base of the PR and between ed9b8a1 and 12779f7.

📒 Files selected for processing (6)
  • docs/decisions/ai-receive-path-scaling.md
  • docs/decisions/experiment-inventory.md
  • docs/decisions/r10-loadgen-topology.md
  • docs/decisions/replication-lag-and-semisync.md
  • loadtest/aws/README.md
  • loadtest/aws/bootstrap.sh

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.

@Khyojae

Khyojae commented Sep 23, 2026

Copy link
Copy Markdown
Member Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Sep 23, 2026 •

Copy link
Copy Markdown
⚠️ Action not completed

Review rate limited.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@Khyojae
Khyojae merged commit c355d4a into main Sep 23, 2026
3 checks passed
@Khyojae
Khyojae deleted the fix/restore-708-reverted-docs branch September 23, 2026 14:53
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.

1 participant