Bug 41846 – Version check upon app installation is incorrect

Bug 41846 - Version check upon app installation is incorrect


Summary:	Version check upon app installation is incorrect

Status:	CLOSED DUPLICATE of bug 43901

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

Importance:	P5 normal (vote)
Target Milestone:	---
Assigned To:	Dirk Wiesenthal
QA Contact:

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2016-07-22 16:41 CEST by Johannes Keiser
Modified:	2017-07-06 14:25 CEST (History)
CC List:	2 users (show)

See Also:
What kind of report is it?:	Bug Report
What type of bug is this?:	3: Simply Wrong: The implementation doesn't match the docu
Who will be affected by this bug?:	1: Will affect a very few installed domains
How will those affected feel about the bug?:	2: A Pain – users won’t like this once they notice it
User Pain:	0.034
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
proposed patch (1.56 KB, patch) 2016-07-22 16:41 CEST, Johannes Keiser	Details \| Diff
app_installation_version_compare_tests (3.58 KB, patch) 2016-07-22 16:54 CEST, Johannes Keiser	Details \| Diff
View All 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 Johannes Keiser

2016-07-22 16:41:18 CEST

Created attachment 7826 [details]
proposed patch

If any of the version number parts (major, minor, patchlevel, errata) is higher than the current UCS version it is seen as not fitting.
This comparison can fail for e.g.

Required: 4.0-2 errata400
Current:  4.1-2 errata200

Even though the app should be installable since the errata level is higher than the current one it is seen as not fitting.

Comment 1 Johannes Keiser

2016-07-22 16:54:33 CEST

Created attachment 7827 [details]
app_installation_version_compare_tests

Comment 2 Dirk Wiesenthal

2016-09-22 06:25:51 CEST

I would qualify this as a wrong ini file. Requiring 4.0-2 for a 4.1 App is wrong. Also errata levels are more or less consecutive through all patchlevel releases.

If this ever happens, it can be fixed by changing the ini file. But yes, it is wrong.

Comment 3 Dirk Wiesenthal

2017-03-22 13:49:02 CET


*** This bug has been marked as a duplicate of bug 43901 ***