Bug 796805

Summary: Merge aeolus-all into aeolus-conductor
Product: [Retired] CloudForms Cloud Engine Reporter: John Eckersberg <jeckersb>
Component: aeolus-allAssignee: John Eckersberg <jeckersb>
Status: CLOSED CURRENTRELEASE QA Contact: wes hayutin <whayutin>
Severity: low Docs Contact:
Priority: unspecified    
Version: 1.0.0CC: slinaber
Target Milestone: rc   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: v0.8.0-36 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-08-30 17:16:11 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description John Eckersberg 2012-02-23 17:13:02 UTC
Description of problem:

Presently within the conductor source, we have two spec files, one for aeolus-all and one for aeolus-conductor.  Both are always the same version, and both always contain the same changelog.  When cutting a new release the process is as follows:

1. Update spec for conductor
2. Copy conductor spec updates into specfile for aeolus-all
3. Run `make rpms`, which generates two srpms, one for aeolus-conductor and one for aeolus-all
4. Use rhpkg to import aeolus-conductor into dist-git, commit, push, build
5. Use rhpkg to import aeolus-all into dist-git, commit, push, build

It makes much more sense to have aeolus-all be a subpackage of aeolus-conductor instead.  This way aeolus-all will inherit the version and changelog from aeolus-conductor automatically.  `make rpms` would only output one srpm, and we would only have to import and build a single srpm.

Comment 1 John Eckersberg 2012-02-23 17:15:14 UTC
I forgot to mention, jgreguske specifically pointed out that standalone meta-packages (which aeolus-all presently is) are frowned upon and the proposed solution is more standard for them.

Comment 2 John Eckersberg 2012-02-23 17:30:14 UTC
Patch on the list:

https://fedorahosted.org/pipermail/aeolus-devel/2012-January/008433.html

Comment 3 John Eckersberg 2012-02-23 17:38:01 UTC
40ced33 on github/master

Comment 4 Steve Linabery 2012-02-24 04:16:14 UTC
f36474d

Comment 5 wes hayutin 2012-02-27 21:19:27 UTC
verified