Add terminate, refine to delete cluster

[vlatte]

• 클러스터 삭제 시, mcis terminate 실행 후 mcis delete 실행 (mcis delete 실패 시, mcis refine 후 다시 mcis delete)
• 삭제하려는 mcis가 존재하지 않을 때 해당 cluster의 meta-data 삭제

[sykim-etri]

@vlatte 번거로우시겠지만 각 커밋 메시지를 아래와 같이 수정 부탁드립니다. 제가 해보려니 PR의 commit message 수정하는 방법을 잘 모르겠네요. 이것저것 해봐도 잘 안되네요..--;

feat: terminate and refine MCIS before deleting a cluster feat: when deleting a cluster, delete the cluster metadata if MCIS does not exist

[seokho-son]

@sykim-etri 참고하세요.. ^^

Changing the message of older or multiple commit messages

If you need to amend the message for multiple commits or an older commit, you can use interactive rebase, then force push to change the commit history.

On the command line, navigate to the repository that contains the commit you want to amend.

Use the git rebase -i HEAD~n command to display a list of the last n commits in your default text editor.

# Displays a list of the last 3 commits on the current branch
$ git rebase -i HEAD~3
The list will look similar to the following:

pick e499d89 Delete CNAME
pick 0c39034 Better README
pick f7fde4a Change the commit message but push the same commit.

# Rebase 9fdb3bd..f7fde4a onto 9fdb3bd
#
# Commands:
# p, pick = use commit
# r, reword = use commit, but edit the commit message
# e, edit = use commit, but stop for amending
# s, squash = use commit, but meld into previous commit
# f, fixup = like "squash", but discard this commit's log message
# x, exec = run command (the rest of the line) using shell
#
# These lines can be re-ordered; they are executed from top to bottom.
#
# If you remove a line here THAT COMMIT WILL BE LOST.
#
# However, if you remove everything, the rebase will be aborted.
#
# Note that empty commits are commented out
Replace pick with reword before each commit message you want to change.

pick e499d89 Delete CNAME
reword 0c39034 Better README
reword f7fde4a Change the commit message but push the same commit.
Save and close the commit list file.

In each resulting commit file, type the new commit message, save the file, and close it.

When you're ready to push your changes to GitHub, use the push --force command to force push over the old commit.

$ git push --force example-branch

[sykim-etri]

@seokho-son 해당 pr 브랜치(upstream/pr/54)를 local(pr-54)로 가져온 뒤에 다시 push하니 pr-54 branch가 push되어서 애매한 상황이 발생하였습니다.--; 해당 pr 브랜치로 push하는 방법이 있을까요? 아래 사이트를 참고하긴 했습니다만. https://www.lesstif.com/gitbook/github-pr-pull-request-merge-local-30704954.html

[seokho-son]

@sykim-etri 일단 해당 작업 (커밋 메시지를 변경하는 등)을 위해서 rebase 하는 것은 커밋 히스토리를 바꾸는 것이기 때문에 force로 푸시를 하셔야 할 것입니다. (누가 작업하시든~)

그리고, 기여자가 자신의 origin에 다시 푸시하는 것이 아니라 다른 사람이 작업하는 경우에는, 메인테이너 권한을 가져야 가능할 것입니다. 가능한 기여자께서 자체 처리하도록 유도하는 것이 바람직합니다.

일단 방법을 공유 드립니다.

$ git fetch upstream pull/54/head:pr-54

업스트림의 PR을 직접 패치(pull/54/head)하고 로컬 브랜치로 만듬(pr-54)

$ git checkout pr-54

$ (work hard… change, commit, squashing?)

$ git push --force git@github.com:vlatte/cb-ladybug pr-54:change_delcluster

git@github.com:vlatte/cb-ladybug 의 change_delcluster 브랜치에 작업된 pr-54 로컬 브랜치를 푸시

→ Upstream의 PR에 자동으로 반영됨.

[seokho-son]

@sykim-etri 음.. 참고로 커밋 메시지 히스토리를 잘 관리하면 좋긴 한데.. 지금 클라우드 바리스타는 FW 마다 커밋 히스토리 관리 상태가 다 다른 상황이라.. (한글 커밋이 포함되고 있는 FW도 있고.. 중복된 커밋 메시지가 많은 경우도 있습니다..) 아직 열심히 관리가 필요할지는 잘 모르겠습니다. ㅠㅠ 한번 판단해보시면 좋을 것 같고, 상황만 공유 드립니다.

[CB-TB 한정 커밋 메시지 관리 가이드] TB의 경우에는 영문 커밋 메시지를 사용하고, 중복된 커밋 메시지를 사용하지 않으며, (이 경우 정정) 1개의 PR에 관련된 커밋들만 가능한 수용하고 있습니다. 특별한 상황이 아니면 커밋 메시지 변경 요청은 하고 있지 않으며 PR에 과하게 의미없는 커밋이 많이 포함된 경우에만, 커밋 스쿼싱을 요청하고 있습니다.

[sykim-etri]

@seokho-son 커밋이 잘 관리되어 있으면 개발에 직접 관여하지 않더라도 비교적 쉽게 코드를 따라갈 수 있는 것 같습니다. 또한 깔끔한 히스토리는 후속 기여자들을 유도하는데 도움이 될 것으로 기대합니다. 현재는 CB-LB에서 적용하는 자체 commit style guide를 지키는 수준에서만 관리가 되고 있습니다.

[sykim-etri]

@vlatte $ git push --force git@github.com:vlatte/cb-ladybug pr-54:change_delcluster에서 먼가 퍼미션 이슈가 있어서 제대로 안되는 것 같습니다. 번거로우시겠지만 해당 commit 메시지 수정을 부탁드립니다.--;

[vlatte]

@sykim-etri 수정했습니다.