Bug 45328 – Registering Non-Docker Apps may fail when updated from 4.1 to 4.2

Bug 45328 - Registering Non-Docker Apps may fail when updated from 4.1 to 4.2


Summary:	Registering Non-Docker Apps may fail when updated from 4.1 to 4.2

Status:	CLOSED FIXED

Product:	UCS
Classification:	Unclassified
Component:	App Center
Version:	UCS 4.2
Hardware:	Other Linux

Importance:	P5 normal (vote)
Target Milestone:	UCS 4.2-2-errata
Assigned To:	Felix Botner
QA Contact:	Dirk Wiesenthal

URL:
Keywords:

Depends on:	44724 44821
Blocks:
	Show dependency tree / graph

Reported:	2017-09-06 11:39 CEST by Dirk Wiesenthal
Modified:	2017-09-26 12:04 CEST (History)
CC List:	2 users (show)

See Also:
What kind of report is it?:	Bug Report
What type of bug is this?:	1: Cosmetic issue or missing function but workaround exists
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.051
Enterprise Customer affected?:	Yes
School Customer affected?:	Yes
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:	2017053021000444
Bug group (optional):
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 Dirk Wiesenthal

2017-09-06 11:39:37 CEST

We may have to workaround this during the upgrade (where the errata updates are not yet installed) by fixing postup.sh, too.

+++ This bug was initially created as a clone of Bug #44724 +++

When ucsschool="4.1 R2 v11" was installed and an update is made,

  univention-app register ucsschool
tries to register the 4.1 App again.

This will eventually lead to an unregistered component.

Workaround here:
  univention-app register 4.2/ucsschool="4.2 v1" --do-it

Comment 1 Dirk Wiesenthal

2017-09-06 11:40:19 CEST

Tagging it 4.2-2 errata... not sure what would be correct here

Comment 3 Felix Botner

2017-09-08 15:49:42 CEST

Added univention-app update; univention-app register to the 4.2-1 postup on updates-test.software-univention.de (and git).

Please test a (direct) update (with school) from 4.1-4 to 4.2-1. The app version should be correct now.

Is this works, please Reopen, we have to copy the now script to updates.software ...

Comment 4 Dirk Wiesenthal

2017-09-12 15:40:07 CEST

As discussed. Works fine wrt UCS@school regsitration.

But I get the following in my logs:
Registering schema /usr/share/univention-appcenter/apps/openproject/openproject.schema
/usr/lib/python2.7/getpass.py:83: GetPassWarning: Can not control echo on the terminal.
  passwd = fallback_getpass(prompt, stream)
Warning: Password input may be echoed.

This is due to some Apps trying to register a schema file, which requires admin credentials. As we cannot pass this in postup.sh, I suggest limiting the registration to:

univention-app register --app


Other than that, it works (actually, due to the ordering of the register command, UCS@school is correctly registered even with the traceback above). Please add it to 4.2-2's postup.sh, though. That way, also those domains already updated to 4.2-1, but not 4.2-2 yet, will get the fix.

Comment 5 Felix Botner

2017-09-12 17:20:10 CEST

* added --app  updates-test.software-univention.de,
  git 5d910b09070742075ddca241c4699cc478bfd7dc
* added to 4.2-2 postup updates-test.software-univention.de,
  git 0876f5de8a77d5af8b7101649ba1afe31a37dfff

Is this works, please Reopen, we have to copy the now script to updates.software ...

Comment 6 Dirk Wiesenthal

2017-09-13 09:59:58 CEST

Works fine

Comment 7 Dirk Wiesenthal

2017-09-18 10:06:07 CEST

Whoops

Comment 8 Felix Botner

2017-09-21 13:20:53 CEST

released (4.2-1 and 4.2-2)

Comment 9 Dirk Wiesenthal

2017-09-26 12:04:47 CEST

OK, works