Bug 39137 - univention-welcome-screen: cleanup script not executable
univention-welcome-screen: cleanup script not executable
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: System setup
UCS 4.0
Other Linux
: P5 normal (vote)
: UCS 4.0-3-errata
Assigned To: Alexander Kramer
Erik Damrose
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2015-08-10 18:57 CEST by Erik Damrose
Modified: 2015-11-04 17:24 CET (History)
3 users (show)

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Erik Damrose univentionstaff 2015-08-10 18:57:53 CEST
setup.log contains a traceback, as the script 98_start_univention_welcome_screen can not be executed - the executable bit is not set.

The welcome screen does start however, as univention-welcome-screen installs its init script with S99... We could simply remove the script.

= Running 98_start_univention_welcome_screen
Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.7/univention/management/console/modules/setup/util.py", line 593, in run_scripts_in_path
    subprocess.call(os.path.join(path, filename), stdout=logfile, stderr=logfile)
  File "/usr/lib/python2.7/subprocess.py", line 493, in call
    return Popen(*popenargs, **kwargs).wait()
  File "/usr/lib/python2.7/subprocess.py", line 679, in __init__
    errread, errwrite)
  File "/usr/lib/python2.7/subprocess.py", line 1259, in _execute_child
    raise child_exception
OSError: [Errno 13] Permission denied
Comment 1 Alexander Kramer univentionstaff 2015-09-08 10:09:06 CEST
As discussed with Erik and Florian I made 98_start_univention_welcome_screen executable instead of deleting it.

There should be no problem with starting the service a second time during the init script.

ucs4.0-3
r 63501
Package: univention-welcome-screen
Version: 0.0.0-12.11.201509080956
r 63503 yaml file

ucs4.1-0
r 63502
Comment 2 Alexander Kramer univentionstaff 2015-09-08 10:52:07 CEST
Package: univention-welcome-screen
Version: 1.0.0-1.12.201509081000
Comment 3 Erik Damrose univentionstaff 2015-09-24 12:25:47 CEST
(In reply to Alexander Kramer from comment #1)
> There should be no problem with starting the service a second time during
> the init script.
Reopen:
The init script seems to ignore other running welcome screen instances. After system setup a firefox with 2 tabs is running, both showing the welcome screen. This is a regression, because one can access the tabs at the top of the screen.

If the init script is called from the console, a second stack of firefox + webserver is started, which overlays the current running one.

The init script should check if a welcome screen is already running, and fail with a message that another instance is already running
Comment 4 Florian Best univentionstaff 2015-09-24 12:31:58 CEST
Why does this check doesn't work:?
 44 »   »   if [ -e "/var/run/univention-welcome-screen.pid" ]; then
 45 »   »   »   pgrep -f firefox -F /var/run/univention-welcome-screen.pid > /dev/null
 46 »   »   »   if [ "0" != "$?" ]; then
 47 »   »   »   »   log_action_end_msg 0
 48 »   »   »   »   exit 0
 49 »   »   »   fi
 50 »   »   fi
Comment 5 Florian Best univentionstaff 2015-10-01 17:15:02 CEST
Please make also sure that the script is not executed after the debian installer!
Comment 6 Alexander Kramer univentionstaff 2015-10-20 16:58:47 CEST
Made sure that the welcome screen is only stared once during DI and System Setup.
One also gets an msg if an instance is already running while calling the init script again.


UCS 4.0-3
r 64614
Adapted debian rules and init script to avoid multiple running instances

r 64628
Changed debian / rules --> dh_installinit defaults back to 99

Successful build
Package: univention-welcome-screen
Version: 0.0.0-14.14.201510201617
Branch: ucs_4.0-0
Scope: errata4.0-3

r 64631 yaml file


UCS4.1-0
r 64632 merged changes

Successful build
Package: univention-welcome-screen
Version: 1.0.0-2.15.201510201657
Branch: ucs_4.1-0
Comment 7 Stefan Gohmann univentionstaff 2015-10-21 17:33:42 CEST
(In reply to Florian Best from comment #5)
> Please make also sure that the script is not executed after the debian
> installer!

This still happens with the latest version in 4.1. 

To reproduce: 
 Just boot via DVD
 run the debian installer
 run system setup
 click on the finish button. 

After that the welcome screen will be shown for a few seconds.
Comment 8 Alexander Kramer univentionstaff 2015-10-22 14:23:29 CEST
Cleanup script starts welcome screen only if debian installer is not running

UCS 4.0-3 / r 64721
Successful build
Package: univention-welcome-screen
Version: 0.0.0-15.17.201510221033
Branch: ucs_4.0-0
Scope: errata4.0-3

r 64748 yaml file


UCS 4.1-0 / r 64751

Package: univention-welcome-screen
Version: 1.0.0-4.18.201510221422
Branch: ucs_4.1-0
Comment 9 Erik Damrose univentionstaff 2015-11-02 12:41:14 CET
OK: improved running instance detection, u-w-s is not run twice
OK: not starting if debian installer is running
OK: I fixed the yaml file in r65083
OK: Merge to UCS 4.1
-> Verified
Comment 10 Janek Walkenhorst univentionstaff 2015-11-04 17:24:54 CET
<http://errata.software-univention.de/ucs/4.0/356.html>