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

Don't Block in AutoYaST Upgrade with Reboot Message [SLE-15-SP5] #1119

Merged
merged 2 commits into from
Aug 27, 2024

Conversation

shundhammer
Copy link
Contributor

@shundhammer shundhammer commented Aug 27, 2024

Target Branch

This is for SLE-15-SP5. A merge to SLE-15-SP6, -SP7 and master / Factory will follow.

Bugzilla

https://bugzilla.suse.com/show_bug.cgi?id=1181625

Trello

https://trello.com/c/jxGwssL6/

Problem

In certain situations, an AutoYaST upgrade may block with a pop-up dialog "The system will reboot now" message that does not continue after a timeout as it should; the user needs to confirm it interactively which is against the idea of an auto-upgrade.

Cause

The pop-up is suppressed only in Mode.autoinst, not also in Mode.autoupgrade.

Fix

Instead of just checking for Mode.autoinst, check for Mode.auto which includes both Mode.autoinst and Mode.autoupgrade.

See also https://github.com/yast/yast-yast2/blob/master/library/general/src/modules/Mode.rb#L283-L286

Related PRs

@coveralls
Copy link

coveralls commented Aug 27, 2024

Coverage Status

coverage: 40.913%. remained the same
when pulling 38b7553 on huha-ay-upgrade-sp5
into ecc4aeb on SLE-15-SP5.

@shundhammer shundhammer changed the title Don't Block in AutoYaST Installation or Upgrade with Reboot Message [SLE-15-SP5] Don't Block in AutoYaST Upgrade with Reboot Message [SLE-15-SP5] Aug 27, 2024
@shundhammer shundhammer merged commit 0a77ab7 into SLE-15-SP5 Aug 27, 2024
12 checks passed
@shundhammer shundhammer deleted the huha-ay-upgrade-sp5 branch August 27, 2024 09:27
@yast-bot
Copy link
Contributor

✔️ Internal Jenkins job #759 successfully finished
✔️ Created IBS submit request #342269

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.

4 participants