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

Revert "use the first group of the cluster UUID as the cluster name" #104

Merged
merged 3 commits into from
Jul 12, 2022

Conversation

zugwan
Copy link
Contributor

@zugwan zugwan commented Jul 11, 2022

클러스터 ID 체계가 변경되어 UUID를 사용하지 않기 때문에 관련 수정 사항을 원복합니다.

@seungkyua
Copy link
Contributor

수정 하는 김에 workflow template parameter 로 들어오는 substring 부분도 제거하는 것이 어떨�까요?
혹시 decapod-flow 에서도 자르는 부분이 있으면 제거하는 것이 좋겠습니다.

  • name: app_prefix
    value: "{{=sprig.substr(0, 8, workflow.parameters.cluster_id)}}"

@ktkfree
Copy link
Contributor

ktkfree commented Jul 11, 2022

이거 main merge 후 tks pr 들로 테스트하겠습니다.
당분간(금일~내일)까지는 e2e 테스트 실패 예정입니다.
위에 승규님 코멘트 확인해주시면, merge 하겠습니다

@zugwan
Copy link
Contributor Author

zugwan commented Jul 12, 2022

수정 하는 김에 workflow template parameter 로 들어오는 substring 부분도 제거하는 것이 어떨�까요? 혹시 decapod-flow 에서도 자르는 부분이 있으면 제거하는 것이 좋겠습니다.

* name: app_prefix
  value: "{{=sprig.substr(0, 8, workflow.parameters.cluster_id)}}"

말씀하신 내역까지 모두 반영하였습니다. decapod-flow에는 관련 사항이 존재하지 않습니다.

@ktkfree
Copy link
Contributor

ktkfree commented Jul 12, 2022

이 내용으로 merge 후에 tks apis 와 같이 테스트 진행하겠습니다.

@ktkfree ktkfree merged commit 3062fb7 into main Jul 12, 2022
@ktkfree ktkfree deleted the revert_clusternaming branch July 12, 2022 06:17
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.

3 participants