Skip to content

move instHandler out#99

Merged
kwonkwonn merged 1 commit intomainfrom
controll_include_instHandler
Mar 3, 2026
Merged

move instHandler out#99
kwonkwonn merged 1 commit intomainfrom
controll_include_instHandler

Conversation

@kwonkwonn
Copy link
Contributor

Type of Change

  • Bug fix
  • New feature
  • Refactor / tech debt
  • Docs / test

Summary

libvirt.Con이 의미없이 인자로 전달되는 것이 아닌 내부 필드로 세팅후 참조할 수 있도록 수정

Related Issue

Closes #98

Test Plan

  • make test passes
  • Manually verified on libvirt environment

@kwonkwonn kwonkwonn requested review from Jbchan01 and Copilot and removed request for Copilot March 2, 2026 08:29
@kwonkwonn
Copy link
Contributor Author

Inst가 Get에서 사용되는 경우는 많지가 않음.
GetAllInstance 나 GetAllbyState등등..
그래서 이것두 interface 화 해서 나중에 테스트할때 사용하게 할것

@kwonkwonn
Copy link
Contributor Author

최종적으론 GenerateFactory*에서도 inst를 인자로 받지 않게 해서, instHandler 를 아예지우는 것이 목표.

@kwonkwonn kwonkwonn merged commit f25a1b3 into main Mar 3, 2026
7 checks passed
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.

Domain Control 마이그레이션

2 participants