Bug 47117 - updating 4.2 to 4.3 the dynamicmaps.cf file is removed even if the update is blocked
updating 4.2 to 4.3 the dynamicmaps.cf file is removed even if the update is ...
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: Mail
UCS 4.3
Other Linux
: P5 normal (vote)
: UCS 4.3-0-errata
Assigned To: Sönke Schwardt-Krummrich
Daniel Tröder
:
Depends on: 45963 45968 46460
Blocks:
  Show dependency treegraph
 
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):
Max CVSS v3 score:


Attachments

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