Bug 40135 - When installing a Docker App fails: Docker container still exists
When installing a Docker App fails: Docker container still exists
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: App Center
UCS 4.1
Other Linux
: P5 normal (vote)
: UCS 4.1-0-errata
Assigned To: Dirk Wiesenthal
Daniel Tröder
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2015-11-30 22:14 CET by Dirk Wiesenthal
Modified: 2015-12-09 16:47 CET (History)
2 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 Dirk Wiesenthal univentionstaff 2015-11-30 22:14:23 CET
When the installation fails, "remove" is called. With a special argument: --do-not-backup.

Because of that the docker container is not stopped (done in the backup routine). Then the container is to be removed. But as it was not stopped, this fails.
Comment 1 Dirk Wiesenthal univentionstaff 2015-11-30 23:44:40 CET
Fixed in
  univention-appcenter 5.0.19-18.77.201511302249

New functions docker.stop() added which bypasses the Stop.call() command - as the superior init.d script is already removed at that point. Here only the real docker stop is important.
Comment 2 Daniel Tröder univentionstaff 2015-12-03 11:49:09 CET
OK: advisory (changed wording in r66072)
OK: Test:
 # univention-app install dudle-docker

****
in second terminal:
****

 # pkill -f univention-join

****
in first terminal:
****


Something went wrong while joining the system
Something went wrong while joining the system

Setup script failed!
Aborting...
Going to remove Dudle-docker (1.0.0-2)
314b75e353b79fdedaa2654608a7bca564c37728839190767f651dc017186648
314b75e353b79fdedaa2654608a7bca564c37728839190767f651dc017186648
 Removing any system startup links for /etc/init.d/docker-app-dudle-docker ...
   /etc/rc0.d/K14docker-app-dudle-docker
   /etc/rc1.d/K14docker-app-dudle-docker
   /etc/rc2.d/S41docker-app-dudle-docker
   /etc/rc3.d/S41docker-app-dudle-docker
   /etc/rc4.d/S41docker-app-dudle-docker
   /etc/rc5.d/S41docker-app-dudle-docker
   /etc/rc6.d/K14docker-app-dudle-docker
Removing localhost from LDAP object
[..]

# docker ps
CONTAINER ID        IMAGE               COMMAND             CREATED             STATUS              PORTS               NAMES
[empty]

# docker images
REPOSITORY                                       TAG                 IMAGE ID            CREATED             VIRTUAL SIZE
docker.software-univention.de/ucs-appbox-amd64   4.1-0               e0cf89397138        3 weeks ago         373.8 MB

[only the base image]
(which btw. still has the atd bug)
Comment 3 Janek Walkenhorst univentionstaff 2015-12-09 16:47:12 CET
<http://errata.software-univention.de/ucs/4.1/27.html>