Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove helm chart #27

Merged
merged 2 commits into from
Oct 30, 2023
Merged

Remove helm chart #27

merged 2 commits into from
Oct 30, 2023

Conversation

NEM-NE
Copy link
Contributor

@NEM-NE NEM-NE commented Oct 17, 2023

helm chart 혼동을 방지하기 위해 구 helm chart는 legacy 브랜치에서만 개발할 수 있도록 main 브랜치에서는 삭제합니다.

context: https://vessl-ai.slack.com/archives/C04DC03NY72/p1697441509239739, https://linear.app/vessl-ai/issue/VSSL-6462/[yonsei-ai]-yonsei-cs-gpu-클러스터-셋업#comment-37a45cf8

@NEM-NE NEM-NE marked this pull request as ready for review October 17, 2023 08:31
@NEM-NE NEM-NE requested a review from Namnamseo October 17, 2023 08:31
@Namnamseo
Copy link
Contributor

helm-chart 폴더를 날린 커밋을 추가하기보다는 main 브랜치 자체를 삭제하는 게 낫지 않을까요? (git push origin -d main)

history를 남기고 싶은 생각도 드는데요, 그러면

  • (모든 파일 삭제) + (여기에 무슨 일이 있었는가를 알리는 친절한 README.md 생성)을 하는 커밋을 마지막으로 추가하고,
  • main 브랜치를 삭제하고
  • deleted-main 브랜치로 푸시하기

이런 수법도 있을 것 같습니다.


그리고 어떻게 하든간에 GitHub 상에서의 default branch 설정이 main으로 되어 있어서,
main을 날릴 거라면 이것도 뭔가 돌려놓으면 좋지 않을까 싶습니다.

얘가 앞으로 legacy <- develop으로만 쓰일 거라면 legacy 아니면 develop을 default branch로..?

@NEM-NE
Copy link
Contributor Author

NEM-NE commented Oct 17, 2023

legacy 브랜치로 옮기고 main 브랜치에서는 삭제하고 싶은 이유는

  1. tools는 계속 사용하고 있음
  2. default branch에 helm-charts를 유지하면 혼동 가능성이 있음 (e.g. yonsei-cs-gpu 셋업 시 혼동하여 디버깅에 어려움이 있었음)

인데 main 브랜치 자체를 삭제하는게 나을까요?

@Namnamseo
Copy link
Contributor

@NEM-NE
아 tools를 쓴다면... tools는 남겨두고, README.md만 하나 만들어 주시면 좋을 거 같아요
그냥 간단하게 "Please refer to PR #27." 이런 내용으로요

@NEM-NE
Copy link
Contributor Author

NEM-NE commented Oct 27, 2023

@Namnamseo 넴네

@NEM-NE NEM-NE merged commit c34dbde into main Oct 30, 2023
@Namnamseo Namnamseo deleted the sungbin/remove/helmchart branch August 23, 2024 07:29
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