[sle][functional][u][medium] test fails in shutdown on minimalx
openQA test in scenario sle-15-SP1-Installer-DVD-x86_64-create_minimalX@64bit fails in
Everytime I try to run the shutdown module on minimalx it gets stuck, the linked test is an create_hdd_gnome with DESKTOP=minimalx.
This is NOT a simple needle issue, I already tried to create some, also simply setting SHUTDOWN_NEEDS_AUTH=1 does not work.
The adaption needs to be done in the power_utils.
We should also discuss why this is not being tested on openQA.
Basically everytime we execute the shutdown module on minimalx
Fails since (at least) Build 136.2
Obviously the shutdown module should work on all desktop managers
- Add shutdown in main.pm for minimalx (0.1h - 0.5h)
- try clicking through the shutdown process (0.5h -1.0h)
- if necessary workaroung bugs (0.0h-2.0h)
#2 Updated by okurz over 2 years ago
Thanks for the report. However, if we never had a test executing "shutdown" on icewm the ticket is rather a request for new test coverage, not a regression, right?
I checked "minimalx" on openSUSE Tumbleweed, https://openqa.opensuse.org/tests/829126, and also there I can not see "shutdown" being scheduled.
#3 Updated by jorauch over 2 years ago
In theory we have the code, but it is never used and it does not work (on SLE15), so I would rather vote for 'enhancement'
My primary intention is to investigate and fix the shutdown module, scheduling it a side bonus to avoid running in something like this again
#9 Updated by jorauch over 2 years ago
First part which is fixing the code
Still needs to be scheduled in main.pm
#11 Updated by jorauch over 2 years ago
This PR schedules the shutdown tests on minimalx:
Now waiting for merges and verification in production to close this ticket.
Setting to feedback meanwhile
#13 Updated by okurz over 2 years ago
- Status changed from Resolved to Feedback
Please stick to our https://progress.opensuse.org/projects/openqatests/wiki#Definition-of-DONE and keep the ticket open until the according PR is merged, deployed and we have verification runs on production.
#16 Updated by jorauch over 2 years ago
#17 Updated by jorauch over 2 years ago
Looks like the test failed to select the console properly:
#19 Updated by jorauch over 2 years ago
- Status changed from Feedback to Workable
- Priority changed from Normal to High
Shutdown does still not work properly:
Without the workaround:
#21 Updated by okurz over 2 years ago
I see shutdown on minimalx previously working on
and now failing in
As currently there seem to be too many failures related to shutdown on minimalx I removed it again from the schedule as we had discussed already on friday but then did not revert:
already merged, will retrigger the failed staging tests.
#22 Updated by okurz over 2 years ago
I retriggered about 100 jobs, mainly "cryptlvm" and "minimalx", not only Leap 15.1 staging but also Factory staging, aarch64 build validation jobs as well as maintenance tests. The outreach was wider than I thought.
jorauch based on this we can now more carefully test the different scenarios when you want to bring back the test module. Let's talk next week how to continue as well.
#24 Updated by okurz over 2 years ago
- Due date deleted (
- Priority changed from High to Normal
- Target version changed from Milestone 23 to Milestone 24
Yes, thank you for that. Actually with another fix, by DimStar, in https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/6887 we can set it to "Normal" and remove the Due-Date. So actually you can relax ;)
#26 Updated by okurz over 2 years ago
Could you please clarify at what point we are?
- Is the shutdown module now scheduled?
yes, in case of
- What needs to be done here at all?
Add it back to the cases of
!INSTALLONLY and make sure it works in a stable manner in the scenarios where it was reported as failing. Also, work around bugs with proper record_soft_failure.
#31 Updated by jorauch over 2 years ago
- Status changed from In Progress to Feedback
#32 Updated by okurz over 2 years ago
Looks promising so far. Do you have a PR where you referenced these tests?
I guess if you can make http://pinky.arch.suse.de/tests/108#step/shutdown/23 not fail you are good to go :)
#34 Updated by jorauch over 2 years ago
Test with PPC and cryptlvm
#36 Updated by jorauch over 2 years ago
New try after history got scrambled:
#39 Updated by jorauch over 2 years ago
Here we have another failure (360sec timeout):
#40 Updated by jorauch over 2 years ago
After sleeping a night over the results I come to the conclusion that we ran in the wrong direction here, we should go back to the initial apporach of detecting unwanted popups instead of wanting to discover a failed shutdown. This avoids a lot of trouble with corner cases
#42 Updated by jorauch over 2 years ago
- Status changed from In Progress to Feedback
#45 Updated by jorauch over 2 years ago
- Status changed from Feedback to Resolved
Fixed the only fail in other_de:
I guess we can consider this resolved?
#46 Updated by jorauch over 2 years ago
- Status changed from Resolved to Workable
Somehow still broken:
Behavior seems weirder than expected
#51 Updated by jorauch over 2 years ago
- Status changed from Feedback to In Progress
Actually it shuts down despite the popup, we could use check_shutdown in the branch, but the timeout would be somehow random
#52 Updated by jorauch over 2 years ago
Opensuse shuts down normally on SLE I have the following behaviour on the latest build:
I really do not understand when it just shuts down, when it shuts down with a popup and when the shutdown gets blocked
Maybe a detection for "Authentication is required to shut down while users are logged in" could help?
#55 Updated by okurz over 2 years ago
- Priority changed from Normal to Low
- Target version changed from Milestone 24 to Milestone 25
We should wait at least until after the RC with this
I guess we can even wait for longer and do not need to hurry. If you think it's safer, we can implement it post-SLE15SP1GM
#56 Updated by jorauch over 2 years ago
- Status changed from In Progress to Workable
I agree, maybe the behaviour has stabilized until then