Bug 48100

Summary: Document support for docker multi container setup per app in the App Center
Product: UCS extended documentation Reporter: Nico Gulden <gulden>
Component: App ProviderAssignee: Nico Gulden <gulden>
Status: CLOSED FIXED QA Contact: Dirk Wiesenthal <wiesenthal>
Severity: normal    
Priority: P5 CC: gulden
Version: unspecified   
Target Milestone: ---   
Hardware: Other   
OS: Linux   
See Also: https://forge.univention.org/bugzilla/show_bug.cgi?id=43612
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?: Yes
Waiting Support: Flags outvoted (downgraded) after PO Review:
Ticket number: Bug group (optional):
Max CVSS v3 score:
Bug Depends on:    
Bug Blocks: 48312    
Attachments: Doc build with changes
New Document build

Description Nico Gulden univentionstaff 2018-11-06 11:16:37 CET
The support for docker multi container setup per app in the App Center should be documented in the App Provider documentation.

The setup currently only allows a single YAML file for the docker compose setup. Environment variables and references to pre-filled volumes are not supported currently.

The life cycle scripts can only deal with one container. As a workaround, the app provider should additionally provide the location to the main docker image from the yaml file to the docker image setting in the provider portal. The scripts will then refer to this image.

Port redirection should still be configured in the portal, as well as the webinterface and port setting.
Comment 1 Nico Gulden univentionstaff 2018-11-07 12:05:16 CET
A new section has been added: http://jenkins.knut.univention.de:8080/view/Doku/job/UCS-4.3/job/UCS-4.3-2/job/HandbookAppProvider/lastSuccessfulBuild/artifact/webroot/app-provider-4.3.html#create-app-with-docker:compose

A reference to the UCS errata version needs to be added.

Currently, the text says it uses Docker Compose 1.8.0 and supports the compose file format in version 2.0, according to the implementation information. https://docs.docker.com/compose/compose-file/#reference-and-guidelines says that compose file format is supported with Docker Compose version 1.10.0+
Comment 2 Nico Gulden univentionstaff 2018-11-07 15:47:18 CET
Missunderstanding. Removed the hint on Docker Compose version and just kept the information on compose file version 2.0.
Comment 3 Nico Gulden univentionstaff 2018-11-28 15:23:33 CET
Reverted the changes from the 4.3-2 branch.
Comment 4 Nico Gulden univentionstaff 2018-11-28 15:38:44 CET
Created attachment 9762 [details]
Doc build with changes

For QA, the attached build includes the changes to the documentation.
Comment 5 Nico Gulden univentionstaff 2018-12-04 12:15:09 CET
Needs to be updated to latest final implementation after QA.
Comment 6 Nico Gulden univentionstaff 2018-12-04 23:22:53 CET
Created attachment 9768 [details]
New Document build
Comment 7 Nico Gulden univentionstaff 2018-12-04 23:24:06 CET
Updated the documentation after the implemtenation has been verified.

See https://git.knut.univention.de/univention/ucs/compare/4.3-2...ngulden%2F48100 for the changes.

For QA: The latest build is attached as PDF.
Comment 8 Nico Gulden univentionstaff 2018-12-05 15:48:21 CET
Bugzilla does not allow me to add a new build and mark the others as obsolete. The last build: http://jenkins.knut.univention.de:8080/job/BuildDocBookBranch/lastSuccessfulBuild/artifact/webroot/app-provider-4.3.html


I added the errata update number.
Comment 9 Dirk Wiesenthal univentionstaff 2018-12-06 13:18:39 CET
Ok, I only made small adjustments.

I deleted the part where it was stated that Volumes and Database integration fields may be left empty. This is true, but technically it does not matter. Should they specify volumes, they are added to the main service. Should they specify a database, the database is created and the environment variables are added. All these fields make little sense, but they work, should anyone use them.

Technically, "--env-file" file integration is currently not available in multi container apps. I guess we can ignore that until someone really fills in something and wonders why it is nowhere applied.
Comment 10 Philipp Hahn univentionstaff 2018-12-11 10:57:36 CET
published