Skip to content

Feat/#13#14

Open
breakpack wants to merge 3 commits into
devfrom
feat/#13
Open

Feat/#13#14
breakpack wants to merge 3 commits into
devfrom
feat/#13

Conversation

@breakpack
Copy link
Copy Markdown
Contributor

@breakpack breakpack commented Aug 18, 2025

🔥 Related Issue

🏃‍ Task

  • dto 작성 📍 관련커밋: {[c9156bb]}
  • Cluster Owner table 이용 로직 작성 및 자원 조회 시 사용자 확인 로직 개발, 가드및 로거 적용 📍 관련커밋: {[ca7a484]}

💬 To Reviews

📄 Reference

  • None

@breakpack breakpack linked an issue Aug 18, 2025 that may be closed by this pull request
4 tasks
@breakpack breakpack requested a review from hyni03 August 18, 2025 09:27
Copy link
Copy Markdown
Collaborator

@hyni03 hyni03 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

자원 테이블을 생성 로직이 필요 없을 것 같습니다!
또한 자원 관련은 조회 로직만 필요하므로 resource.query로 하나로 묶으면 좋을 것 같아요~

@breakpack
Copy link
Copy Markdown
Contributor Author

breakpack commented Aug 19, 2025

api 만들면서 생긴 문젠데 클러스터를 조회할수 있는 유저가 여러명인 경우가 있습니다 이경우 어떻게 해야할까요?
예시로 1번 클러스터를 user1, user2가 조회한다고 했을때, 현재 db구조에서는 클러스터가 하나의 uid만 가지고 있게 설계되어 있어 조회가 불가능 하지 않을까요?

생각해보니 저번에 클러스터에서 유저를 조회할 수 있게 한 이유가 이거였어여....

@breakpack
Copy link
Copy Markdown
Contributor Author

조회관련해서 DB 구성 수정있습니다 노션 확인 부탁드려요 :D
자원 조회하는데 있어서 사용자 확인이 필요하다 판단되어 AuthGarud 활용해서 사용자 파악할 수 있게 만들었습니다

@breakpack
Copy link
Copy Markdown
Contributor Author

네임스페이스 기준 Pod 조회도 만들어 둘게요

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

중복 파일이 있는 것 같습니다!
삭제 부탁드려요~

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DB 관련 Query는 DB/query에 정의 후 사용해주세요.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

clusterowner -> cluster_user 로 해서 cu_cid가 좋을 것 같아요

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.

Feat: API 개발_자원조회

2 participants