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

auto detect kubevirt install #4791

Merged
merged 6 commits into from
Dec 6, 2024
Merged

auto detect kubevirt install #4791

merged 6 commits into from
Dec 6, 2024

Conversation

changluyi
Copy link
Collaborator

@changluyi changluyi commented Dec 4, 2024

Pull Request

What type of this PR

Examples of user facing changes:

  • Features
    有可能kubevirt 比 kube-ovn 晚安装,这个迁移优化的功能就没有及时激活,不太好用,所以想加一个监听kubevirt CRD安装,一旦监听到就开启kubevirt相关informer

本来想做成能够在kubevirt 卸载后,kube-ovn自动关闭informerfactory,然后重新安装,再次启动。但
碰到了一些问题,informerFactory start了以后,通过给stopChan 发消息来关闭,执行shutdown。然后再start ,informer似乎就不会再去监听了。

所以这个PR 就暂时没有考虑 kubevirt 卸载的情况。

  • Bug fixes
  • Docs
  • Tests

Which issue(s) this PR fixes

Fixes #(issue-number)

@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Dec 4, 2024
@coveralls
Copy link

coveralls commented Dec 4, 2024

Pull Request Test Coverage Report for Build 12191235176

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 20 (0.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.003%) to 18.675%

Changes Missing Coverage Covered Lines Changed/Added Lines %
pkg/controller/controller.go 0 3 0.0%
pkg/controller/kubevirt.go 0 17 0.0%
Totals Coverage Status
Change from base Build 12153017747: -0.003%
Covered Lines: 8682
Relevant Lines: 46490

💛 - Coveralls

@changluyi changluyi requested a review from bobz965 December 4, 2024 08:53
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Dec 4, 2024
@changluyi changluyi requested a review from zhangzujian December 5, 2024 02:38
Signed-off-by: clyi <[email protected]>
Signed-off-by: clyi <[email protected]>
Signed-off-by: clyi <[email protected]>
@changluyi changluyi merged commit 322fb8e into master Dec 6, 2024
68 checks passed
@changluyi changluyi deleted the monitor_kubevirt_migration branch December 6, 2024 08:20
changluyi added a commit that referenced this pull request Dec 27, 2024
* auto detect kubevirt install

---------

Signed-off-by: clyi <[email protected]>
changluyi added a commit that referenced this pull request Dec 27, 2024
* auto detect kubevirt install

---------

Signed-off-by: clyi <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm This PR has been approved by a maintainer size:M This PR changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants