Bug 46850

Summary:	preup.sh should check, if a failed.ldif exists
Product:	UCS	Reporter:	Christina Scheinig <scheinig>
Component:	Update - Release updates	Assignee:	Jürn Brodersen <brodersen>
Status:	CLOSED FIXED	QA Contact:	Felix Botner <botner>
Severity:	normal
Priority:	P5	CC:	gohmann
Version:	UCS 4.2
Target Milestone:	UCS 4.3-0-errata
Hardware:	Other
OS:	Linux
What kind of report is it?:	Bug Report	What type of bug is this?:	2: Improvement: Would be a product improvement
Who will be affected by this bug?:	2: Will only affect a few installed domains	How will those affected feel about the bug?:	5: Blocking further progress on the daily work
User Pain:	0.114	Enterprise Customer affected?:
School Customer affected?:	Yes	ISV affected?:
Waiting Support:		Flags outvoted (downgraded) after PO Review:
Ticket number:	2018041821000155	Bug group (optional):
Max CVSS v3 score:

Description Christina Scheinig

2018-04-19 10:46:40 CEST

In a customer environment the update to UCS 4.2 broke, because a failed.ldif exist and could not be applied.
After fixing the failed.ldif, still an unconfigured slapd package was left behind, and the update could not be restarted.

Maybe we can check for a failed.ldif in the preup script?

Comment 1 Jürn Brodersen

2018-05-09 18:43:30 CEST

[4.3-0 575138c183] Bug #46850: check for failed.ldif in preup.sh
[4.3-0 0d89c46be4] Bug #46850: YAML
[4.3-0 f98772fd9a] Bug #46850: YAML

I didn't add an ucr variable because moving the failed.ldif shouldn't be a problem if someone wants to ignore the error.

Comment 2 Felix Botner

2018-05-14 14:50:43 CEST

please make this change also to the current 4.3-0 preup.sh, there is no automatic way to change this file post release

see https://forge.univention.org/bugzilla/show_bug.cgi?id=46669#c1

Comment 3 Jürn Brodersen

2018-05-14 15:02:07 CEST

(In reply to Felix Botner from comment #2)
> please make this change also to the current 4.3-0 preup.sh, there is no
> automatic way to change this file post release
> 
> see https://forge.univention.org/bugzilla/show_bug.cgi?id=46669#c1

Done. The new preup.sh should be signed and on the test mirror.

After QA but before closing see: https://forge.univention.org/bugzilla/show_bug.cgi?id=46669#c3

Comment 4 Felix Botner

2018-05-14 16:50:54 CEST

OK, looks good

Comment 5 Jürn Brodersen

2018-05-14 17:11:38 CEST

preup.sh is now on "updates.software-univention.de"

Comment 6 Felix Botner

2018-05-14 17:46:48 CEST

OK - https://updates.software-univention.de/
OK - update fails with existing failed.ldif
OK - update starts without failed.ldif
OK - signature
OK - YAML

Comment 7 Arvid Requate

2018-05-16 17:04:24 CEST

<http://errata.software-univention.de/ucs/4.3/40.html>