Bug 1381198

Summary:	[RFE] Let's get rid of upgrade path from module directory name
Product:	Red Hat Enterprise Linux 6	Reporter:	Petr Hracek <phracek>
Component:	preupgrade-assistant	Assignee:	Michal Bocek <mbocek>
Status:	CLOSED ERRATA	QA Contact:	Alois Mahdal <amahdal>
Severity:	unspecified	Docs Contact:	Jiri Hornicek <jhornice>
Priority:	unspecified
Version:	6.9	CC:	amahdal, fkluknav, jhornice, jmazanek, jreznik, mbocek, ovasik, phracek, pstodulk, tcerna
Target Milestone:	rc	Keywords:	Extras, FutureFeature
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	preupgrade-assistant-2.5.0-1.el6	Doc Type:	Release Note
Doc Text:	Previously, each set of modules had to contain its upgrade path in the directory name, which limited the usability of the Preupgrade Assistant. With this update, the name of the module set directory can be arbitrary, and the properties.ini file containing the upgrade path is now required in the root of the directory.	Story Points:	---
Clone Of:		Environment:
Last Closed:	2017-11-14 20:40:21 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	1429926

Description Petr Hracek 2016-10-03 11:50:07 UTC

Description of problem:

Nowadays each set of modules has to contain upgrade path in directory name.
This causes several problems like module creation and testing them etc.

New proposal is to include a file like upgrade-file which will be on top of directory. Like RHEL/upgrade_path.

The file will contain source and target versions like:

# cat upgrade_path
6
7
#

or
# cat upgrade_path
6_7
#

Upstream Issue: https://github.com/upgrades-migrations/preupgrade-assistant/issues/124
Upstream Pull Request: https://github.com/upgrades-migrations/preupgrade-assistant/pull/148

Comment 2 Michal Bocek 2017-05-03 12:58:59 UTC

*** Bug 1330762 has been marked as a duplicate of this bug. ***

Comment 3 Michal Bocek 2017-10-11 14:10:51 UTC

This has been implemented in the upstream and will be included in the next release.
Related PRs:
https://github.com/upgrades-migrations/preupgrade-assistant/pull/233
https://github.com/upgrades-migrations/preupgrade-assistant/pull/282
https://github.com/upgrades-migrations/preupgrade-assistant/pull/287
https://github.com/upgrades-migrations/preupgrade-assistant/pull/297
https://github.com/upgrades-migrations/preupgrade-assistant/pull/302

Comment 5 Alois Mahdal 2017-11-01 16:31:44 UTC

Sorry for spam, fixing Github references in External Trackers section.

Comment 6 Alois Mahdal 2017-11-03 19:16:38 UTC

Veriried: test preupg/result-files now uses arbitrary name for module set and still passes with the new build.  I've also manually checked the result; looks OK.

Comment 8 errata-xmlrpc 2017-11-14 20:40:21 UTC

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2017:3211