-
Notifications
You must be signed in to change notification settings - Fork 23
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
Fix migration sp5 #684
Fix migration sp5 #684
Conversation
@@ -226,7 +226,8 @@ def propose_for_update(force_reset) | |||
# old one is grub2, but mismatch of EFI and non-EFI (bsc#1081355) | |||
elsif old_bootloader =~ /grub2/ && old_bootloader != current_bl.name | |||
raise MismatchBootloader.new(old_bootloader, current_bl.name) | |||
elsif !current_bl.proposed? || force_reset | |||
elsif force_reset | |||
log.warn "forced proposel during migration. Will change existing config" |
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.
Typo: s/proposel/proposal/
Are we also going to tell the user about this forced reset?
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.
LGTM
✔️ Internal Jenkins job #754 successfully finished |
Problem
User selection was ignored during migration.
Solution
Drop legacy code for SLE11 migration and reset proposal only if it is forced.
Testing