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

[WIP] reduce cluster_id length #94

Merged
merged 1 commit into from
Jul 7, 2022

Conversation

seungkyua
Copy link
Contributor

user cluster 에 접근하기 위해서 user cluster 의 kubeconfig 를 받아오는 부분이 있는데, admin 에서 관리하는 user cluster 의 namespace 가 cluster id 8자리로 줄어들면서 namespace 를 못찾는 문제가 발생했다.
cluster id 를 substr 으로 8자리로 자르는 로직을 추가했음.
e2e 테스트가 성공하는 것을 확인하고 merge 해야 함.

@mrchypark
Copy link

최근 논의에서 클러스터 id를 "C+대소영문자와숫자 8자리" 로 변경하였으므로 8자리로 자르면 안되고 처음부터 9자리로 제공될 것으로 보입니다.

@seungkyua
Copy link
Contributor Author

최근 논의에서 클러스터 id를 "C+대소영문자와숫자 8자리" 로 변경하였으므로 8자리로 자르면 안되고 처음부터 9자리로 제공될 것으로 보입니다.

일단 e2e 테스트를 통과해야 하기 때문에 먼저 수정했습니다. 이게 되면 나중에 다른 곳 포함하여 cluster id 바꿀 때 한 번에 수정해야 합니다.

@ktkfree ktkfree mentioned this pull request Jul 7, 2022
@ktkfree ktkfree merged commit 255bc82 into openinfradev:main Jul 7, 2022
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.

None yet

3 participants