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

[AdminWeb] Need to improve the name shown in resource zone settings #1415

Closed
Jeon-Jinhyeok opened this issue Jan 9, 2025 · 4 comments · Fixed by #1416
Closed

[AdminWeb] Need to improve the name shown in resource zone settings #1415

Jeon-Jinhyeok opened this issue Jan 9, 2025 · 4 comments · Fixed by #1416
Assignees
Labels
AdminWeb enhancement New feature or request

Comments

@Jeon-Jinhyeok
Copy link
Member

Jeon-Jinhyeok commented Jan 9, 2025

What would you like to be enhanced:
adminweb에서 리소스에 대한 zone 설정시에, zone의 display name이 보이는 상황으로, 모두 같은 이름을 하고있어 구분이 어려운 상황입니다.

Zone 1 Display Name Zone 2 Display Name

Why is this needed
: Zone Name을 사용하거나, Display Name을 적절히 변경하여 사용자로 하여금 zone을 구분하기 용이하도록 할 필요가 있어 보입니다.

Proposed solution
: Display Name(Zone Name) 처럼 결합하여 보여주면 어떨지 생각해봅니다.

@Jeon-Jinhyeok Jeon-Jinhyeok added the enhancement New feature or request label Jan 9, 2025
@Jeon-Jinhyeok Jeon-Jinhyeok changed the title [adminweb] Need to improve the name shown in resource zone settings [AdminWeb] Need to improve the name shown in resource zone settings Jan 9, 2025
@powerkimhub
Copy link
Member

@Jeon-Jinhyeok

  • Thanks for your proposal
  • Related issue: [Metadata: Region/Zone] Enhance Supported Region and Zone Info #1389
  • 다음 링크에서
    • API 출력 결과 중 CSPDisplayName 값을 참고해보시기 바랍니다.
    • 전체 CSP(일부 미반영 CSP 제외)의 CSPDisplayName을 확인해보신 후에
  • CSPDisplayName 값으로 issue 해결에 적합한지 아니면, 정보 보완이 필요한지 확인 후 의견 부탁드립니다.

@Jeon-Jinhyeok
Copy link
Member Author

Jeon-Jinhyeok commented Jan 9, 2025

image
image
{csp}_region_meta.yaml 파일을 다음과 같이 수정했을 때 명확하게 구분이 가는 것 같습니다.
하드코딩으로 변경할 것인지, Display Name과 Zone Name을 조합하여 보여주는 함수를 추가하는 것이 좋을지 고민해봐야할 것 같습니다.

@powerkimhub
Copy link
Member

@Jeon-Jinhyeok


  • 현재 Rest API는 유지 하고, AdminWeb 관련 페이지에서,
  • Name(DisplayName)처럼 조합해서 제공하는 것이 좋을 것 같습니다.
    • 추가로, Zone Status 정보(Available) 삭제 포함
      • 사유: 실제로 Status 정보가 잘 활용되지 않는 것으로 보임
  • 화면 출력 예시
  us-east-2a(United States, Ohio)
  us-east-2b(United States, Ohio)
  ...

@Jeon-Jinhyeok
Copy link
Member Author

생성 시 Zone 정보를 선택해야 하는 리소스의 경우 다음과 같이 보일 수 있도록 수정하였습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AdminWeb enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants