Bug 49960 - Write exams without restarting client PCs
Write exams without restarting client PCs
Status: CLOSED FIXED
Product: UCS@school
Classification: Unclassified
Component: UMC - Exam mode
UCS@school 4.4
Other Linux
: P5 normal (vote)
: UCS@school 4.4 v3-errata
Assigned To: Ole Schwiegert
Daniel Tröder
:
Depends on: 49961
Blocks:
  Show dependency treegraph
 
Reported: 2019-08-05 08:43 CEST by Ole Schwiegert
Modified: 2019-11-14 17:57 CET (History)
1 user (show)

See Also:
What kind of report is it?: Feature Request
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 Ole Schwiegert univentionstaff 2019-08-05 08:43:43 CEST
As a teacher I don't want to have to restart all computers before I start the class test to save time and effort.

Requirements:
- There is a UCR variable that allows domain administrators to enable the "Restart Student Machines" view.
- This is necessary if computer group policies are to be stored.

Suggestion:
- Exams users are allowed to log in to all computers in the computer room.
- For original users the login is limited to the school server.

Acceptance criteria:
1 Assuming I start a class test, unless the domain administrator configures something else, the computer room overview appears directly (no view "Restart student computers").
Assuming I start a class test, if the domain administrator has configured it with a UCR variable, the "Restart Student Machines" view appears.
If this is implemented, the manual must explicitly state that the UCR variable must be set when setting up the class work mode or computer policies.
4 This UCR variable is set by default for new installations (=no restart).
5. it is enough if it works with Windows10

Are we destroying the UCS function ("Log on to these computers only")? Do we have to remember the old values on the user and restore them afterwards to avoid destroying the feature? -> Don't remember at first. If there is a problem, it will be added in a later update

-> WHY should the function be removed? Teachers say: This is too time-consuming to check if all computers are restarted, causes stress before class tests.
Comment 1 Ole Schwiegert univentionstaff 2019-08-08 15:03:10 CEST
UCRV is handled in Bug #49961

The login restriction is implemented in oschwieg/4.4/49960
Comment 2 Ole Schwiegert univentionstaff 2019-08-09 10:06:48 CEST
exam students get sambaUserWorkstations set to the computers from the computer room

original users get their sambaUserWorkstations set to one computer that is generated as $ws1$ws2... or just $ if they do not have any workstations set

after the exam the original users get their old settings reactivated

test was added

please reopen for merge and build
Comment 3 Ole Schwiegert univentionstaff 2019-08-09 15:12:17 CEST
The original users now get the $ prepended to their configured workstations during exams or just one workstation set to $ if non was specified before

Documentation is still missing. Especially admins need to know about the changes regarding exam mode and the need to restart/GPOs
Comment 4 Ole Schwiegert univentionstaff 2019-08-12 09:17:53 CEST
Documentation was added.
Please reopen for merge&build
Comment 5 Daniel Tröder univentionstaff 2019-08-12 10:25:29 CEST
I commented a commit: https://git.knut.univention.de/univention/ucsschool/commit/26be4bbe899d02593d946054c396c17b7abc624f

The rest of the code and the test looks good to me.

Please review my comment, merge and build.
Comment 6 Ole Schwiegert univentionstaff 2019-08-12 13:37:35 CEST
Changes were made as discussed (no move in lib, catch len==0)
Comment 7 Ole Schwiegert univentionstaff 2019-08-12 13:38:16 CEST
reopen for merge and build
Comment 8 Ole Schwiegert univentionstaff 2019-08-14 09:26:03 CEST
The code is not backwards compatible if school slave systems are not updated yet (new required argument in UMC call -> https://git.knut.univention.de/univention/ucsschool/commit/c2bcf0416fa105c5e526129ef244488554d8412b)
Comment 9 Ole Schwiegert univentionstaff 2019-08-22 09:58:29 CEST
Backwards compatibility was added by removing required=True from new parameter.

Please reopen for merge and build
Comment 10 Daniel Tröder univentionstaff 2019-09-04 12:00:13 CEST
OK: documentation
OK: limiting access through sambaUserWorkstations
OK: restoring previous value of sambaUserWorkstations
OK: backwards compatible code change
OK: tested on singlemaster
OK: tested on multiserver

Please merge and build.
Comment 11 Ole Schwiegert univentionstaff 2019-09-04 15:19:11 CEST
Package: ucs-test-ucsschool
Version: 6.0.45A~4.4.0.201909041515
Branch: ucs_4.4-0
Scope: ucs-school-4.4

Package: ucs-school-umc-exam
Version: 9.0.1-15A~4.4.0.201909041517
Branch: ucs_4.4-0
Scope: ucs-school-4.4
Comment 12 Daniel Tröder univentionstaff 2019-09-05 10:07:10 CEST
OK: Jenkins tests succeeded
OK: advisory
Comment 13 Sönke Schwardt-Krummrich univentionstaff 2019-11-14 17:57:06 CET
UCS@school 4.4 v4 has been released.

https://docs.software-univention.de/changelog-ucsschool-4.4v4-de.html

If this error occurs again, please clone this bug.