Bug 29933 - univention-ucc-sync-persistent-files failed to re-sync the UCR settings
univention-ucc-sync-persistent-files failed to re-sync the UCR settings
Status: CLOSED FIXED
Product: Z_Univention Corporate Client (UCC)
Classification: Unclassified
Component: General
unspecified
Other Linux
: P5 normal
: UCC 1.0
Assigned To: Stefan Gohmann
Lukas Walter
: interim-3
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-01-04 23:41 CET by Stefan Gohmann
Modified: 2013-03-26 09:14 CET (History)
1 user (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 Stefan Gohmann univentionstaff 2013-01-04 23:41:49 CET
The re-sync of UCR failed with the following error:

root@ucc:~# /usr/sbin/univention-ucc-sync-persistent-files --sync-to-ucc
Copying /ucc_root/persistent-files/tmp/none to UCC system
Setting ucc/persistent/ucr
/bin/sh: 1: ucc/persistent/files: not found
/bin/sh: 1: ucc/persistent/ucr: not found
root@ucc:~# 
root@ucc:~# cat /ucc_root/persistent-ucr.conf
ucc/persistent/ucr:;ucc/persistent/files;ucc/persistent/ucr

The reason is the subprocess call with shell=True
 subprocess.call("/usr/sbin/univention-config-registry set " + vname + "=" + vvalue, shell=True)

In this example it will execute this (three shell commands because of the ;): 
/usr/sbin/univention-config-registry set ucc/persistent/ucr=;ucc/persistent/files;ucc/persistent/ucr

univention-ucc-sync-persistent-files should use the python interface of UCR.
Comment 1 Stefan Gohmann univentionstaff 2013-01-05 00:13:08 CET
The value of an UCR variable will be cut at the first ":"

The key=value pair
ucc/perupdate_doc_releasenotes_url=http://download.univention.de/doc/

will be changed to
ucc/perupdate_doc_releasenotes_url=http
Comment 2 Stefan Gohmann univentionstaff 2013-01-05 00:13:38 CET
Both issues have been fixed.
Comment 3 Lukas Walter univentionstaff 2013-01-31 16:50:37 CET
OK -> The described error does not occur anymore.


Verified.
Comment 4 Moritz Muehlenhoff univentionstaff 2013-03-26 09:14:30 CET
UCC 1.0 has been released: 
http://forum.univention.de/viewtopic.php?f=26&t=2417
http://forum.univention.de/viewtopic.php?f=54&t=2418

If this error occurs again, please use "Clone This Bug".