-
Notifications
You must be signed in to change notification settings - Fork 27
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
Adding OVA migration flow #459
Conversation
c048dc9
to
be2a49f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we have some TODOs but let's deal with them later on.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@bkhizgiy the changes look good now but please take a look at the code-smell issues that are reported for this PR, specifically the "Define a constant instead of duplicating this literal" ones (and maybe other "easy catches") - the more things (that we're not going to backport) we solve now, the better as future backports would be easier
Signed-off-by: Bella Khizgiyaev <bkhizgiy@redhat.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dropping unused DefaultWindows and DefaultLinux - good!
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Adding OVA flow for migration