[CUBRIDQA-1517] Advance submodule bump PRs one commit at a time - #7702
Merged
Merged
Conversation
✅ TC Merge Gate — Merge AllowedAll TC PRs are merged, closed, or not present. TC Repositories & Branches:
|
🧪 TC Test Environment ReadyCircleCI Testing:
TC Repositories & Branches:
Next Steps:
|
Member
Author
|
/run all |
Srltas
marked this pull request as ready for review
August 14, 2026 06:55
Contributor
|
Member
Author
|
/run all |
kisoo-han
approved these changes
Aug 14, 2026
|
✅ TC Branch Finalized for Engine PR was merged. Cleanup Results:
TC base branch is ready for the next PR. |
|
✅ TC Branch Finalized for Engine PR was merged. Cleanup Results:
TC base branch is ready for the next PR. |
hyunikn
added a commit
to hyunikn/cubrid
that referenced
this pull request
Aug 18, 2026
hyunikn
added a commit
to hyunikn/cubrid
that referenced
this pull request
Aug 18, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
http://jira.cubrid.org/browse/CUBRIDQA-1517
Purpose
기존 bump PR은 서브모듈 포인터를 생성 시점의 최신 커밋까지 한 번에 올립니다. PR이 열린 상태에서 서브모듈에 새로운 커밋이 추가되면 해당 커밋들도 같은 PR에 계속 누적됩니다. 이 때문에 CI가 실패할 경우, 여러 커밋 중 어떤 커밋이 원인인지 별도로 확인해야 했습니다.
이를 개선하기 위해 서브모듈 포인터를 한 번에 한 커밋씩만 올리도록 변경했습니다. 각 서브모듈에는 동시에 하나의 bump PR만 열리며, 각 PR에는 하나의 커밋만 포함됩니다.
Implementation
동작이 이렇게 바뀝니다.
commit_message,pushertarget_sha(필수, 40자),reanchor[키] Update <서브모듈> submodule서브모듈 develop의 커밋이 대기열이고, 부모가 기록한 커밋 SHA가 어디까지 반영했는지 가리키는 포인터입니다.
flowchart LR A[트리거] --> B{열린 bump PR 있나} B -->|있음| C[종료] B -->|없음| D[포인터 다음 커밋 1건만 반영] D --> E[PR 생성] E --> F[머지되면 포인터 이동] F --> A수동 실행은 입력 받은 SHA로 포인터를 옮깁니다. 한 PR에 여러 커밋을 한 번에 넣고 싶을 때 사용하는 기능입니다. 이미 열린 bump PR이 있으면 수동 실행 시 새 PR 대신 bump PR이 갱신됩니다.
수동 실행 방법
CUBRID/cubrid → Actions → Submodule bump (receiver) → Run workflow
developAllow a non-fast-forward re-anchor
서브모듈에서 강제 푸시로 커밋 이력이 다시 쓰으면, 기록해둔 그 커밋이 develop에서 사라지게 되므로 다음 커밋을 계산할 수 없습니다. 그 때 새 이력에 대응하는 커밋의 SHA를 다시 넣는 기능으로 평소에 쓸 일은 없습니다.
Remarks
N/A