Skip to content

[CUBRIDQA-1517] Advance submodule bump PRs one commit at a time - #7702

Merged
Srltas merged 3 commits into
CUBRID:developfrom
Srltas:CUBRIDQA-1517
Aug 18, 2026
Merged

Srltas merged 3 commits into
CUBRID:developfrom
Srltas:CUBRIDQA-1517

Conversation

@Srltas

@Srltas Srltas commented Aug 14, 2026

Copy link
Copy Markdown
Member

http://jira.cubrid.org/browse/CUBRIDQA-1517

Purpose

기존 bump PR은 서브모듈 포인터를 생성 시점의 최신 커밋까지 한 번에 올립니다. PR이 열린 상태에서 서브모듈에 새로운 커밋이 추가되면 해당 커밋들도 같은 PR에 계속 누적됩니다. 이 때문에 CI가 실패할 경우, 여러 커밋 중 어떤 커밋이 원인인지 별도로 확인해야 했습니다.

이를 개선하기 위해 서브모듈 포인터를 한 번에 한 커밋씩만 올리도록 변경했습니다. 각 서브모듈에는 동시에 하나의 bump PR만 열리며, 각 PR에는 하나의 커밋만 포함됩니다.

Implementation

동작이 이렇게 바뀝니다.

대상 현재 변경 후
포인터 전진 폭 서브모듈 최신까지 한 번에 한 커밋씩
트리거 서브모듈 신호, 수동 실행 서브모듈 신호, 부모의 포인터 변경, 수동 실행
수동 실행 입력 commit_message, pusher target_sha(필수, 40자), reanchor
PR 제목 [키] Update <서브모듈> submodule 끝에 SHA 7자 추가
커밋 메시지 제목 한 줄 비교 링크 + 포함 커밋 전체 메시지

서브모듈 develop의 커밋이 대기열이고, 부모가 기록한 커밋 SHA가 어디까지 반영했는지 가리키는 포인터입니다.

flowchart LR
  A[트리거] --> B{열린 bump PR 있나}
  B -->|있음| C[종료]
  B -->|없음| D[포인터 다음 커밋 1건만 반영]
  D --> E[PR 생성]
  E --> F[머지되면 포인터 이동]
  F --> A
Loading

수동 실행은 입력 받은 SHA로 포인터를 옮깁니다. 한 PR에 여러 커밋을 한 번에 넣고 싶을 때 사용하는 기능입니다. 이미 열린 bump PR이 있으면 수동 실행 시 새 PR 대신 bump PR이 갱신됩니다.

수동 실행 방법
CUBRID/cubrid → Actions → Submodule bump (receiver)Run workflow

입력
Use workflow from develop
Target submodule 대상 서브모듈 선택
Submodule commit to pin 반영할 커밋의 40자 SHA
Allow a non-fast-forward re-anchor 평소에는 체크하지 않음

Allow a non-fast-forward re-anchor
서브모듈에서 강제 푸시로 커밋 이력이 다시 쓰으면, 기록해둔 그 커밋이 develop에서 사라지게 되므로 다음 커밋을 계산할 수 없습니다. 그 때 새 이력에 대응하는 커밋의 SHA를 다시 넣는 기능으로 평소에 쓸 일은 없습니다.

Remarks

N/A

@Srltas Srltas self-assigned this Aug 14, 2026
@github-actions

github-actions Bot commented Aug 14, 2026

Copy link
Copy Markdown

✅ TC Merge Gate — Merge Allowed

All TC PRs are merged, closed, or not present.

TC Repositories & Branches:

  • cubrid-testcases: No open TC PR (merged, closed, or not created)
  • cubrid-testcases-private-ex: No open TC PR (merged, closed, or not created)

@github-actions

Copy link
Copy Markdown

🧪 TC Test Environment Ready

CircleCI Testing:

  • CircleCI will automatically test using the branches below.

TC Repositories & Branches:

Next Steps:

  1. Wait for CircleCI tests to complete
  2. If CircleCI tests failed, please check the test results and fix the issues.
  3. When ready to merge this PR, please merge the TC PR first, then merge this PR.

@Srltas

Srltas commented Aug 14, 2026

Copy link
Copy Markdown
Member Author

/run all

@Srltas
Srltas marked this pull request as ready for review August 14, 2026 06:55
@greptile-apps

greptile-apps Bot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Security Review

쓰기 권한 및 GitHub App 개인 키에 접근하는 actions/create-github-app-token을 변경 가능한 major-version 태그로 참조합니다. 검토한 불변 commit SHA로 고정해 공급망 변조 시 자격 증명 탈취와 저장소 변경 위험을 차단해야 합니다.

Reviews (1): Last reviewed commit: "feat(submodule-bump): advance one commit..." | Re-trigger Greptile

Comment thread .github/workflows/submodule-bump-receiver.yml Outdated
Comment thread .github/workflows/submodule-bump-receiver.yml
@Srltas

Srltas commented Aug 14, 2026

Copy link
Copy Markdown
Member Author

/run all

@Srltas
Srltas merged commit 8c77c46 into CUBRID:develop Aug 18, 2026
12 checks passed
@Srltas
Srltas deleted the CUBRIDQA-1517 branch August 18, 2026 04:22
@github-actions

Copy link
Copy Markdown

TC Branch Finalized for cubrid-testcases-private-ex

Engine PR was merged.

Cleanup Results:

  • Branch tc/pr-7702: Deleted

TC base branch is ready for the next PR.

@github-actions

Copy link
Copy Markdown

TC Branch Finalized for cubrid-testcases

Engine PR was merged.

Cleanup Results:

  • Branch tc/pr-7702: Deleted

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
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