Bug 39192 - Docker App uninstallation
Docker App uninstallation
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: App Center
UCS 4.1
Other Linux
: P5 normal (vote)
: UCS 4.1
Assigned To: Dirk Wiesenthal
Stefan Gohmann
: interim-2
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2015-08-17 09:26 CEST by Stefan Gohmann
Modified: 2015-11-17 12:11 CET (History)
0 users

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 2015-08-17 09:26:28 CEST
A Docker App can be uninstalled via the UMC App Center module.

The user can decide if all data of the App should be deleted from the hard drive. If the data should be deleted, they will be deleted. If the data should not be deleted, the data will be moved into a backup folder, for example /var/llib/docker/appcenter/removed_apps/$app_20150812. Afterwards, a hint to the directory is shown.
Comment 1 Dirk Wiesenthal univentionstaff 2015-10-22 13:08:43 CEST
Fixed in
  univention-appcenter 5.0.6-55.1.201510221245

When deleting or image upgrading an app, docker commit makes a backup of the then stopped container.

Nothing else is done, no files are back upped anywhere. But docker images lists appcenter-backup-$appid:$timestamp

This should be enough to get to the old data.

Furthermore, store_data is called before uninstallation.
Comment 2 Stefan Gohmann univentionstaff 2015-10-28 16:31:47 CET
OK, but I think the app data should be moved to a backup folder. Otherwise the data will be used again if the App is re-installed.
Comment 3 Dirk Wiesenthal univentionstaff 2015-10-29 00:03:04 CET
Okay, data is saved in
  /var/lib/univention-appcenter/backups/

I also disabled backups of containers while failing to install a new app (which reverts by removing the container, which eventually triggered the backup)
Comment 4 Stefan Gohmann univentionstaff 2015-10-29 06:59:40 CET
Yes, it looks good now. I've also added a test case:
 80_docker/58_app_uninstallation
Comment 5 Stefan Gohmann univentionstaff 2015-11-17 12:11:58 CET
UCS 4.1 has been released:
 https://docs.software-univention.de/release-notes-4.1-0-en.html
 https://docs.software-univention.de/release-notes-4.1-0-de.html

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