Bug 1191419

Summary: Move config classes into subpackage
Product: [oVirt] ovirt-node Reporter: Fabian Deutsch <fdeutsch>
Component: GeneralAssignee: Douglas Schilling Landgraf <dougsland>
Status: CLOSED CURRENTRELEASE QA Contact: Ying Cui <ycui>
Severity: medium Docs Contact:
Priority: high    
Version: ---CC: bazulay, bugs, cshao, gklein, leiwang, lsurette, mgoldboi, ovirt-bugs, rbalakri, ycui, yeylon, ykaul
Target Milestone: ovirt-3.6.0-rcFlags: ykaul: needinfo? (bazulay)
rule-engine: ovirt-3.6.0+
ylavi: planning_ack+
rule-engine: devel_ack+
ycui: testing_ack+
Target Release: 3.2   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: node
Fixed In Version: ovirt-node-3.6.0-0.24.20151209gitc0fa931.el7ev Doc Type: Bug Fix
Doc Text:
http://www.ovirt.org/Features/Node/subpackages
 Story Points: ---
Clone Of: Environment:
Last Closed: 2016-02-02 14:40:00 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Node RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1190758    

Description Fabian Deutsch 2015-02-11 10:06:42 UTC 
Description of problem:
Currently the ovirt-node package includes everything from the TUI and installer.
Because in future we want to use cockpit, we should split the "backend" config.defaults package into a subpackage, to be able to use it standalone.

That package has a couple of dependencies (like utils) which also need to get into that core package.

Version-Release number of selected component (if applicable):
3.5
Comment 1 Fabian Deutsch 2015-02-11 10:07:41 UTC 
The idea is to separate the TUI/installer from the actual classes doing the configuration.
Comment 2 Fabian Deutsch 2015-02-12 11:52:06 UTC 
After moving the configuration modules into a separate package, some important things to remember are:
1. Add necessary Requires on affected packages
2. Ensure that necessary dependencies  are also included (ovirt-node.utils)
3. Ensure that NO old code (ovirtnode.*) is required as a dependency.
Comment 3 Fabian Deutsch 2015-02-26 16:09:20 UTC 
The premise is to only make packaging changes, no code changes for now.