Comment 1 Felix Botner 2016-03-14 12:31:13 CET

OK, added dudle_20160201 on appcenter.test

during the update from non-docker to the docker app, all user data is migrated.

Comment 2 Dirk Wiesenthal 2016-04-19 17:18:15 CEST

I think that after
  DockerVolumes=/var/www/dudle:/var/www/dudle

you could remove preinst, store_data, restore_data

You increased the version of dudle.deb. Has it seen a bugfix release? If so, could you adjust the README_UPDATE? Can you put the sources in the source/ dir of the App Center? Daniel did it for 1.0.1.

Comment 3 Daniel Tröder 2016-04-20 08:49:02 CEST

Just a sidenote: Dudle is now hosted Github. The author has just tagged two v1 releases: https://github.com/kellerben/dudle/releases
He wants release a v2 soonish.

Comment 4 Felix Botner 2016-04-29 10:51:24 CEST

(In reply to Daniel Tröder from comment #3)
> Just a sidenote: Dudle is now hosted Github. The author has just tagged two
> v1 releases: https://github.com/kellerben/dudle/releases
> He wants release a v2 soonish.

ok, built version v1.1

(In reply to Dirk Wiesenthal from comment #2)
> I think that after
>   DockerVolumes=/var/www/dudle:/var/www/dudle
> 
> you could remove preinst, store_data, restore_data
> 
> You increased the version of dudle.deb. Has it seen a bugfix release? If so,
> could you adjust the README_UPDATE? Can you put the sources in the source/
> dir of the App Center? Daniel did it for 1.0.1.

This DockerVolumes=/var/www/dudle:/var/www/dudle does not work. The dudle package installs it's scripts into /var/www/dudle too, so after removing the package from the docker host all the dudle cgi scripts are gone in the container too (cause they share the install directory).

So, instead, in the preinst i move the data from the docker host into the standard share directory (/var/lib/univention-appcenter/apps/dudle/data) and in the setup script i create a link from /var/www/dudle to /var/lib/univention-appcenter/apps/dudle/data. Now everything (data and progs) goes to /var/lib/univention-appcenter/apps/dudle/data (the persistent store).

Comment 5 Dirk Wiesenthal 2016-05-13 11:29:23 CEST

Please wait for Bug#41178 to be released. Then set the errata level as RequiredUCSVersion.

Also, these "errors" show up in a UMC install when installing (not upgrading) the latest version:

    ls: Zugriff auf /var/www/dudle nicht möglich: Datei oder Verzeichnis nicht gefunden
    + DATADIR=/var/lib/univention-appcenter/apps/dudle/data
    + '[' '!' -e /var/lib/univention-appcenter/apps/dudle/data ']'
    + ln -s /var/lib/univention-appcenter/apps/dudle/data /var/www/dudle
    + /usr/share/univention-docker-container-mode/setup --username Administrator --password-file /tmp/tmp.mPXZ4xxzV2 --app dudle --app-version 1.1.0-1 --error-file /tmp/tmp.aYzyyuZP5B
    + exit 0

Please remove the -x option and check for an installed non-docker app:
  preinst would be called with --old-version 1.0.1-1
  or you just do a "ls 2>/dev/null" so that the warning does not show up.

Comment 6 Felix Botner 2016-05-13 12:29:56 CEST

fixed

Comment 7 Stefan Gohmann 2016-09-05 07:09:50 CEST

I think this issue should be re-checked and it can be released with the other Docker Apps.

Comment 8 Felix Botner 2016-09-15 16:36:37 CEST

works for me

Comment 9 Dirk Wiesenthal 2016-09-30 13:23:05 CEST

OK, works as expected

Comment 10 Dirk Wiesenthal 2016-10-19 12:12:32 CEST

Published with RequiredUCSVersion=4.1-3 errata293

Comment 11 Felix Botner 2016-12-14 12:06:35 CET

TODO docker migration

Comment 12 Felix Botner 2016-12-14 12:21:58 CET

http://wiki.univention.de/index.php?title=Dudle_App_Migration

added "Dockermigrationlink" to dudle_20160201.ini

Comment 13 Dirk Wiesenthal 2017-01-19 14:54:09 CET

Works