Bug 47117 - updating 4.2 to 4.3 the dynamicmaps.cf file is removed even if the update is blocked
Summary: updating 4.2 to 4.3 the dynamicmaps.cf file is removed even if the update is ...
Status: CLOSED FIXED
Alias: None
Product: UCS
Classification: Unclassified
Component: Mail
Version: UCS 4.3
Hardware: Other Linux
: P5 normal
Target Milestone: UCS 4.3-0-errata
Assignee: Sönke Schwardt-Krummrich
QA Contact: Daniel Tröder
URL:
Keywords:
Depends on: 45963 45968 46460
Blocks:
  Show dependency tree / graph
 
Reported: 2018-06-04 08:29 CEST by Daniel Tröder
Modified: 2018-06-13 08:38 CEST (History)
4 users (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 5: Major Usability: Impairs usability in key scenarios
Who will be affected by this bug?: 3: Will affect average number of installed domains
How will those affected feel about the bug?: 3: A User would likely not purchase the product
User Pain: 0.257
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Customer ID:
Max CVSS v3 score:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Tröder univentionstaff 2018-06-04 08:29:43 CEST 
+++ This bug was initially created as a clone of Bug #45968 +++

The pre-up script removes the dynamicmaps.cf file before aborting the update because of a found Cyrus installation. The resulting system has a broken mail system.

See: https://help.univention.com/t/postfix-error-unsupported-dictionary-type-ldap-4-3-upgrade-pre-up-removes-dynamicmaps-cf/8900
Comment 1 Sönke Schwardt-Krummrich univentionstaff 2018-06-04 12:14:41 CEST 
Moved the section for removing the postfix file after the cyrus-installed-check.

No YAML as we only need the new preup.sh[.pgp]

omar:
cd /var/univention/buildsystem2/test_mirror/ftp/4.3/maintained/4.3-0/all/
cp /home/phahn/REPOS/ucs/base/univention-updater/script/preup.sh preup.sh
repo-ng-sign-release-file -p /etc/archive-keys/ucs4.0.txt -k 6B8BFD3C -i preup.sh -o preup.sh.gpg
cp --reflink=auto -fv preup.sh* ../../../../../../mirror/testing/4.3/maintained/4.3-0/all/
sudo update_mirror.sh -v testing/4.3/maintained/4.3-0/all/

After QA: same for mirror/ftp/
Comment 2 Daniel Tröder univentionstaff 2018-06-12 14:14:39 CEST 
OK: code change
OK: manual test: upgrade from 4.2 to 4.3 with Cyrus installed
    → preup.sh stopped update because of Cyrus
    → dynamicmaps.cf is unmodified
OK: preup.sh and preup.sh.gpg are published on updates-test.software-univention.de
OK: tested signature verification

Assignee: please publish to updates.software-univention.de and I will CLOSE the bug.
Comment 3 Sönke Schwardt-Krummrich univentionstaff 2018-06-12 21:34:22 CEST 
$ ssh omar
$ cd /var/univention/buildsystem2/test_mirror/ftp/4.3/maintained/4.3-0/all
$ cp --reflink=auto -fv preup.sh* ../../../../../../mirror/ftp/4.3/maintained/4.3-0/all/
$ sudo update_mirror.sh -v ftp/4.3/maintained/4.3-0/all/
$ curl -s http://updates.software-univention.de/4.3/maintained/4.3-0/all/preup.sh | egrep "46102|45968"
# Bug #46102: Remove Cyrus integration
# Bug #45968: let Postfix3 extension packages recreate /etc/postfix/dynamicmaps.cf with new format
$
Comment 4 Daniel Tröder univentionstaff 2018-06-13 08:38:29 CEST 
OK: preup.sh and preup.sh.gpg are published on updates.software-univention.de