Bug 29931 - sync_to_persistent code in sync_to_ucc mode
sync_to_persistent code in sync_to_ucc mode
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
: interim-3
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-01-04 22:12 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 22:12:32 CET
While debugging the persistent file handling, I found this code block in univention-ucc-sync-persistent-files which confused me. I think it must be removed to prevent errors. This block is part of the sync_to_ucc mode, but the code syncs to the persistent storage:

Index: univention-ucc-sync-persistent-files
===================================================================
--- univention-ucc-sync-persistent-files        (Revision 38051)
+++ univention-ucc-sync-persistent-files        (Arbeitskopie)
@@ -107,17 +107,3 @@
        else:
                print "Storage directory for persistent Univention Configuration Registry configuration not found"
 
-       if configRegistry.has_key("ucc/persistent/files"):
-
-               if os.path.exists("/ucc_root/persistent-files.SAVE"):
-                       shutil.rmtree("/ucc_root/persistent-files.SAVE", True)
-
-               if os.path.exists("/ucc_root/persistent-files"):
-                       os.renames("/ucc_root/persistent-files", "/ucc_root/persistent-files.SAVE")
-                       os.makedirs("/ucc_root/persistent-files")
-
-               for i in configRegistry["ucc/persistent/files"].split(";"):
-                       if os.path.exists(i):
-                               shutil.copy(i, "/ucc_root/persistent-files/")
-                               print "Copying", i, " from persistent storage partition to UCC system"
-
Comment 1 Stefan Gohmann univentionstaff 2013-01-04 22:32:34 CET
The code has been removed.
Comment 2 Moritz Muehlenhoff univentionstaff 2013-01-24 14:08:05 CET
The code is removed.
Comment 3 Moritz Muehlenhoff univentionstaff 2013-03-26 09:14:11 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".