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.

Sign up for GitHub

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

Jump to bottom

[Alibaba, Tencent] Bug fix for Tag Test #1308

Merged
merged 5 commits into from
Aug 29, 2024
Merged

[Alibaba, Tencent] Bug fix for Tag Test #1308

merged 5 commits into from
Aug 29, 2024

Conversation

SungWoongz
Copy link
Contributor

@SungWoongz SungWoongz commented Aug 28, 2024 •
edited
Loading

@powerkimhub

  • Tag Test 진행 중 발견된 버그들에 대한 픽스입니다.

  • Alibaba

    • CommonHander.go
      • IID를 읽지 못함 -> request 수정
    • DiskHandler.go
      • resourceType=disk를 읽지 못함 -> irs.RSTYPE 수정
    • SecurityHandler.go
      • TagList 추출하여 반환하도록 코드 추가

  • Tencent

    • NLBHandler.go
      • TagList 추출하여 반환하도록 코드 추가

@powerkimhub
Copy link
Member

@SungWoongz

  • 현 PR로 해결되는 기존 이슈가 있을 까요?

@SungWoongz
Copy link
Contributor Author

@powerkimhub

  • 현재 tag test와 zone based테스트를 위해 merge가 필요합니다.
  • 확인부탁드립니다.

@powerkimhub powerkimhub merged commit 9e9e5b2 into cloud-barista:master Aug 29, 2024
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@SungWoongz @powerkimhub