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

refactor(vm): wrap KVMI ready reasons #565

Merged
merged 2 commits into from
Dec 11, 2024
Merged

Conversation

danilrwx
Copy link
Contributor

@danilrwx danilrwx commented Dec 10, 2024

Description

Wrap reason for VirtualMachineInstanceReady condition

Why do we need it, and what problem does it solve?

Remove deprecated code

What is the expected result?

Same behavior, but with our types

Checklist

  • The code is covered by unit tests.
  • e2e tests passed.
  • Documentation updated according to the changes.
  • Changes were tested in the Kubernetes cluster manually.

Signed-off-by: Daniil Antoshin <[email protected]>

fix

Signed-off-by: Daniil Antoshin <[email protected]>
@danilrwx danilrwx force-pushed the feat/vm/wrap-ready-reasons branch from ed11356 to b9acb12 Compare December 10, 2024 14:02
Signed-off-by: Daniil Antoshin <[email protected]>
@danilrwx danilrwx changed the title feat(vm): wrap VM ready reasons refactor(vm): wrap VM ready reasons Dec 11, 2024
@danilrwx danilrwx changed the title refactor(vm): wrap VM ready reasons refactor(vm): wrap KVMI ready reasons Dec 11, 2024
@danilrwx danilrwx merged commit afe682a into main Dec 11, 2024
12 checks passed
@danilrwx danilrwx deleted the feat/vm/wrap-ready-reasons branch December 11, 2024 09:52
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.

2 participants