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

[Cloudit] AddRule, RemoveRule 기능 추가 및 핸들러 기능 개선 #610

Merged
merged 1 commit into from
Apr 21, 2022
Merged

[Cloudit] AddRule, RemoveRule 기능 추가 및 핸들러 기능 개선 #610

merged 1 commit into from
Apr 21, 2022

Conversation

inno-cloudbarista
Copy link
Contributor

  • AddRule, RemoveRule 기능 추가
    • 시험 항목에 대한 테스트
      • Cloudit은 TCP/UDP 2가지만을 제공(ICMP 미제공)
      • ICMP 외 테스트 완료
    • portRange CB <-> Cloudit 포매팅 수정
      • portRange의 all CB => -1 or 1-65535 => Cloudit 1-65535
    • Rule 이름 변경
      • 기존 index 방식 시 추가 삭제시 중복 가능성 존재 => 랜덤 값으로 변경
      • Rule 이름 중복이여도 Error는 발생하지 않음
  • 핸들러 기능 개선
    • 핸들러 별로 SystemId, NameId 기준이 다른 경우 개선
      • 테스트 용이성 및 기능 통일
      • 적용 핸들러 VM, VPC, SecurityGroup

@tjeom-inno
[Cloudit] AddRule, RemoveRule 기능 추가 및 핸들러 기능 개선
4d32ed6 
- AddRule, RemoveRule 기능 추가
   - 시험 항목에 대한 테스트
	   - Cloudit은 TCP/UDP 2가지만을 제공(ICMP 미제공)
	   - ICMP 외 테스트 완료
   - portRange CB <-> Cloudit 포매팅 수정
      -  portRange의 all CB => -1 or 1-65535 => Cloudit 1-65535
	- Rule 이름 변경 
       - 기존 index 방식 시 추가 삭제시 중복 가능성 존재 => 랜덤 값으로 변경
       - Rule 이름 중복이여도 Error는 발생하지 않음
 - 핸들러 기능 개선 
	- 핸들러 별로 SystemId, NameId 기준이 다른 경우 개선
		- 테스트 용이성 및 기능 통일 
		-  적용 핸들러 VM, VPC, SecurityGroup
@powerkimhub powerkimhub self-requested a review April 21, 2022 00:07
@powerkimhub powerkimhub merged commit e249595 into cloud-barista:master Apr 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@powerkimhub powerkimhub powerkimhub approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@inno-cloudbarista @powerkimhub @tjeom-inno